Page MenuHomePhabricator

Standardize 'mediawiki.ui' variables to CSS variables naming scheme in preparation for WikimediaUI Base variables takeover
Closed, ResolvedPublic2 Estimated Story Points

Description

As of current, MW core's deprecated mediawiki.ui variables rely on a mix of variables that don't follow the CSS naming scheme guidelines.

While we're moving forward to rely on a standardized set of variables to enable using general variables with an override/call of WikimediaUI Base variables for Wikimedia projects to be relied on,

  • it's necessary to deprecate mediawiki.ui's variables and
  • rename the remaining variables to new naming scheme resulting in equal variables like in WikimediaUI Base in a go

This task is limited to MediaWiki core. Skins and extensions are targeted in separate tasks.

Event Timeline

Volker_E renamed this task from Standardize to Standardize 'mediawiki.ui' variables to CSS variables naming scheme in preparation for WikimediaUI Base variables takeover.Jul 7 2020, 12:22 AM
Volker_E updated the task description. (Show Details)

Change 610438 had a related patch set uploaded (by VolkerE; owner: VolkerE):
[mediawiki/core@master] mediawiki.ui: Deprecate all '@colorButton*' and @colorDisabledText variables

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

Change 610601 had a related patch set uploaded (by VolkerE; owner: VolkerE):
[mediawiki/core@master] mediawiki.ui: Deprecate @sizeInputBinary and replace with @size-input-binary

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

Change 610688 had a related patch set uploaded (by VolkerE; owner: VolkerE):
[mediawiki/core@master] mediawiki.ui: Deprecate *InputBinary* variables

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

Change 610438 merged by jenkins-bot:
[mediawiki/core@master] mediawiki.ui: Deprecate all '@colorButton*' and @colorDisabledText variables

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

Change 610601 merged by jenkins-bot:
[mediawiki/core@master] mediawiki.ui: Deprecate @sizeInputBinary and replace with @size-input-binary

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

Change 610688 merged by jenkins-bot:
[mediawiki/core@master] mediawiki.ui: Deprecate *InputBinary* variables

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

Change 610937 had a related patch set uploaded (by VolkerE; owner: VolkerE):
[mediawiki/core@master] mediawiki.ui: Deprecate @colorShadow

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

Change 610939 had a related patch set uploaded (by VolkerE; owner: VolkerE):
[mediawiki/core@master] mediawiki.ui: Deprecate @borderWidthRadioChecked var and replace it

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

Change 610937 merged by jenkins-bot:
[mediawiki/core@master] mediawiki.ui: Deprecate @colorShadow

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

Change 610939 merged by jenkins-bot:
[mediawiki/core@master] mediawiki.ui: Deprecate @borderWidthRadioChecked var and replace it

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

Change 611013 had a related patch set uploaded (by VolkerE; owner: VolkerE):
[mediawiki/core@master] mediawiki.ui: Deprecate @icon* vars and replace them

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

Change 611091 had a related patch set uploaded (by VolkerE; owner: VolkerE):
[mediawiki/core@master] mediawiki.ui: Deprecate @colorRegressive

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

Change 611013 merged by jenkins-bot:
[mediawiki/core@master] mediawiki.ui: Deprecate @icon* vars and replace them

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

Change 611414 had a related patch set uploaded (by VolkerE; owner: VolkerE):
[mediawiki/core@master] mediawiki.ui: Deprecate @boxShadow* variables

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

Change 611414 merged by jenkins-bot:
[mediawiki/core@master] mediawiki.ui: Deprecate @boxShadow* variables

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

Change 611091 merged by jenkins-bot:
[mediawiki/core@master] mediawiki.ui: Deprecate @colorRegressive

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

Change 611865 had a related patch set uploaded (by VolkerE; owner: VolkerE):
[mediawiki/core@master] mediawiki.ui: Deprecate @colorProgressive & @colorDestructive* variables

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

Change 611954 had a related patch set uploaded (by VolkerE; owner: VolkerE):
[mediawiki/core@master] mediawiki.ui: Deprecate message (boxes) related variables

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

Change 611956 had a related patch set uploaded (by VolkerE; owner: VolkerE):
[mediawiki/core@master] mediawiki.ui: Deprecate @colorMediumSevere & colorLowSevere vars

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

ovasileva triaged this task as Medium priority.Jul 13 2020, 10:13 AM

Change 611865 merged by jenkins-bot:
[mediawiki/core@master] mediawiki.ui: Deprecate @colorProgressive* & @colorDestructive* variables

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

Change 611954 merged by jenkins-bot:
[mediawiki/core@master] mediawiki.ui: Deprecate message (boxes) related variables

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

Change 611956 merged by jenkins-bot:
[mediawiki/core@master] mediawiki.ui: Deprecate @colorMediumSevere & @colorLowSevere vars

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

The gray colors do not seem to meet the standardized names and do not appear to be deprecated in 1.35. Was this intentional? If not should this be addressed and backported?

(@colorGray1..15)

@Jdlrobson The comment states for several months, and in time for MW 1.35, that those colors are only for reference and will be replaced.
The only thing that might be disputed here, is to not have the term “deprecated” in this comment. I think that's debatable and possible neglibe, as CSS/Less is more of a social contract then a technical contract on its deprecation policy.

@Volker_E it was a little unclear, but I agree reading the comments more closely it should be clear that they are not to be used. I think at minimum I would however recommend adding a note about the deprecated LESS variables in the 1.35 release notes out of courtesy to 3rd parties with a timeline for their eventual removal so 3rd parties know to look at the file in the first place.

@Jdlrobson Are you aiming at a backported addition to RELEASE-NOTES-1.35?

That's my suggestion. That way nobody can accuse you of not going through the normal deprecation channels.

MBinder_WMF set the point value for this task to 2.Jul 29 2020, 5:18 PM

Change 618111 had a related patch set uploaded (by VolkerE; owner: VolkerE):
[mediawiki/core@master] Add mediawiki.ui Less variable deprecation note

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

Change 618036 had a related patch set uploaded (by VolkerE; owner: VolkerE):
[mediawiki/core@REL1_35] Add mediawiki.ui Less variable deprecation note

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

Change 618111 merged by jenkins-bot:
[mediawiki/core@master] Add mediawiki.ui Less variable deprecation note

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

Change 618036 merged by jenkins-bot:
[mediawiki/core@REL1_35] Add mediawiki.ui Less variable deprecation note

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