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)

defect

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};

https://searchfox.org/mozilla-central/rev/5f81c5091d442d829120e19477ba869ae5219709/browser/components/newtab/content-src/styles/_theme.scss#111,119

This patch also renames --customize-menu-primary-action-text to --customize-menu-slider-fill, since that's the only place its used.

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

Set release status flags based on info from the regressing bug 1726432

Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → 94 Branch
Whiteboard: [fidefe-theme]

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
Has Regression Range: --- → yes
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: