From e6aff692b48f99d9531ac7d9e38fbdf949ac7bf6 Mon Sep 17 00:00:00 2001
From: bymyself <cbyrne@comfy.org>
Date: Sun, 31 Aug 2025 19:44:19 -0700
Subject: [PATCH] [fix] Correct Install All button visibility for manager UI
 states
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

- Install All Missing Custom Nodes button only shows for NEW_UI state
- Legacy UI state only shows Open Manager button
- Disabled state shows no buttons
- Matches original PR #5063 behavior exactly

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
---
 src/components/dialog/content/LoadWorkflowWarning.vue | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/src/components/dialog/content/LoadWorkflowWarning.vue b/src/components/dialog/content/LoadWorkflowWarning.vue
index ea35130e8..32d1add85 100644
--- a/src/components/dialog/content/LoadWorkflowWarning.vue
+++ b/src/components/dialog/content/LoadWorkflowWarning.vue
@@ -33,6 +33,7 @@
   </ListBox>
   <div v-if="showManagerButtons" class="flex justify-end py-3">
     <PackInstallButton
+      v-if="showInstallAllButton"
       :disabled="
         isLoading || !!error || missingNodePacks.length === 0 || isInstalling
       "
@@ -115,6 +116,11 @@ const showManagerButtons = computed(() => {
   return managerStateStore.managerUIState !== ManagerUIState.DISABLED
 })
 
+// Only show Install All button for NEW_UI (new manager with v4 support)
+const showInstallAllButton = computed(() => {
+  return managerStateStore.managerUIState === ManagerUIState.NEW_UI
+})
+
 const openManager = async () => {
   const state = managerStateStore.managerUIState