Skip to content
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

All: Remove usage of jQuery positional selectors #1904

Merged
merged 4 commits into from
Jan 22, 2020
Merged

Conversation

mgol
Copy link
Member

@mgol mgol commented Dec 30, 2019

jQuery positional selectors () have been deprecated in jQuery 3.4.0 and they'll be removed in jQuery 4.0.0. This PR removes their usage.

Most of the changes were possible without changing public API. However, dropping :even usage required a change to the header option of the accordion widget. I made it an optional function.

The polyfill for `.even()` & `.odd()` is added for jQuery <3.5.0.

There was no usage of the :odd selector in the code but the `.odd()` method
is also polyfilled for completeness.
Copy link
Member

@timmywil timmywil left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM if the tests pass

@mgol mgol merged commit 0c860b0 into jquery:master Jan 22, 2020
@mgol mgol deleted the pos-removal branch January 22, 2020 15:44
@mgol mgol added this to the 1.13 milestone Jun 10, 2020
mgol added a commit to mgol/api.jqueryui.com that referenced this pull request Sep 2, 2021
mgol added a commit to jquery/api.jqueryui.com that referenced this pull request Sep 2, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
4 participants