Configurator fixes for handwired/concertina/64key

[veikman]

Description

Fixes and enhancements for the use of QMK Configurator specifically with the Concertina, following a tip by @noroadsleft.

Types of Changes

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

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).

[noroadsleft]

Does this look correct? (Disregarding that I have QWERTY assignments here instead of Colemak)

[veikman]

Yes, that’s correct as far as Configurator is able to render what’s really going on under Colemak. The biggest invisible customization is this snippet from keyboards/handwired/concertina/64key/keymaps/default/config.h:

#define LCPO_KEYS KC_LCTL, KC_TRNS, KC_LBRACKET
#define RCPC_KEYS KC_RCTL, KC_TRNS, KC_RBRACKET
#define LAPO_KEYS KC_LALT, KC_LSFT, KC_LBRACKET
#define RAPC_KEYS KC_RALT, KC_RSFT, KC_RBRACKET

Those custom space-cadet-style modifier-and-tap combinations all render as /( and /), but are really brackets and braces. I know of no better (i.e. renderable) way to implement the functionality.

[noroadsleft]

Ah, Configurator's API doesn't have any support for being able to change the default Space Cadet keys, which are:

#define LCPO_KEYS KC_LCTL, KC_LSFT, KC_9
#define RCPC_KEYS KC_RCTL, KC_RSFT, KC_0
#define LAPO_KEYS KC_LALT, KC_LSFT, KC_9
#define RAPC_KEYS KC_RALT, KC_RSFT, KC_0

[noroadsleft]

Thanks!