Closed
Bug 1730560
Opened 3 years ago
Closed 3 years ago
New tab customize checkbox checkmark is hard to see on light theme
Categories
(Firefox :: New Tab Page, defect)
Firefox
New Tab Page
Tracking
()
VERIFIED
FIXED
94 Branch
Tracking | Status | |
---|---|---|
firefox-esr78 | --- | unaffected |
firefox-esr91 | --- | unaffected |
firefox92 | --- | unaffected |
firefox93 | --- | unaffected |
firefox94 | --- | verified |
People
(Reporter: Mardak, Assigned: bugzilla)
References
(Regression)
Details
(Keywords: regression, Whiteboard: [fidefe-theme])
Attachments
(2 files)
Looks like the checkbox check got switched but not the slider toggle. Unclear if they should be the same as each other?
--customize-menu-check-fill: #{$newtab-card-primary-text-color};
--customize-menu-primary-action-text: #{$white};
Assignee | ||
Comment 1•3 years ago
|
||
This patch also renames --customize-menu-primary-action-text to --customize-menu-slider-fill, since that's the only place its used.
Updated•3 years ago
|
Assignee: nobody → htwyford
Status: NEW → ASSIGNED
Pushed by htwyford@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/9b1f394b6cb2
Fix newtab customize checkbox color. r=Mardak
Updated•3 years ago
|
Keywords: regression
Comment 3•3 years ago
|
||
Set release status flags based on info from the regressing bug 1726432
status-firefox92:
--- → unaffected
status-firefox93:
--- → unaffected
status-firefox94:
--- → affected
status-firefox-esr78:
--- → unaffected
status-firefox-esr91:
--- → unaffected
Comment 4•3 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → 94 Branch
Assignee | ||
Updated•3 years ago
|
Whiteboard: [fidefe-theme]
Comment 5•3 years ago
|
||
I have verified that the checkmark is white on a blue background for the Light theme, and the checkmark is black on a light blue background on Dark theme using Firefox 94.0a1 (Build ID: 20210923214525) on Windows 10 x64, macOS 11.6, and Ubuntu Linux 20.04 x64.
Status: RESOLVED → VERIFIED
Updated•3 years ago
|
Has Regression Range: --- → yes
Updated•3 years ago
|
You need to log in
before you can comment on or make changes to this bug.
Description
•