Skip to content

Commit b357fe8

Browse files
bchapuisclaude
andcommitted
Clean up property field visibility toggle and simplify condition formatting
Remove unnecessary disabled prop and styling from the visibility toggle button, and collapse multi-line condition into a single line for readability. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent 62a2fd1 commit b357fe8

2 files changed

Lines changed: 2 additions & 8 deletions

File tree

apps/app/src/components/workflow/fields/property-field.tsx

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -153,10 +153,7 @@ export function PropertyField({
153153
pressed={parameter.hidden}
154154
onPressedChange={onToggleVisibility}
155155
aria-label={`Toggle visibility for ${parameter.name}`}
156-
className={`group px-1 h-8 w-8 bg-transparent data-[state=on]:bg-transparent hover:bg-transparent ${
157-
disabled ? "opacity-70 cursor-not-allowed" : ""
158-
}`}
159-
disabled={disabled}
156+
className="group px-1 h-8 w-8 bg-transparent data-[state=on]:bg-transparent hover:bg-transparent"
160157
>
161158
{parameter.hidden ? (
162159
<EyeOffIcon className="h-3 w-3 text-neutral-400 group-hover:text-neutral-700 dark:text-neutral-500 dark:group-hover:text-neutral-300" />

apps/app/src/components/workflow/use-workflow-execution-state.ts

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -190,10 +190,7 @@ export function useWorkflowExecutionState({
190190

191191
// Check if we need to reset node states before updating status
192192
// (must happen outside the state updater to avoid side effects)
193-
if (
194-
!eagerStart &&
195-
statusRef.current === "idle"
196-
) {
193+
if (!eagerStart && statusRef.current === "idle") {
197194
resetNodeStates("executing");
198195
}
199196

0 commit comments

Comments
 (0)