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

possible race condition when hovering on the next element/parent #75

Open
Offirmo opened this issue Jul 31, 2018 · 4 comments
Open

possible race condition when hovering on the next element/parent #75

Offirmo opened this issue Jul 31, 2018 · 4 comments

Comments

@Offirmo
Copy link

Offirmo commented Jul 31, 2018

Salut! Awesome lib filling exactly my needs, thanks for sharing.

A bug: I have 3 buttons on top of each other, with a bit of space between them.

Basically it's 3 times this example:

const button = <span>Hover me to display the tooltip</span>

return (
  <StatefulToolTip parent={ button }>
    Stateful Tooltip content here!
  </StatefulToolTip>
)

Here is what happen when I'm quickly hovering above them: http://www.offirmo.net/assets/share/2018-07-31_17-53-16.mp4

May be related to #39

@Offirmo
Copy link
Author

Offirmo commented Jul 31, 2018

Precision: I'm using StatefulToolTip

  • tried changing the timeout = no effect
  • tried giving a different group to each tooltip = no effect

@Offirmo Offirmo changed the title possible race condition when switcheng to the next element possible race condition when hovering on the next element/parent Jul 31, 2018
@Offirmo
Copy link
Author

Offirmo commented Aug 30, 2018

We devs are always complaining but sometimes it works...

I got a successful setup using ToolTip and not StatefulToolTip. I have to manage some state myself though, so this bug report still holds.

However, it works! https://www.online-adventur.es/apps/the-boring-rpg/index.html (bottom right -> inventory then hover on the equipment)

Thanks a LOT for sharing this lib!

@Offirmo
Copy link
Author

Offirmo commented Aug 30, 2018

🙇

@Redmega
Copy link
Collaborator

Redmega commented Aug 31, 2018

@Offirmo Sorry I didn't have the chance to check this out before. Glad you resolved your issue!

Cool little game there! Gonna leave this open since, as you said, stateful tooltip still has that issue.

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

No branches or pull requests

2 participants