Volume control broken in ar.wp
Closed, ResolvedPublicBUG REPORT
Actions

Assigned To

Authored By

	TheDJ
	May 2 2022, 3:32 PM

Description

video-js .vjs-volume-panel .vjs-volume-control.vjs-volume-vertical
left: -3000em

	Subject	Repo	Branch	Lines +/-
	Don't push volume slider off screen in rtl	mediawiki/extensions/TimedMediaHandler	master	+7 -0

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptMay 2 2022, 3:32 PM

Aklapper moved this task from Backlog to Reported Upstream on the Upstream board.

Restricted Application added a project: I18n. · View Herald TranscriptMay 2 2022, 7:43 PM

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

[mediawiki/extensions/TimedMediaHandler@master] Don't push volume slider off screen in rtl

I can. Do you want it now or later once the patch is merged and deployed?

You can do it now, it's already not working (it was for when we were still using an inline volume control, instead of a vertical one.)

Change 788394 merged by jenkins-bot:

[mediawiki/extensions/TimedMediaHandler@master] Don't push volume slider off screen in rtl

Done. Can you double check?

Looks good. should be fixed after next deploy cycle