When suspending, we want to set all LEDs off & sync first, and only then pause
LED operations. When the code was originally written, `LEDControl.syncLeds()`
synced LEDs even when `LEDControl.paused` was set - it no longer does so, so we
need to pause after we synced.
This addresses keyboardio/Kaleidoscope#516, at least partially.
Signed-off-by: Gergely Nagy <algernon@keyboard.io>
I hate to be that guy, but I was reading these on github's page when
deciding whether or not to order a keyboard.io, and the extra long lines
here are not automatically wrapped when browsing the repo on github.
Signed-off-by: Tycho Andersen <tycho@tycho.ws>
This adds the necessary code to support having five additional layers in EEPROM.
Also adds support for Focus, so these layers can be changed, and the default
layer set, too.
We also enable the EEPROM commands, which can be helpful in debugging and
backing up one's EEPROM contents.
Signed-off-by: Gergely Nagy <algernon@keyboard.io>
Since we use a dedicated function for the single magic combo we have, using an
enum in this case is just more confusing than if we didn't. For this reason,
drop the use of enum, and just specify the array directly.
Thanks @obra for the suggestion!
Signed-off-by: Gergely Nagy <algernon@keyboard.io>
Make use of the MagicCombo and USB-Quirks plugins in order to allow one to
toggle between Boot and NKRO mode of the keyboard, simply by pressing Left Fn +
Shift + Esc.
Signed-off-by: Gergely Nagy <algernon@keyboard.io>
It is not necessary anymore, as BootKeyboard supports wakeup itself now. As
such, also updated the comments regarding the plugin in
KALEIDOSCOPE_INIT_PLUGINS.
Signed-off-by: Gergely Nagy <algernon@keyboard.io>
Because the key two rows below `NumLock` is `Key_Quote` on the base layer too,
and is not a numpad-specific key either, make it transparent, so it does not get
highlighted erroneously.
Thanks to @ImmaculatePotato and Eddie Jinks for reporting the issue, and
proposing the fix.
Fixes#48.
Signed-off-by: Gergely Nagy <algernon@keyboard.io>
I added a couple items to the list of things that should be documented.
I also realize I have no idea what the ANY key does, but I'm intrigued. I don't understand from these code comments, and don't have time to puzzle it out now.
Otherwise, though, this is so much better than the file I've been poking at for weeks! It's clear and clean and much more welcoming. Thanks for all the behind the scenes work that made this possible, and thank you Jesse for the work writing up comments!
This lass is not fond of backslash, so she made the sketch use backslash
less, so much so that it is now backslash-less.
Signed-off-by: Csilla Nagyné Martinák <csilla@csillger.hu>
For a while now, `Kaleidoscope.use` does not require a NULL sentinel at
the end. This drops the sentinel instead of adding documentation that
would explain why that NULL is there.
Signed-off-by: Csilla Nagyné Martinák <csilla@csillger.hu>
