Page MenuHomePhabricator

Media alignment broken with MinervaNeue since disabling wgParserEnableLegacyMediaDOM
Closed, ResolvedPublic

Description

In T314318, cswiki and enwikivoyage opted in to the new media structure. However, that seems to have exposed that not all deployed skins were updated beforehand.

For example, see https://en.m.wikivoyage.org/wiki/Dresden

This should have been fixed in T287965 but seems to have been overlooked.

Event Timeline

Restricted Application added a subscriber: Aklapper. · View Herald Transcript
Arlolra renamed this task from Media alignment broken since disabling wgParserEnableLegacyMediaDOM to Media alignment broken with MinervaNeue since disabling wgParserEnableLegacyMediaDOM.Sep 22 2022, 12:11 AM
Jdlrobson subscribed.

The ResourceLoaderSkinModules are optional. They should be seen as boilerplate styles. In the case of Minerva, given its desire to keep the minimum possible CSS payload it currently ships its own styles.

I assume you'll need to add a minimum set of relevant rules to https://github.com/wikimedia/mediawiki-skins-MinervaNeue/blob/master/resources/skins.minerva.base.styles/content/images.less and https://github.com/wikimedia/mediawiki-skins-MinervaNeue/blob/master/resources/skins.minerva.base.styles/content/thumbnails.less

Change 834342 had a related patch set uploaded (by Arlolra; author: Arlolra):

[mediawiki/skins/MinervaNeue@master] [WIP] Fix media alignment since disabling wgParserEnableLegacyMediaDOM

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

Change 834350 had a related patch set uploaded (by Arlolra; author: Arlolra):

[mediawiki/skins/MinervaNeue@master] Restrict figure to the size of the media

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

Change 834350 merged by jenkins-bot:

[mediawiki/skins/MinervaNeue@master] Restrict figure to the size of the media

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

Change 834364 had a related patch set uploaded (by Arlolra; author: Arlolra):

[mediawiki/skins/MinervaNeue@wmf/1.40.0-wmf.2] Restrict figure to the size of the media

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

Change 834342 merged by jenkins-bot:

[mediawiki/skins/MinervaNeue@master] Fix media alignment since disabling wgParserEnableLegacyMediaDOM

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

Change 834366 had a related patch set uploaded (by Arlolra; author: Arlolra):

[mediawiki/skins/MinervaNeue@wmf/1.40.0-wmf.2] Fix media alignment since disabling wgParserEnableLegacyMediaDOM

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

Change 834364 merged by jenkins-bot:

[mediawiki/skins/MinervaNeue@wmf/1.40.0-wmf.2] Restrict figure to the size of the media

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

Change 834366 merged by jenkins-bot:

[mediawiki/skins/MinervaNeue@wmf/1.40.0-wmf.2] Fix media alignment since disabling wgParserEnableLegacyMediaDOM

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

Mentioned in SAL (#wikimedia-operations) [2022-09-22T20:47:42Z] <brennen@deploy1002> Started scap: Backport for [[gerrit:834364|Restrict figure to the size of the media (T305357 T318300)]], [[gerrit:834366|Fix media alignment since disabling wgParserEnableLegacyMediaDOM (T318300)]]

Mentioned in SAL (#wikimedia-operations) [2022-09-22T20:48:02Z] <brennen@deploy1002> brennen and arlolra: Backport for [[gerrit:834364|Restrict figure to the size of the media (T305357 T318300)]], [[gerrit:834366|Fix media alignment since disabling wgParserEnableLegacyMediaDOM (T318300)]] synced to the testservers: mwdebug1001.eqiad.wmnet, mwdebug2002.codfw.wmnet, mwdebug1002.eqiad.wmnet, mwdebug2001.codfw.wmnet

Mentioned in SAL (#wikimedia-operations) [2022-09-22T20:54:16Z] <brennen@deploy1002> Finished scap: Backport for [[gerrit:834364|Restrict figure to the size of the media (T305357 T318300)]], [[gerrit:834366|Fix media alignment since disabling wgParserEnableLegacyMediaDOM (T318300)]] (duration: 06m 33s)

Change 834407 had a related patch set uploaded (by Arlolra; author: Arlolra):

[mediawiki/core@master] [WIP] Separate new gallery styles from content-media feature

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

Change 834602 had a related patch set uploaded (by Arlolra; author: Arlolra):

[mediawiki/skins/MinervaNeue@master] Remove redundant styling rules and fix center/none halign for figures

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

Change 834606 had a related patch set uploaded (by Arlolra; author: Arlolra):

[mediawiki/skins/MinervaNeue@master] Hide figcaptions for plain and frameless figures

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

Change 834602 merged by jenkins-bot:

[mediawiki/skins/MinervaNeue@master] Remove redundant styling rules and fix center/none halign for figures

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

Change 834606 merged by jenkins-bot:

[mediawiki/skins/MinervaNeue@master] Hide figcaptions for plain and frameless figures

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

We are going to let the last set of changes to MinervaNeue ride next week's train. If someone sees an urgency and needs this to be backported to this week's train, please chime in and we'll make it happen.

Change 834407 merged by jenkins-bot:

[mediawiki/core@master] Separate gallery styles from the content-media feature

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

Change 844078 had a related patch set uploaded (by Arlolra; author: Arlolra):

[mediawiki/skins/MinervaNeue@master] [WIP] max-width only applies to thumbs

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

Change 844078 merged by jenkins-bot:

[mediawiki/skins/MinervaNeue@master] max-width only applies to thumbs

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

Change 852243 had a related patch set uploaded (by Arlolra; author: Arlolra):

[mediawiki/skins/MinervaNeue@master] Media border option applies to the media element, not the wrapper

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

Change 852243 merged by jenkins-bot:

[mediawiki/skins/MinervaNeue@master] Media border option applies to the media element, not the wrapper

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

Change 852314 had a related patch set uploaded (by Arlolra; author: Arlolra):

[mediawiki/skins/MinervaNeue@wmf/1.40.0-wmf.8] Media border option applies to the media element, not the wrapper

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

Change 852314 merged by jenkins-bot:

[mediawiki/skins/MinervaNeue@wmf/1.40.0-wmf.8] Media border option applies to the media element, not the wrapper

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

Mentioned in SAL (#wikimedia-operations) [2022-11-03T13:50:02Z] <lucaswerkmeister-wmde@deploy1002> Started scap: Backport for [[gerrit:852314|Media border option applies to the media element, not the wrapper (T318300)]]

Mentioned in SAL (#wikimedia-operations) [2022-11-03T13:50:26Z] <lucaswerkmeister-wmde@deploy1002> lucaswerkmeister-wmde and arlolra: Backport for [[gerrit:852314|Media border option applies to the media element, not the wrapper (T318300)]] synced to the testservers: mwdebug1002.eqiad.wmnet, mwdebug2002.codfw.wmnet, mwdebug1001.eqiad.wmnet, mwdebug2001.codfw.wmnet

Mentioned in SAL (#wikimedia-operations) [2022-11-03T13:56:34Z] <lucaswerkmeister-wmde@deploy1002> Finished scap: Backport for [[gerrit:852314|Media border option applies to the media element, not the wrapper (T318300)]] (duration: 06m 31s)