Skip to content

Tooltip: Register event handlers before content is loaded #1456

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

Closed

Conversation

scottgonzalez
Copy link
Member

Fixes #8740
Closes gh-1053

This replaces gh-1053, which won't apply cleanly because of 8825d93. I've kept @mziech as the author since this is 90% his work. I'd like to get a review from him since it does contain some changes.

}
});
element.trigger( "mouseover" );
});
Copy link
Member Author

Choose a reason for hiding this comment

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

This test was untouched.

@mziech
Copy link
Contributor

mziech commented Feb 24, 2015

Thank you for rebasing my changes on the current master!

Although I can hardly remember what was actually causing the problem, the changes look fine to me. However, tests are failing not only on Travis but locally as well. Maybe you want to have a look.

@scottgonzalez
Copy link
Member Author

Wonderful. The test passes in isolation, but not in the full suite. I'll dig in.

@scottgonzalez
Copy link
Member Author

The mouseleave event was being fired too early (I believe it was actually in the same loop as mouseover. Splitting that out into a different spin of the event loop fixed the failure.

scottgonzalez pushed a commit that referenced this pull request Mar 10, 2015
Fixes #8740
Closes gh-1053
Closes gh-1456
(cherry picked from commit c4e367b)
@scottgonzalez scottgonzalez deleted the tooltip-close-early branch April 27, 2015 20:19
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.

3 participants