New issue
Advanced search Search tips

Issue 628071 link

Starred by 3 users

Issue metadata

Status: Fixed
Owner:
Closed: Oct 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug



Sign in to add a comment

Spinner for updating device has visual glitch with md-settings

Reported by ringosta...@gmail.com, Jul 14 2016

Issue description

bugs: Update your device and the spinner has a dot in it.

 
Screenshot 2016-07-13 at 8.36.34 PM.png
16.5 KB View Download
Labels: Hotlist-MD-Settings-About
Owner: dpa...@chromium.org
Status: Assigned (was: Unconfirmed)

Comment 2 by dpa...@chromium.org, Jul 18 2016

This is unfortunately caused by https://github.com/PolymerElements/iron-icon/issues/67. We use the "src" attribute to display the throbber, and the "icon" attribute to display the remaining icons. When setting "icon" to null, it unfortunately does not fully clear the previously displayed icon (see attachment where the checkmark icon is visible if the zoom level is increased quite a bit).


unwanted_icon.png
20.7 KB View Download

Comment 3 by dpa...@chromium.org, Jul 18 2016

FYI, I have a candidate fix. Will send a PR.

Comment 4 by dpa...@chromium.org, Jul 19 2016

Status: ExternalDependency (was: Assigned)
There is already a PR that fixes the issue at https://github.com/PolymerElements/iron-iconset-svg/pull/44. Need to ensure that this PR lands, since it has been sitting around for a few months.
 Issue 630336  has been merged into this issue.

Comment 6 by dbeam@chromium.org, Oct 12 2016

Cc: dpa...@chromium.org
Owner: dbeam@chromium.org
Status: Started (was: ExternalDependency)
i'll try to roll in the github PR
Project Member

Comment 7 by bugdroid1@chromium.org, Oct 12 2016

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

commit 621fabaed32c36e94621eef310fda26830235f0d
Author: dbeam <dbeam@chromium.org>
Date: Wed Oct 12 19:07:43 2016

Roll Polymer element iron-iconset-svg, 1.0.9 -> 1.0.11 to pick up:
https://github.com/PolymerElements/iron-iconset-svg/pull/44

TBR=dpapad@chromium.org
BUG= 628071 
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

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

[modify] https://crrev.com/621fabaed32c36e94621eef310fda26830235f0d/chrome/browser/resources/md_downloads/crisper.js
[modify] https://crrev.com/621fabaed32c36e94621eef310fda26830235f0d/chrome/browser/resources/md_history/app.crisper.js
[modify] https://crrev.com/621fabaed32c36e94621eef310fda26830235f0d/third_party/polymer/v1_0/bower.json
[modify] https://crrev.com/621fabaed32c36e94621eef310fda26830235f0d/third_party/polymer/v1_0/components-chromium/iron-iconset-svg/bower.json
[modify] https://crrev.com/621fabaed32c36e94621eef310fda26830235f0d/third_party/polymer/v1_0/components-chromium/iron-iconset-svg/iron-iconset-svg-extracted.js
[modify] https://crrev.com/621fabaed32c36e94621eef310fda26830235f0d/third_party/polymer/v1_0/components-chromium/paper-icon-button/paper-icon-button-light.html
[modify] https://crrev.com/621fabaed32c36e94621eef310fda26830235f0d/third_party/polymer/v1_0/components_summary.txt

Comment 8 by dpa...@chromium.org, Oct 13 2016

Status: Fixed (was: Started)

Sign in to add a comment