Page MenuHomePhabricator

Audio player too wide
Closed, ResolvedPublic

Description

Screenshot 2022-03-21 at 12.55.03.png (108×258 px, 3 KB)

This is because the thumb spacing styling has moved onto the 'thumb image' class (the border is now on the content, instead of on the wrapper of the contents. Here too, after the content replacement for the JS player, the <audio> element is wrapped inside the span, but the span has a fixed size, which is expecting the dimensions without the padding and the border.

Filed as part of T304010

Event Timeline

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

[mediawiki/core@master] [WIP] Add styles for a class on the mw-file-element

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

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

[mediawiki/extensions/TimedMediaHandler@master] [WIP] Make span the mw-file-element

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

Arlolra triaged this task as Medium priority.Jul 8 2022, 5:48 PM

There's also an issue with the line height.

This is the current rendering of an audio thumb ([[File:1951-test.wav|thumb|wave!]]),

Screen Shot 2022-07-19 at 6.04.14 PM.png (214×696 px, 11 KB)

This is what it looks like with $wgParserEnableLegacyMediaDOM = false;

Screen Shot 2022-07-19 at 6.04.25 PM.png (198×682 px, 10 KB)

This is after applying the change in https://gerrit.wikimedia.org/r/c/mediawiki/core/+/812383

Screen Shot 2022-07-19 at 6.04.32 PM.png (170×664 px, 10 KB)

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

[mediawiki/core@master] Suppress line-height inside thumbs

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

Change 812384 merged by jenkins-bot:

[mediawiki/core@master] Use a universal selector (*) to match the media element

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

Change 812387 merged by jenkins-bot:

[mediawiki/extensions/TimedMediaHandler@master] Swap the media element with a span

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

Change 820544 had a related patch set uploaded (by Zabe; author: Zabe):

[mediawiki/extensions/TimedMediaHandler@master] Stop requiring resources/config.json

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

Change 820544 merged by jenkins-bot:

[mediawiki/extensions/TimedMediaHandler@master] Stop requiring resources/config.json

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

Change 812383 merged by jenkins-bot:

[mediawiki/core@master] Suppress line-height inside thumbs

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

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

[mediawiki/skins/Timeless@master] Copy upstream change to suppress line-height in thumbs

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

Change 930276 merged by jenkins-bot:

[mediawiki/skins/Timeless@master] Copy upstream change to suppress line-height in thumbs

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

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

[mediawiki/extensions/TimedMediaHandler@master] Fix audio player being too wide when !parserEnableLegacyMediaDOM

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

Change 932352 merged by jenkins-bot:

[mediawiki/extensions/TimedMediaHandler@master] Fix audio player being too wide when !parserEnableLegacyMediaDOM

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