identity sync state - incorrect text color |
|||||||
Issue descriptionWhat is the expected result? Google blue 600 - paused Google red 600 - error What happens instead? Google blue 500
,
Jul 17
,
Jul 17
,
Jul 18
This got off my radar as it wasn't assigned, sorry. Alan what's the alpha of {GR,GB}600 on white that yields G{R,B}50? I'd like to use the same alpha for both red and blue states.
,
Jul 18
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/26723c8e856a06d2e727f27803739da0f8467201 commit 26723c8e856a06d2e727f27803739da0f8467201 Author: Peter Boström <pbos@chromium.org> Date: Wed Jul 18 18:43:23 2018 Use Google blue/red 600 for avatar error states Aligns values used for text with the ones requested by the spec. Bug: chromium:863586 Change-Id: I0e9b4c32bd672e38ab2b6a7831fc62fdefe0f650 Reviewed-on: https://chromium-review.googlesource.com/1141359 Commit-Queue: Bret Sepulveda <bsep@chromium.org> Reviewed-by: Bret Sepulveda <bsep@chromium.org> Cr-Commit-Position: refs/heads/master@{#576149} [modify] https://crrev.com/26723c8e856a06d2e727f27803739da0f8467201/chrome/browser/ui/views/profiles/avatar_toolbar_button.cc
,
Jul 18
These states are both using 600 now.
,
Jul 19
pbos@ Request you to provide the steps to reproduce this issue, which will help us to verify the fix on the latest Canary 69.0.3496.0. Thanks..
,
Jul 19
This is best verified by a designer as it's a subtle shift in color. I think this can be ignored by TE, thanks. |
|||||||
►
Sign in to add a comment |
|||||||
Comment 1 by bettes@chromium.org
, Jul 13