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

Updates for jQuery 3.0+ and deprecations #4221

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

smartexcan
Copy link

@smartexcan smartexcan commented Nov 15, 2022

jQuery migrate warnings

  • Replace deprecated $.type with typeof, Array.isArray or $.isPlainObject where appropriate.
  • Fix invalid self-closing html tags (div, button, ...)
  • Fix hard-coded number values in calls to jQuery.fn.css( (property), value )
  • Replace deprecated document ready signature in index.html

slick.min.js was not changed since it was still missing some previous changes to main file, it should probably just be re-minified.

replace $.type with `typeof`, `Array.isArray` or `$.isPlainObject` where needed
Fix invalid self-closing html tags
@ribeiroeder
Copy link

@ahmadalfy can you check this? jQuery 4 completely removed support for the "$.type" and "$.isArray" functions used in slick's current code and this PR fixes them with their native equivalents.

@andrewcbi
Copy link

@pablo67340 This should be merged into next build. I've been using this in production since Dec 2022.

This was referenced May 2, 2024
@markvantilburg
Copy link
Contributor

@ahmadalfy might be a good idea to merge this one and then check what is still missing comparing with #4007

#4071 is difficult to diff due to the changed spaces.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants