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

Keybinding not mapped correctly depending on Keyboard Layout #13874

Open
Tracked by #13192
jfaltermeier opened this issue Jul 2, 2024 · 4 comments
Open
Tracked by #13192

Keybinding not mapped correctly depending on Keyboard Layout #13874

jfaltermeier opened this issue Jul 2, 2024 · 4 comments

Comments

@jfaltermeier
Copy link
Contributor

Bug Description:

Keybindings including AltGr do not work (on German keyboard layout at least)

Steps to Reproduce:

  1. Use German keyboard layout
  2. Add a keybinding for e.g. "Ctrl+["
  3. Keybinding is not working. In the UI it shows as "Ctrl+ü", which actually works. (I think ü is [ on US layout)
  4. When I click to edit the binding again, it shows "Ctrl+[" though.

Additional Information

I'm not sure if there is a general issue with detecting Alt Gr/Figuring out the required keys for each layout.
However I think we shouldn't map to wrong keybindings at least.

  • Operating System: Ubuntu
  • Theia Version: master
@JonasHelming JonasHelming mentioned this issue Jul 3, 2024
63 tasks
@rschnekenbu
Copy link
Contributor

I don't know if this is related, but I get a lot of errors at startup about key bindings: [1175888:0709/134222.061193:ERROR:accelerator_util.cc(23)] The accelerator string can only contain ASCII characters, invalid string: "Ctrl+Shift+à"
I can reproduce the issue with a French keyboard too, with the keybindings containing ².

@tsmaeder
Copy link
Contributor

tsmaeder commented Jul 17, 2024

Any keybinding to "ä" in a swiss-german keyboard will provoke the above message, as well. If we can't support those accelerators, we should at least catch them from going to the electron code.
Btw: VS Code can handle accelerators like "Ctrl+§" no Problem, so it must be possible somehow.

@tYYGH
Copy link

tYYGH commented Aug 24, 2024

Same issue for me. France has 2 official layouts: AZERTY and BÉPO. I’m using the BÉPO layout, on Archlinux.
With it:

  • Ctrl+z (i.e. left Ctrl + ^ physical key) does an outdent instead of undo (they indeed share the same shortcut), but even removing all Ctrl+z bindings except the one for undo does not let me use the undo feature.
  • Ctrl+w (i.e. left Ctrl + $ physical key) does an indent instead of closing the tab; same problem as undo.

It’s my first day using Theia, and so far I did not use other key-bindings (except Ctrl+Shift+P —physical key E—, which works). So I tried to open the keybinding (Ctrl+Alt+,, i.e. left Ctrl and Alt + G physical key) and it works. Maybe those that work are those associated to an alphabetical physical keys…

For your information, I explicitly specify “left” above, because on French keyboards, left and right Alt keys have a totally different role, and are never interchangeable.

@sgraband
Copy link
Contributor

sgraband commented Nov 5, 2024

To add to this:
I am using a german keyboard layout and the following keybinding do also not work:

  • Ctrl + + does not zoom in
  • Ctrl + Shift + ´ does not open the terminal but rather Ctrl + Shift + ^ does

These issues seem to only occur in the electron variant.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants