Footer font color in the new dropdown should be GoogleGrey700 |
||
Issue descriptionChrome Version: M69 OS: desktop Footer font color is GG900, but it should be GG700.
,
Jul 9
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/27ecaa37e93ad2a0eea2ec4a4eac69742f576ffd commit 27ecaa37e93ad2a0eea2ec4a4eac69742f576ffd Author: Tommy Martino <tmartino@chromium.org> Date: Mon Jul 09 23:45:24 2018 [Autofill Views] Fixing Label Colors This change solves two problems related to font colors: * Changes passed down from UX have requested the use of the subtext/hint color for all text in footer rows. * A bug causes text colors to change (permanently) to incorrect values when hover styles are applied. This was observed on Linux under GTK+ theme only. We don't need to modify text colors during RefreshStyle anyway, so the simplest fix is to define these at initialization and not touch them again. Change-Id: I80d2de91bb7067b3dc8fea6f369c83658e15c22a Bug: 861883 , 859887 Reviewed-on: https://chromium-review.googlesource.com/1130015 Commit-Queue: Tommy Martino <tmartino@chromium.org> Reviewed-by: Peter Kasting <pkasting@chromium.org> Reviewed-by: Fabio Tirelo <ftirelo@chromium.org> Cr-Commit-Position: refs/heads/master@{#573532} [modify] https://crrev.com/27ecaa37e93ad2a0eea2ec4a4eac69742f576ffd/chrome/browser/ui/views/autofill/autofill_popup_view_native_views.cc
,
Jul 10
|
||
►
Sign in to add a comment |
||
Comment 1 by ftirelo@chromium.org
, Jul 3Labels: M-69