Page MenuHomePhabricator

Deprecate and replace mediawiki.ui components
Open, Needs TriagePublic

Description

Coming from MinervaNeue default payload question:

We're still loading mediawiki.ui component code per default.
We should evaluate and remove styles from rendering-critical CSS

  • Deprecate mediawiki.ui

Buttons:
@Jdlrobson brought up the idea of reducing the CSS to load only normal state of button elements. When looking into mediawiki.ui/components/buttons.less it seems reasonably framed to progress. T193821
mediawiki.ui button-group
core

  • Special:GlobalRenameQueue

extensions:

  • Html2Wiki
  • ULS per JS

Exemplified:
mediawiki.ui checkbox is currently still in use in:

  • core
    • HTMLMultiSelectField.php
    • HTMLCheckMatrix.php
    • Xml.php
  • Extension:InputBox
  • Extension:MobileFrontend
  • Extension:Translate
  • Extension:UniversalLanguageSelector

mediawiki.ui radio:

Those should be replaced by similar OOUI PHP widgets and removed from loaded modules.

mediawiki.ui utilities

  • .mw-ui-flush-left
  • .mw-ui-flush-right
  • .mw-ui-center-block

Related Objects

Event Timeline

mediawiki.ui inputs of type=search should be deprecated already. Everyone who wants to use such apart from OOUI needs to be aware of the vendor specific extensions of this HTML5 input type. AFAIK there's currently no usage of .mw-ui-input together with type=search any more.

Change 429160 had a related patch set uploaded (by Esanders; owner: Esanders):
[mediawiki/extensions/Flow@master] Remove unused CSS for MWUI components

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

Change 429160 merged by jenkins-bot:
[mediawiki/extensions/Flow@master] Remove unused CSS for MWUI components

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

Volker_E updated the task description. (Show Details)

Change 612397 had a related patch set uploaded (by VolkerE; owner: VolkerE):
[mediawiki/core@master] mediawiki.ui: Deprecate all sub-modules

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

Change 612397 abandoned by VolkerE:
[mediawiki/core@master] mediawiki.ui: Deprecate all sub-modules

Reason:

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

Change 919360 had a related patch set uploaded (by Jdlrobson; author: Jdlrobson):

[mediawiki/core@master] Remove mediawiki ui styles on OOUI forms

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

Change 919361 had a related patch set uploaded (by Jdlrobson; author: Jdlrobson):

[mediawiki/core@master] [WIP] Login page: Use Codex CSS components

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

Change 919392 had a related patch set uploaded (by Jdlrobson; author: Jdlrobson):

[mediawiki/core@master] Drop mw-ui-radio markup

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

Change 919393 had a related patch set uploaded (by Jdlrobson; author: Jdlrobson):

[mediawiki/core@master] VFormHTMLForm: Remove dead code

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

Change 919394 had a related patch set uploaded (by Jdlrobson; author: Jdlrobson):

[mediawiki/core@master] Drop UseMediaWikiUIEverywhere

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

Change 919398 had a related patch set uploaded (by Jdlrobson; author: Jdlrobson):

[mediawiki/core@master] Use quiet button rather than mw-ui-button

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

Change 919392 merged by jenkins-bot:

[mediawiki/core@master] Drop mw-ui-radio markup

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

Change 919360 merged by jenkins-bot:

[mediawiki/core@master] Remove mediawiki ui styles on OOUI forms

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

Change 919393 merged by jenkins-bot:

[mediawiki/core@master] VFormHTMLForm: Remove dead code

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

Change 919398 merged by jenkins-bot:

[mediawiki/core@master] EditPage: Use plain link rather than mw-ui-button for help link

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

Change 922957 had a related patch set uploaded (by VolkerE; author: VolkerE):

[mediawiki/skins/Example@master] Remove 'mediawiki.ui' dependencies

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

Change 922957 abandoned by Jdlrobson:

[mediawiki/skins/Example@master] Remove 'mediawiki.ui' dependencies

Reason:

Done in https://gerrit.wikimedia.org/r/c/mediawiki/skins/Example/+/928887

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

Test wiki created on Patch demo by Roan Kattouw (WMF) using patch(es) linked to this task:
https://patchdemo.wmflabs.org/wikis/bc272f38d8/w

Change 919394 abandoned by Jdlrobson:

[mediawiki/core@master] Drop UseMediaWikiUIEverywhere

Reason:

Lots of things to do beforehand so abandoning for now :-)

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

Test wiki created on Patch demo by Roan Kattouw (WMF) using patch(es) linked to this task:
https://patchdemo.wmflabs.org/wikis/427782b8dc/w

Test wiki created on Patch demo by Jdlrobson using patch(es) linked to this task:
https://patchdemo.wmflabs.org/wikis/f5539e0094/w

Test wiki created on Patch demo by Roan Kattouw (WMF) using patch(es) linked to this task:
https://patchdemo.wmflabs.org/wikis/a1e0f00ea6/w

Change 919361 merged by jenkins-bot:

[mediawiki/core@master] Login page: Use Codex CSS components

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

@Jdlrobson seems like a follow-up patch is needed to address this (form descriptions stick to the next elements in the form):

Looking at https://patchdemo.wmflabs.org/wikis/bc272f38d8/w/index.php?title=Special:CreateAccount&returnto=Main+Page

image.png (665×393 px, 48 KB)

@stjn are you only seeing that on the patchdemo? I'm not seeing on https://en.wikipedia.beta.wmflabs.org/w/index.php?title=Special:CreateAccount&campaign=loginCTA (which is the most up to date version of the code)
Looks like https://patchdemo.wmflabs.org/wikis/bc272f38d8/wiki/Main_Page is running patchset 6 of https://gerrit.wikimedia.org/r/c/mediawiki/core/+/919361/ and we merged patchset 24 so the patchdemo is very outdated code.

Ah. Assumed that since patchdemo was created not long ago, that’s what it looks like still. My apologies.

Jdlrobson updated the task description. (Show Details)