qmk/qmk_firmware

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

Open

#9.230 aberto em 29 de mai. de 2020

Ver no GitHub
 (8 comments) (2 reactions) (0 assignees)C (43.867 forks)batch import
bughelp wanted

Métricas do repositório

Stars
 (20.368 stars)
Métricas de merge de PR
 (Mesclagem média 20d 9h) (27 fundiu PRs em 30d)

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.

Guia do colaborador

Direção de pesquisa
Inspecione a implementação do RETRO TAPPING no QMK para entender por que o ALT T não o respeita, enquanto o SFT T e o CTL T respeitam.
Pilha de tecnologia
c
Domain
backend
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
GitC
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
75