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

[BUG] After upgrading to 4.4.3 some shortcuts stopped working #1110

Closed
damusnet opened this issue Jan 4, 2024 · 5 comments
Closed

[BUG] After upgrading to 4.4.3 some shortcuts stopped working #1110

damusnet opened this issue Jan 4, 2024 · 5 comments
Labels
bug Something isn't working

Comments

@damusnet
Copy link

damusnet commented Jan 4, 2024

Describe the bug
shit+? and shift+plus,shift+bracketright used to work, now they don't

To Reproduce
Steps to reproduce the behavior:

  1. Go to CodeSandbox repro
  2. Make sure to open the console
  3. Try hitting shift+?, shift+plus, shift+minus (with + and - keys respectively)
  4. Only shift+minus will log to the console

Expected behavior
Existing shortcuts should still work

Screenshots
image

Desktop (please complete the following information):

  • OS: macOS
  • Browser: chrome
  • Version: Version 120.0.6099.129 (Official Build) (arm64)
@JohannesKlauss JohannesKlauss added the bug Something isn't working label Jan 4, 2024
@rawpixel-vincent
Copy link

same here, commenting out that line:

 // if (!(keys != null && keys.includes(keyCode)) && !['ctrl', 'control', 'unknown', 'meta', 'alt', 'shift', 'os'].includes(keyCode)) {
  //   console.log("elmoo");
  //   return false;
  // }

in isHotkeyMatchingKeyboardEvent()
fix the issue, if that helps

happens is firefox / chrome - linux and macos

I've managed to log that before this condition execute, and I'm not sure why "right" is in keys, maybe some memoization issue 🤷🏻
{"pressedKey":"delete","keyCode":"","keys":["right"]}

downgrading to 4.4.1 fixed the issue, all good 👍🏻

@JohannesKlauss
Copy link
Owner

Hmh, weird. I'll have a look into it.

@JohannesKlauss
Copy link
Owner

Weirdly enough only Shift+? works for me, the other two don't register.

@JohannesKlauss
Copy link
Owner

Fixed in 4.4.4

@damusnet
Copy link
Author

Thank you for looking into this.

After updating the CodeSandbox to 4.4.4: shift+? and shift+minus now both work, but shift+plus still doesn't.

Note also that on a extended keyboard with separate numpad, neither shift+plus nor shift+minus work.

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

No branches or pull requests

3 participants