Page MenuHomePhabricator

[Darkmode] Find mapping for #72777d in `h1 .wikibase-title-id`
Closed, ResolvedPublic

Description

Per comment in the Figma file, we need to find a mapping for the item ID in the page headings. Currently hard-coded to #72777d, which potentially maps to color-placeholder

2025-03-19-121727_370x139_scrot.png (370×139 px, 10 KB)

Acceptance Criteria

  • h1 .wikibase-title-id is mapped to a codex token
  • other hard-coded references to #72777d are mapped to appropriate codex tokens.

Event Timeline

Change #1129235 had a related patch set uploaded (by Arthur taylor; author: Arthur taylor):

[mediawiki/extensions/Wikibase@master] Update colour for .wikibase-title-id to a codex token

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

Heading looks somewhat better now:

2025-03-19-132742_475x163_scrot.png (475×163 px, 11 KB)

There are a couple of other references to the colour value (.wikibase-toolbarbutton-disabled, .wikibase-toolbarlabel.ui-state-disabled and .wikibase-statementview-references-heading), but they don't appear to correspond to elements in the UX.

There are a couple of other references to the colour value (.wikibase-toolbarbutton-disabled, .wikibase-toolbarlabel.ui-state-disabled and .wikibase-statementview-references-heading), but they don't appear to correspond to elements in the UX.

  • .wikibase-toolbarbutton-disabled is e.g. the “save” button right after you’ve clicked “edit” (it becomes disabled once you make changes): image.png (143×27 px, 2 KB)
  • .wikibase-toolbarlabel appears to have become unused with Refactored toolbars, a mere decade ago. Let’s remove those styles.
  • .wikibase-statementview-references-heading can probably have its color removed, AFAICT the only contents is the “x reference(s)” toggler that has its own color (which was fixed in T389345).

Change #1129316 had a related patch set uploaded (by Lucas Werkmeister (WMDE); author: Lucas Werkmeister (WMDE)):

[mediawiki/extensions/Wikibase@master] Remove long-unused .wikibase-toolbarlabel styles

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

Change #1129318 had a related patch set uploaded (by Lucas Werkmeister (WMDE); author: Lucas Werkmeister (WMDE)):

[mediawiki/extensions/Wikibase@master] Use Codex token for .wikibase-toolbarbutton-disabled

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

  • .wikibase-statementview-references-heading can probably have its color removed, AFAICT the only contents is the “x reference(s)” toggler that has its own color (which was fixed in T389345).

Aha, the color is still in effect when JavaScript is disabled! (Found in this 2016 commit via git blame. Note that the “diff and old revisions” part mentioned there no longer applies, those also get functional togglers nowadays.) So we should keep it and ask UX what the right value is.

Change #1129235 merged by jenkins-bot:

[mediawiki/extensions/Wikibase@master] Update colour for .wikibase-title-id to a codex token

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

Change #1129316 merged by jenkins-bot:

[mediawiki/extensions/Wikibase@master] Remove long-unused .wikibase-toolbarlabel styles

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

Change #1129851 had a related patch set uploaded (by Lucas Werkmeister (WMDE); author: Lucas Werkmeister (WMDE)):

[mediawiki/extensions/Wikibase@master] Use @color-disabled for references heading

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

Change #1129851 merged by jenkins-bot:

[mediawiki/extensions/Wikibase@master] Use @color-disabled for references heading

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

Change #1129318 merged by jenkins-bot:

[mediawiki/extensions/Wikibase@master] Use Codex token for .wikibase-toolbarbutton-disabled

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