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

Search keyboard shortcut ⌘ + K not working in Dvorak #1521

Closed
gabalafou opened this issue Oct 18, 2023 · 4 comments · Fixed by #1525
Closed

Search keyboard shortcut ⌘ + K not working in Dvorak #1521

gabalafou opened this issue Oct 18, 2023 · 4 comments · Fixed by #1525
Labels
kind: bug Something isn't working

Comments

@gabalafou
Copy link
Collaborator

Steps to reproduce

The following steps will assume that your physical keyboard is labelled according to the QWERTY layout. For this test, it is important to keep the following key map in mind:

QWERTY Dvorak
K T
V K
  1. At the OS level, choose the Dvorak keyboard layout
  2. Using Chrome browser, go to PyData Theme
  3. Press control + V (Dvorak maps this key to the letter "K")
    Result: nothing
  4. Press control + K (Dvorak maps this key to the letter "T")
    Result: opens new browser tab
  5. On Mac, press control (not command) + K
    Result: opens search dialog

Expected behavior

Pressing control + the key mapped to "K" in Dvorak (rather than the key mapped to "K" in QWERTY) should open the search dialog.

@12rambau
Copy link
Collaborator

I have been using AZERTY (fr-iso) layout my all life and didn't experience any issues even though the k is 1 letter to the left of the en-iso mapping.

@12rambau 12rambau added kind: enhancement New feature or request needs: discussion Needs discussion before an implementation can be made kind: bug Something isn't working and removed kind: enhancement New feature or request needs: discussion Needs discussion before an implementation can be made labels Oct 18, 2023
@gabalafou
Copy link
Collaborator Author

Wait what? If I do an image search for AZERTY keyboards, I see the K in the exact same position as QWERTY. See for example the Wikipedia entry on AZERTY and compare with QWERTY. In both, I see that the K key is 8 keys to right of the caps lock key.

@12rambau
Copy link
Collaborator

I was counting the number of letters from "Enter" but you effectively have 1 extra key for special characters,

@gabalafou
Copy link
Collaborator Author

gabalafou commented Oct 18, 2023

Mystery solved. The K key on your keyboard gets reported as event.code === 'KeyK' (check your key's code), and as #1525 shows, the code handling the keyboard shortcut was erroneously using event.code (should use event.key instead).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind: bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants