Skip to content

Commit 7690264

Browse files
committed
Fix checkpoint popover not opening due to StandardTooltip wrapper conflict
The StandardTooltip component was incorrectly wrapping the PopoverTrigger, which prevented the popover from opening. This was introduced in PR #5098 when standardizing tooltip delays. The fix moves StandardTooltip inside PopoverTrigger (with asChild prop) so that PopoverTrigger can properly control its child element while still providing tooltip functionality. This pattern is consistent with other components like ShareButton.
1 parent c58b963 commit 7690264

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

webview-ui/src/components/chat/checkpoints/CheckpointMenu.tsx

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -61,13 +61,13 @@ export const CheckpointMenu = ({ ts, commitHash, currentHash, checkpoint }: Chec
6161
setIsOpen(open)
6262
setIsConfirming(false)
6363
}}>
64-
<PopoverTrigger asChild>
65-
<StandardTooltip content={t("chat:checkpoint.menu.restore")}>
64+
<StandardTooltip content={t("chat:checkpoint.menu.restore")}>
65+
<PopoverTrigger asChild>
6666
<Button variant="ghost" size="icon">
6767
<span className="codicon codicon-history" />
6868
</Button>
69-
</StandardTooltip>
70-
</PopoverTrigger>
69+
</PopoverTrigger>
70+
</StandardTooltip>
7171
<PopoverContent align="end" container={portalContainer}>
7272
<div className="flex flex-col gap-2">
7373
{!isCurrent && (

0 commit comments

Comments
 (0)