Page MenuHomePhabricator

tokens: Remove deprecated `min-size-base`
Closed, ResolvedPublic

Description

Background

In https://gerrit.wikimedia.org/r/c/design/codex/+/923445 we deprecated the min-size-base token in favor of min-size-interactive-pointer, but we haven't yet removed it, and it's still being used in various places. We should migrate existing uses of it and then remove it (or rather, move it to the deprecated aliases file so that it's still exported for wikimedia-ui-base but no longer for Codex users).

It's more straightforward to get rid of all deprecated WikimediaUI Base variables in Codex (the 'deprecated-aliases-wikimedia-ui-base.json' layer) at once than to pick out 1:1. That's why we're focusing just on the downstream consumer side of things here.

Acceptance

Event Timeline

Restricted Application added a subscriber: Aklapper. · View Herald Transcript
Volker_E renamed this task from Remove deprecated token min-size-base to tokens: Remove deprecated `min-size-base`.May 31 2023, 4:30 AM
Volker_E updated the task description. (Show Details)
Volker_E subscribed.

Change 928947 had a related patch set uploaded (by VolkerE; author: VolkerE):

[mediawiki/extensions/MediaSearch@master] styles: Replace deprecated `min-size-base` token

https://gerrit.wikimedia.org/r/928947

Change 928948 had a related patch set uploaded (by VolkerE; author: VolkerE):

[mediawiki/extensions/Kartographer@master] styles: Replace deprecated `min-size-base` token

https://gerrit.wikimedia.org/r/928948

Change 928950 had a related patch set uploaded (by VolkerE; author: VolkerE):

[mediawiki/extensions/GrowthExperiments@master] styles: Replace deprecated `min-size-base` token

https://gerrit.wikimedia.org/r/928950

Change 928947 merged by jenkins-bot:

[mediawiki/extensions/MediaSearch@master] styles: Replace deprecated `min-size-base` token

https://gerrit.wikimedia.org/r/928947

Change 928948 merged by jenkins-bot:

[mediawiki/extensions/Kartographer@master] styles: Replace deprecated `min-size-base` token

https://gerrit.wikimedia.org/r/928948

Change 928950 merged by jenkins-bot:

[mediawiki/extensions/GrowthExperiments@master] styles: Replace deprecated `min-size-base` token

https://gerrit.wikimedia.org/r/928950

Change 945686 had a related patch set uploaded (by Catrope; author: Catrope):

[design/codex@main] tokens: Remove min-size-base, make it a deprecated alias

https://gerrit.wikimedia.org/r/945686

Change 945686 merged by jenkins-bot:

[design/codex@main] tokens: Remove min-size-base, make it a deprecated alias

https://gerrit.wikimedia.org/r/945686

Catrope claimed this task.
Catrope reassigned this task from Catrope to Volker_E.

Change 946638 had a related patch set uploaded (by VolkerE; author: VolkerE):

[mediawiki/core@master] mediawiki.less: Hard remove deprecated `min-size-base` token

https://gerrit.wikimedia.org/r/946638

Quickly reopening for the skin variables piece.

Change 949555 had a related patch set uploaded (by Anne Tomasevich; author: Anne Tomasevich):

[mediawiki/core@master] Update Codex from v0.16.1 to v0.17.0

https://gerrit.wikimedia.org/r/949555

Test wiki created on Patch demo by ATomasevich (WMF) using patch(es) linked to this task:
https://patchdemo.wmflabs.org/wikis/6188b4e5fe/w

Change 949555 merged by jenkins-bot:

[mediawiki/core@master] Update Codex from v0.16.1 to v0.17.0

https://gerrit.wikimedia.org/r/949555

Change 946638 merged by jenkins-bot:

[mediawiki/core@master] mediawiki.less: Hard remove deprecated `min-size-base` token

https://gerrit.wikimedia.org/r/946638

Test wiki on Patch demo by ATomasevich (WMF) using patch(es) linked to this task was deleted:

https://patchdemo.wmflabs.org/wikis/6188b4e5fe/w/

Change 970878 had a related patch set uploaded (by VolkerE; author: VolkerE):

[oojs/ui@master] styles, themes: Replace deprecated `@min-size-*` vars

https://gerrit.wikimedia.org/r/970878

Change 970878 merged by jenkins-bot:

[oojs/ui@master] styles, themes: Replace deprecated `@min-size-*` vars

https://gerrit.wikimedia.org/r/970878

The final step of removing the token itself is tackled in T350447.

Change 980958 had a related patch set uploaded (by Anne Tomasevich; author: Anne Tomasevich):

[mediawiki/core@master] Update OOUI to v0.48.3

https://gerrit.wikimedia.org/r/980958

Change 980958 merged by jenkins-bot:

[mediawiki/core@master] Update OOUI to v0.48.3

https://gerrit.wikimedia.org/r/980958