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

Re-ordering mouseout and mouseover #34

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

Conversation

jgphilpott
Copy link

I was having an issue in my project when the mouse moves directly from one object to another. I have a div on the page that is supposed to be updated with metadata on a mouseover event and cleared on a mouseout event. However, when the mouse moves directly from one object to another the mouseover event for the new object would fire before the mouseout event for the old object. As a result the div would be updated with the new metadata but then immediately cleared.

I felt like the mouseout event for the old object should fire before the mouseover event for the new object so that the div can be cleared of the old metadata and then updated with the new. I made these changes to the code and now it works as expected.

You may have alternative reasons for wanting to keep it as it was but for my usecase I felt like this should be the default behaviour. I just thought I would share this incase others think so also, if you like it merge it, if not then delete it. Etherway thanks so much for the awesome code!

Cheers! 🍺🍺

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