MacViews: Adding shortcuts to bookmarks via System Preferences doesn't work |
||
Issue descriptionVia this public report: https://www.reddit.com/r/chrome/comments/9gy8bb/recent_chrome_update_prevents_mac_keyboard/ This is a regression from Cocoa. Chrome Version: 71.0.3555.0 OS: MacOS What steps will reproduce the problem? (1) Add a top-level bookmark with an easily typable name, so that it shows up in the Bookmarks menu (see first attached screenshot). (2) Add a shortcut to Chrome in System Preferences > Keyboard > Shortcuts with the bookmarks' name (see second attached screenshot). (3) Verify that the shortcut is displayed in the menu (see third attached screenshot). (4) Type the shortcut. What is the expected result? Chrome navigates to the bookmark What happens instead? The default shortcut is triggered.
,
Sep 20
The shortcut "cmd 1" is reserved by Chrome to navigate to the first tab in the window. I tried another shortcut -- e.g. cmd + shift + U and it worked fine.
,
Sep 20
Sure, but it's still a regression, no?
,
Sep 20
For example, I can rename Command+N in Finder to "Zoom"
,
Sep 20
This is an intended behavior change. This causes Chrome to match the behavior of Safari -- where special hotkeys that don't show up in the menu like "cmd 1" [to switch to the first tab] cannot be overridden. I just tried this -- Safari has the same behavior as Chrome. The root problem is that we cannot simultaneously satisfy all desired ordering constraints. In general, we want 1) Chrome special hotkeys to have greater priority than Web Contents. 2) Web Contents to have greater priority than macOS Menu shortcuts 3) some macOS Menu shortcuts to have higher priority than Chrome special hotkeys [this case]. This is an over simplification of the set of constraints we need to simultaneously satisfy. Every consumer of hotkeys wants to be the "highest priority" consumer. We can't make this happen everywhere. |
||
►
Sign in to add a comment |
||
Comment 1 by ellyjo...@chromium.org
, Sep 20Owner: erikc...@chromium.org
Status: Assigned (was: Untriaged)