solidjs-community/solid-primitives

[keyboard] Pressing Command-[key] breaks key sequences & shortcuts

Ouverte

#553 ouverte le 23 déc. 2023

Voir sur GitHubGuide contributeur
 (3 commentaires) (4 réactions) (0 personne assignée)TypeScript (153 forks)auto 404
help wanted

Métriques du dépôt

Stars
 (1 541 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

Description

Describe the bug

When using useKeyDownSequence and createShortcut if you at any time use a shortcut such as Command-A, they will stop working as expected. This issue can be observed on the example provided in the keyboard package's readme.

  1. Press Command + A.
  2. Observe that the A stays in "Pressed Keys".
  3. Press other keys and see they are added to the list with A. Notice the Control + E + R example does not work anymore.
  4. Blur the window and notice the A is removed. Pressing A will also remove it.

I've observed this behavior on MacOS Chrome and Safari, have not tried on Windows.

Minimal Reproduction Link

https://codesandbox.io/s/solid-primitives-keyboard-demo-s2l84k?file=/index.tsx

Guide contributeur

Direction de recherche
Examinez comment la touche Commande (Meta) interrompt la détection des séquences de touches dans le package clavier. Vérifiez handleKeyDown et handleKeyUp, en particulier la logique de suivi des touches enfoncées et de correspondance des séquences.
Stack technique
typescriptsolidjs
Domaine
frontend
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
TypeScriptSolid.js
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70