From 27a6f9e2abeeb864f35d1df2ea157fe4b508f950 Mon Sep 17 00:00:00 2001
From: space-nuko <24979496+space-nuko@users.noreply.github.com>
Date: Sat, 6 May 2023 11:24:19 -0500
Subject: [PATCH] Fix properties

---
 src/lib/components/ComfyComboProperty.svelte  |  7 ++++++-
 src/lib/components/ComfyNumberProperty.svelte |  6 +++++-
 src/lib/components/ComfyProperties.svelte     | 16 ++++++++++++++++
 src/lib/components/ComfyUIPane.svelte         |  2 +-
 4 files changed, 28 insertions(+), 3 deletions(-)

diff --git a/src/lib/components/ComfyComboProperty.svelte b/src/lib/components/ComfyComboProperty.svelte
index 11b746e..5b9b671 100644
--- a/src/lib/components/ComfyComboProperty.svelte
+++ b/src/lib/components/ComfyComboProperty.svelte
@@ -5,6 +5,7 @@
  export let value: string = "";
  export let values: string[] = [""];
  export let name: string = "";
+ export let disabled: boolean = false;
  let value_: string = ""
 
  $: value;
@@ -27,7 +28,7 @@
 <label class="select-wrapper">
     <BlockTitle>{name}</BlockTitle>
     <div class="select">
-        <select on:blur bind:value>
+        <select on:blur bind:value {disabled}>
             {#each values as value}
                 <option {value}>
                     {value}
@@ -53,4 +54,8 @@
  .select-title {
      padding: 0.2rem;
  }
+
+ input:disabled {
+     cursor: not-allowed;
+ }
 </style>
diff --git a/src/lib/components/ComfyNumberProperty.svelte b/src/lib/components/ComfyNumberProperty.svelte
index 5e96516..d4451b5 100644
--- a/src/lib/components/ComfyNumberProperty.svelte
+++ b/src/lib/components/ComfyNumberProperty.svelte
@@ -5,6 +5,7 @@
  export let value: number = 0;
  export let step: number = 1;
  export let name: string = "";
+ export let disabled: boolean = false;
  let value_: number = 0;
 
  $: value;
@@ -26,7 +27,7 @@
 <label class="number-wrapper">
     <BlockTitle>{name}</BlockTitle>
     <div class="number">
-        <input type="number" bind:value {step}>
+        <input type="number" bind:value {step} {disabled}>
     </div>
 </label>
 
@@ -42,4 +43,7 @@
          }
      }
  }
+ input[disabled] {
+     cursor: not-allowed;
+ }
 </style>
diff --git a/src/lib/components/ComfyProperties.svelte b/src/lib/components/ComfyProperties.svelte
index 251f624..055f352 100644
--- a/src/lib/components/ComfyProperties.svelte
+++ b/src/lib/components/ComfyProperties.svelte
@@ -158,6 +158,7 @@
                                 value={target.attrs[spec.name] || spec.defaultValue}
                                 on:change={(e) => updateAttribute(spec, target, e.detail)}
                                 on:input={(e) => updateAttribute(spec, target, e.detail)}
+                                disabled={!$uiState.uiUnlocked || !spec.editable}
                                 label={spec.name}
                                 max_lines={1}
                                 />
@@ -165,6 +166,7 @@
                                 <Checkbox
                                     value={target.attrs[spec.name] || spec.defaultValue}
                                     on:change={(e) => updateAttribute(spec, target, e.detail)}
+                                    disabled={!$uiState.uiUnlocked || !spec.editable}
                                     label={spec.name}
                                     />
                         {:else if spec.type === "number"}
@@ -172,6 +174,7 @@
                                         name={spec.name}
                                         value={target.attrs[spec.name] || spec.defaultValue}
                                         step={1}
+                                        disabled={!$uiState.uiUnlocked || !spec.editable}
                                         on:change={(e) => updateAttribute(spec, target, e.detail)}
                                         />
                         {:else if spec.type === "enum"}
@@ -179,6 +182,7 @@
                                             name={spec.name}
                                             value={target.attrs[spec.name] || spec.defaultValue}
                                             values={spec.values}
+                                            disabled={!$uiState.uiUnlocked || !spec.editable}
                                             on:change={(e) => updateAttribute(spec, target, e.detail)}
                                             />
                         {/if}
@@ -192,12 +196,14 @@
                                     on:change={(e) => updateProperty(spec, e.detail)}
                                     on:input={(e) => updateProperty(spec, e.detail)}
                                     label={spec.name}
+                                    disabled={!$uiState.uiUnlocked || !spec.editable}
                                     max_lines={1}
                                     />
                             {:else if spec.type === "boolean"}
                                     <Checkbox
                                         value={node.properties[spec.name] || spec.defaultValue}
                                         label={spec.name}
+                                        disabled={!$uiState.uiUnlocked || !spec.editable}
                                         on:change={(e) => updateProperty(spec, e.detail)}
                                         />
                             {:else if spec.type === "number"}
@@ -205,6 +211,7 @@
                                             name={spec.name}
                                             value={node.properties[spec.name] || spec.defaultValue}
                                             step={1}
+                                            disabled={!$uiState.uiUnlocked || !spec.editable}
                                             on:change={(e) => updateProperty(spec, e.detail)}
                                             />
                             {:else if spec.type === "enum"}
@@ -212,6 +219,7 @@
                                                 name={spec.name}
                                                 value={node.properties[spec.name] || spec.defaultValue}
                                                 values={spec.values}
+                                                disabled={!$uiState.uiUnlocked || !spec.editable}
                                                 on:change={(e) => updateProperty(spec, e.detail)}
                                                 />
                             {/if}
@@ -224,12 +232,14 @@
                                     on:change={(e) => updateVar(spec, e.detail)}
                                     on:input={(e) => updateVar(spec, e.detail)}
                                     label={spec.name}
+                                    disabled={!$uiState.uiUnlocked || !spec.editable}
                                     max_lines={1}
                                     />
                             {:else if spec.type === "boolean"}
                                     <Checkbox
                                         value={getVar(node, spec)}
                                         on:change={(e) => updateVar(spec, e.detail)}
+                                        disabled={!$uiState.uiUnlocked || !spec.editable}
                                         label={spec.name}
                                         />
                             {:else if spec.type === "number"}
@@ -237,6 +247,7 @@
                                             name={spec.name}
                                             value={getVar(node, spec)}
                                             step={1}
+                                            disabled={!$uiState.uiUnlocked || !spec.editable}
                                             on:change={(e) => updateVar(spec, e.detail)}
                                             />
                             {:else if spec.type === "enum"}
@@ -244,6 +255,7 @@
                                                 name={spec.name}
                                                 value={getVar(node, spec)}
                                                 values={spec.values}
+                                                disabled={!$uiState.uiUnlocked || !spec.editable}
                                                 on:change={(e) => updateVar(spec, e.detail)}
                                                 />
                             {/if}
@@ -257,12 +269,14 @@
                                 on:change={(e) => updateWorkflowAttribute(spec, e.detail)}
                                 on:input={(e) => updateWorkflowAttribute(spec, e.detail)}
                                 label={spec.name}
+                                disabled={!$uiState.uiUnlocked || !spec.editable}
                                 max_lines={1}
                                 />
                         {:else if spec.type === "boolean"}
                                 <Checkbox
                                     value={$layoutState.attrs[spec.name] || spec.defaultValue}
                                     on:change={(e) => updateWorkflowAttribute(spec, e.detail)}
+                                    disabled={!$uiState.uiUnlocked || !spec.editable}
                                     label={spec.name}
                                     />
                         {:else if spec.type === "number"}
@@ -270,6 +284,7 @@
                                         name={spec.name}
                                         value={$layoutState.attrs[spec.name] || spec.defaultValue}
                                         step={1}
+                                        disabled={!$uiState.uiUnlocked || !spec.editable}
                                         on:change={(e) => updateWorkflowAttribute(spec, e.detail)}
                                         />
                         {:else if spec.type === "enum"}
@@ -277,6 +292,7 @@
                                             name={spec.name}
                                             value={$layoutState.attrs[spec.name] || spec.defaultValue}
                                             values={spec.values}
+                                            disabled={!$uiState.uiUnlocked || !spec.editable}
                                             on:change={(e) => updateWorkflowAttribute(spec, e.detail)}
                                             />
                         {/if}
diff --git a/src/lib/components/ComfyUIPane.svelte b/src/lib/components/ComfyUIPane.svelte
index a6b7415..e430f98 100644
--- a/src/lib/components/ComfyUIPane.svelte
+++ b/src/lib/components/ComfyUIPane.svelte
@@ -50,7 +50,7 @@
 
  function ungroup() {
      const item = layoutState.getCurrentSelection()[0]
-     if (item.type !== "container")
+     if (!item || item.type !== "container")
          return;
 
      $layoutState.currentSelection = []