Closed
Bug 1504852
Opened 6 years ago
Closed 6 years ago
Turn off Content Blocking label is barely visible in the dark theme
Categories
(Firefox :: Site Identity, defect, P2)
Tracking
()
VERIFIED
FIXED
Firefox 65
Tracking | Status | |
---|---|---|
firefox-esr60 | --- | unaffected |
firefox63 | --- | unaffected |
firefox64 | --- | unaffected |
firefox65 | --- | verified |
People
(Reporter: ehsan.akhgari, Assigned: ewright)
References
(Blocks 1 open bug)
Details
(Keywords: regression, Whiteboard: [privacy65])
Attachments
(2 files)
Please see the screenshot.
Updated•6 years ago
|
Summary: Turn on Content Blocking label is barely visible in the dark theme → Turn off Content Blocking label is barely visible in the dark theme
Updated•6 years ago
|
Component: Theme → Site Identity and Permission Panels
Comment 1•6 years ago
|
||
Seems like this is specific to Linux, at least I can't reproduce on OSX. Erica, do you want to take a look at this whenever you have time? :)
status-firefox64:
--- → unaffected
status-firefox65:
--- → affected
Flags: needinfo?(ewright)
Keywords: regression
OS: Unspecified → Linux
Priority: -- → P2
Whiteboard: [privacy65]
Comment 2•6 years ago
|
||
Judging by the CSS, this should be an issue on Windows too. Set color: inherit to fix this.
OS: Linux → All
Assignee | ||
Updated•6 years ago
|
Assignee: nobody → ewright
Status: NEW → ASSIGNED
Flags: needinfo?(ewright)
Assignee | ||
Comment 3•6 years ago
|
||
In windows and linux, ensure that the text color inside the content blocking button is visible. Add 3 px of margin besode the icon to separate it from the text.
Pushed by ewright@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/6901bde94374
Turn off Content Blocking label color is visible in dark theme and icon is spaced from text. r=dao
Comment 5•6 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 65
Comment 6•6 years ago
|
||
I verified this issue on mac OS, Ubuntu 16.04 and Windows 10, Nightly 65.0a1 (2018-11-07). Marking it accordingly.
Status: RESOLVED → VERIFIED
Updated•6 years ago
|
status-firefox63:
--- → unaffected
status-firefox-esr60:
--- → unaffected
You need to log in
before you can comment on or make changes to this bug.
Description
•