From 213066915a024dbd95d63fc9addd23f4b90317b6 Mon Sep 17 00:00:00 2001
From: vnegnev <4116364+vnegnev@users.noreply.github.com>
Date: Sun, 23 Jun 2024 23:49:08 +0100
Subject: [PATCH] Put new keycodes at the end to avoid breaking keymaps; add
 comments

---
 keyboards/svalboard/keymaps/keymap_support.h |  5 +++--
 keyboards/svalboard/keymaps/vial/vial.json   | 21 ++++++++++----------
 2 files changed, 14 insertions(+), 12 deletions(-)

diff --git a/keyboards/svalboard/keymaps/keymap_support.h b/keyboards/svalboard/keymaps/keymap_support.h
index 7c1d30eb1fc..9c065776c31 100644
--- a/keyboards/svalboard/keymaps/keymap_support.h
+++ b/keyboards/svalboard/keymaps/keymap_support.h
@@ -28,8 +28,6 @@ enum my_keycodes {
     SV_RIGHT_DPI_DEC,
     SV_LEFT_SCROLL_TOGGLE,
     SV_RIGHT_SCROLL_TOGGLE,
-    SV_LEFT_SCROLL_HOLD,
-    SV_RIGHT_SCROLL_HOLD,
     SV_RECALIBRATE_POINTER,
     SV_MH_CHANGE_TIMEOUTS,
     SV_CAPS_WORD,
@@ -39,6 +37,9 @@ enum my_keycodes {
     SV_SNIPER_2,
     SV_SNIPER_3,
     SV_SNIPER_5,
+    SV_LEFT_SCROLL_HOLD,
+    SV_RIGHT_SCROLL_HOLD,
+    // New keycodes should go here, to avoid breaking existing keymaps - order must match vial.json
     KC_NORMAL_HOLD = SAFE_RANGE,
     KC_FUNC_HOLD,
     SV_SAFE_RANGE, // Keycodes over this are safe on Svalboard.
diff --git a/keyboards/svalboard/keymaps/vial/vial.json b/keyboards/svalboard/keymaps/vial/vial.json
index 6a6ebcf4d22..208ef37db67 100644
--- a/keyboards/svalboard/keymaps/vial/vial.json
+++ b/keyboards/svalboard/keymaps/vial/vial.json
@@ -36,16 +36,6 @@
         "title": "Toggle if the right pointer is scroll or pointer.",
         "name": "SV_RIGHT_SCROLL_TOGGLE"
       },
-      {
-        "shortName": "Scroll\nLeft\nHold",
-        "title": "Hold to make the left pointing device scroll",
-        "name": "SV_LEFT_SCROLL_HOLD"
-      },
-      {
-        "shortName": "Scroll\nRight\nHold",
-        "title": "Hold to make the right pointing device scroll",
-        "name": "SV_RIGHT_SCROLL_HOLD"
-      },
       {
         "shortName": "Fix\nDrift",
         "title": "Reset the calibration of the pointing device",
@@ -90,7 +80,18 @@
         "shortName": "Sniper\n5x",
         "title": "Slow the cursor 5x more.",
         "name": "SV_SNIPER_5"
+      },
+      {
+        "shortName": "Scroll\nLeft\nHold",
+        "title": "Hold to make the left pointing device scroll",
+        "name": "SV_LEFT_SCROLL_HOLD"
+      },
+      {
+        "shortName": "Scroll\nRight\nHold",
+        "title": "Hold to make the right pointing device scroll",
+        "name": "SV_RIGHT_SCROLL_HOLD"
       }
+      // New keycodes should go here, to avoid breaking existing keymaps - order must match keymap_support.h
     ],
     "layouts": {
       "keymap": [