jquery-migrate-3.4.1.source.js:137 JQMIGRATE: jQuery.fn.blur() event …

vakata / jstree

jquery tree plugin

http://jstree.com

MIT License

5.15k stars 1.38k forks source link

jquery-migrate-3.4.1.source.js:137 JQMIGRATE: jQuery.fn.blur() event … #2767

Closed pandamouse closed 1 month ago

pandamouse commented 9 months ago

…shorthand is deprecated

jquery-migrate-3.4.1.source.js:137 JQMIGRATE: jQuery.fn.keydown() event shorthand is deprecated etc etc

put the event related options into 'on' option

vakata commented 1 month ago

Sorry for taking such a long time to review this. DO you have any idea if this will break older jQuery versions?

pandamouse commented 1 month ago

Sorry for taking such a long time to review this. DO you have any idea if this will break older jQuery versions?

It will not break older jQuery version.

I'm just putting those event properties inside the 'on' property. You can see them use that as an example for jQuery 1.8 https://api.jquery.com/jQuery/#jQuery-html-attributes (when they first allow this kind of declaration where you chuck in any jQuery.fn as a property).