-
Notifications
You must be signed in to change notification settings - Fork 5.3k
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
2nd Reopen - Suggested fix for #8740 #1053
Conversation
…0 - Tooltip: Does not hide consistently with dynamically loaded content.
This looks good to me. There are some jshint issues with your second test, but we might be able to just drop that one. |
@scottgonzalez Any concerns with landing this? @mziech addressed your concerns from #992 and I verified this indeed fixes the issue. @jzaefferer you may also want to take a look. |
…ery-ui that includes pull-request jquery/jquery-ui#1053
var element = $( "#tooltipped1" ).tooltip({ | ||
content: function( response ) { | ||
element.trigger( "mouseleave" ); | ||
setTimeout(function () { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No space between function
and ()
.
Conflicts: ui/tooltip.js
…ot hide consistently with dynamically loaded content.
…de consistently with dynamically loaded content.
Ok, I incorporated the recent change request and updated the pull request. |
Any idea when this will be in a release? It fixes the issue for me of multiple tooltips showing when I have AJAX content. |
Bump. |
Fixes #8740 Closes jquerygh-1053
Yay progress 👍 |
Tooltip: Register event handlers before content is loaded. Fixes #8740 - Tooltip: Does not hide consistently with dynamically loaded content.
See http://bugs.jqueryui.com/ticket/8740
This pull request incorporates all suggested changes from #992 including the fixed commit message.