New issue
Advanced search Search tips

Issue 747087 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Closed: Jul 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Chrome , Mac
Pri: 3
Type: Bug



Sign in to add a comment

Wrong blue color on checkmark icon for chrome://settings/help

Project Member Reported by bettes@chromium.org, Jul 20 2017

Issue description

Actual: paper blue 600
Expected: google blue 500
 
Screen Shot 2017-07-20 at 12.28.56 PM.png
16.7 KB View Download

Comment 1 by dpa...@chromium.org, Jul 20 2017

Status: Started (was: Assigned)
@bettes: I don't see a google-blue-600 color in Polymer, only 500 and 700 exist, see [1]. Attaching screenshot of both 500 and 700, which one do you prefer?

[1] https://cs.chromium.org/chromium/src/third_party/catapult/third_party/polymer/components/paper-styles/color.html?l=24-27.
google_blue_500.png
4.2 KB View Download
google_blue_700.png
6.5 KB View Download

Comment 2 by bettes@chromium.org, Jul 20 2017

500 please :) 

>> Expected: google blue 500

Comment 3 by dpa...@chromium.org, Jul 20 2017

>> Expected: google blue 500
Heh, ok. Sorry for the confusion (I guess this is proof of how the brain does not actually read every single letter when there is a lot of redundancy in a sentence).
Project Member

Comment 4 by bugdroid1@chromium.org, Jul 22 2017

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

commit 6dc97ff2473f4394182a75cffabc6b7ae3460bfa
Author: dpapad <dpapad@chromium.org>
Date: Sat Jul 22 02:37:32 2017

About page: Tweak color of checkmark icon on chrome://settings/help

Bug:  747087 
Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: Id7633ba0090fb8142de1a63bcca6e9626738f5c0
Reviewed-on: https://chromium-review.googlesource.com/580582
Reviewed-by: Dave Schuyler <dschuyler@chromium.org>
Commit-Queue: Demetrios Papadopoulos <dpapad@chromium.org>
Cr-Commit-Position: refs/heads/master@{#488843}
[modify] https://crrev.com/6dc97ff2473f4394182a75cffabc6b7ae3460bfa/chrome/browser/resources/settings/about_page/about_page.html

Comment 5 by dpa...@chromium.org, Jul 24 2017

Status: Fixed (was: Started)
Status: Verified (was: Fixed)
9807.0.0, 62.0.3176.0
on ChromeOS

Sign in to add a comment