Skip to content

Commit 11d92e3

Browse files
committed
fix: properly handle tooltip overflow with collision detection
- Move collisionPadding and avoidCollisions to base TooltipContent component - This ensures all tooltips in the app respect viewport boundaries - Remove duplicate props from StandardTooltip
1 parent 3f2d367 commit 11d92e3

File tree

2 files changed

+3
-7
lines changed

2 files changed

+3
-7
lines changed

webview-ui/src/components/ui/standard-tooltip.tsx

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -50,13 +50,7 @@ export function StandardTooltip({
5050
return (
5151
<Tooltip>
5252
<TooltipTrigger asChild={asChild}>{children}</TooltipTrigger>
53-
<TooltipContent
54-
side={side}
55-
align={align}
56-
sideOffset={sideOffset}
57-
className={className}
58-
collisionPadding={10}
59-
avoidCollisions={true}>
53+
<TooltipContent side={side} align={align} sideOffset={sideOffset} className={className}>
6054
{content}
6155
</TooltipContent>
6256
</Tooltip>

webview-ui/src/components/ui/tooltip.tsx

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,8 @@ const TooltipContent = React.forwardRef<
1717
<TooltipPrimitive.Content
1818
ref={ref}
1919
sideOffset={sideOffset}
20+
collisionPadding={10}
21+
avoidCollisions={true}
2022
className={cn(
2123
"z-50 overflow-hidden rounded-xs bg-vscode-notifications-background border border-vscode-notifications-border px-3 py-1.5 text-xs text-vscode-notifications-foreground animate-in fade-in-0 zoom-in-95 data-[state=closed]:animate-out data-[state=closed]:fade-out-0 data-[state=closed]:zoom-out-95 data-[side=bottom]:slide-in-from-top-2 data-[side=left]:slide-in-from-right-2 data-[side=right]:slide-in-from-left-2 data-[side=top]:slide-in-from-bottom-2",
2224
className,

0 commit comments

Comments
 (0)