Skip to content

Commit 76e5714

Browse files
gabrieljablonskiGabriel Jablonski
authored andcommitted
fix: TooltipWrapper and click event behavior
1 parent b1b50cf commit 76e5714

File tree

1 file changed

+8
-11
lines changed

1 file changed

+8
-11
lines changed

src/components/Tooltip/Tooltip.tsx

Lines changed: 8 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -81,10 +81,11 @@ const Tooltip = ({
8181
} else {
8282
handleShow(true)
8383
}
84+
const target = event.currentTarget ?? event.target
8485
setActiveAnchor((anchor) =>
85-
anchor.current === event.target ? anchor : { current: event.target as HTMLElement },
86+
anchor.current === target ? anchor : { current: target as HTMLElement },
8687
)
87-
setProviderActiveAnchor({ current: event.target as HTMLElement })
88+
setProviderActiveAnchor({ current: target as HTMLElement })
8889

8990
if (tooltipHideDelayTimerRef.current) {
9091
clearTimeout(tooltipHideDelayTimerRef.current)
@@ -150,19 +151,15 @@ const Tooltip = ({
150151
lastFloatPosition.current = mousePosition
151152
}
152153

153-
const handleClickTooltipAnchor = () => {
154-
if (setIsOpen) {
155-
setIsOpen(!isOpen)
156-
} else if (!setIsOpen && isOpen === undefined) {
157-
setShow(true)
158-
if (delayHide) {
159-
handleHideTooltipDelayed()
160-
}
154+
const handleClickTooltipAnchor = (event?: Event) => {
155+
handleShowTooltip(event)
156+
if (delayHide) {
157+
handleHideTooltipDelayed()
161158
}
162159
}
163160

164161
const handleClickOutsideAnchor = (event: MouseEvent) => {
165-
if (event.target === activeAnchor.current) {
162+
if (activeAnchor.current?.contains(event.target as HTMLElement)) {
166163
return
167164
}
168165
setShow(false)

0 commit comments

Comments
 (0)