qmk / qmk_firmware

Open-source keyboard firmware for Atmel AVR and Arm USB families
https://qmk.fm
GNU General Public License v2.0
18.09k stars 38.89k forks source link

[Bug] Keychron Q6: F keys in macro not working #19311

Closed mrtnRitter closed 1 year ago

mrtnRitter commented 1 year ago

Describe the Bug

When a macro contains a F key, the macro is not send.

Steps to reproduce: Fork this repo, compile using qmk compile -kb keychron/q6/iso_encoder -km keychron, open VIA, add macro {KC_LALT,KC_F4}, assign macro to a key, press the key and nothing happens.

Further investigation:

Keyboard Used

keychron/q6/iso_encoder

Link to product page (if applicable)

No response

Operating System

Windows 10

qmk doctor Output

Ψ QMK Doctor is checking your environment.
Ψ CLI version: 1.1.1
Ψ QMK home: C:/Users/Martin/Documents/GitHub/qmk_firmware
Ψ Detected Windows 10 (10.0.19045).
Ψ Git branch: master
⚠ The official repository does not seem to be configured as git remote "upstream".
Ψ All dependencies are installed.
Ψ Found arm-none-eabi-gcc version 10.1.0
Ψ Found avr-gcc version 8.5.0
Ψ Found avrdude version 7.0
Ψ Found dfu-util version 0.11
Ψ Found dfu-programmer version 0.7.2
Ψ Submodules are up to date.
Ψ Submodule status:
Ψ - lib/chibios: 2022-09-18 10:01:17 +0000 --  (0e9d558b5)
Ψ - lib/chibios-contrib: 2022-10-03 18:09:41 +0200 --  (bb8356fb)
Ψ - lib/googletest: 2021-06-11 06:37:43 -0700 --  (e2239ee6)
Ψ - lib/lufa: 2022-08-26 12:09:55 +1000 --  (549b97320)
Ψ - lib/pico-sdk: 2022-09-19 18:02:44 +0200 --  (8d56ea3)
Ψ - lib/printf: 2022-06-29 23:59:58 +0300 --  (c2e3b4e)
Ψ - lib/vusb: 2022-06-13 09:18:17 +1000 --  (819dbc1)
Ψ QMK is ready to go, but minor problems were found

Is AutoHotKey / Karabiner installed

Other keyboard-related software installed

No response

Additional Context

No response

tzarc commented 1 year ago

Likely a bug in VIA, not QMK. As per the reply on your reddit post (here) it seems VIA hasn't fully updated to the new keycode definitions within QMK.

Suggest posting an issue on their GitHub repo instead.

mrtnRitter commented 1 year ago

Thanks to your help on Reddit, I can confirm that the macro works fine when implemented in QMK keyboard.c. This bug report is forwared to VIA.

Thanks!