From e4cc642a79492ee5b04223c970a3ae9f0d22fba6 Mon Sep 17 00:00:00 2001 From: Joshua Moses Diamond Date: Fri, 10 Apr 2020 20:10:37 -0400 Subject: Some refinements of 75_ansi/spidey3 layout (#8581) * First cut at Josh Diamond's KBD75 customizations. Includes: * My unique keymap with ChromeOS specific keys * Use RGB underglow to indicate Caps Lock * Some unicode bindings * Some changes to make debugging easier * Updated spidey3 to be applicable to all 75_ansi boards * Sadly, ChromeOS doesn't pay attention to most consumer codes * Add mac layer; fix flakeyness in CAPS_LOCK underglow. * Make layers.json match the keymap (to the extent possible) * Major cleanup; fix broken debug persistence * Cleanup some whitespace issues * Fix incorrect log message. * Rework layer indication to user RGBLIGHT_LAYERS * Update layouts/community/75_ansi/spidey3/keymap.c Co-Authored-By: Drashna Jaelre * Rename users/spidey3/rgblight.c to layer_rgb.c per suggestion * Refactor to use set_single_persistant_default_layer(). * Use dprint/f to make logging more elegant. * Update users/spidey3/config.h Co-Authored-By: Drashna Jaelre * Update users/spidey3/config.h Co-Authored-By: Drashna Jaelre * Update layouts/community/75_ansi/spidey3/rules.mk Co-Authored-By: Ryan * Update users/spidey3/spidey3.c Co-Authored-By: Ryan * Update users/spidey3/layer_rgb.c Co-Authored-By: Ryan * Update users/spidey3/init.c Co-Authored-By: Ryan * Changes from code review * Numpad layer, various keys for 75_ansi/spidey3 * Add Fn-B to toggle NKRO * Blink rgb to acknowledge some setting changes * Updated media control & reset key location * Minor cleanup Co-authored-by: Joshua Diamond Co-authored-by: Drashna Jaelre Co-authored-by: Ryan --- users/spidey3/init.c | 15 ++++++++-- users/spidey3/layer_rgb.c | 76 +++++++++++++++++++++++++++++++++++++++-------- users/spidey3/spidey3.c | 18 +++++------ users/spidey3/spidey3.h | 5 +++- 4 files changed, 88 insertions(+), 26 deletions(-) (limited to 'users/spidey3') diff --git a/users/spidey3/init.c b/users/spidey3/init.c index a4be6113f4..b923946093 100644 --- a/users/spidey3/init.c +++ b/users/spidey3/init.c @@ -1,15 +1,15 @@ #include "spidey3.h" void keyboard_post_init_user(void) { - print("SPIDEY3: keyboard_post_init_user\n"); - uprintf(" debug_enable=%u\n", debug_enable); + print("keyboard_post_init_user\n"); + uprintf("\tdebug_enable=%u\n", debug_enable); #ifdef RGBLIGHT_ENABLE keyboard_post_init_user_rgb(); #endif } void eeconfig_init_user(void) { - print("SPIDEY3: eeconfig_init_user\n"); + print("eeconfig_init_user\n"); set_single_persistent_default_layer(_BASE); #ifdef UNICODEMAP_ENABLE eeconfig_init_user_unicode(); @@ -19,3 +19,12 @@ void eeconfig_init_user(void) { eeconfig_init_user_rgb(); #endif } + +void shutdown_user() { +#ifdef RGBLIGHT_ENABLE + clear_rgb_layers(); + rgblight_enable(); + rgblight_mode_noeeprom(RGBLIGHT_MODE_STATIC_LIGHT); + rgblight_sethsv_noeeprom(HSV_RED); +#endif +} diff --git a/users/spidey3/layer_rgb.c b/users/spidey3/layer_rgb.c index d80eb20a31..75e66384e2 100644 --- a/users/spidey3/layer_rgb.c +++ b/users/spidey3/layer_rgb.c @@ -10,7 +10,7 @@ uint8_t rgb_val; bool rgb_saved = 0; void spidey_swirl(void) { - dprint("SPIDEY3: Setting Spidey Swirl!\n"); + dprint("Setting Spidey Swirl!\n"); rgblight_enable(); rgblight_mode(RGBLIGHT_MODE_RAINBOW_SWIRL); rgblight_sethsv(213, 255, 128); @@ -27,54 +27,104 @@ void eeconfig_init_user_rgb(void) const rgblight_segment_t PROGMEM _capslock_layer[] = RGBLIGHT_LAYER_SEGMENTS( {0, 2, HSV_AZURE}, {14, 2, HSV_AZURE} ); const rgblight_segment_t PROGMEM _layer1_layer[] = RGBLIGHT_LAYER_SEGMENTS( {7, 1, HSV_PURPLE} ); -const rgblight_segment_t PROGMEM _layer2_layer[] = RGBLIGHT_LAYER_SEGMENTS( {9, 2, HSV_GREEN} ); +const rgblight_segment_t PROGMEM _layer2_layer[] = RGBLIGHT_LAYER_SEGMENTS( {10, 3, HSV_MAGENTA} ); +const rgblight_segment_t PROGMEM _layer3_layer[] = RGBLIGHT_LAYER_SEGMENTS( {10, 1, HSV_GREEN} ); +const rgblight_segment_t PROGMEM _yes_layer[] = RGBLIGHT_LAYER_SEGMENTS( {9, 6, HSV_GREEN} ); +const rgblight_segment_t PROGMEM _no_layer[] = RGBLIGHT_LAYER_SEGMENTS( {9, 6, HSV_RED} ); // Now define the array of layers. Later layers take precedence -const rgblight_segment_t* const PROGMEM _rgb_layers[] = RGBLIGHT_LAYERS_LIST( _capslock_layer, _layer1_layer, _layer2_layer ); +const rgblight_segment_t* const PROGMEM _rgb_layers[] = + RGBLIGHT_LAYERS_LIST( _capslock_layer, _layer1_layer, _layer2_layer, _layer3_layer, _yes_layer, _no_layer ); const uint8_t PROGMEM _n_rgb_layers = sizeof(_rgb_layers) / sizeof(_rgb_layers[0]) - 1; +void clear_rgb_layers() { + for (uint8_t i=0; i<_n_rgb_layers; i++) { + rgblight_set_layer_state(i, false); + } +} + void do_rgb_layers(layer_state_t state, uint8_t start, uint8_t end) { - dprint("SPIDEY3: do_rgb_layers()\n"); - for (uint8_t i=start; ievent.pressed; switch (keycode) { case SPI_GLO: - if (record->event.pressed) { + if (pressed) { spidey_swirl(); } - break; + return false; + + // Acks follow... + case DEBUG: + rgb_layer_ack(debug_enable, pressed); + return false; + + case SPI_LNX: + case SPI_OSX: + case SPI_WIN: + rgb_layer_ack(true, pressed); + return false; + + // Tricky! + // For these, on press the toggle hasn't happened yet, + // so we need a little logic to invert, assuming that + // on key press the flag WILL be toggled, and on key + // release the flag has already been toggled. + +#ifdef VELOCIKEY_ENABLE + case VLK_TOG: + rgb_layer_ack(pressed != velocikey_enabled(), pressed); + return true; +#endif + +#ifdef NKRO_ENABLE + case NK_TOGG: + case NK_ON: + case NK_OFF: + rgb_layer_ack(pressed != keymap_config.nkro, pressed); + return true; +#endif } return true; diff --git a/users/spidey3/spidey3.c b/users/spidey3/spidey3.c index d0b7947043..8be99dea04 100644 --- a/users/spidey3/spidey3.c +++ b/users/spidey3/spidey3.c @@ -20,39 +20,39 @@ bool process_record_user(uint16_t keycode, keyrecord_t *record) { } eeconfig_update_debug(debug_config.raw); #endif - return false; + break; case SPI_LNX: - dprint("SPIDEY3: SPI_LNX\n"); + dprint("SPI_LNX\n"); set_single_persistent_default_layer(_BASE); layer_off(_OSX); #if defined(UNICODE_ENABLE) || defined(UNICODEMAP_ENABLE) set_unicode_input_mode(UC_LNX); #endif - return false; + break; case SPI_OSX: - dprint("SPIDEY3: SPI_OSX\n"); + dprint("SPI_OSX\n"); set_single_persistent_default_layer(_OSX); #if defined(UNICODE_ENABLE) || defined(UNICODEMAP_ENABLE) set_unicode_input_mode(UC_OSX); #endif - return false; + break; case SPI_WIN: - dprint("SPIDEY3: SPI_WIN\n"); + dprint("SPI_WIN\n"); set_single_persistent_default_layer(_BASE); layer_off(_OSX); #if defined(UNICODE_ENABLE) || defined(UNICODEMAP_ENABLE) set_unicode_input_mode(UC_WINC); #endif - return false; + break; } } #ifdef RGBLIGHT_ENABLE bool res = process_record_user_rgb(keycode, record); - if (!res) return false; + if (res) return true; #endif - return true; + return false; } layer_state_t default_layer_state_set_user(layer_state_t state) { diff --git a/users/spidey3/spidey3.h b/users/spidey3/spidey3.h index 125f70a9ac..224858391d 100644 --- a/users/spidey3/spidey3.h +++ b/users/spidey3/spidey3.h @@ -9,12 +9,14 @@ enum userspace_layers { _BASE = 0, _OSX, + _NUMPAD, _FN, }; enum rgb_base_layer { RGB_LAYER_BASE_DEFAULT = _BASE, - RGB_LAYER_BASE_REGULAR = _FN, + RGB_LAYER_BASE_REGULAR = _NUMPAD, + RGB_LAYER_BASE_ACKS = _FN+1, }; enum custom_keycodes { @@ -32,6 +34,7 @@ bool process_record_user_rgb(uint16_t keycode, keyrecord_t *record); layer_state_t layer_state_set_user_rgb(layer_state_t state); layer_state_t default_layer_state_set_user_rgb(layer_state_t state); bool led_update_user_rgb(led_t led_state); +void clear_rgb_layers(void); #endif #ifdef UNICODEMAP_ENABLE -- cgit v1.2.1