Page MenuHomePhabricator

Remove all mediawiki.ui components and libraries in favor of Codex
Open, HighPublic

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

mediawiki.ui ResourceLoader modules

done

radios only

inputs only

buttons only

inputs and checkboxes

buttons and radios and checkboxes

  • Extension:UniversalLanguageSelector (T340255)

buttons and icons

  • Extension:ContentTranslation (T340259)

buttons and inputs

Core

  • mediawiki.ui utilities; provides the CSS layout classes: .mw-ui-flush-left, .mw-ui-flush-right & .mw-ui-center-block
    • Fix remaining usages in T212095
    • Ensure that classes are not used in scripts (via GlobalSearch, example) & remove mixin code from MediaWiki core.

Details

Related Changes in Gerrit:
SubjectRepoBranchLines +/-
mediawiki/extensions/Flowmaster+454 -8
mediawiki/extensions/MobileFrontendmaster+2 -3
mediawiki/extensions/Translatemaster+69 -57
mediawiki/extensions/GuidedTourmaster+29 -22
mediawiki/coremaster+1 -48
mediawiki/coremaster+20 -222
mediawiki/coremaster+1 -32
mediawiki/extensions/ExternalGuidancemaster+13 -6
mediawiki/coremaster+4 -9
mediawiki/extensions/OAuthmaster+0 -2
mediawiki/extensions/MobileFrontendmaster+0 -43
mediawiki/coremaster+4 -1
mediawiki/extensions/MobileFrontendmaster+2 -1
mediawiki/extensions/WikimediaMessagesmaster+0 -2
mediawiki/coremaster+208 -53
mediawiki/coremaster+42 -80
mediawiki/coremaster+13 -8
mediawiki/coremaster+0 -84
mediawiki/coremaster+1 -27
mediawiki/coremaster+0 -6
mediawiki/coremaster+13 -0
mediawiki/extensions/Flowmaster+0 -50
Show related patches Customize query in gerrit

Related Objects

StatusSubtypeAssignedTask
OpenJdlrobson
Resolvedmatmarex
ResolvedBUG REPORTmatmarex
OpenNone
ResolvedBUG REPORTNone
ResolvedBUG REPORTJdlrobson
ResolvedCatrope
ResolvedBUG REPORTngkountas
ResolvedJdlrobson
ResolvedVolker_E
ResolvedJdlrobson
ResolvedVolker_E
ResolvedBUG REPORTJdlrobson
ResolvedBUG REPORTNone
ResolvedBUG REPORTJdlrobson
DuplicateNone
ResolvedJdlrobson
ResolvedKSarabia-WMF
OpenBUG REPORTNone
ResolvedNone
ResolvedBUG REPORTSBisson
ResolvedJdlrobson
ResolvedBUG REPORTJdlrobson
ResolvedJdlrobson
ResolvedBUG REPORTJdlrobson
ResolvedNone
ResolvedJdlrobson
DuplicateNone
OpenNone
OpenNone
ResolvedWangombe
OpenNone
Resolvedmatmarex
ResolvedVolker_E
OpenNone
ResolvedVolker_E
OpenNone
OpenNone
OpenNone
Openabi_

Event Timeline

There are a very large number of changes, so older changes are hidden. Show Older Changes

@Tacsipacsi thanks for flagging the bug - this is why code review is important! I appreciate you looking at it. I re-ran the script before the holidays.
I also got 518 when I tried the revised script which I have compiled into: https://docs.google.com/spreadsheets/d/19xK_rBTx2dhFQ67TOsXAJGG3O6FwPL09N6D9SMbMGI4/edit#gid=0
I got 1218 on main namespace but 1146 of those were subpages. Many of these are translations as I pointed out before. Looking closely https://meta.wikimedia.org/wiki/Wikimedia_Affiliates_Data_Portal and translations doesn't even use the library (it's improved without it). After the holidays and 5 edits by myself, it seems some of those fixes have propagated to translations as I expected - and that number has reduced to 968.

I'll be having some conversations with the design systems team and web team over the next few weeks about how we want to proceed with this deprecation.
It seems like it might also be useful for us to re-ignite the discussion around communication and impact to improve our shared understanding around communication between consumers and providers.

Jdlrobson updated the task description. (Show Details)

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

[mediawiki/extensions/Flow@master] Flow should maintain a local version of MediaWiki UI

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

Jdlrobson claimed this task.

Seems we missed a few @Volker_E

Sorry wrong ticket :)

Change 995375 had a related patch set uploaded (by Bartosz Dziewoński; author: Bartosz Dziewoński):

[mediawiki/core@master] Remove uses of $wgUseMediaWikiUIEverywhere

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

Change 995376 had a related patch set uploaded (by Bartosz Dziewoński; author: Bartosz Dziewoński):

[mediawiki/core@master] Remove $wgUseMediaWikiUIEverywhere

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

Change 995375 merged by jenkins-bot:

[mediawiki/core@master] Remove uses of $wgUseMediaWikiUIEverywhere

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

Change 995376 merged by jenkins-bot:

[mediawiki/core@master] Remove $wgUseMediaWikiUIEverywhere

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

Change 987868 abandoned by Jdlrobson:

[mediawiki/extensions/Flow@master] Flow should maintain a local version of MediaWiki UI

Reason:

Abandoning this as Design Systems Team will now be overseeing this now.

FWIW I think continuing to tie Flow to Codex design tokens is not worth the effort. We should be looking to undeploy this extension and applying pressure by degrading it as much as possible - even if that means it goes out of sync with other parts of the interface. Taking snapshots of the current CSS preserves it as it was. If we continue to support design tokens here we are effectively maintaining it and keeping it up to date with the rest of the codebase IMO and that doesn't seem like a great use of our time.

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

Change 982911 merged by jenkins-bot:

[mediawiki/extensions/GuidedTour@master] Removes deprecated mediawiki.ui.button in favor of Codex

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

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

[mediawiki/core@master] mediawiki.ui: Remove 'utilities.less'

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

Change 1011267 merged by jenkins-bot:

[mediawiki/core@master] mediawiki.ui: Remove 'utilities.less'

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

Test wiki on Patch demo by Jdlrobson using patch(es) linked to this task was deleted:

https://patchdemo.wmflabs.org/wikis/f5539e0094/w/

Test wiki on Patch demo by Roan Kattouw (WMF) using patch(es) linked to this task was deleted:

https://patchdemo.wmflabs.org/wikis/a1e0f00ea6/w/

Change #1082216 had a related patch set uploaded (by Abijeet Patro; author: Ebrahim):

[mediawiki/extensions/Translate@master] Adopt codex provided colors in more place

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

Change #1082216 had a related patch set uploaded (by Ebrahim; author: Ebrahim):

[mediawiki/extensions/Translate@master] Adopt codex provided colors in more place

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

Change #1082216 abandoned by Ebrahim:

[mediawiki/extensions/Translate@master] Adopt codex provided colors in more place

Reason:

There were some CSS files I've overlooked, let me retract this hopefully temporarily

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

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

[mediawiki/extensions/MobileFrontend@master] Remove unused CSS selectors and classes

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

Change #1251556 merged by jenkins-bot:

[mediawiki/extensions/MobileFrontend@master] Remove unused CSS selectors and classes

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

Change #922957 restored by VolkerE:

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

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

Change #987868 restored by VolkerE:

[mediawiki/extensions/Flow@master] Flow should maintain a local version of MediaWiki UI

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

Change #987868 abandoned by Jdlrobson:

[mediawiki/extensions/Flow@master] Flow should maintain a local version of MediaWiki UI

Reason:

Volker I don't have bandwidth for this any more - if you want to continue down this road would you mind re-posting it under your account?

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