Page MenuHomePhabricator

Migrate all skin styles for .oo-ui-windowManager and .oo-ui-defaultOverlay to #mw-teleport-target
Closed, ResolvedPublic

Description

All skins that currently override the styles for .oo-ui-windowManager or .oo-ui-defaultOverlay should override them for #mw-teleport-target instead, once the OOUI nodes are moved inside the MW node. This will allow them to also style font size of dialogs etc. provided by Codex and any future libraries. See T347199: Apply skin body styles to teleported elements for context.

This work can be done at any point after T348286: Move the default OOUI overlay and window manager inside the MediaWiki teleport target (it doesn't have to be done immediately).

This task covers WMF-deployed skins (Codesearch):

Other skins (Codesearch) that could be changed in the same way (but it's not required) include: BlueSpiceCalumma, BlueSpiceDiscovery, Cosmos, DarkVector, FemiwikiSkin, GreyStuff, Metrolook, Mirage, Nimbus, Splash; and the Theme extension.

Related Objects

Event Timeline

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

[mediawiki/skins/CologneBlue@master] Migrate OOUI overlay styles to #mw-teleport-target

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

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

[mediawiki/skins/Modern@master] Migrate OOUI overlay styles to #mw-teleport-target

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

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

[mediawiki/skins/MonoBook@master] Migrate OOUI overlay styles to #mw-teleport-target

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

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

[mediawiki/skins/Timeless@master] Migrate OOUI overlay styles to #mw-teleport-target

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

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

Change 970447 merged by jenkins-bot:

[mediawiki/skins/Timeless@master] Migrate OOUI overlay styles to #mw-teleport-target

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

Change 970446 merged by jenkins-bot:

[mediawiki/skins/MonoBook@master] Migrate OOUI overlay styles to #mw-teleport-target

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

Change 970445 merged by jenkins-bot:

[mediawiki/skins/Modern@master] Migrate OOUI overlay styles to #mw-teleport-target

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

Change 970444 merged by jenkins-bot:

[mediawiki/skins/CologneBlue@master] Migrate OOUI overlay styles to #mw-teleport-target

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

matmarex removed a project: Patch-For-Review.
matmarex updated the task description. (Show Details)

@Nardog I'll swap this back, to group bugs caused by this work on the "opposite" side of the task tree. It's easier for me to track them that way, and I hope you don't mind.

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

https://patchdemo.wmflabs.org/wikis/084901cf7c/w/