search

briancherne / jquery-hoverIntent

hoverIntent jQuery Plug-in
https://briancherne.github.io/jquery-hoverIntent/
MIT License
826 stars 253 forks source link

Add destroy method? #43

Open ktoh opened 9 years ago

ktoh commented 9 years ago

It would be good (also good plugin practice) to add a destroy() method to remove the hoverIntent bindings on an element or set of elements.

This would take a minimimal amount (3 unminified lines) of code, and the code has already been written here:

While it's possible to just direct users to the StackOverflow question, I think it would be better to include the method inside the plugin to avoid versioning issues in the future, as well as for the sake of promoting plugin modularity (allow for revokable methods).

thanks, this is a great plugin!

NAlbuquerque commented 9 years ago

Agree!

usmonster commented 9 years ago

With the newest (yet-unreleased) version of the plugin, the instructions found on stackoverflow are actually no longer valid, so perhaps a proper .destroy() method (or similar) is a good idea. It's definitely under consideration--thanks for the suggestion!