Page MenuHomePhabricator

JSON uses non-standard Codex design tokens
Closed, ResolvedPublicBUG REPORT

Description

Steps to replicate the issue (include links if applicable):

Visit https://it.wikivoyage.org/wiki/MediaWiki:Gadget-ListingEditor.json

Screenshot 2025-12-09 at 4.56.45 PM.png (801×1 px, 189 KB)

Switch to dark mode

Screenshot 2025-12-09 at 4.57.01 PM.png (712×1 px, 88 KB)

What happens?:

  • The use of green seems inappropriate here but might be a good example of expanding Codex's design tokens for presentation needs.
  • The color in dark mode does not correspond with the one in light mode
  • This requires additional and unnecessary CSS for dark mode support that we'd like to clean up.

What should have happened instead?:
There seem to be several options here:

  1. Per @esander's suggestion switch to closest color (@background-color-success-subtle;) possibly with a rename to describe it's use semantically https://gerrit.wikimedia.org/r/c/mediawiki/core/+/1216771
  2. Replace color with a different color e.g.@background-color-backdrop-light

Screenshot 2025-12-09 at 5.01.49 PM.png (881×936 px, 102 KB)

  1. Create a new token
  2. ...

Software version (on Special:Version page; skip for WMF-hosted wikis like Wikipedia):

Other information (browser name/version, screenshots, etc.):

Details

Other Assignee
Volker_E
Related Changes in Gerrit:

Event Timeline

Change #1216771 had a related patch set uploaded (by Jdlrobson; author: Esanders):

[mediawiki/core@master] JsonContent: Use background-color-success-subtle for JSON tables

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

What is the reason for the coloring of the cells? Before we go into token application action it would be helpful to understand the why better:

  • For signaling a certain value quality? No.
  • For signaling a value over a value container. Yes.

For latter and in order to not confuse with validation, I'd recommend

image.png (1×1 px, 188 KB)

treatment.

In case we're going to have table highlights design tokens, I'd opt for a light violet instead of a green hue.

That seems like a reasonable follow-up. I just want to merge the least controversial change that would remove the dark-mode hacks.

Volker_E assigned this task to Esanders.
Volker_E triaged this task as Medium priority.
Volker_E removed a project: Patch-For-Review.
Volker_E updated Other Assignee, added: Volker_E.

Change #1216771 merged by jenkins-bot:

[mediawiki/core@master] mediawiki.content.json: Use neutral tokens for JSON tables

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