From 4bf749fd5f48072257bae3ff65897910d626fb4b Mon Sep 17 00:00:00 2001
From: alelpoan <alelpoan@proton.me>
Date: Sat, 4 Jul 2026 01:42:08 +0300
Subject: [PATCH] fix(desktop): add tooltip and fix scrollbar overlap on tool
 output copy button

---
 .../components/assistant-ui/tool/fallback.tsx |  3 +-
 .../desktop/src/components/ui/copy-button.tsx | 30 +++++++++++--------
 2 files changed, 19 insertions(+), 14 deletions(-)

diff --git a/apps/desktop/src/components/assistant-ui/tool/fallback.tsx b/apps/desktop/src/components/assistant-ui/tool/fallback.tsx
index 63e81c8c8..a501b7c9e 100644
--- a/apps/desktop/src/components/assistant-ui/tool/fallback.tsx
+++ b/apps/desktop/src/components/assistant-ui/tool/fallback.tsx
@@ -486,10 +486,11 @@ function ToolEntry({ part }: ToolEntryProps) {
           {copyAction.text && (
             <CopyButton
               appearance="inline"
-              className="absolute right-1.5 top-1.5 z-10 h-5 gap-0 rounded-md px-1 opacity-5 transition-opacity group-hover/tool-block:opacity-100 hover:opacity-100 focus-visible:opacity-100"
+              className="absolute right-4 top-1.5 z-10 h-5 gap-0 rounded-md px-1 opacity-5 transition-opacity group-hover/tool-block:opacity-100 hover:opacity-100 focus-visible:opacity-100"
               iconClassName="size-3"
               label={copyAction.label}
               showLabel={false}
+              side="left"
               stopPropagation
               text={copyAction.text}
             />
diff --git a/apps/desktop/src/components/ui/copy-button.tsx b/apps/desktop/src/components/ui/copy-button.tsx
index f7eed235d..ff7663ff9 100644
--- a/apps/desktop/src/components/ui/copy-button.tsx
+++ b/apps/desktop/src/components/ui/copy-button.tsx
@@ -49,6 +49,7 @@ export interface CopyButtonProps {
   onCopyError?: (error: unknown) => void
   preventDefault?: boolean
   showLabel?: boolean
+  side?: React.ComponentProps<typeof Tip>['side']
   stopPropagation?: boolean
   text: CopyPayload
   title?: string
@@ -69,6 +70,7 @@ export function CopyButton({
   onCopyError,
   preventDefault = false,
   showLabel,
+  side,
   stopPropagation = false,
   text,
   title
@@ -180,18 +182,20 @@ export function CopyButton({
 
   if (appearance === 'inline') {
     return (
-      <button
-        aria-label={ariaLabel}
-        className={cn(
-          'inline-flex items-center gap-1 rounded-sm px-1.5 py-0.5 text-[0.75rem] text-muted-foreground transition-colors hover:bg-accent hover:text-foreground disabled:opacity-40',
-          className
-        )}
-        disabled={disabled}
-        onClick={event => void copy(event)}
-        type="button"
-      >
-        {content}
-      </button>
+      <Tip label={feedbackLabel} side={side}>
+        <button
+          aria-label={ariaLabel}
+          className={cn(
+            'inline-flex items-center gap-1 rounded-sm px-1.5 py-0.5 text-[0.75rem] text-muted-foreground transition-colors hover:bg-accent hover:text-foreground disabled:opacity-40',
+            className
+          )}
+          disabled={disabled}
+          onClick={event => void copy(event)}
+          type="button"
+        >
+          {content}
+        </button>
+      </Tip>
     )
   }
 
@@ -229,5 +233,5 @@ export function CopyButton({
   )
 
   // Only icon-only buttons need a tooltip; the text variant already shows its label.
-  return appearance === 'icon' ? <Tip label={feedbackLabel}>{button}</Tip> : button
+  return appearance === 'icon' ? <Tip label={feedbackLabel} side={side ?? 'bottom'}>{button}</Tip> : button
 }