Skip to content

All: Drop support for jQuery 1.6.x #1422

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

Closed
wants to merge 2 commits into from
Closed

All: Drop support for jQuery 1.6.x #1422

wants to merge 2 commits into from

Conversation

jzaefferer
Copy link
Member

Affects core, effects, position and widget. Only position has unit tests that fail with jQuery 1.6 without the workaround.

Drops the 1.6.x copies jQuery of jQuery and removes them from the select in the testsuites.

Fixes #10723

Affects core, effects, position and widget. Only position has unit tests
that fail with jQuery 1.6 without the workaround.

Drops the 1.6.x copies jQuery of jQuery and removes them from the select
in the testsuites.

Fixes #10723
@mikesherov
Copy link
Member

I suppose bind to on is for another PR?

@mikesherov
Copy link
Member

LGTM

@arschmitz
Copy link
Member

@mikesherov i have an old branch im going to update and submit that removes our dependency on the event alias core module might make sense to do it at the same time. Since now in all supported versions bind is an event alias?

@tjvantoll
Copy link
Member

👍

@scottgonzalez
Copy link
Member

There are support comments in tests too.

@jzaefferer
Copy link
Member Author

Added a commit that updates core and tooltip tests. Verified both suites tests pass everywhere using browserstack-runner locally.

@scottgonzalez
Copy link
Member

Looks good.

@jzaefferer jzaefferer deleted the drop-jquery1-6 branch January 13, 2015 14:07
@jzaefferer
Copy link
Member Author

Landed. @mikesherov @arschmitz yeah, updating bind/on should be done elsewhere. There's currently no ticket (for 1.12) for that.

jzaefferer pushed a commit that referenced this pull request Jan 22, 2015
Removes 1.6 versions from "All" tests and from the TestSwarm job.

Ref #10723
Ref gh-1422
Closes gh-1434
Closes gh-1435
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

5 participants