New issue
Advanced search Search tips

Issue 638467 link

Starred by 2 users
Status: Fixed
Owner:
mahmadi@chromium.org
Closed: Oct 2016
Cc:
tommycli@chromium.org
dpa...@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug



Sign in to add a comment

[People] Update sync icon from checkmark to sync icon

Project Member Reported by bettes@chromium.org, Aug 17 2016 
Let's use the iron-icons "notification:sync" to represent sync and not the checkmark

size: 20px
color: google green 700

Mocks updated: 
https://folio.googleplex.com/chrome-ux-specs-and-sources/Chrome%20Inner%20Pages/03-Settings/preview/cards#%2FPREVIEW-People-account.png%3Fz=width
Screen Shot 2016-08-16 at 7.43.21 PM.png
76.7 KB View Download

Comment 1 by dpa...@chromium.org, Sep 14 2016

Cc: tommycli@chromium.org
Owner: mahmadi@chromium.org

Comment 2 by ew...@chromium.org, Oct 3 2016 

Issue 642963 has been merged into this issue.

Comment 3 by mahmadi@chromium.org, Oct 3 2016

Status: Started (was: Assigned)
Project Member

Comment 4 by bugdroid1@chromium.org, Oct 3 2016 

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

commit c2897611468f3615ff9297281c9b88f8a9852861
Author: mahmadi <mahmadi@chromium.org>
Date: Mon Oct 03 22:10:12 2016

[Settings][People] sync icon instead of checkmark to represent successful sync

BUG= 638467 
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2387063003
Cr-Commit-Position: refs/heads/master@{#422557}

[modify] https://crrev.com/c2897611468f3615ff9297281c9b88f8a9852861/chrome/browser/resources/settings/people_page/people_page.html
[modify] https://crrev.com/c2897611468f3615ff9297281c9b88f8a9852861/chrome/browser/resources/settings/people_page/people_page.js

Comment 5 by mahmadi@chromium.org, Oct 4 2016

Status: Fixed (was: Started)
►

Sign in to add a comment