New issue
Advanced search Search tips

Issue 630619 link

Starred by 2 users

Issue metadata

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



Sign in to add a comment

[MD Downloads] Button shows focus state after clicking button

Reported by ugurcan....@gmail.com, Jul 22 2016

Issue description

What steps will reproduce the problem?
(1) Open chrome://downloads/
(2) Press 'Open downloads folder' to open downloads folder
(3) Go back to chrome

What is the expected result?
Focus state should not be seen.

What happens instead?
Focus state shown.

Please provide any additional information below. Attach a screenshot if possible.
Focus state can be seen in attached screenshot.
Also it happens in chrome://settings/ as well.

Version: 54.0.2804.0

 
Ekran Resmi 2016-07-22 17.23.30.png
70.5 KB View Download

Comment 1 by dbeam@chromium.org, Jul 25 2016

Components: -UI>Settings UI>Browser>Downloads
Owner: dbeam@chromium.org
Status: Assigned (was: Unconfirmed)
Summary: [MD Downloads] Button shows focus state after clicking button (was: [MD Downloads] Button show focus state after clicking button)
yeah, we should probably drop the bolding / background color when the menu closes (so it doesn't appear when re-shown)

thanks for the bug!

Comment 2 by dbeam@chromium.org, Jul 26 2016

Status: Started (was: Assigned)
Project Member

Comment 3 by bugdroid1@chromium.org, Jul 26 2016

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

commit dc04659f0a5c33a5baaef1910e4b03650f754544
Author: dbeam <dbeam@chromium.org>
Date: Tue Jul 26 04:40:08 2016

MD Downloads: remove bold selected item style

R=tsergeant@chromium.org
BUG= 630619 
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

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

[modify] https://crrev.com/dc04659f0a5c33a5baaef1910e4b03650f754544/chrome/browser/resources/md_downloads/toolbar.css
[modify] https://crrev.com/dc04659f0a5c33a5baaef1910e4b03650f754544/chrome/browser/resources/md_downloads/vulcanized.html

Comment 4 by dbeam@chromium.org, Jul 26 2016

Status: Fixed (was: Started)

Sign in to add a comment