Skip to content

[PB-3521]: improve Tooltip accessibility, API, and maintainability#87

Open
terrerox wants to merge 2 commits intomasterfrom
feat/tooltip
Open

[PB-3521]: improve Tooltip accessibility, API, and maintainability#87
terrerox wants to merge 2 commits intomasterfrom
feat/tooltip

Conversation

@terrerox
Copy link
Contributor

@terrerox terrerox commented Mar 5, 2026

  • Add ARIA support (role, aria-describedby, aria-hidden) and keyboard handling (focus/blur, Escape to dismiss)
  • Add new props: content, delayShowInMs, tooltipClassName, titleClassName, animation
  • Rename delayInMs to delayHideInMs for clarity
  • Allow hovering tooltip bubble when delayHideInMs is set
  • Extract position config into a constant and refactor show/hide logic
  • Replace lineHeight hack with flex layout
  • Update tests to use data-testid and role queries

  - Add ARIA support (role, aria-describedby, aria-hidden) and keyboard
    handling (focus/blur, Escape to dismiss)
  - Add new props: content, delayShowInMs, tooltipClassName, titleClassName,
    animation
  - Rename delayInMs to delayHideInMs for clarity
  - Allow hovering tooltip bubble when delayHideInMs is set
  - Extract position config into a constant and refactor show/hide logic
  - Replace lineHeight hack with flex layout
  - Update tests to use data-testid and role queries
@terrerox terrerox self-assigned this Mar 5, 2026
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.

1 participant