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

Issue 632271 link

Starred by 2 users
Status: WontFix
Owner:
mahmadi@chromium.org
Closed: Aug 2016
Cc:
nyerramilli@chromium.org
█ msrchandra@chromium.org
michae...@chromium.org
dpa...@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Mac
Pri: 1
Type: Bug-Regression



Sign in to add a comment

Regression: Unnecessary focus ring is seen on 'Search engine' link of md-settings page.

Reported by rk...@etouch.net, Jul 28 2016 
Chrome Version:54.0.2810.0 (Official Build) 24d606bb2a3e6290b97d9731c1dfd4dbfcb948e7-refs/heads/master@{#408294}
OS: Mac (10.10.5,10.11.4), Windows (7,8,10)

What steps will reproduce the problem?
(1) Launch chrome, switch to 'Guest' mode and navigate to chrome://md-settings page.
(2) Click on Menu icon to open side nav menu and press Tab key, observe

Unnecessary focus ring is seen on 'Search engine' link.

No such a focus ring is seen on 'Search engine' link.

This is regression issue, broken in ‘M 54’ will soon update the other info

Comment 1 by rk...@etouch.net, Jul 28 2016

Labels: hasbisect OS-Linux
Owner: mahmadi@chromium.org
Status: Assigned (was: Unconfirmed)
Good build:54.0.2809.0 
Bad build:54.0.2810.2

Narrow Bisect:
This is regression issue, broken in ‘M 54’ and below is narrow bisect :
https://chromium.googlesource.com/chromium/src/+log/804b612a628958aef2f1f21fdda3c56c827cb2cb..90cefaaf2c2dc1fe7f136edc6a673228525649b3?pretty=fuller&n=100

Suspecting: r408112
Actual_Guest.mp4
428 KB View Download

Comment 2 by mahmadi@chromium.org, Aug 1 2016

Cc: michae...@chromium.org
michaelpg@, reason the focus ring is showing seems to be that the first menu item that has a non-negative tab-index (/people) is hidden. Not sure why the focus ring shows here but maybe hiding the menu items isn't optimal here. wdyt?

Comment 3 by mahmadi@chromium.org, Aug 1 2016 

or perhaps we can disable the menu items that we would like to hide and hide them using css.

Comment 4 by mahmadi@chromium.org, Aug 3 2016

Status: Started (was: Assigned)

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

Status: WontFix (was: Started)
After discussion with dbema@, we decided this has to be fixed in Polymer.

https://github.com/PolymerElements/iron-menu-behavior/issues/72
►

Sign in to add a comment