fix: disallow null inputs when editing

feat: allow special inputs while creating a chord input
fixes #93

src/lib/assets/keymaps/keyboard.yml

@@ -6,41 +6,49 @@ actions:
     id: "LEFT_CTRL"
     display: CTRL
     title: Control Keyboard Modifier
+    keyCode: ControlLeft
     variant: left
   513: &left_shift
     id: "LEFT_SHIFT"
     title: Shift Keyboard Modifier
+    keyCode: ShiftLeft
     variant: left
     icon: shift
   514: &left_alt
     id: "LEFT_ALT"
     display: ALT
     title: Alt Keyboard Modifier
+    keyCode: AltLeft
     variant: left
   515: &left_gui
     id: "LEFT_GUI"
     title: GUI Keyboard Modifier
+    keyCode: MetaLeft
     icon: apps
     variant: left
   516:
     variationOf: 512
     <<: *left_ctrl
     id: "RIGHT_CTRL"
+    keyCode: ControlRight
     variant: right
   517:
     variationOf: 513
     <<: *left_shift
     id: "RIGHT_SHIFT"
+    keyCode: ShiftRight
     variant: right
   518:
     variationOf: 514
     <<: *left_alt
     id: "RIGHT_ALT"
+    keyCode: AltRight
     variant: right
   519:
     variationOf: 515
     <<: *left_gui
     id: "RIGHT_GUI"
+    keyCode: MetaRight
     variant: right
   520:
     id: "RELEASE_MOD"

src/lib/assets/keymaps/scancode.yml

@@ -920,35 +920,27 @@ actions:
     description: Not required to be supported by any OS
   480:
     id: "KSC_E0"
-    keyCode: "ControlLeft"
     title: Keyboard Left Control
   481:
     id: "KSC_E1"
-    keyCode: "ShiftLeft"
     title: Keyboard Left Shift
   482:
     id: "KSC_E2"
-    keyCode: "AltLeft"
     title: Keyboard Left Alt
   483:
     id: "KSC_E3"
-    keyCode: "MetaLeft"
     title: Keyboard Left GUI
   484:
     id: "KSC_E4"
-    keyCode: "ControlRight"
     title: Keyboard Right Control
   485:
     id: "KSC_E5"
-    keyCode: "ShiftRight"
     title: Keyboard Right Shift
   486:
     id: "KSC_E6"
-    keyCode: "AltRight"
     title: Keyboard Right Alt
   487:
     id: "KSC_E7"
-    keyCode: "MetaRight"
     title: Keyboard Right GUI
   488:
     id: "KSC_E8"

src/lib/serial/keymap-codes.ts

@@ -33,7 +33,3 @@ export const KEYMAP_IDS: Map<string, KeyInfo> = new Map(
     ),
   ).filter(([id]) => id !== undefined),
 )
-
-export const specialKeycodes = new Map([
-  [" ", 32], // Space
-])

src/routes/config/chords/ChordActionEdit.svelte

@@ -28,7 +28,12 @@
   function keydown(event: KeyboardEvent) {
     if (!editing) return
     event.preventDefault()
-    pressedKeys.add(inputToAction(event, get(serialPort)?.device === "X")!)
+    const input = inputToAction(event, get(serialPort)?.device === "X")
+    if (input == undefined) {
+      alert("Invalid key")
+      return
+    }
+    pressedKeys.add(input)
     pressedKeys = pressedKeys
   }
 

src/routes/config/chords/ChordPhraseEdit.svelte

@@ -1,7 +1,5 @@
 <script lang="ts">
-  import {KEYMAP_IDS, specialKeycodes} from "$lib/serial/keymap-codes"
   import {tick} from "svelte"
-  import ActionSelector from "$lib/components/layout/ActionSelector.svelte"
   import {changes, ChangeType} from "$lib/undo-redo"
   import type {ChordInfo} from "$lib/undo-redo"
   import {scale} from "svelte/transition"
@@ -26,6 +24,7 @@
     } else if (event.key === "Delete") {
       deleteAction(cursorPosition)
     } else {
+      if (event.key === "Shift") return
       const action = inputToAction(event, get(serialPort)?.device === "X")
       if (action !== undefined) {
         insertAction(cursorPosition, action)

src/routes/config/chords/input-converter.ts

@@ -1,9 +1,9 @@
-import {KEYMAP_IDS, KEYMAP_KEYCODES, specialKeycodes} from "$lib/serial/keymap-codes"
+import {KEYMAP_IDS, KEYMAP_KEYCODES} from "$lib/serial/keymap-codes"
 
 export function inputToAction(event: KeyboardEvent, useKeycodes?: boolean): number | undefined {
   if (useKeycodes) {
     return KEYMAP_KEYCODES.get(event.code)
   } else {
-    return KEYMAP_IDS.get(event.key)?.code ?? specialKeycodes.get(event.key)
+    return KEYMAP_IDS.get(event.key)?.code ?? KEYMAP_KEYCODES.get(event.code)
   }
 }