Fix slider position of the tracking protection toggle button in RTL locales

ASSIGNED
Assigned to

Status

()

Firefox
Private Browsing
P3
normal
ASSIGNED
26 days ago
7 days ago

People

(Reporter: magicp, Assigned: groovecoder, Mentored)

Tracking

({good-first-bug, regression})

Trunk
good-first-bug, regression
Points:
---
Bug Flags:
qe-verify +

Firefox Tracking Flags

(firefox56 wontfix, firefox57 wontfix, firefox58 affected, firefox59 affected)

Details

MozReview Requests

()

Submitter Diff Changes Open Issues Last Updated
Loading...
Error loading review requests:

Attachments

(2 attachments)

(Reporter)

Description

26 days ago
Created attachment 8923316 [details]
tracking-protection-toggle-button-in-rtl.png

Steps to reproduce:
1. Start Firefox in RTL locales.
2. Open a new private window.
3. Check the tracking protection toggle button

Actual Results:
checked slider position is wrong.

Expected Results:
[aboutprivatebrowsing.css]
.toggle:checked + .toggle-btn:dir(rtl)::after {
  left: auto;
  right: 16px; <= should be 21px
}
(Reporter)

Updated

26 days ago
Blocks: 1378799
status-firefox56: --- → affected
status-firefox57: --- → affected
The fix for this problem is described in comment 0. If you're not on an rtl language, you can just paste

document.body.dir = "rtl"

into the console on about:privatebrowsing to flip it.

The line to change is here: https://searchfox.org/mozilla-central/rev/af86a58b157fbed26b0e86fcd81f1b421e80e60a/browser/themes/shared/privatebrowsing/aboutPrivateBrowsing.css#161
Mentor: jhofmann@mozilla.com
status-firefox56: affected → wontfix
status-firefox57: affected → wontfix
Flags: qe-verify+
Keywords: good-first-bug, regression
Priority: -- → P3

Comment 2

20 days ago
If offset-inline-start is enabled by default, we could also use that.
status-firefox59: --- → affected
Comment hidden (mozreview-request)
Assignee: nobody → lcrouch
Status: NEW → ASSIGNED

Comment 4

7 days ago
mozreview-review
Comment on attachment 8929526 [details]
fix tracking protection slider position in rtl (bug 1412789)

https://reviewboard.mozilla.org/r/200800/#review206136

Looks perfect, thank you!

::: commit-message-45715:1
(Diff revision 1)
> +fix tracking protection slider position in rtl (bug 1412789) r?johannh

Nit: The preferred commit message style is

Bug 1412789 - Fix tracking protection slider position in rtl. r?johannh
Attachment #8929526 - Flags: review?(jhofmann) → review+
You need to log in before you can comment on or make changes to this bug.