Closed
Bug 1317804
Opened 8 years ago
Closed 8 years ago
Private browsing about page has non-functioning checkbox appended to 'Tracking Protection' label
Categories
(Firefox :: Private Browsing, defect)
Firefox
Private Browsing
Tracking
()
VERIFIED
FIXED
Firefox 53
People
(Reporter: jimm, Assigned: mconley)
References
Details
(Keywords: regression)
Attachments
(3 files)
(deleted),
image/jpeg
|
Details | |
(deleted),
image/png
|
Details | |
(deleted),
text/x-review-board-request
|
johannh
:
review+
jcristau
:
approval-mozilla-aurora+
|
Details |
This appears to be an oversized web form checkbox. Clicking on it doesn't change the state though.
Comment 1•8 years ago
|
||
Huh, first of all this checkbox should not be shown. I can reproduce on Nightly, I think that regressed with bug 1309316. Should be easy to fix.
However, I can't reproduce
> Clicking on it doesn't change the state though.
and I'm also confused that you don't seem to have the green/gray "toggle" next to the checkbox. Can you check for errors in the page console and the browser console?
Blocks: 1309316
status-firefox50:
--- → unaffected
status-firefox51:
--- → unaffected
status-firefox52:
--- → affected
Keywords: regression
Comment 2•8 years ago
|
||
Duplicate of bug #1317795 ?
BTW I'm able to change status of Tracking Protection using both checkboxes in Private Mode (Windows 7 [64bit] + Nightly [64it]).
Comment 3•8 years ago
|
||
> Duplicate of bug #1317795 ?
Looks like it, thanks!
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → DUPLICATE
Updated•8 years ago
|
No longer blocks: 1309316
status-firefox50:
unaffected → ---
status-firefox51:
unaffected → ---
status-firefox52:
affected → ---
status-firefox53:
affected → ---
Assignee | ||
Comment 4•8 years ago
|
||
Actually, I think this is kinda separate from bug 1317795. The checkbox is _never_ supposed to be visible.
I think the situation jimm was in was occurs when privacy.trackingprotection.enabled is set to true, which enables global tracking protection. I am easily able to reproduce the issue with that pref set.
I think I can fix this separately from the checkbox restyling that's going to occur elsewhere once bug 418833 lands.
Status: RESOLVED → REOPENED
Resolution: DUPLICATE → ---
Comment hidden (mozreview-request) |
Comment 6•8 years ago
|
||
(In reply to Mike Conley (:mconley) from comment #4)
> I think the situation jimm was in was occurs when
> privacy.trackingprotection.enabled is set to true, which enables global
> tracking protection. I am easily able to reproduce the issue with that pref
> set.
>
> I think I can fix this separately from the checkbox restyling that's going
> to occur elsewhere once bug 418833 lands.
Ah, good find. I don't think that this needs any more fixing except hiding the checkbox though :)
Assignee | ||
Updated•8 years ago
|
Attachment #8812490 -
Flags: review?(past) → review?(jhofmann)
Comment 7•8 years ago
|
||
mozreview-review |
Comment on attachment 8812490 [details]
Bug 1317804 - Hide the checkbox toggle in about:privatebrowsing.
https://reviewboard.mozilla.org/r/94216/#review94402
Thanks, this seems to work fine!
::: browser/themes/shared/privatebrowsing/aboutPrivateBrowsing.css:142
(Diff revision 1)
>
> +.toggle-input {
> + opacity: 0;
> + width: 0;
> + pointer-events: none;
> + position: absolute;
A minor note:
As a relative newcomer to this code, I had to spend some time reading through blame and old bugs to find out why exactly this is hidden like that and not just using "display:none" (apparently to preserve keyboard focus).
Can you add a really short comment explaining that? The recent changes will make it much harder to get the blame info in the future.
Attachment #8812490 -
Flags: review?(jhofmann) → review+
Assignee | ||
Comment 8•8 years ago
|
||
mozreview-review-reply |
Comment on attachment 8812490 [details]
Bug 1317804 - Hide the checkbox toggle in about:privatebrowsing.
https://reviewboard.mozilla.org/r/94216/#review94402
> A minor note:
>
> As a relative newcomer to this code, I had to spend some time reading through blame and old bugs to find out why exactly this is hidden like that and not just using "display:none" (apparently to preserve keyboard focus).
>
> Can you add a really short comment explaining that? The recent changes will make it much harder to get the blame info in the future.
Thanks for the review! That's a really good call - thanks johannh! (And welcome back! :D)
Comment hidden (mozreview-request) |
Comment 10•8 years ago
|
||
Pushed by mconley@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/9b5e7c9b5c8c
Hide the checkbox toggle in about:privatebrowsing. r=johannh
Comment 11•8 years ago
|
||
bugherder |
Status: REOPENED → RESOLVED
Closed: 8 years ago → 8 years ago
status-firefox53:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 53
Assignee | ||
Comment 12•8 years ago
|
||
Comment on attachment 8812490 [details]
Bug 1317804 - Hide the checkbox toggle in about:privatebrowsing.
Approval Request Comment
[Feature/regressing bug #]:
Bug 1309316.
[User impact if declined]:
A non-native checkbox (sometimes non-functional in a non-default case) will be displayed in about:privatebrowsing.
[Describe test coverage new/current, TreeHerder]:
Testing was manual, but this is a page styling thing, so that's about par for the course here.
[Risks and why]:
Very low risk. We're just hiding the checkbox.
[String/UUID change made/needed]:
None.
Attachment #8812490 -
Flags: approval-mozilla-aurora?
Updated•8 years ago
|
Assignee: nobody → mconley
Comment 14•8 years ago
|
||
Comment on attachment 8812490 [details]
Bug 1317804 - Hide the checkbox toggle in about:privatebrowsing.
styling regression fix in about:privatebrowsing, take in aurora52
Attachment #8812490 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Comment 15•8 years ago
|
||
bugherder uplift |
status-firefox52:
--- → fixed
Comment 16•8 years ago
|
||
I have reproduced this bug with Nightly 53.0a1 (2016-11-15) on Windows 8.1 , 64 Bit !
Build ID 20161115030213
User Agent Mozilla/5.0 (Windows NT 6.3; WOW64; rv:53.0) Gecko/20100101 Firefox/53.0
This bug's fix is Verified with latest Aurora and latest Nightly !
Build ID 20161206004003
User Agent Mozilla/5.0 (Windows NT 6.3; WOW64; rv:52.0) Gecko/20100101 Firefox/52.0
Build ID 20161205030204
User Agent Mozilla/5.0 (Windows NT 6.3; WOW64; rv:53.0) Gecko/20100101 Firefox/53.0
[bugday-20161207]
Comment 18•8 years ago
|
||
[bugday-20170104] Bug verified
Comment 19•8 years ago
|
||
[bugday-20170104] Bug verified
Comment 20•8 years ago
|
||
[bugday-1317804] bug verified
sorry about previous comment
Updated•8 years ago
|
status-firefox52:
fixed → ---
Updated•8 years ago
|
status-firefox52:
--- → verified
Updated•6 years ago
|
Flags: in-qa-testsuite+
You need to log in
before you can comment on or make changes to this bug.
Description
•