Skip to content

Commit d425290

Browse files
authored
fix(richtext-slate, ui): use PointerEvents to show tooltips on enabled / disabled buttons (#9006)
Fixes #9005 Note: I did not replace all instances of `onMouseEnter`, just the ones that can be disabled and have `tooltip` set.
1 parent 4b4ecb3 commit d425290

File tree

2 files changed

+4
-4
lines changed

2 files changed

+4
-4
lines changed

packages/richtext-slate/src/field/elements/Button.tsx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -54,8 +54,8 @@ export const ElementButton: React.FC<ButtonProps> = (props) => {
5454
.filter(Boolean)
5555
.join(' ')}
5656
onClick={onClick || defaultOnClick}
57-
onMouseEnter={() => setShowTooltip(true)}
58-
onMouseLeave={() => setShowTooltip(false)}
57+
onPointerEnter={() => setShowTooltip(true)}
58+
onPointerLeave={() => setShowTooltip(false)}
5959
>
6060
{tooltip && <Tooltip show={showTooltip}>{tooltip}</Tooltip>}
6161
{children}

packages/ui/src/elements/Button/index.tsx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -114,8 +114,8 @@ export const Button = forwardRef<HTMLAnchorElement | HTMLButtonElement, Props>((
114114
className: !SubMenuPopupContent ? [classes, styleClasses].join(' ') : classes,
115115
disabled,
116116
onClick: !disabled ? handleClick : undefined,
117-
onMouseEnter: tooltip ? () => setShowTooltip(true) : undefined,
118-
onMouseLeave: tooltip ? () => setShowTooltip(false) : undefined,
117+
onPointerEnter: tooltip ? () => setShowTooltip(true) : undefined,
118+
onPointerLeave: tooltip ? () => setShowTooltip(false) : undefined,
119119
rel: newTab ? 'noopener noreferrer' : undefined,
120120
target: newTab ? '_blank' : undefined,
121121
}

0 commit comments

Comments
 (0)