Page MenuHomePhabricator

Progress bar hard to use on mobile because of all the other controls
Closed, ResolvedPublic

Description

As reported on https://en.wikipedia.org/wiki/Wikipedia:Village_pump_(technical)#Progress_bar_on_mobile_media_player_is_practically_unusable

YLukQaw.png (2×1 px, 852 KB)

In portrait orientation the progress bar control is hard to use because it gets crowded out by the other controls. There are two reasons for this. Mostly there are already a lot of controls, we could choose another layout perhaps. But secondly, I think I didn't modify the control layout cut offs when we increased the font size of the controls.

Another layout with the progress bar above the other controls is what I experimented with in https://twitter.com/dj_hartman/status/1517400646604967937
https://gerrit.wikimedia.org/r/c/mediawiki/extensions/TimedMediaHandler/+/785202

Event Timeline

Change 785202 had a related patch set uploaded (by TheDJ; author: TheDJ):

[mediawiki/extensions/TimedMediaHandler@master] Position progress control above the other controls.

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

Some quick screenshots:

Screenshot 2022-04-25 at 17.13.47.png (2×1 px, 1 MB)

image.png (2×4 px, 3 MB)

LGTM. Should we just ship it?

The discord was also enthusiastic. And we can always revert of course.

Change 785202 merged by jenkins-bot:

[mediawiki/extensions/TimedMediaHandler@master] Position progress control above the other controls.

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

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

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