fix: disable reset to default layout by timeout

Author: meskill

keyboards/splitkb/kyria/keymaps/meskill/config.h

@@ -33,7 +33,6 @@
 #ifdef RGBLIGHT_ENABLE
     // hardware settings
     #define RGBLIGHT_SLEEP
-    #define RGB_DISABLE_WHEN_USB_SUSPENDED
     #define RGBLIGHT_TIMEOUT 600000
     // only underglow leds
     #undef RGBLIGHT_LED_COUNT

keyboards/splitkb/kyria/keymaps/meskill/features/lang_switch.c

@@ -17,17 +17,10 @@
 #include "lang_switch.h"
 #include "debug.h"
 
-static uint32_t timer = 0;
-
-void reset_lang_switch_timer(void) {
-    timer = timer_read32() + LANG_SWITCH_TIMEOUT;
-}
-
 bool process_lang_switch(uint16_t keycode, keyrecord_t *record) {
     bool down = record->event.pressed;
     int mods = get_mods() | get_oneshot_mods();
 
-    reset_lang_switch_timer();
     uint16_t lang_layer_keycode = keymap_key_to_keycode(LANG_SWITCH_LAYER, record->event.key);
 
     if (keycode != lang_layer_keycode || !mods || mods & MOD_MASK_SHIFT) {
@@ -48,12 +41,3 @@ bool process_lang_switch(uint16_t keycode, keyrecord_t *record) {
 
     return true;
 }
-
-void lang_switch_task(void) {
-    if (timer_expired32(timer_read32(), timer)) {
-        dprint("lang_switch_timer expired\n");
-        layer_move(0);
-        SWTC_EN();
-        reset_lang_switch_timer();
-    }
-}

keyboards/splitkb/kyria/keymaps/meskill/features/lang_switch.h

@@ -19,10 +19,8 @@
 #include QMK_KEYBOARD_H
 #include "layout.h"
 
-#ifndef LANG_SWITCH_TIMEOUT
-#    define LANG_SWITCH_TIMEOUT 600000 // default is 10 minutes
-#endif
-
+// Processes the modifier keys for hotkeys when using the additional language layer
+// by converting the keycode to the keycode of the base layer.
+// That fixes the issues for some programs that separate hotkeys by language and not just by keycode.
+//
 bool process_lang_switch(uint16_t keycode, keyrecord_t *record);
-
-void lang_switch_task(void);

keyboards/splitkb/kyria/keymaps/meskill/keymap.c

@@ -41,7 +41,6 @@ void keyboard_post_init_user(void) {
 
 void matrix_scan_user(void) {
     lang_word_task();
-    lang_switch_task();
 }
 
 bool process_record_user(uint16_t keycode, keyrecord_t *record) {

keyboards/splitkb/kyria/keymaps/meskill/readme.md

@@ -3,6 +3,5 @@
 ## Notable features
 
 - **Two languages support** - support english and russian language as separate layers to make punctuation consistent between languages. Also, implement the language switch with language in the OS. Full support, currently will work only on Windows after installing [these keyboard layouts](https://github.com/meskill/settings-windows/tree/master/keyboard/layouts)
-- **Lang switch** - automatically switch from russian to english timeout 
 - **Lang word** - feature similar to CapsWord that allows to switch temporarily to another language to type single word and after typing switches back to the original language
 - rgb lighting based on layers with visual markers

keyboards/splitkb/kyria/keymaps/meskill/rgb.c

@@ -24,7 +24,7 @@ enum {
     // and I need to overwrite ru layer with en so it should be higher in rgb sense
     RGB_RU = EN,
     RGB_EN = RU,
-    RGB_DBG = MDA + 1,
+    RGB_DBG = STP + 1,
     RGB_CWRD,
     RGB_RESET
 };
@@ -145,7 +145,7 @@ void layer_state_set_rgb(layer_state_t state) {
 }
 
 static uint32_t rgb_timer;
-bool is_rgb_timeout = false;
+static bool is_rgb_timeout = false;
 
 void refresh_rgb(void) {
     rgb_timer = timer_read32();