qmk/qmk_firmware

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

Open

#9230 aperta il 29 mag 2020

Vedi su GitHub
 (8 commenti) (2 reazioni) (0 assegnatari)C (43.867 fork)batch import
bughelp wanted

Metriche repository

Star
 (20.368 star)
Metriche merge PR
 (Merge medio 20g 9h) (27 PR mergiate in 30 g)

Descrizione

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.

Guida contributor