New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 907229 link

Starred by 4 users

Issue metadata

Status: Fixed
Owner:
Closed: Dec 5
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 3
Type: Bug



Sign in to add a comment

Cros-Adapta: White button borders in dark theme too bright

Reported by justin.b...@gmail.com, Nov 20

Issue description

Chrome Version       : 72.0.3611.0
OS Version: 11265.0.0
URLs (if applicable) :
Other browsers tested:
  Add OK or FAIL after other browsers where you have tested this issue:
     Safari:
    Firefox:
    IE/Edge:

What steps will reproduce the problem?
1. Install Linux for ChromeOS
2. Install any of the following applications: gnome-terminal, terminator, tilix.
3. Launch application.

What is the expected result?

Applications will launch with a consistent UI theme.

What happens instead of that?

There's a white border around application UI controls that seem out of place for the theme - contrast with an install of the same applications in, say, Ubuntu 18.04. Not all UI controls have this border - it seems like a quirk of the in-use theme. If this isn't a defect, it would be helpful to the community to know if there is a setting that adjusts the color of these borders.

Please provide any additional information below. Attach a screenshot if
possible.

UserAgentString: Mozilla/5.0 (X11; CrOS x86_64 11265.0.0) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/72.0.3611.0 Safari/537.36



 
gnome-terminal.png
147 KB View Download
tilix.png
44.1 KB View Download
Components: OS>Systems>Containers
N.B. This was tested on a Pixelbook in current stable and dev channels.
Cc: sbroslawsky@chromium.org
Owner: jopra@chromium.org
Status: Assigned (was: Unconfirmed)
Summary: Cros-Adapta: White button borders in dark theme too bright (was: Unexpected white border around UI controls in Crostini GUI applications)
Thanks, this is a problem with the dark theme variant.

I debated using a darker color when I implemented button borders but had run out of colors in the pallet I was working off.

I'll work with our UI designers to find an appropriate grey.
Thank you, jopra@chromium.org !
Cc: sgabr...@chromium.org
Add sgabriel@ to the thread who might have a recommendation here. Thanks!
Something we've been exploring so far for dark buttons is having a 1dip border of #5F6368 100%. This might work for this as well (preview attached)
dark buttons.png
36.5 KB View Download
In addition, I am not sure if it's customizable here but the shade of blue we are using when it is displayed on a dark background is #8AB4F8, it enables better contrast.
Status: Started (was: Assigned)
Thanks sgabriel + sbroslawsky. I'll try to get both changes into a single CL (should only be a color scheme update without other code changes).
Thanks sgabriel!
Adding screenshots of proposed change.
before.png
93.4 KB View Download
after.png
87.1 KB View Download
Better :)
Project Member

Comment 12 by bugdroid1@chromium.org, Dec 4

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/cros-adapta/+/eb6d8c1832b9181926df107faf41a80887fd982c

commit eb6d8c1832b9181926df107faf41a80887fd982c
Author: Josh Pratt <jopra@chromium.org>
Date: Tue Dec 04 08:11:39 2018

Cros-Adapta: Update suggestion color, selection color and border colors

BUG= chromium:907229 
TEST=manual (build_packages, build_image, flash, test)
Change-Id: I29cd8b417576caf531e987796fe2c725b23c099d
Reviewed-on: https://chromium-review.googlesource.com/1351861
Commit-Ready: Josh Pratt <jopra@chromium.org>
Tested-by: Josh Pratt <jopra@chromium.org>
Reviewed-by: Raymes Khoury <raymes@chromium.org>
Reviewed-by: Nicholas Verne <nverne@chromium.org>
Reviewed-by: Thomas Anderson <thomasanderson@chromium.org>

[modify] https://crrev.com/eb6d8c1832b9181926df107faf41a80887fd982c/gtk-3.0/colors-dark.css

Status: Fixed (was: Started)
Should be fixed in m72 dependant on the timing of Termina updates.

Sign in to add a comment