New issue
Advanced search Search tips

Issue 906075 link

Starred by 2 users
Status: Fixed
Owner:
ellyjo...@chromium.org
Closed: Dec 4
Cc:
mac-bugs-priority@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Mac
Pri: 1
Type: Bug-Regression



Sign in to add a comment

Pressing Command key when a bookmark/settings menu is open closes it

Reported by kwpol...@gmail.com, Nov 16 
UserAgent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_14_1) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/72.0.3610.2 Safari/537.36

Steps to reproduce the problem:
1. Open a menu — either a bookmarks folder, the bookmarks overflow, or the Chrome ellipsis menu.
2. Press the Command key (intending to open a bookmark in a new tab, for example)

What is the expected behavior?

What went wrong?
The menu just disappears.

Did this work before? Yes A few days ago.

Chrome version: 72.0.3610.2  Channel: dev
OS Version: OS X 10.14.1
Flash Version:

Comment 1 by meh...@chromium.org, Nov 16

Components: -UI UI>Input>KeyboardShortcuts UI>Browser
Labels: -Pri-2 Needs-Bisect Pri-1
Status: Untriaged (was: Unconfirmed)
Thanks for the report. That's a regression. -> Needs-Bisect.

Comment 2 by robliao@chromium.org, Nov 17

Labels: Hotlist-DesktopUIConsider

Comment 3 by meh...@chromium.org, Nov 18

Labels: -Needs-Bisect M-72
Owner: ellyjo...@chromium.org
Status: Assigned (was: Untriaged)
The Regression Range is https://chromium.googlesource.com/chromium/src/+log/ce46de13a7df955e9acf21c436aa374543679e07..6eeb7d038d2afb6661dbea710a616dba51785825

ellyjones@: Probably caused by your CL: https://chromium-review.googlesource.com/c/chromium/src/+/1334147

Can you please take a look at this regression?

The problem is that we need the accelerators key press in some cases when the Menus are open :(

Thanks in advance :)

Comment 4 by ellyjo...@chromium.org, Nov 19 

Ah, rats, it is. Thanks for reporting this. I'll revert my CL.

Comment 6 by robliao@chromium.org, Nov 22

Labels: Group-Toolbar

Comment 7 by ellyjo...@chromium.org, Dec 4

Status: Fixed (was: Assigned)
This actually got fixed by the linked revert (yay!)
►

Sign in to add a comment