This code is still using bind() and unbind()

Hi,

According to jQuery doc's .bind() is not used anymore and you should switch to .on() as per doc's found here: https://github.com/jquery/jquery-migrate/blob/master/warnings.md#jqmigrate-jqueryfnbind-is-deprecated

Your code line is here:

jquery-mousewheel/jquery.mousewheel.js

Line 84 in 33dc8f1

return fn ? this.bind('mousewheel', fn) : this.trigger('mousewheel');

Also according to jQuery doc's .unbind() is not used anymore and you should switch to .off() as per doc's found here: https://github.com/jquery/jquery-migrate/blob/master/warnings.md#jqmigrate-jqueryfnunbind-is-deprecated

Your code line is here:

jquery-mousewheel/jquery.mousewheel.js

Line 88 in 33dc8f1

return this.unbind('mousewheel', fn);

[dmethvin]

Fixed in f7d64c7

@dmethvin Thanks for adding my pr. Is it possible to update the version number of this repo as we are using jquery-mousewheel in our repo. The last release is 3.1.13 dated 19 Jun 2015 (which is probably wrong and has not been updated). Thanks in advance.

[dmethvin]

Yes, I am working on updating all the build and test scripts now. Once that is done I will publish 3.2.0 to npm. That will also fix the issue with the minified file being missing.

Cool, look forward to it. 👍