Checkbox looks inconsistent (10 px too wide) in "Insecure Connection" warning, compared to the other Firefox checkboxes

VERIFIED FIXED in Firefox 53

Status

()

defect
VERIFIED FIXED
3 years ago
3 months ago

People

(Reporter: Virtual, Assigned: dao)

Tracking

({nightly-community, regression, ux-consistency})

53 Branch
Firefox 53
x86_64
Windows 7
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox-esr45 unaffected, firefox50 unaffected, firefox51 unaffected, firefox52 unaffected, firefox53 verified)

Details

()

Attachments

(3 attachments)

Posted image bad.png β€”
+++ This bug was initially created as a clone of Bug #1317795 +++

[Tracking Requested - why for this release]: Regression

STR:
1. Open this website page - https://forum.doom9.org/showthread.php?t=166689
2. See that checkbox is rendered differently (10 px too wide) compared to the other Firefox checkboxes
Flags: needinfo?(mconley)
Blocks: 1317795
Has Regression Range: --- → yes
Has STR: --- → yes
No longer depends on: 1317795
Removing the whole block here (from line 111 to 117): https://dxr.mozilla.org/mozilla-central/source/browser/themes/shared/aboutNetError.css#111

fixes the issue
Component: General → Theme
Posted patch patch β€” β€” Splinter Review
Assignee: nobody → dao+bmo
Status: NEW → ASSIGNED
Flags: needinfo?(mconley)
Attachment #8814368 - Flags: review?(mconley)
Comment on attachment 8814368 [details] [diff] [review]
patch

Review of attachment 8814368 [details] [diff] [review]:
-----------------------------------------------------------------

Thanks!
Attachment #8814368 - Flags: review?(mconley) → review+
Pushed by dgottwald@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/46a20ea6799d
Remove obsolete checkbox styling. r=mconley
https://hg.mozilla.org/mozilla-central/rev/46a20ea6799d
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 53
With no reply from DΓ£o in 3 weeks, maybe Mike can help?  If you think this should be fixed in 52, can you request the uplift?
Flags: needinfo?(mconley)
With bug 1321302 landed in 52 (which effectively undoes the styling that was done to move us to proper checkboxes as opposed to pseudoelements) this bug is no longer reproducible for me, so I don't believe this uplift is necessary. Settings the flags.
Flags: needinfo?(mconley)
Flags: needinfo?(dao+bmo)
You need to log in before you can comment on or make changes to this bug.