qmk/qmk_firmware

[Bug] `RETRO_TAPPING` does not work with `ALT_T(*)`

Open

#9 230 ouverte le 29 mai 2020

Voir sur GitHub
 (8 commentaires) (2 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

Even when RETRO_TAPPING is defined, ALT_T(*) works as same as without RETRO_TAPPING.

I reproduced this issue as follows:

  • Add the following to crkbd/keymaps/default/config.h:
    #define RETRO_TAPPING
    
  • Add the following to crkbd/keymaps/default/keymap.c:
    #define KC_SandA ALT_T(KC_SPC)
    
  • Replace KC_RALT with KC_SandA to _QWERTY layer in crkbd/keymaps/default/keymap.c.
  • Compile and write the firmware to crkbd
  • Press the KC_SandA key over TAPPING_TERM and release it.
    • Expected behavior: A space is sent (we will see a space).
    • Actual behavior: Alt is sent (I saw that a window menu is focused on) rather than a space is sent.

System Information

  • Keyboard: crkbd
    • Revision (if applicable): default
  • Operating system: Windows 10
  • AVR GCC version: avr-gcc (GCC) 5.4.0
  • ARM GCC version: arm-none-eabi-gcc (15:6.3.1+svn253039-1build1) 6.3.1 20170620
  • QMK Firmware version: 0.8.192
  • Any keyboard related software installed?
    • AutoHotKey
    • Karabiner
    • Other: Nothing

Additional Context

I confirmed that SFT_T and CTL_T work as intended.

Guide contributeur