Keychron q6

[KeychronMacro]

Add new keyboard q6 which contains four sub versions.

Description

• Add keyboard rev_0160, ansi version of q6.
• Add keyboard rev_0161, ansi version of q6 and enable encoder ec11.
• Add keyboard rev_0162, iso version of q6.
• Add keyboard rev_0163, iso version of q6 and enable encoder ec11.

Types of Changes

• Core
• Bugfix
• New feature
• Enhancement/optimization
• Keyboard (addition or update)
• Keymap/layout/userspace (addition or update)
• Documentation

Issues Fixed or Closed by This PR

Checklist

• My code follows the code style of this project: C, Python
• I have read the PR Checklist document and have made the appropriate changes.
• My change requires a change to the documentation.
• I have updated the documentation accordingly.
• I have read the CONTRIBUTING document.
• I have added tests to cover my changes.
• I have tested the changes and verified that they work and don't break anything (as well as I can manage).

[DoppelKakao]

Hi, just bought a q6 and was wandering what the status of this pr is?

[eldorel]

Is there anything actually blocking this merge, or are we just waiting on someone to push the button at this point?

[tzarc]

Is there anything actually blocking this merge, or are we just waiting on someone to push the button at this point?

We're in discussion to help simplify long-term maintenance of the Keychron submissions. Bear with us.

[kroegerama]

Are you sure, these layouts (layout macro names) are correct? The Q6 is Fullsize (+F13) + 4 Buttons above the Numpad. Does not seem to be a standard layout you can find here: https://github.com/qmk/qmk_firmware/tree/master/layouts/default

Here is an image of the non-knob version (no F13): https://www.keychron.com/pages/keychron-q6-keycap-size

[eldorel]

Are you sure, these layouts (layout macro names) are correct? The Q6 is Fullsize (+F13) + 4 Buttons above the Numpad. Does not seem to be a standard layout you can find here: https://github.com/qmk/qmk_firmware/tree/master/layouts/default

Here is an image of the non-knob version (no F13): https://www.keychron.com/pages/keychron-q6-keycap-size

I can confirm that the Q6 is not a TKL layout. It has the layout shown in your link.
(and I'm typing this on one, so i'm pretty confident of that.)

It's basically fullsize_ISO/ansi with 4 additional keys above the numpad.

[kroegerama]

RGB_MATRIX_ENABLE is already checked in line 42

[KeychronMacro]

Changed, thanks.

[kroegerama]

I just flashed a build of this firmware (VIA version) on a Q6 ISO Knob. Now I am unable to remap the rotation of the encoder. Remapping of pressing the knob works, though.

Edit I was using this keymap: https://cdn.shopify.com/s/files/1/0059/0630/1017/files/Q6_UK_ISO_Knob_V1.2.json.zip

[kroegerama]

I was able to fix this by adding a proper definition of the encoder to the via json:
Q6_UK_ISO_Knob_V1.2-mod.zip

Result:

[adophoxia]

I was able to fix this by adding a proper definition of the encoder to the via json:
Q6_UK_ISO_Knob_V1.2-mod.zip

Result:

This was because they were using the action_exec hack that was used before encoder mapping to mimic encoder actions, which in this case, was probably why your knob wasn't working since the JSON used was for that hack, @kroegerama

[keychron-dev]

can this be merged now?