From b369ed5821d7b9a79784aeb4408f66723fe7d3d4 Mon Sep 17 00:00:00 2001
From: Bohdan Triapitsyn <artmore@protonmail.com>
Date: Tue, 28 Jul 2026 19:28:59 +0300
Subject: [PATCH] fix: make sync action tooltip trigger focusable

Wraps the sync button in a focusable trigger when rebase sync is blocked
Preserves the existing sync button behavior and labels
Improves tooltip access for keyboard users
---
 .../src/components/views/git/SyncActions.tsx  | 36 ++++++++++---------
 1 file changed, 19 insertions(+), 17 deletions(-)

diff --git a/packages/ui/src/components/views/git/SyncActions.tsx b/packages/ui/src/components/views/git/SyncActions.tsx
index 710b2ee2..b5e37233 100644
--- a/packages/ui/src/components/views/git/SyncActions.tsx
+++ b/packages/ui/src/components/views/git/SyncActions.tsx
@@ -74,23 +74,25 @@ export const SyncActions: React.FC<SyncActionsProps> = ({
     <div className="inline-flex items-center rounded-[9px] [corner-shape:squircle] supports-[corner-shape:squircle]:rounded-[50px] border border-border/60 bg-[var(--surface-elevated)] overflow-hidden">
       <Tooltip>
         <TooltipTrigger asChild>
-          <button
-            type="button"
-            onClick={handleSync}
-            disabled={isPrimaryDisabled}
-            className={cn(
-              'inline-flex h-7 items-center gap-1.5 px-2 typography-ui-label font-medium text-foreground',
-              'transition-colors hover:bg-interactive-hover focus-visible:outline-none focus-visible:ring-2 focus-visible:ring-primary disabled:cursor-not-allowed disabled:opacity-50'
-            )}
-            aria-label={t('gitView.sync.syncChanges')}
-          >
-            {syncAction === 'sync' ? (
-              <Icon name="loader-4" className="size-4 animate-spin" />
-            ) : (
-              <Icon name="refresh" className="size-4" />
-            )}
-            <span className="whitespace-nowrap tabular-nums">{primaryLabel}</span>
-          </button>
+          <span className="inline-flex" tabIndex={blocksRebaseSync ? 0 : undefined}>
+            <button
+              type="button"
+              onClick={handleSync}
+              disabled={isPrimaryDisabled}
+              className={cn(
+                'inline-flex h-7 items-center gap-1.5 px-2 typography-ui-label font-medium text-foreground',
+                'transition-colors hover:bg-interactive-hover focus-visible:outline-none focus-visible:ring-2 focus-visible:ring-primary disabled:cursor-not-allowed disabled:opacity-50'
+              )}
+              aria-label={t('gitView.sync.syncChanges')}
+            >
+              {syncAction === 'sync' ? (
+                <Icon name="loader-4" className="size-4 animate-spin" />
+              ) : (
+                <Icon name="refresh" className="size-4" />
+              )}
+              <span className="whitespace-nowrap tabular-nums">{primaryLabel}</span>
+            </button>
+          </span>
         </TooltipTrigger>
         <TooltipContent sideOffset={8}>{tooltipLabel}</TooltipContent>
       </Tooltip>