New issue
Advanced search Search tips

Issue 767149 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Oct 2017
Components:
EstimatedDays: ----
NextAction: ----
OS: Mac
Pri: 2
Type: Bug

Blocking:
issue 589943
issue 767116



Sign in to add a comment

Error icon

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

Issue description

Questions: 

- whats the name and dimensions of this icon? Link to chromium if possible
- Is it used anywhere else in chrome? 
- If not, there's other alternatives we could use

https://chrome-assets.googleplex.com/?search=error&platform=0&order=0&sort=asc&grouped=true&flagged=false

error.svg: 
https://chrome-assets.googleplex.com/?search=error.svg&path=src/ui/webui/resources/images&platform=0&grouped=true&flagged=false&shared=1
 

Comment 1 by bettes@chromium.org, Sep 21 2017

warning_icon - 24px

Comment 2 by bettes@chromium.org, Sep 23 2017

Only action item here is to resize the warning_icon down to 16px. 
P - Download manager Error - MacViews.png
99 KB View Download

Comment 3 by sdy@chromium.org, Oct 6 2017

Blocking: 589943

Comment 4 by sdy@chromium.org, Oct 6 2017

Blockedon: -767116

Comment 5 by sdy@chromium.org, Oct 6 2017

Blocking: 767116

Comment 6 by sdy@chromium.org, Oct 6 2017

Labels: M-64
Project Member

Comment 7 by bugdroid1@chromium.org, Oct 16 2017

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

commit 3f13d8f08d0ad3ec2144cef681636ca5ef10a009
Author: Sidney San Martín <sdy@chromium.org>
Date: Mon Oct 16 15:33:05 2017

Resize MD downloads danger icon per design changes.

Bug:  767149 
Change-Id: I3f109ea28f221ae6f800fcee23bd2744c923705e
Reviewed-on: https://chromium-review.googlesource.com/719998
Reviewed-by: Elly Fong-Jones <ellyjones@chromium.org>
Commit-Queue: Sidney San Martín <sdy@chromium.org>
Cr-Commit-Position: refs/heads/master@{#509053}
[modify] https://crrev.com/3f13d8f08d0ad3ec2144cef681636ca5ef10a009/chrome/browser/ui/cocoa/download/md_download_item_view.mm

Comment 8 by sdy@chromium.org, Oct 16 2017

Status: Fixed (was: Assigned)

Sign in to add a comment