Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Incorrect keyboard shortcut shown in menu. #85798

Open
ksze opened this issue Nov 29, 2019 · 6 comments
Open

Incorrect keyboard shortcut shown in menu. #85798

ksze opened this issue Nov 29, 2019 · 6 comments
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug chromium Issues and items related to Chromium confirmed Issue has been confirmed by VS Code Team member help wanted Issues identified as good community contribution opportunities linux Issues with VS Code on Linux menus Menu items and widget issues upstream Issue identified as 'upstream' component related (exists outside of VS Code)

Comments

@ksze
Copy link

ksze commented Nov 29, 2019

  • VSCode Version: 1.40.2 (also happens in 1.41.0-insider)
  • OS Version: Pop!_OS 19.10 64-bit (which is very much like Ubuntu 19.10 Eoan Ermine)

Steps to Reproduce:

  1. Press Ctrl+K, Ctrl+S to open the keyboard shortcut editor;
  2. Find the entry for toggling the Activity Bar (workbench.action.toggleActivityBarVisibility)
  3. Set its keybinding to "Ctrl + Super + A", which shouldn't be used by anything else by default; "Super" is the "Windows" key on PC keyboards, also known as the meta key in Linux.
  4. Close the keyboard shortcut editor;
  5. Open the "View" menu --> "Appearance"
  6. See that the keyboard shortcut displayed next to "Show Activity Bar" is "Ctrl+A" instead of "Ctrl+Super+A"

Does this issue occur when all extensions are disabled?: Yes

@sandy081
Copy link
Member

sandy081 commented Dec 1, 2019

Does the assigned keybinding works?

@sandy081 sandy081 added the info-needed Issue requires more information from poster label Dec 1, 2019
@ksze
Copy link
Author

ksze commented Dec 2, 2019

The keybinding (Ctrl+Super+A) does work.

@sandy081 sandy081 removed the info-needed Issue requires more information from poster label Dec 2, 2019
@sbatten sbatten added bug Issue identified by VS Code Team member as probable bug linux Issues with VS Code on Linux menus Menu items and widget issues upstream Issue identified as 'upstream' component related (exists outside of VS Code) electron Issues and items related to Electron and removed bug Issue identified by VS Code Team member as probable bug labels Dec 11, 2019
@sbatten sbatten assigned deepak1556 and unassigned sbatten Dec 11, 2019
@sbatten
Copy link
Member

sbatten commented Dec 11, 2019

It seems that electron is just ignoring the super part of the shortcut even though is documented as supported. @deepak1556 let me know if we can't fix this on electron side and I will patch with our "non-native keybind" workaround

@deepak1556 deepak1556 added the confirmed Issue has been confirmed by VS Code Team member label Dec 12, 2019
@deepak1556 deepak1556 added bug Issue identified by VS Code Team member as probable bug help wanted Issues identified as good community contribution opportunities labels Dec 12, 2022
@yaitskov
Copy link

I struggle with that issue. I use Emacs keybindings, but menu still shows default shortcuts. So it is totally messed up!

@RichardFevrier
Copy link

Any Super/Meta keybind is ignored. Super/Meta + C to copy instead of Ctrl + C becomes just C.
On linux no matters xorg or wayland it is completely broken. Unusable for me.

@deepak1556
Copy link
Contributor

This requires patch in https://source.chromium.org/chromium/chromium/src/+/main:ui/base/accelerators/accelerator.cc;l=396-405 and cannot be made easily. @sbatten a workaround on our end would be better here.

@deepak1556 deepak1556 added chromium Issues and items related to Chromium and removed electron Issues and items related to Electron labels Dec 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Issue identified by VS Code Team member as probable bug chromium Issues and items related to Chromium confirmed Issue has been confirmed by VS Code Team member help wanted Issues identified as good community contribution opportunities linux Issues with VS Code on Linux menus Menu items and widget issues upstream Issue identified as 'upstream' component related (exists outside of VS Code)
Projects
None yet
Development

No branches or pull requests

6 participants