Regression : Weird blue focus ring is seen in iron drop down list under ‘Language’ section in md-settings.
Reported by
yfulgaon...@etouch.net,
Sep 30 2016
|
||||||
Issue descriptionChrome Version : 55.0.2876.0 (Official Build) 5c0e46df680c3284a99ae05aafaf52078674ea2d-refs/heads/master@{#421986} (64-bit) OS: Mac(10.10.5)(10.11.5) What steps will reproduce the problem? 1. Launch chrome, navigate to chrome://md-settings page and go to ‘Language’ section. 2. Expand ‘Language’ section and click on iron drop down list seen next to default language. 3. Observe the blue focus ring in drop down list. Actual : Weird blue focus ring is seen in iron drop down list under ‘Language’ section. Expected : Blue focus ring should be seen properly in iron drop down list under ‘Language’ section. This is a regression issue broken in ‘M-55’, below is the Manual Regression and will soon update bisect info. Good build : 55.0.2875.0 Bad build : 55.0.2876.0
,
Sep 30 2016
Using the per-revision bisect providing the bisect results, Good build:55.0.2875.0(Revision: 421703). Bad build: 55.0.2876.0 (Revision: 421986). You are probably looking for a change made after 421776 (known good), but no later than 421777 (first known bad). CHANGELOG URL: ----------------- https://chromium.googlesource.com/chromium/src/+log/864454fb784c52c1202410dc28799c8dc0f9ad0c..8cf11a02e256b61c86a23e9c6126ff3183a628c5 @tsergeant - Could you please check whether this is caused with respect to your change, if not please help us in assigning it to the right owner. Review-Url: https://codereview.chromium.org/2272553002 Thanks!
,
Sep 30 2016
,
Oct 3 2016
yep, that CL is the cause (https://codereview.chromium.org/2272553002) we can work around this by making the <hr> disabled. but maybe we should just do the separator in CSS, with a dynamic class name on a particular item (the line should only show up under certain conditions) tsergeant -- can you check the other uses of cr-shared-menu for similar regressions? Then assign this to me for the Settings > Language section specific fix.
,
Oct 3 2016
Interesting. We're now using a <paper-listbox> in <cr-shared-menu>, so that's where this issue comes from. Languages is the only place that currently uses non-button elements in a cr-shared-menu.
,
Oct 18 2016
Updating the latest behavior of the bug, Issue is still seen on chrome latest canary M56-56.0.2891.0. michaelpg@ - Could you please let us know is there any latest update available for this issue? Thanks!
,
Oct 18 2016
I don't see this in canary. What platform were you using?
,
Nov 16 2016
Not reproducible. |
||||||
►
Sign in to add a comment |
||||||
Comment 1 by yfulgaon...@etouch.net
, Sep 30 201641.1 KB
41.1 KB View Download