-
Notifications
You must be signed in to change notification settings - Fork 263
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
3.2/3.3 Deprecations #972
Comments
For the record, this should include |
From the Dave's list only jQuery.holdReady was deprecated in |
There's history in this one, I'll just edit it. |
Fixes jquery#947 Fixes jquery#949 Fixes jquery#950 Fixes jquery#1142 Ref jquery#970 Ref jquery#972
Fixes jquery#947 Fixes jquery#949 Fixes jquery#950 Fixes jquery#1142 Ref jquery#970 Ref jquery#972
Fixes jquery#947 Fixes jquery#949 Fixes jquery#950 Fixes jquery#1142 Ref jquery#970 Ref jquery#972
Most of the issues for recent jQueries have been handled in #1160. There are three left. The first is event shorthand methods; this is a bit tricky as the pages also serve as event descriptions. We could go by what @dmethvin suggested & just rewrite those pages, noting the deprecated signatures but generally using the full ones in the description. The other two: are not technically deprecated, though, it seems - they were never mentioned in a jQuery release blog post. If we want to deprecate, let's try to do it in jQuery 3.6.0. I'll create dedicated issues for them soon. |
It would be good to deprecate both of those for 4.0. We can easily patch On |
This commit replaces all instances in WooCommerce codebase (except included third-party libraries) where jQuery.fn.click( handler ) event shorthand was used. This shorthand was deprecated in jQuery 3.3 (see jquery/jquery#3214). The jQuery documentation was not updated yet (see jquery/jquery-migrate#288 and jquery/api.jquery.com#972). jQuery.click() was not deprecated and so it was not replaced.
This commit replaces all instances in WooCommerce codebase (except included third-party libraries) where jQuery.fn.click( handler ) event shorthand was used. This shorthand was deprecated in jQuery 3.3 (see jquery/jquery#3214). The jQuery documentation was not updated yet (see jquery/jquery-migrate#288 and jquery/api.jquery.com#972). jQuery.click() was not deprecated and so it was not replaced.
From the quoted text above, it is a bit unclear to me whether the intent is to mark these with a deprecation flag for 3.x or not. Would appreciate clarification - i.e. if it is preferred to not use the shorthand notation, that is one thing. If they will be marked as deprecated, then I take that to mean this functionality is expected to be removed in the next major version. Or, if these aliases are to be marked as deprecated, but the intent is to keep the aliases around even in jquery 4, I'd appreciate it if that is made clear as well. We are in the process of doing a jquery migration, and it would be helpful just to know exactly what decision has been made in this regard. |
`focus` and `delegate` have been deprecated respectively in jQuery 3.3.0 and 3.0.0 This commit replaces `focus` with `trigger` (available since v1.0.0) and `delegate` with `on` (available since v1.7.0) Ref: - https://api.jquery.com/category/deprecated/deprecated-3.0/ - jquery/api.jquery.com#972
`focus` and `delegate` have been deprecated respectively in jQuery 3.3.0 and 3.0.0 This commit replaces `focus` with `trigger` (available since v1.0.0) and `delegate` with `on` (available since v1.7.0) Ref: - https://api.jquery.com/category/deprecated/deprecated-3.0/ - jquery/api.jquery.com#972
Everything here has now been documented. I've also verified all deprecated APIs from 3.4-3.6 have also been documented. Closing! |
Similar to #970 this is the rollup of all the deprecations planned for jQuery 3.2 and 3.3 to be sure we don't miss any. We have tickets in the Core tracker for these and a few have their own docs tickets already.
jQuery 3.2
jQuery 3.3
event.which (see also Deprecate(EDIT by @mgol: this API got undeprecated)which
property of the event object jquery#2337)For the event shorthand methods (
.click()
,.mousemove()
, etc.) I think we should use the current API entries and explain that.click()
is now best done as.on("click", fn)
or.trigger("click")
. However let's just rewrite the examples and discussion in those pages to use.on()
or.trigger()
and describe the behavior of the event since I think people still want/need those event-specific details.The text was updated successfully, but these errors were encountered: