From 2eb6c26c42cd1c24924370cb9c1ec1dacc928215 Mon Sep 17 00:00:00 2001
From: ZacharyZcR <zacharyzcr1984@gmail.com>
Date: Sun, 9 Nov 2025 17:38:21 +0800
Subject: [PATCH] fix: Allow Host Manager button to switch to existing tab
 instead of being disabled

Changes Host Manager button behavior to match Admin and User Profile tabs:
- Button is always clickable (except during split screen mode)
- If Host Manager tab already exists, clicking switches to it
- If tab doesn't exist, creates new tab and switches to it
- Removes 'Already Open' disabled state and tooltip

This improves UX by allowing users to quickly return to Host Manager tab without needing to manually find and click the tab.
---
 src/ui/desktop/navigation/LeftSidebar.tsx | 16 +++++++++-------
 1 file changed, 9 insertions(+), 7 deletions(-)

diff --git a/src/ui/desktop/navigation/LeftSidebar.tsx b/src/ui/desktop/navigation/LeftSidebar.tsx
index 9669b788..faa0cae9 100644
--- a/src/ui/desktop/navigation/LeftSidebar.tsx
+++ b/src/ui/desktop/navigation/LeftSidebar.tsx
@@ -115,7 +115,11 @@ export function LeftSidebar({
     Array.isArray(allSplitScreenTab) && allSplitScreenTab.length > 0;
   const sshManagerTab = tabList.find((t) => t.type === "ssh_manager");
   const openSshManagerTab = () => {
-    if (sshManagerTab || isSplitScreenActive) return;
+    if (isSplitScreenActive) return;
+    if (sshManagerTab) {
+      setCurrentTab(sshManagerTab.id);
+      return;
+    }
     const id = addTab({ type: "ssh_manager", title: "Host Manager" });
     setCurrentTab(id);
   };
@@ -416,13 +420,11 @@ export function LeftSidebar({
                   className="m-2 flex flex-row font-semibold border-2 !border-dark-border"
                   variant="outline"
                   onClick={openSshManagerTab}
-                  disabled={!!sshManagerTab || isSplitScreenActive}
+                  disabled={isSplitScreenActive}
                   title={
-                    sshManagerTab
-                      ? t("interface.sshManagerAlreadyOpen")
-                      : isSplitScreenActive
-                        ? t("interface.disabledDuringSplitScreen")
-                        : undefined
+                    isSplitScreenActive
+                      ? t("interface.disabledDuringSplitScreen")
+                      : undefined
                   }
                 >
                   <HardDrive strokeWidth="2.5" />