update on ready event listeners to jquery 3 compliant syntax #2690

Merged
merged 1 commit into from Jan 10, 2017

Projects

None yet

2 participants

@leggomuhgreggo
Collaborator
leggomuhgreggo commented Jan 10, 2017 edited

Addresses #2676 created by @KolyaKorruptis, which points out that jQuery 3 has deprecated .on('ready')

There were only two instances where slick used, and one was (I think superfluously?) leveraging jQuery's .off method. I deleted that line and updated the other to .ready(...function...)

Tests w/ latest jquery

Let me know if there's any issues!

Thanks

@kenwheeler kenwheeler merged commit 851da94 into kenwheeler:master Jan 10, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment