Page MenuHomePhabricator
Create Task
Maniphest T165650

Directly use the WikimediaUI values in the WikimediaUI theme in OOUI, rather than via copy-paste
Closed, ResolvedPublic
Actions

Description

In order to implement WikimediaUI Base in WikimediaUI theme, we need to minimize differences between rWMUI WikimediaUI Base /wikimedia-ui-base.less and the existing theme variables in common.less. (continuing work from @Prtksxna comment)

SAME variable name

  • @color-placeholder
  • @color-destructive
  • @background-color-framed
  • @background-color-destructive
  • @background-color-tabs
  • @box-shadow-dialog
  • ...some more shadow stuff...

VALUE is slightly different

MISTAKE in rWMUI?

  • Should @background-color-primary be @wmui-color-accent90?

Yes. Fixed in D660.

  • Should @color-primary--focus be @color-primary instead of @wmui-color-accent50?

tl;dr Yes. I'm not having a strong conviction on what is the more insightful way for devs, but consistency is important so let's follow the OOUI way.

MISSING variables

  • @background-color-base--disabled added
  • @background-color-filled--disabled added
  • min-size-* and size-base
  • Toolbar and Tools specific variables
  • @border-style-base
  • Transitions

OOUI specific variables, that we shouldn't put into WikimediaUI Base right now

  • A bunch of font stuff
  • @oo-ui-font-size-browser – debatable if we should generalize this
  • @oo-ui-font-size-default – debatable if we should generalize this
  • @color-destructive-non-primary – OOUI specific
  • @opacity-disabled-indicator– OOUI specific
  • @opacity-disabled-tool – debatable if we should generalize this
  • @border-radius-frameless – debatable if we should generalize this
  • …

Details

Show related patches Customize query in gerrit

Related Objects
Search...

Event Timeline

• Prtksxna created this task.May 18 2017, 7:13 AM
• Prtksxna created subtask T165652: Package and publish WikimediaUI to npm.May 18 2017, 7:17 AM
• Prtksxna added a comment.May 18 2017, 10:08 AM

I am attempting to document things that are needed by OOUI /src/themes/mediawiki/common.less and aren't present in rWMUI WikimediaUI Base /wikimedia-ui-base.less. Some of them might not even need to be in in rWMUI WikimediaUI Base and will stay in OOUI, but I'll try to document all.

SAME variable name

  • @color-placeholder
  • @color-destructive
  • @background-color-framed
  • @background-color-destructive
  • @background-color-tabs
  • @box-shadow-dialog
  • @box-shadow-toolbar*
  • @box-shadow-widget
  • @box-shadow-frameless-focus
  • ...some more shadow stuff...

VALUE is slightly different

  • @background-color-destructive

MISTAKE in rWMUI?

  • Should @background-color-primary be @wmui-color-accent90?
  • Should @color-primary--focus be @color-primary instead of @wmui-color-accent50?
  • Should @border-color-filled--disabled be @wmui-color-base70 instead of base30?

MISSING variables

  • Most size stuff
  • A bunch of font stuff
  • @oo-ui-font-size-browser
  • @oo-ui-font-size-default
  • @color-destructive-non-primary
  • @opacity-disabled-indicator
  • @opacity-disabled-tool
  • @border-radius-frameless
  • @border-width-medium

I haven't documented these, but there are some cases in which common.less is assigning a variable to a variable instead of a value, and wikimedia-ui-base.less is doing the same. In such cases do we want to use the WMUI variable or let it be. For example in @color-erroneous. Not sure what should REF what.

These changes can be seen in Gerrit - https://gerrit.wikimedia.org/r/354205. I wont be updating this comment as that patch or rWMUI get updated.

• Prtksxna assigned this task to Volker_E.May 18 2017, 6:04 PM
Jdforrester-WMF renamed this task from Use the WikimediaUI values in the Mediawiki theme in OOjs UI to Directly use the WikimediaUI values in the WikimediaUI theme in OOjs UI, rather than via copy-paste.May 20 2017, 8:22 AM
Volker_E updated the task description. (Show Details)May 21 2017, 2:37 PM
In T165650#3272934, @Prtksxna wrote:
  • Should @border-color-filled--disabled be @wmui-color-base70 instead of base30?

Not sure if I understand. It's referring to @wmui-color-base100 right now and that's intended

Jdlrobson awarded a token.May 22 2017, 6:50 AM
gerritbot added a comment.May 24 2017, 2:59 PM

Change 355433 had a related patch set uploaded (by VolkerE; owner: VolkerE):
[oojs/ui@master] Align @background-color-destructive to WikimediaUI Base

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

gerritbot added a project: Patch-For-Review.May 24 2017, 2:59 PM
Volker_E updated the task description. (Show Details)May 24 2017, 2:59 PM
Volker_E updated the task description. (Show Details)May 24 2017, 6:28 PM
Volker_E moved this task from Backlog to Next-up on the OOUI board.May 24 2017, 9:13 PM
SamanthaNguyen added a project: Technical-Debt.May 25 2017, 1:36 AM
gerritbot added a comment.May 25 2017, 12:35 PM

Change 355433 merged by jenkins-bot:
[oojs/ui@master] Align @background-color-destructive to WikimediaUI Base

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

Volker_E closed subtask T165652: Package and publish WikimediaUI to npm as Resolved.May 26 2017, 4:57 PM
Volker_E moved this task from Backlog to In Dev/Progress on the Wikimedia Design Style Guide board.Jun 6 2017, 6:29 PM
gerritbot added a comment.Jun 9 2017, 2:45 AM

Change 357965 had a related patch set uploaded (by Jforrester; owner: Jforrester):
[oojs/ui@master] WikimedaUI: Pull in and use upstream WikimediaUI package

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

Jdforrester-WMF moved this task from Next-up to OOjs-UI-0.22.2 on the OOUI board.Jun 12 2017, 11:04 PM
Jdforrester-WMF edited projects, added OOUI (OOjs-UI-0.22.2); removed OOUI.
Jdforrester-WMF moved this task from OOjs-UI-0.22.2 to OOjs-UI-0.22.3 on the OOUI board.Jun 28 2017, 7:38 PM
Jdforrester-WMF edited projects, added OOUI (OOjs-UI-0.22.3); removed OOUI (OOjs-UI-0.22.2).
gerritbot added a comment.Jul 2 2017, 1:57 PM

Change 360328 had a related patch set uploaded (by VolkerE; owner: VolkerE):
[oojs/ui@master] themes: Align variable names to WikimediaUI Base scheme

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

gerritbot added a comment.Jul 3 2017, 9:08 AM

Change 362800 had a related patch set uploaded (by VolkerE; owner: VolkerE):
[oojs/ui@master] WikimediaUI theme: Align variable pseudo classes names to WikimediaUI Base

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

gerritbot added a comment.Jul 3 2017, 9:27 AM

Change 362801 had a related patch set uploaded (by VolkerE; owner: VolkerE):
[oojs/ui@master] themes: Align read-only variable names to pseudo-class selector scheme

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

gerritbot added a comment.Jul 3 2017, 9:36 AM

Change 362802 had a related patch set uploaded (by VolkerE; owner: VolkerE):
[oojs/ui@master] WikimediaUI theme: Align checked variable names to pseudo-class scheme

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

gerritbot added a comment.Jul 3 2017, 9:44 AM

Change 362806 had a related patch set uploaded (by VolkerE; owner: VolkerE):
[oojs/ui@master] WikimediaUI theme: Align disabled variable names to pseudo-class scheme

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

gerritbot added a comment.Jul 3 2017, 9:53 AM

Change 362956 had a related patch set uploaded (by VolkerE; owner: VolkerE):
[oojs/ui@master] WikimediaUI theme: Align opacity-icon* variable names to WikimediaUI Base

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

Volker_E updated the task description. (Show Details)Jul 3 2017, 9:55 AM
gerritbot added a comment.Jul 5 2017, 8:22 PM

Change 363416 had a related patch set uploaded (by VolkerE; owner: VolkerE):
[oojs/ui@master] WikimediaUI theme: Replace @color-light with @color-base--inverted

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

gerritbot added a comment.Jul 11 2017, 3:14 PM

Change 360328 merged by jenkins-bot:
[oojs/ui@master] themes: Align variable names to WikimediaUI Base scheme

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

gerritbot added a comment.Jul 11 2017, 3:15 PM

Change 362800 merged by jenkins-bot:
[oojs/ui@master] WikimediaUI theme: Align variable pseudo classes names to WikimediaUI Base

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

gerritbot added a comment.Jul 11 2017, 3:16 PM

Change 362801 merged by jenkins-bot:
[oojs/ui@master] themes: Align read-only variable names to pseudo-class selector scheme

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

gerritbot added a comment.Jul 11 2017, 3:16 PM

Change 362802 merged by jenkins-bot:
[oojs/ui@master] WikimediaUI theme: Align checked variable names to pseudo-class scheme

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

gerritbot added a comment.Jul 11 2017, 3:17 PM

Change 362806 merged by jenkins-bot:
[oojs/ui@master] WikimediaUI theme: Align disabled variable names to pseudo-class scheme

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

gerritbot added a comment.Jul 11 2017, 3:18 PM

Change 362956 merged by jenkins-bot:
[oojs/ui@master] WikimediaUI theme: Align opacity-icon* variable names to WikimediaUI Base

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

gerritbot added a comment.Jul 11 2017, 3:19 PM

Change 363416 merged by jenkins-bot:
[oojs/ui@master] WikimediaUI theme: Replace @color-base-light with @color-base--inverted

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

gerritbot added a comment.Jul 11 2017, 8:17 PM

Change 357965 merged by jenkins-bot:
[oojs/ui@master] WikimediaUI: Pull in the upstream WikimediaUI package

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

gerritbot added a comment.Jul 12 2017, 1:53 AM

Change 364634 had a related patch set uploaded (by Jforrester; owner: Jforrester):
[oojs/ui@master] [WIP] WikimediaUI theme: Directly use the LESS values rather than via copy-paste

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

gerritbot added a comment.Aug 1 2017, 7:35 PM

Change 364634 merged by jenkins-bot:
[oojs/ui@master] WikimediaUI theme: Directly use the Less values rather than via copy-paste

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

Jdforrester-WMF added a comment.Aug 1 2017, 8:53 PM

Is this now Resolved enough?

Volker_E added a comment.Aug 1 2017, 9:03 PM

I'd like to add a few more aligned before closing this.

gerritbot added a comment.Aug 2 2017, 6:02 PM

Change 369703 had a related patch set uploaded (by VolkerE; owner: VolkerE):
[oojs/ui@master] Align vars to WikimediaUI Base and remove them as OOjs UI vars

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

gerritbot added a comment.Aug 2 2017, 7:05 PM

Change 369703 merged by jenkins-bot:
[oojs/ui@master] Align vars to WikimediaUI Base and remove them as OOjs UI vars

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

Milimetric unsubscribed.Aug 5 2017, 3:34 PM
Krinkle unsubscribed.Aug 16 2017, 12:32 AM
Jdforrester-WMF added a comment.Aug 21 2017, 11:54 PM
In T165650#3490935, @Volker_E wrote:

I'd like to add a few more aligned before closing this.

How many more? :-)

gerritbot added a comment.Sep 20 2017, 4:23 AM

Change 379165 had a related patch set uploaded (by VolkerE; owner: VolkerE):
[oojs/ui@master] Remove vars covered by WikimediaUI Base

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

gerritbot added a comment.Sep 22 2017, 10:39 AM

Change 379165 merged by jenkins-bot:
[oojs/ui@master] Remove vars covered by WikimediaUI Base

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

Volker_E renamed this task from Directly use the WikimediaUI values in the WikimediaUI theme in OOjs UI, rather than via copy-paste to Directly use the WikimediaUI values in the WikimediaUI theme in OOUI, rather than via copy-paste.Dec 21 2017, 4:47 PM
Volker_E moved this task from Unsorted to OOUI on the UI-Standardization board.Mar 19 2018, 5:10 AM
Volker_E moved this task from In Dev/Progress to Stalled / Waiting on the Wikimedia Design Style Guide board.Mar 27 2018, 4:20 PM
Volker_E added a comment.Aug 15 2018, 3:51 PM

That was the task I was searching for earlier.
Further variables reused in https://gerrit.wikimedia.org/r/#/c/oojs/ui/+/452842/

Volker_E updated the task description. (Show Details)Aug 15 2018, 3:52 PM
Jdforrester-WMF added a comment.Aug 16 2018, 4:35 PM
In T165650#4505121, @Volker_E wrote:

That was the task I was searching for earlier.
Further variables reused in https://gerrit.wikimedia.org/r/#/c/oojs/ui/+/452842/

Does that mean we can close it now? It's been a year since you said you wanted to do a little more. :-)

Volker_E updated the task description. (Show Details)Aug 16 2018, 9:00 PM
Volker_E updated the task description. (Show Details)Aug 17 2018, 12:48 AM
Volker_E updated the task description. (Show Details)Aug 17 2018, 4:40 AM
Volker_E updated the task description. (Show Details)
gerritbot added a comment.Aug 17 2018, 5:32 AM

Change 453350 had a related patch set uploaded (by VolkerE; owner: VolkerE):
[oojs/ui@master] WikimediaUI theme: Use WikimediaUI Base size* variables

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

gerritbot added a comment.Oct 2 2018, 9:30 PM

Change 464060 had a related patch set uploaded (by VolkerE; owner: VolkerE):
[oojs/ui@master] WikimediaUI theme: Remove vars covered by WikimediaUI Base vars

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

gerritbot added a comment.Oct 2 2018, 10:16 PM

Change 464060 merged by jenkins-bot:
[oojs/ui@master] WikimediaUI theme: Remove vars covered by WikimediaUI Base vars

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

gerritbot added a comment.Oct 3 2018, 5:31 PM

Change 464370 had a related patch set uploaded (by VolkerE; owner: VolkerE):
[mediawiki/core@master] Update OOUI to v0.29.0

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

gerritbot added a comment.Oct 4 2018, 5:14 PM

Change 464607 had a related patch set uploaded (by Jforrester; owner: Jforrester):
[mediawiki/core@master] Update OOUI to v0.29.1

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

gerritbot added a comment.Oct 4 2018, 5:14 PM

Change 464370 abandoned by Jforrester:
Update OOUI to v0.29.0

Reason:
Replaced by Ie847465af6762.

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

gerritbot added a comment.Oct 4 2018, 6:31 PM

Change 464607 merged by jenkins-bot:
[mediawiki/core@master] Update OOUI to v0.29.1

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

Jdforrester-WMF edited projects, added MW-1.32-notes (WMF-deploy-2018-10-16 (1.32.0-wmf.26)); removed Patch-For-Review.Oct 4 2018, 7:26 PM
• Demian subscribed.Feb 16 2020, 3:08 AM
Aklapper removed Volker_E as the assignee of this task.Jun 19 2020, 4:20 PM

This task has been assigned to the same task owner for more than two years. Resetting task assignee due to inactivity, to decrease task cookie-licking and to get a slightly more realistic overview of plans. Please feel free to assign this task to yourself again if you still realistically work or plan to work on this task - it would be welcome!

For tips how to manage individual work in Phabricator (noisy notifications, lists of task, etc.), see https://phabricator.wikimedia.org/T228575#6237124 for available options.
(For the records, two emails were sent to assignee addresses before resetting assignees. See T228575 for more info and for potential feedback. Thanks!)

gerritbot added a comment.Jul 8 2020, 12:28 AM

Change 453350 had a related patch set uploaded (by VolkerE; owner: VolkerE):
[oojs/ui@master] WikimediaUI theme: Use WikimediaUI Base size* variables

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

gerritbot added a project: Patch-For-Review.Jul 8 2020, 12:28 AM
gerritbot added a comment.Jul 9 2020, 8:54 PM

Change 610924 had a related patch set uploaded (by VolkerE; owner: VolkerE):
[wikimedia-ui-base@master] Add min-size-base variable

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

gerritbot added a comment.Jul 9 2020, 9:24 PM

Change 610924 merged by jenkins-bot:
[wikimedia-ui-base@master] Add min-size-base variable

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

gerritbot added a comment.Jul 10 2020, 5:42 AM

Change 453350 merged by jenkins-bot:
[oojs/ui@master] WikimediaUI theme: Use WikimediaUI Base size* variables

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

Volker_E closed this task as Resolved.Jul 10 2020, 6:24 AM
Volker_E claimed this task.
Volker_E removed a project: Patch-For-Review.
Volker_E updated the task description. (Show Details)
gerritbot added a comment.Jul 10 2020, 7:09 AM

Change 611174 had a related patch set uploaded (by VolkerE; owner: VolkerE):
[mediawiki/core@master] Update OOUI to v0.39.3

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

gerritbot added a project: Patch-For-Review.Jul 10 2020, 7:09 AM
gerritbot added a comment.Jul 10 2020, 10:32 AM

Change 611174 merged by jenkins-bot:
[mediawiki/core@master] Update OOUI to v0.39.3

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

Volker_E moved this task from Stalled / Waiting to Done on the Wikimedia Design Style Guide board.Aug 9 2021, 4:41 PM
Volker_E removed a project: Patch-For-Review.
Content licensed under Creative Commons Attribution-ShareAlike (CC BY-SA) 4.0 unless otherwise noted; code licensed under GNU General Public License (GPL) 2.0 or later and other open source licenses. By using this site, you agree to the Terms of Use, Privacy Policy, and Code of Conduct. · Wikimedia Foundation · Privacy Policy · Code of Conduct · Terms of Use · Disclaimer · CC-BY-SA · GPL