qmk/qmk_firmware

[Bug] When holding multiple mod-tap keys, only one gets registered as a modifier

Open

#13 346 ouverte le 26 juin 2021

Voir sur GitHub
 (2 commentaires) (0 réactions) (0 assignés)C (43 867 forks)batch import
bughelp wanted

Métriques du dépôt

Stars
 (20 368 stars)
Métriques de merge PR
 (Merge moyen 20j 9h) (27 PRs mergées en 30 j)

Description

Describe the Bug

I have home-row modifiers set up with mod-tap like so:

#define HM_A LALT_T(KC_A)
#define HM_R LGUI_T(KC_R)
#define HM_S LSFT_T(KC_S)
#define HM_T LCTL_T(KC_T)

My relevant configuration is as follows:

#define TAPPING_TERM 250
#define IGNORE_MOD_TAP_INTERRUPT
#define TAPPING_FORCE_HOLD
#define PERMISSIVE_HOLD

When I input this key sequence:

HM_R down, HM_S down, KC_SPACE down, KC_SPACE up, HM_S up, HM_R up

My computer sees the following:

gui down, s down, space down, space up, s up, gui up

But I would expect to see this:

gui down, shift down, space down, space up, shift up, gui up

Basically, I would expect the second mod-tap key to also be treated as a modifier since space was pressed and released between when it was pressed and released.

System Information

  • Keyboard: Keyboardio Atreus
    • Revision (if applicable):
  • Operating system: Arch Linux
  • AVR GCC version: 11.1.0
  • ARM GCC version: 11.1.0
  • QMK Firmware version: 0.13.2
  • Any keyboard related software installed?
    • AutoHotKey
    • Karabinerk
    • Other:

Guide contributeur

Direction de recherche
Étudiez l'implémentation de mod tap dans le firmware QMK, en particulier dans des fichiers comme quantum/process keycode/process mod tap.c. Le problème suggère que lorsque plusieurs touches mod tap sont maintenues enfoncées, une seule est enregistrée comme modificateur. Examinez comment le firmware gère les interruptions et les événements de maintien. Vérifiez les options de configuration comme IGNORE MOD TAP INTERRUPT et PERMISSIVE HOLD. Le comportement attendu est que les deux modificateurs soient enregistrés lorsqu'une touche espace est pressée entre eux. Aucune PR n'est liée, donc la correction nécessiterait de modifier la logique pour autoriser plusieurs maintiens mod tap simultanés.
Stack technique
c
Domaine
backend
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.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Une demi journée
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
CGit
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
55