Skip to content

fixed calls to jQuery on() & off event handlers to match ver10.7 syntax#60

Open
furiousgreencloud wants to merge 1 commit into
jmar777:masterfrom
furiousgreencloud:master
Open

fixed calls to jQuery on() & off event handlers to match ver10.7 syntax#60
furiousgreencloud wants to merge 1 commit into
jmar777:masterfrom
furiousgreencloud:master

Conversation

@furiousgreencloud

Copy link
Copy Markdown

without this all the "resize" handlers/functions are removed

@jmar777

jmar777 commented Mar 9, 2017

Copy link
Copy Markdown
Owner

@furiousgreencloud thanks for the PR!

I'm a little confused, though... what's the version 10.7 you referenced in your commit message? The latest jQuery version is 3.1.1 AFAICT, and the api for .on() seems to have been stable for quite awhile (the selector and data parameters in particular are still optional).

@furiousgreencloud

Copy link
Copy Markdown
Author

the version 10.7 is a typo, I was trying to reference "version added: 1.7.off( events [, selector ] [, handler ] )" from http://api.jquery.com/off/

@jmar777

jmar777 commented Mar 9, 2017

Copy link
Copy Markdown
Owner

Gotcha. Do you have an example of where the previous version didn't work and the modified version does? As best I can tell, they should do the same thing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants