Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

#6703: tooltip content interaction. #6732

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

oneironautics
Copy link

First time contributor checklist:

Contributor checklist:

  • My contribution is not related to translations.
  • My commits are in nice logical chunks with good commit messages
  • My changes are rebased on the latest main branch
  • A yarn ready run passes successfully (more about tests here)
  • My changes are ready to be shipped to users

Description

Fixes #6703.

  • Mouse enter/leave events are now captured on the popper and taken into account when evaluating the overall tooltip hover state.
  • An optional hide delay has been added. The purpose of this is to prevent the tooltip hover state from changing when the cursor transitions from the tooltip element to the popper element (or vice versa).

These changes allow the user to interact with the tooltip popper content (e.g., clicking a hyperlink to update Signal).

This was tested on MacOS 14.2.1 (23C71) using various combinations of show/hide delay values. No new tests were written. Three new storybook examples were added for the tooltip component.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

Notice about failed update not interactable when sidebar is collapsed
2 participants