Fix Copy-Paste from a text control also duplicates the node #352

newcat · Jan 1, 2024 · b772527 · b772527
1 parent d6bc70c
commit b772527
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 8 deletions.
diff --git a/packages/renderer-vue/src/commands/hotkeyHandler.ts b/packages/renderer-vue/src/commands/hotkeyHandler.ts
@@ -1,5 +1,7 @@
 import { ref } from "vue";
 
+const INPUT_ELEMENT_TAGS = ["INPUT", "TEXTAREA", "SELECT"];
+
 export function useHotkeyHandler(executeCommand: (name: string) => void) {
     const pressedKeys = ref<string[]>([]);
     const handlers = ref<Array<{ keys: string[]; commandName: string }>>([]);
@@ -9,6 +11,10 @@ export function useHotkeyHandler(executeCommand: (name: string) => void) {
             pressedKeys.value.push(ev.key);
         }
 
+        if (INPUT_ELEMENT_TAGS.includes(document.activeElement?.tagName ?? "")) {
+            return;
+        }
+
         handlers.value.forEach((h) => {
             if (h.keys.every((k) => pressedKeys.value.includes(k))) {
                 executeCommand(h.commandName);

diff --git a/packages/renderer-vue/src/node/Node.vue b/packages/renderer-vue/src/node/Node.vue
@@ -34,7 +34,6 @@
                     placeholder="Node Name"
                     @blur="doneRenaming"
                     @keydown.enter="doneRenaming"
-                    @keydown.delete.stop
                 />
             </div>
         </slot>

diff --git a/packages/renderer-vue/src/sidebar/Sidebar.vue b/packages/renderer-vue/src/sidebar/Sidebar.vue
@@ -1,11 +1,5 @@
 <template>
-    <div
-        ref="el"
-        class="baklava-sidebar"
-        :class="{ '--open': graph.sidebar.visible }"
-        :style="styles"
-        @keydown.delete.stop
-    >
+    <div ref="el" class="baklava-sidebar" :class="{ '--open': graph.sidebar.visible }" :style="styles">
         <div v-if="resizable" class="__resizer" @mousedown="startResize" />
 
         <div class="__header">