Page MenuHomePhabricator

[WtC-M3] [QB] Use Codex tokens to style QB’s interface colors
Closed, ResolvedPublic5 Estimated Story Points

Description

Problem

The Query Builder's user interface was styled using WiKit tokens. With the WiKit design system being superseded by Codex, the official Wikimedia design system. The utilization of WiKit tokens is not recommended, and they should be replaced in order to reduce maintenance costs and ensure consistency.

Solution

We have to replace the WiKit background and border color tokens used to style the Query Builder's interface by their Codex equivalents.

To support the replacement, here's a WiKit to Codex tokens mapping. You can find the WiKit tokens matched with their corresponding Codex token.

Considerations

Files where color tokens and/or variables are used (please beware the list might be incomplete!):

Acceptance criteria
  • WiKit tokens used to define the colors of the Query Builder app are replaced by Codex tokens

Event Timeline

Arian_Bozorg renamed this task from Use Codex tokens to style QB’s interface colors to [SW] [WtC-M3] [QB] Use Codex tokens to style QB’s interface colors.Mar 27 2024, 9:37 AM
Arian_Bozorg renamed this task from [SW] [WtC-M3] [QB] Use Codex tokens to style QB’s interface colors to [WtC-M3] [QB] Use Codex tokens to style QB’s interface colors.Mar 27 2024, 9:50 AM
karapayneWMDE set the point value for this task to 5.Apr 16 2024, 8:21 AM

Change #1038297 had a related patch set uploaded (by Guergana Tzatchkova; author: Guergana Tzatchkova):

[wikidata/query-builder@master] Use Codex tokens to style QB’s interface colors

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

Change #1038297 merged by jenkins-bot:

[wikidata/query-builder@master] Use Codex tokens to style QB’s interface colors

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

Hey there! Just a nitpick that I didn't catch while looking at the patch, sorry about that! We should use $background-color-neutral-subtle instead of $background-color-interactive-subtle (it's a more semantically correct token) in the QueryBuilder.vue file. That would be all.

Thanks for the replacements!

Hey @HasanAkgun_WMDE 👋🏻 Just a heads-up that there's a tiny fix pending to be applied in relation to this task. Thank you!

Change #1056217 had a related patch set uploaded (by Hasan Akgün (WMDE); author: Hasan Akgün (WMDE)):

[wikidata/query-builder@master] Fix background color code

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

Hey @HasanAkgun_WMDE 👋🏻 Just a heads-up that there's a tiny fix pending to be applied in relation to this task. Thank you!

Hey @Sarai-WMDE , fixed that one! Thanks for heads-up!

Thank you, Hasan! All looking good 🎉

Change #1056217 merged by jenkins-bot:

[wikidata/query-builder@master] Fix background color code

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

Design verification done! ✅ No other fixes are needed. Thank you 🙏🏻