Skip to content

fix: interactivity for components within tooltip body [INS-4773]#8239

Merged
godfrzero merged 1 commit intodevelopfrom
fix/tooltip-interaction
Dec 11, 2024
Merged

fix: interactivity for components within tooltip body [INS-4773]#8239
godfrzero merged 1 commit intodevelopfrom
fix/tooltip-interaction

Conversation

@godfrzero
Copy link
Copy Markdown
Contributor

@godfrzero godfrzero commented Dec 9, 2024

Attempting to click on links within tooltips would lead to the tooltip being dismissed. This PR fixes the issue by moving the tooltip to be a sibling of the tooltip trigger, which is the correct structure for react-aria tooltips. Ref: https://react-spectrum.adobe.com/react-aria/useTooltipTrigger.html

Before After
Kapture 2024-12-07 at 09 31 28 Kapture 2024-12-07 at 09 33 55

The Tooltip component is used in 45 locations throughout the codebase, and in 23 additional locations via the HelpTooltip component.

@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Dec 9, 2024

CLA assistant check
All committers have signed the CLA.

@godfrzero godfrzero marked this pull request as ready for review December 11, 2024 06:23
@godfrzero godfrzero requested a review from a team December 11, 2024 06:23
Attempting to click in links within tooltips would lead to the tooltip being dismissed earlier. This fixes the issue by moving the tooltip to be a sibling of the tooltip trigger, which is the correct structure for react-aria tooltips. Ref: https://react-spectrum.adobe.com/react-aria/useTooltipTrigger.html
@godfrzero godfrzero force-pushed the fix/tooltip-interaction branch from 3f21b94 to ea75864 Compare December 11, 2024 15:54
@godfrzero godfrzero merged commit eca35d8 into develop Dec 11, 2024
@godfrzero godfrzero deleted the fix/tooltip-interaction branch December 11, 2024 21:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants