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

Undo shortcut - wrong hotkey mapping #1477

Closed
peet86 opened this issue Apr 1, 2020 · 3 comments
Closed

Undo shortcut - wrong hotkey mapping #1477

peet86 opened this issue Apr 1, 2020 · 3 comments
Labels
waiting-for-info Waiting for more information from submitter

Comments

@peet86
Copy link

peet86 commented Apr 1, 2020

code-server version: 3.0
OS Version: macOS
Browser: Safari

Description
On certain keyboard layouts chars are not mapped to the same key like on the English one.
Example: Z and Y keys are switched on DE/HU/etc. layouts.

It seems to me you are using key codes to catch the pressed shortcuts.. which is an issue for all users who try to use different layouts than EN. Very popular key combinations like CMD+Z (undo) are mapped to wrong keys (CMD+Y in this case)..

Steps to Reproduce
Use a DE layout keyboard / change in your OS to another layout and use the Keyboard Viewer utility

@peet86 peet86 changed the title Undo shortcut - wrong key mapping Undo shortcut - wrong hotkey mapping Apr 1, 2020
@peet86
Copy link
Author

peet86 commented Apr 1, 2020

Just to make it clear: the mapping in vscode seems to be correct, but on a DE/HU/etc. keyboard I need to press CMD+Y to tigger undo.

{ "key": "cmd+z",                 "command": "editor.action.customEditor.undo",
                                     "when": "focusedCustomEditorIsEditable && !inputFocus" },

@code-asher
Copy link
Member

code-asher commented Apr 1, 2020

I had a similar problem (with local VS Code as well) and changing the keyboard.dispatch setting to keyCode fixed it for me. The default is code which is always the same no matter what the layout is.

@code-asher code-asher added the waiting-for-info Waiting for more information from submitter label Apr 7, 2020
@kylecarbs
Copy link
Member

Closing for now. Let us know if that doesn't fix the issue @peet86 !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
waiting-for-info Waiting for more information from submitter
Projects
None yet
Development

No branches or pull requests

3 participants