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

FIX added timeout to show action, so when it have a delay, it won't show unexpectedly #19307

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

gabrielduumont
Copy link

@gabrielduumont gabrielduumont commented Nov 16, 2023

Brief Information

This pull request is in the type of:

  • bug fixing
  • new feature
  • others

What does this PR do?

added timeout to show action, so when it have a delay, it won't show unexpectedly

Fixed issues

Essencially, if you change the showDelay and hideDelay options for a tooltip and you have multiple charts on a page, once you hover an item and get a tooltip, if you move fast to the other chart, that tooltip on the first chart would show up again

After: How does it behave after the fixing?

Just added a _showTimeout (same way you already do for the hideTimeout), that can be cancelled after hiding once

Copy link
Contributor

@Ovilia Ovilia left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please provide a test case. This feature seems not only relate to rich content tooltip but also with html content so extra changes should be made.

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

Successfully merging this pull request may close these issues.

2 participants