qmk/qmk_firmware

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

Open

#9,230 建立於 2020年5月29日

在 GitHub 查看
 (8 留言) (2 反應) (0 負責人)C (43,867 fork)batch import
bughelp wanted

倉庫指標

Star
 (20,368 star)
PR 合併指標
 (平均合併 20天 9小時) (30 天內合併 27 個 PR)

描述

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.

貢獻者指南