New issue
Advanced search Search tips

Issue 863586 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Jul 18
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Chrome , Mac
Pri: 2
Type: Bug



Sign in to add a comment

identity sync state - incorrect text color

Project Member Reported by bettes@chromium.org, Jul 13

Issue description

What is the expected result?
Google blue 600 - paused
Google red 600 - error

What happens instead?
Google blue 500
 
Screen Shot 2018-07-13 at 2.40.33 PM.png
20.3 KB View Download
And the pill should be #E8F0FE, Google Blue 50
Labels: -Hotlist-Teamfood-Feedback
Owner: ----
Status: Available (was: Accepted)
Cc: pbos@chromium.org
Labels: -Pri-3 Pri-2
Owner: pbos@chromium.org
Status: Assigned (was: Available)
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.
Project Member

Comment 5 by bugdroid1@chromium.org, 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

Status: Fixed (was: Assigned)
These states are both using 600 now.
Cc: susan.boorgula@chromium.org
Labels: Needs-Feedback
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..
Labels: -Needs-Feedback
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