Closed Bug 1670094 Opened 5 months ago Closed 1 month ago

[RTL] Picture-in-Picture explainer text is in reverse

Categories

(Toolkit :: Video/Audio Controls, defect, P3)

defect

Tracking

()

VERIFIED FIXED
86 Branch
Tracking Status
firefox86 --- verified

People

(Reporter: itiel_yn8, Assigned: katkoor2)

References

(Blocks 2 open bugs)

Details

(Keywords: rtl)

Attachments

(4 files)

Attached image Current state

With media.videocontrols.picture-in-picture.video-toggle.mode = 2 and media.videocontrols.picture-in-picture.video-toggle.has-used = false in RTL localized builds, the PIP explainer text appears in reverse.

Attached image How it should look like

This should be easier to fix after bug 1666637.

See Also: → 1666637

Note to self:
After in bug 1666637 a .rtl class (or something similar) will be added, a .pip-explainer.rtl rule can be added somewhere around
https://searchfox.org/mozilla-central/source/toolkit/themes/shared/media/pictureinpicture-mode-2.css with a simple direction: rtl.

Assignee: nobody → jack1391
Attached image image.png

Direction also needs fixing, as it is LTR (Nightly should appear on the left side of the paragraph)

Assignee: jack1391 → nobody
Depends on: 1666637
Severity: -- → S2
Priority: -- → P3
Assignee: nobody → katkoor2
Status: NEW → ASSIGNED
Attachment #9198845 - Attachment description: Bug 1670094 - Add RTL support for PIP explainer. r=mconley → Bug 1670094 - Add RTL support for PIP explainer. r=mtigley,mhowell
Pushed by mhowell@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/515a66b2e43c
Add RTL support for PIP explainer. r=mtigley,mhowell
Status: ASSIGNED → RESOLVED
Closed: 1 month ago
Resolution: --- → FIXED
Target Milestone: --- → 86 Branch

Reproduced on Nightly 83.0a1 (20201008210150) on Windows 10 from 2020-10-08.
Confirmed as fixed on 86.0b3 on Windows 10, Ubuntu 20.04 and MacOS 10.15.

Status: RESOLVED → VERIFIED
Flags: qe-verify+
You need to log in before you can comment on or make changes to this bug.