jQuery 1.8 deprecations

[dangoor]

This issue is tracking the verification that we're not going to be bitten by any of the following jQuery 1.8 deprecations/removals/changes. For details on any of the items, look at the jQuery 1.8 release announcement

• $(element).data("events") – undocumented, not likely in use
• Deferred.isResolved() and Deferred.isRejected() – replaced with Deferred.state()
• $(element).closest(Array) returning Array – only use the normal versions that return jQuery objects
• $.curCSS – should not be in use anyhow. just an alias for jQuery.css()
• $.attrFn – undocumented, should not be in use
• jqXHR.success, jqXHR.error
• Deferred with sync $.ajax
• jQuery#size() in favor of jQuery#length
• exclusive events (containing a !) on trigger
• $(document).bind("ready or $(document).on("ready
• .load, .unload, .error
• :text, :radio, :checkbox, :image, :password, :reset, :submit – shorthand for [type="text"] when searching

[dangoor]

I found the following deprecated function uses:

• jstree uses :checkbox
• jstree uses .sze()
• jstree has jqXHR.success
• jasmine-jquery uses .size()
• jquery.mockjax has jqXHR.success
• smart autocomplete has jqXHR.success

[pthiess]

Reviewed - This is part of this sprint story card to Upgrade jQuery. @dangoor Instead of putting a priority the importance is given by the milestone already which should be accurate, do you agree?

[dangoor]

Note that @TuckerWhitehouse has also spotted use of Deferred.pipe which was deprecated in 1.8:

https://github.com/adobe/brackets/wiki/Research:-jQuery-2.0-Upgrade

(I wish 1.8 had an upgrade guide! That one was not very obvious from the release notes.)

[dangoor]

jstree's use of .success is fine (it's not .success on a promise from jquery).

[dangoor]

We don't hit the code path of Smart Autocomplete's deprecated .success call. That's a good thing, because it's currently brought in as a submodule from its original source.

[dangoor]

I've put up a pull request for Smart Autocomplete's use of .success, but this need not be a blocker for our work here.

[njx]

@dangoor - looks like we can close this now since the remaining issue doesn't really need to be tracked. Feel free to reopen if necessary.