Merge pull request #363 from skadefro/master

Fix delete not always removing all selected nodes
newcat · Jan 13, 2024 · f8bcbf5 · f8bcbf5
2 parents 3635f04 + aa23d9a
commit f8bcbf5
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/packages/renderer-vue/src/graph/deleteNodes.command.ts b/packages/renderer-vue/src/graph/deleteNodes.command.ts
@@ -9,7 +9,10 @@ export function registerDeleteNodesCommand(displayedGraph: Ref<Graph>, handler:
     handler.registerCommand(DELETE_NODES_COMMAND, {
         canExecute: () => displayedGraph.value.selectedNodes.length > 0,
         execute() {
-            displayedGraph.value.selectedNodes.forEach((n) => displayedGraph.value.removeNode(n));
+            for (let i = displayedGraph.value.selectedNodes.length - 1; i >= 0; i--) {
+                const n = displayedGraph.value.selectedNodes[i];
+                displayedGraph.value.removeNode(n);
+            }
         },
     });
     handler.registerHotkey(["Delete"], DELETE_NODES_COMMAND);