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
Align jQuery.merge & push.apply calls #5298
Comments
mgol
added a commit
to mgol/jquery
that referenced
this issue
Jul 12, 2023
…5298 jquerygh-5298 is about analysing `push.apply` & `jQuery.merge` usage more comprehensively.
2 tasks
More interesting context about the limits of |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Description
We have a number
jQuery.merge
&push.apply
calls that seem interchangeable in many cases. We should investigate their usage and perhaps align to one of them. Ifpush.apply
works fine, we could investigate simplifying thejQuery.merge
implementation by leveraging it.We used to have a comment about that in
domManip.js
but we should investigate all places.When investigating, take #4320 into account.
Link to test case
The text was updated successfully, but these errors were encountered: