Fix pin not respected by multi-select move (#284)

Comfy-Org · Nov 7, 2024 · 1c40aad · 1c40aad
1 parent 80cbe49
commit 1c40aad
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 1 deletion.
diff --git a/src/LGraphCanvas.ts b/src/LGraphCanvas.ts
@@ -2444,7 +2444,7 @@ export class LGraphCanvas {
                 this.#dirty()
 
                 function addToSetRecursively(item: Positionable, items: Set<Positionable>): void {
-                    if (items.has(item)) return
+                    if (items.has(item) || item.pinned) return
                     items.add(item)
                     item.children?.forEach(x => addToSetRecursively(x, items))
                 }

diff --git a/src/LGraphNode.ts b/src/LGraphNode.ts
@@ -1437,6 +1437,8 @@ export class LGraphNode implements Positionable, IPinnable {
     }
 
     move(deltaX: number, deltaY: number): void {
+        if (this.pinned) return
+
         this.pos[0] += deltaX
         this.pos[1] += deltaY
     }

diff --git a/src/interfaces.ts b/src/interfaces.ts
@@ -24,6 +24,9 @@ export interface Positionable {
     /** true if this object is part of the selection, otherwise false. */
     selected?: boolean
 
+    /** See {@link IPinnable.pinned} */
+    readonly pinned?: boolean
+
     readonly children?: ReadonlySet<Positionable>
 
     /**