New issue
Advanced search Search tips

Issue 908486 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Dec 12
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Mac
Pri: 2
Type: Bug

Blocked on:
issue 831603

Blocking:
issue 893598



Sign in to add a comment

Various Content Controls (Buttons, Autocomplete) Use a White Foreground on White Background in Dark Mode

Project Member Reported by robliao@chromium.org, Nov 26

Issue description

1) Fire up Chrome with --force-dark-mode
2) Navigate to crbug.com
3) Create a new issue.

ACTUAL:
Buttons and autocomplete text foreground are white / low contrast.

EXPECTED:
Buttons and autocomplete text foreground have high contrast.
 
White Text.png
9.6 KB View Download
Autocomplete.png
21.3 KB View Download
What's your Mojave version? I can't repro :/

Are you by any chance running the rest of the OS in light mode and just forcing Dark Mode for Chrome? I could see that doing this as well.
I'm running Mojave 10.14.1 My Mojave is certainly in Dark Mode (the menubar is dark and  Finder renders in Dark Mode).
And now it repros. Who knows!?

Looks like WebKit uses a scoped appearance helper when pulling system colors, I'm guessing using the prefers-color-scheme media query (Issue 889087), though I didn't dig far enough to be sure if this is the specific thing that gets plumbed down. This seems like a fine approach to me.
Labels: Hotlist-DesktopUIConsider
Project Member

Comment 5 by bugdroid1@chromium.org, Nov 30

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/92e3fdd3a77ca97233868ca01b7a4888399e8253

commit 92e3fdd3a77ca97233868ca01b7a4888399e8253
Author: Leonard Grey <lgrey@chromium.org>
Date: Fri Nov 30 19:42:50 2018

Mac: Ensure web content system colors use light appearance in Mojave

Bug:  908486 
Change-Id: I762b8a74fcb98fb99218cee590da7cf003dec8cf
Reviewed-on: https://chromium-review.googlesource.com/c/1356870
Reviewed-by: Avi Drissman <avi@chromium.org>
Commit-Queue: Leonard Grey <lgrey@chromium.org>
Cr-Commit-Position: refs/heads/master@{#612732}
[modify] https://crrev.com/92e3fdd3a77ca97233868ca01b7a4888399e8253/content/browser/theme_helper_mac.mm

Buttons fixed, dropdowns: not yet
Cc: dbeam@chromium.org
Labels: Group-Dark_Mode
Labels: -Hotlist-DesktopUIConsider Hotlist-DesktopUITriaged
Blockedon: 831603
Labels: M-73 Target-73
Status: Fixed (was: Assigned)
Should be fixed as of https://chromium-review.googlesource.com/c/chromium/src/+/1367936

Sign in to add a comment