New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 640289 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Aug 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Windows , Mac
Pri: 2
Type: Bug



Sign in to add a comment

Close all windows icon 1px too big

Project Member Reported by bettes@chromium.org, Aug 23 2016

Issue description

In the screenshot, the implemented 'close all' icon is 1px too large. On the left is expected. 

New SVG: 
https://drive.google.com/open?id=0BxMIIGI80eU-d2Q1QXRlTnp0M2M
 
Screen Shot 2016-08-23 at 12.07.40 PM.png
25.4 KB View Download
Project Member

Comment 1 by bugdroid1@chromium.org, Aug 24 2016

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

commit b37d9ea9bfeccc11482eb81d16ff29f84e652181
Author: janeliulwq <janeliulwq@google.com>
Date: Wed Aug 24 01:03:14 2016

[MD User Menu] Updated the close_all icon

Updated the close_all icon with a newly tweaked svg source.
See first bug.

See source at: https://drive.google.com/open?id=0BxMIIGI80eU-d2Q1QXRlTnp0M2M

BUG= 640289 
BUG= 615893 

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

[modify] https://crrev.com/b37d9ea9bfeccc11482eb81d16ff29f84e652181/ui/gfx/vector_icons/close_all.icon

The CL above replaced the old icon with the new one.

Comment 3 by ew...@chromium.org, Aug 24 2016

Status: Fixed (was: Assigned)

Sign in to add a comment