Skip to content

Commit

Permalink
fix: prevent form submit on tooltip click, ref #5258
Browse files Browse the repository at this point in the history
  • Loading branch information
pete-watters committed Apr 19, 2024
1 parent 06aef60 commit 64bb8c2
Show file tree
Hide file tree
Showing 2 changed files with 18 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,11 @@ export function AvailableBalance({
<styled.span color="ink.text-subdued" textStyle="caption.01">
Available balance
</styled.span>
<BasicTooltip label={balanceTooltipLabel} side="top">
<BasicTooltip
label={balanceTooltipLabel}
side="top"
onClickTrigger={e => e.preventDefault()}
>
<Box>
<InfoCircleIcon color="ink.text-subdued" variant="small" />
</Box>
Expand Down
15 changes: 13 additions & 2 deletions src/app/ui/components/tooltip/basic-tooltip.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -10,12 +10,23 @@ interface BasicTooltipProps {
disabled?: boolean;
side?: RadixTooltip.TooltipContentProps['side'];
asChild?: boolean;
onClickTrigger?(e: React.MouseEvent<HTMLElement>): void;
}
export function BasicTooltip({ children, label, disabled, side, asChild }: BasicTooltipProps) {

export function BasicTooltip({
children,
label,
disabled,
side,
asChild,
onClickTrigger,
}: BasicTooltipProps) {
const isDisabled = !label || disabled;
return (
<Tooltip.Root>
<Tooltip.Trigger asChild={asChild}>{children}</Tooltip.Trigger>
<Tooltip.Trigger asChild={asChild} onClick={onClickTrigger}>
{children}
</Tooltip.Trigger>
<Tooltip.Portal>
<Tooltip.Content hidden={isDisabled} side={side} sideOffset={5}>
{label}
Expand Down

0 comments on commit 64bb8c2

Please sign in to comment.