From 7690837f38c56b79d26f61b16fc60ad671dcc1ee Mon Sep 17 00:00:00 2001
From: Julius Marminge <julius0216@outlook.com>
Date: Sat, 14 Feb 2026 22:38:54 -0800
Subject: [PATCH] Fix disabled git action tooltip hover handling

- Use the quick action button itself as the popover trigger
- Keep hover tooltips working on disabled controls via pointer-events override
- Remove focus/blur tab-stop handlers from disabled tooltip wrappers
---
 apps/web/src/components/GitActionsControl.tsx | 25 ++++++-------------
 1 file changed, 7 insertions(+), 18 deletions(-)

diff --git a/apps/web/src/components/GitActionsControl.tsx b/apps/web/src/components/GitActionsControl.tsx
index c855729272e2694657ca0cb5ee3a7d23cf9c29e5..8d0e19658e60d4f7ebafaef07140ea85a53265b7 100644
--- a/apps/web/src/components/GitActionsControl.tsx
+++ b/apps/web/src/components/GitActionsControl.tsx
@@ -459,26 +459,18 @@ export default function GitActionsControl({ api, gitCwd }: GitActionsControlProp
             <Popover open={isQuickActionPopoverOpen} onOpenChange={setIsQuickActionPopoverOpen}>
               <PopoverTrigger
                 render={
-                  <span
-                    className="inline-flex cursor-not-allowed"
-                    onBlur={() => setIsQuickActionPopoverOpen(false)}
-                    onFocus={() => setIsQuickActionPopoverOpen(true)}
+                  <Button
+                    variant="outline"
+                    size="xs"
+                    disabled={isGitActionRunning || quickAction.disabled}
+                    className="cursor-not-allowed disabled:cursor-not-allowed disabled:pointer-events-auto"
                     onMouseEnter={() => setIsQuickActionPopoverOpen(true)}
                     onMouseLeave={() => setIsQuickActionPopoverOpen(false)}
-                    tabIndex={0}
                   />
                 }
               >
-                <Button
-                  variant="outline"
-                  size="xs"
-                  disabled={isGitActionRunning || quickAction.disabled}
-                  className="cursor-not-allowed disabled:cursor-not-allowed"
-                  onClick={runQuickAction}
-                >
-                  <GitQuickActionIcon quickAction={quickAction} />
-                  {quickAction.label}
-                </Button>
+                <GitQuickActionIcon quickAction={quickAction} />
+                {quickAction.label}
               </PopoverTrigger>
               <PopoverPopup tooltipStyle side="bottom" align="start">
                 {quickActionDisabledReason}
@@ -522,11 +514,8 @@ export default function GitActionsControl({ api, gitCwd }: GitActionsControlProp
                         render={
                           <span
                             className="inline-flex w-full cursor-not-allowed"
-                            onBlur={() => setOpenDisabledMenuItemId(null)}
-                            onFocus={() => setOpenDisabledMenuItemId(item.id)}
                             onMouseEnter={() => setOpenDisabledMenuItemId(item.id)}
                             onMouseLeave={() => setOpenDisabledMenuItemId(null)}
-                            tabIndex={0}
                           />
                         }
                       >
-- 
2.54.0