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

Warn about jQuery.nodeName removal #246

Closed
timmywil opened this issue Jan 16, 2017 · 5 comments
Closed

Warn about jQuery.nodeName removal #246

timmywil opened this issue Jan 16, 2017 · 5 comments
Assignees
Milestone

Comments

@timmywil
Copy link
Member

timmywil commented Jan 16, 2017

See jquery/jquery#3475. To be removed in 4.0.

@dmethvin
Copy link
Member

This can't be implemented until core refactors to eliminate internal calls to jQuery.nodeName. Otherwise it will throw warnings even if there are no uses in user code.

@Krinkle
Copy link
Member

Krinkle commented Mar 20, 2017

Internal callers have been fixed in jquery/jquery@ac9e301 and released in 3.2.0.

@dmethvin
Copy link
Member

Right but in general we have said that jQuery Migrate 3.x works with jQuery 3.x. If you used an older jQuery than 3.2 it would be throwing warnings frequently. Is that considered okay? In the past we tried to avoid that.

@dmethvin
Copy link
Member

It also complicates our test matrix since we couldn't test Migrate 3.2 with a jQuery older than 3.2 and assert no warnings.

@Krinkle
Copy link
Member

Krinkle commented Mar 20, 2017

@dmethvin I agree 3.x/3.x compat should be kept. It's our responsibility to keep that flexibility supported to reduce user confusion and upgrade smoothness. If we do want to add the warnings before Migrate 4, it would have to involve a version check.

Personally, I the complexity of a version check is worth it. The earlier the better. It would give platforms like WordPress and MediaWiki the opportunity to warn about these methods much earlier to end-users; instead of only after jQuery 4 is released.

The more users we (indirectly) enable to comfortably migrate early, the better our position is to change things in the jQuery 4 API for the benefit of everyone.


⇒ Continuing at #254

dmethvin added a commit to dmethvin/jquery-migrate that referenced this issue Sep 12, 2017
@dmethvin dmethvin self-assigned this Mar 22, 2018
dmethvin added a commit to dmethvin/jquery-migrate that referenced this issue Mar 30, 2018
@dmethvin dmethvin added this to the 3.1.0 milestone May 29, 2019
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

No branches or pull requests

3 participants