-
Notifications
You must be signed in to change notification settings - Fork 255
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
Check over/out methods gracefully. #52
Comments
Why would you want to throw exceptions? HoverIntent will work fine without declaring an |
@roydukkey I said "can we not" as in, we are and shouldn't be - hence "gracefully" in title.
with version 1.8.1 from Bower. We should probably check that it's defined before attempting to call it here: jquery-hoverIntent/jquery.hoverIntent.js Line 86 in 2790012
Other similar bugs may exist, e.g. jquery-hoverIntent/jquery.hoverIntent.js Line 74 in 2790012
|
Hello @adjohnson916! I'm closing this as a duplicate of #42, which was fixed in #51 and will be available in the upcoming release (or on the master branch now, if you want a preview). Please let me know if I've misunderstood the issue here. Thanks for reporting the issue and for using the plugin! |
@usmonster Thanks. I'm not sure I want |
@adjohnson916 An undefined As for the behavior when there is an It's true that this is the only place where it differs in jQuery
Hope this answer your questions! |
If either
out
orover
methods aren't provided or aren't functions, can we not throw exceptions?The text was updated successfully, but these errors were encountered: