Hub16 - Bug removal + clean up code (#8227)

* bugfixes + cleaned up code

* typo

* Update keyboards/hub16/keymaps/no_mod/keymap.c

Co-Authored-By: Ryan <fauxpark@gmail.com>

Co-authored-by: Ryan <fauxpark@gmail.com>

1 files changed, 12 insertions, 7 deletions

diff --git a/keyboards/hub16/keymaps/default/keymap.c b/keyboards/hub16/keymaps/default/keymap.c
index a77c8bc78..2d12dba0d 100755
--- a/keyboards/hub16/keymaps/default/keymap.c
+++ b/keyboards/hub16/keymaps/default/keymap.c
@@ -18,16 +18,22 @@
 // Function key we are 'wrapping' usual key presses in
 #define KC_WRAP KC_F24
 
+// Keyboard Layers
+enum keyboard_layers{
+    _BASE = 0,
+    _CONTROL
+};
+
 // Tap Dance Declarations
-enum { TD_TO_LED = 0, TD_TO_DEFAULT = 1 };
+enum tap_dance { TD_TO_LED = 0, TD_TO_DEFAULT = 1 };
 
 qk_tap_dance_action_t tap_dance_actions[] = {
-    // Tap once for HID, twice to toggle layer 1
-    [TD_TO_LED]     = ACTION_TAP_DANCE_DUAL_ROLE(_______, 1),
-    [TD_TO_DEFAULT] = ACTION_TAP_DANCE_DUAL_ROLE(_______, 0)};
+    // Tap once for standard key, twice to toggle to control layer
+    [TD_TO_LED]     = ACTION_TAP_DANCE_DUAL_ROLE(KC_P, _CONTROL),
+    [TD_TO_DEFAULT] = ACTION_TAP_DANCE_DUAL_ROLE(KC_P, _BASE)};
 
 const uint16_t PROGMEM keymaps[][MATRIX_ROWS][MATRIX_COLS] = {
-  [0] = LAYOUT( /* Base */
+  [_BASE] = LAYOUT( /* Base */
         KC_S,      KC_V,    
     KC_A, KC_B, KC_C, KC_D, 
     KC_E, KC_F, KC_G, KC_H, 
@@ -35,7 +41,7 @@ const uint16_t PROGMEM keymaps[][MATRIX_ROWS][MATRIX_COLS] = {
     KC_M, KC_N, KC_O, TD(TD_TO_LED) 
   ),
 
-  [1] = LAYOUT( /* LED Control */
+  [_CONTROL] = LAYOUT( /* LED Control */
           KC_NO,            KC_NO,        
     _______, RGB_MOD, RGB_RMOD, RGB_TOG,
     RGB_VAD, RGB_VAI, RGB_HUD,  RGB_HUI, 
@@ -72,7 +78,6 @@ void encoder_update_user(uint8_t index, bool clockwise) {
 
 // Below stolen from TaranVH (https://github.com/TaranVH/2nd-keyboard/blob/master/HASU_USB/F24/keymap.c)
 // Shoutout to drashna on the QMK discord for basically writing this for me.... :P
-
 bool process_record_user(uint16_t keycode, keyrecord_t *record) {
     static uint8_t f24_tracker;
     switch (keycode) {