feat(KeyBoard): optimize user interact

src/components/keyboard/KeyBoard.vue

@@ -12,9 +12,6 @@ import { onBeforeRouteLeave } from "vue-router";
 import KeyObservation from "./KeyObservation.vue";
 import { useKeyboardStore } from "../../store/keyboard";
 
-// TODO 打开设置时要关闭active，建议将各种数据打包到一个对象中共享，省的麻烦
-// TODO 切换按键方案时提示未保存，然后修改edit
-// TODO 方向轮盘具体按键还没激活时也会触发按键检查
 // TODO 右键空白区域添加按键
 // TODO 设置界面添加本地数据编辑器（类似utools）
 // TODO 添加开发者工具打开按钮
@@ -50,12 +47,24 @@ function isKeyUnique(curKey: string): boolean {
 }
 
 function setCurButtonKey(curKey: string) {
+  if (
+    keyboardStore.activeButtonIndex === -1 ||
+    keyboardStore.activeButtonIndex >= store.editKeyMappingList.length
+  )
+    return;
+
+  const keyMapping = store.editKeyMappingList[keyboardStore.activeButtonIndex];
+  if (
+    keyMapping.type === "SteeringWheel" &&
+    keyboardStore.activeSteeringWheelButtonKeyIndex === -1
+  )
+    return;
+
   if (!isKeyUnique(curKey)) {
     message.error("按键重复：" + curKey);
     return;
   }
 
-  const keyMapping = store.editKeyMappingList[keyboardStore.activeButtonIndex];
   if (keyMapping.type === "SteeringWheel") {
     const keyObject = keyMapping.key as {
       left: string;
@@ -100,65 +109,39 @@ function handleClick(event: MouseEvent) {
     // right click
     if (event.target === document.getElementById("keyboardElement")) {
       // add button
-      if (keyboardStore.showSettingFlag) keyboardStore.showSettingFlag = false;
+      keyboardStore.showSettingFlag = false;
       keyboardStore.activeButtonIndex = -1;
       keyboardStore.activeSteeringWheelButtonKeyIndex = -1;
 
       console.log("弹出新增");
-    } else if (
-      // modify key
-      keyboardStore.activeButtonIndex !== -1 &&
-      keyboardStore.activeButtonIndex < store.editKeyMappingList.length &&
-      !keyboardStore.showButtonSettingFlag
-    ) {
-      const curKey = `M${event.button}`;
-      setCurButtonKey(curKey);
+    } else {
+      setCurButtonKey(`M${event.button}`);
     }
   } else {
     // other click
     event.preventDefault();
-    if (
-      keyboardStore.activeButtonIndex !== -1 &&
-      keyboardStore.activeButtonIndex < store.editKeyMappingList.length &&
-      !keyboardStore.showButtonSettingFlag
-    ) {
-      const curKey = `M${event.button}`;
-      setCurButtonKey(curKey);
-    }
+    setCurButtonKey(`M${event.button}`);
   }
 }
 
 function handleKeyUp(event: KeyboardEvent) {
-  if (
-    keyboardStore.activeButtonIndex !== -1 &&
-    keyboardStore.activeButtonIndex < store.editKeyMappingList.length &&
-    !keyboardStore.showButtonSettingFlag
-  ) {
-    const curKey = event.code;
-    setCurButtonKey(curKey);
-  }
+  setCurButtonKey(event.code);
 }
 
 function handleMouseWheel(event: WheelEvent) {
-  if (
-    keyboardStore.activeButtonIndex !== -1 &&
-    keyboardStore.activeButtonIndex < store.editKeyMappingList.length &&
-    !keyboardStore.showButtonSettingFlag
-  ) {
-    if (event.deltaY > 0) {
-      // WheelDown
-      setCurButtonKey("WheelDown");
-    } else if (event.deltaY < 0) {
-      // WheelUp
-      setCurButtonKey("WheelUp");
-    }
+  if (event.deltaY > 0) {
+    // WheelDown
+    setCurButtonKey("WheelDown");
+  } else if (event.deltaY < 0) {
+    // WheelUp
+    setCurButtonKey("WheelUp");
   }
 }
 
-function resetEditKeyMappingList() {
-  keyboardStore.showSettingFlag = false;
+function resetKeyMappingConfig() {
   keyboardStore.activeButtonIndex = -1;
   keyboardStore.activeSteeringWheelButtonKeyIndex = -1;
+  keyboardStore.showSettingFlag = false;
   store.resetEditKeyMappingList();
   keyboardStore.edited = false;
 }
@@ -185,7 +168,7 @@ onBeforeRouteLeave(() => {
         }
       },
       onNegativeClick: () => {
-        resetEditKeyMappingList();
+        resetKeyMappingConfig();
       },
     });
   }
@@ -200,7 +183,7 @@ onBeforeRouteLeave(() => {
     @mousedown="handleClick"
     @contextmenu.prevent
   >
-    <KeySetting @reset-edit-key-mapping-list="resetEditKeyMappingList" />
+    <KeySetting />
     <KeyInfo />
     <template v-for="(_, index) in store.editKeyMappingList">
       <KeySteeringWheel

src/components/keyboard/KeySetting.vue

@@ -20,10 +20,6 @@ import { loadDefaultKeyconfig } from "../../invoke";
 import { KeyMappingConfig } from "../../keyMappingConfig";
 import { useKeyboardStore } from "../../store/keyboard";
 
-const emit = defineEmits<{
-  resetEditKeyMappingList: [];
-}>();
-
 const store = useGlobalStore();
 const keyboardStore = useKeyboardStore();
 const localStore = new Store("store.bin");
@@ -74,7 +70,7 @@ onMounted(async () => {
 
 onActivated(() => {
   // reset editKeyMappingList as the same as keyMappingList
-  store.resetEditKeyMappingList();
+  resetKeyMappingConfig();
   // check config relative size
   checkConfigSize();
 });
@@ -124,6 +120,8 @@ function dragHandler(downEvent: MouseEvent) {
       localStore.set("keySettingPos", keySettingPos.value);
     } else {
       // click up
+      keyboardStore.activeButtonIndex = -1;
+      keyboardStore.activeSteeringWheelButtonKeyIndex = -1;
       keyboardStore.showSettingFlag = !keyboardStore.showSettingFlag;
     }
   };
@@ -167,6 +165,11 @@ async function importDefaultKeyMappingConfig() {
 }
 
 function createKeyMappingConfig() {
+  if (keyboardStore.edited) {
+    message.error("请先保存或还原当前方案");
+    return;
+  }
+
   const keyboardElement = document.getElementById(
     "keyboardElement"
   ) as HTMLElement;
@@ -185,6 +188,11 @@ function createKeyMappingConfig() {
 }
 
 function copyCurKeyMappingConfig() {
+  if (keyboardStore.edited) {
+    message.error("请先保存或还原当前方案");
+    return;
+  }
+
   const curConfig = store.keyMappingConfigList[store.curKeyMappingIndex];
   const newConfig: KeyMappingConfig = {
     title: curConfig.title + "-副本",
@@ -192,6 +200,8 @@ function copyCurKeyMappingConfig() {
     list: curConfig.list,
   };
   store.keyMappingConfigList.push(newConfig);
+  keyboardStore.activeButtonIndex = -1;
+  keyboardStore.activeSteeringWheelButtonKeyIndex = -1;
   store.setKeyMappingIndex(store.keyMappingConfigList.length - 1);
   localStore.set("keyMappingConfigList", store.keyMappingConfigList);
   message.success("方案已复制为：" + curConfig.title + "-副本");
@@ -204,6 +214,11 @@ function delCurKeyMappingConfig() {
   }
   const title = store.keyMappingConfigList[store.curKeyMappingIndex].title;
   store.keyMappingConfigList.splice(store.curKeyMappingIndex, 1);
+
+  // reset active and edit status
+  keyboardStore.activeButtonIndex = -1;
+  keyboardStore.activeSteeringWheelButtonKeyIndex = -1;
+  keyboardStore.edited = false;
   store.setKeyMappingIndex(
     store.curKeyMappingIndex > 0 ? store.curKeyMappingIndex - 1 : 0
   );
@@ -263,6 +278,11 @@ function checkConfigSize() {
 }
 
 function migrateKeyMappingConfig() {
+  if (keyboardStore.edited) {
+    message.error("请先保存或还原当前按键方案");
+    return;
+  }
+
   const keyboardElement = document.getElementById(
     "keyboardElement"
   ) as HTMLElement;
@@ -294,12 +314,32 @@ function migrateKeyMappingConfig() {
       0,
       newConfig
     );
-    store.setKeyMappingIndex(store.curKeyMappingIndex + 1);
     message.success("已迁移到新方案：" + newConfig.title);
+    keyboardStore.activeButtonIndex = -1;
+    keyboardStore.activeSteeringWheelButtonKeyIndex = -1;
+    store.setKeyMappingIndex(store.curKeyMappingIndex + 1);
   } else {
     message.info("当前方案符合蒙版尺寸，无需迁移");
   }
 }
+
+function selectKeyMappingConfig(index: number) {
+  if (keyboardStore.edited) {
+    message.error("请先保存或还原当前按键方案");
+    return;
+  }
+
+  keyboardStore.activeButtonIndex = -1;
+  keyboardStore.activeSteeringWheelButtonKeyIndex = -1;
+  store.setKeyMappingIndex(index);
+}
+
+function resetKeyMappingConfig() {
+  keyboardStore.activeButtonIndex = -1;
+  keyboardStore.activeSteeringWheelButtonKeyIndex = -1;
+  store.resetEditKeyMappingList();
+  keyboardStore.edited = false;
+}
 </script>
 
 <template>
@@ -320,7 +360,14 @@ function migrateKeyMappingConfig() {
       <NIcon><Settings /></NIcon>
     </template>
   </NButton>
-  <div class="key-setting" v-show="keyboardStore.showSettingFlag">
+  <div
+    class="key-setting"
+    v-show="keyboardStore.showSettingFlag"
+    @mousedown="
+      keyboardStore.activeButtonIndex = -1;
+      keyboardStore.activeSteeringWheelButtonKeyIndex = -1;
+    "
+  >
     <NButton
       text
       class="key-setting-close"
@@ -331,16 +378,14 @@ function migrateKeyMappingConfig() {
     <NH4 prefix="bar">按键方案</NH4>
     <NSelect
       :value="store.curKeyMappingIndex"
-      @update:value="(value: number)=>store.setKeyMappingIndex(value)"
+      @update:value="selectKeyMappingConfig"
       :options="keyMappingNameOptions"
     />
     <NP> Relative Size:{{ curRelativeSize.w }}x{{ curRelativeSize.h }} </NP>
     <NFlex style="margin-top: 20px">
       <template v-if="keyboardStore.edited">
         <NButton type="success" @click="saveKeyMappingConfig">保存方案</NButton>
-        <NButton type="error" @click="emit('resetEditKeyMappingList')"
-          >还原方案</NButton
-        >
+        <NButton type="error" @click="resetKeyMappingConfig">还原方案</NButton>
       </template>
       <NButton @click="createKeyMappingConfig">新建方案</NButton>
       <NButton @click="copyCurKeyMappingConfig">复制方案</NButton>