Skip to content

refactor(frontend): hover tooltips on react-aria-components - #4000

Open
Kadrian wants to merge 10 commits into
refactor/info-panefrom
refactor/tooltip-react-aria
Open

refactor(frontend): hover tooltips on react-aria-components#4000
Kadrian wants to merge 10 commits into
refactor/info-panefrom
refactor/tooltip-react-aria

Conversation

@Kadrian

@Kadrian Kadrian commented Sep 2, 2026

Copy link
Copy Markdown
Collaborator

Design-system prep, step 2: hover tooltips on react-aria-components

Stacked on #3999. WithTooltip (tippy) becomes ui-components/Tooltip, in react-aria's composition shape so call sites won't change again as more components move to react-aria:

<TooltipTrigger>
  <IconButton  />
  <Tooltip>{t("common.delete")}</Tooltip>
</TooltipTrigger>
  • react-aria-components and react-aria added, pinned to the same release: a second copy of react-aria would carry its own React contexts and silently break the trigger wiring
  • buttons are triggers on their own; TooltipTarget marks static content and keeps it out of the tab order
  • timing per Spectrum's guideline via tooltipDelay: immediate for help icons, short (300 ms) by default, long (1500 ms) for hints on larger surfaces; after the warm-up, neighboring tooltips open instantly; a tooltip closes 300 ms after the pointer leaves (a hover-revealed trigger stays invisible rather than hidden, so the closing tooltip keeps its anchor)
  • look unchanged; storybook loads index.css now; Tooltip stories
  • tippy stays for the three click popovers until the menus step

@Kadrian
Kadrian marked this pull request as ready for review September 3, 2026 09:00
@Kadrian
Kadrian force-pushed the refactor/tooltip-react-aria branch from 248b5a9 to 06ff19a Compare September 3, 2026 09:16
@Kadrian
Kadrian marked this pull request as draft September 3, 2026 09:39
@Kadrian
Kadrian marked this pull request as ready for review September 3, 2026 10:10
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.

2 participants