Opened 10 years ago
Closed 9 years ago
#14343 closed feature (migrated)
Remove size & andSelf methods deprecated in jQuery 1.8
| Reported by: | m_gol | Owned by: | m_gol |
|---|---|---|---|
| Priority: | low | Milestone: | 1.next/2.next |
| Component: | core | Version: | 1.10.2 |
| Keywords: | Cc: | ||
| Blocked by: | Blocking: |
Description
It's high time to remove jQuery.fn.size and jQuery.fn.andSelf methods. They've been deprecated since 1.8 so removing them in 2.1/1.11 seems fine.
Change History (15)
comment:1 Changed 10 years ago by
| Owner: | set to m_gol |
|---|---|
| Status: | new → assigned |
comment:2 Changed 10 years ago by
| Milestone: | None → 1.11/2.1 |
|---|---|
| Priority: | undecided → low |
comment:3 Changed 10 years ago by
comment:4 Changed 10 years ago by
This seems like a lot of risk for very little benefit. Those two have been deprecated for a while I suspect there is still quite a bit of usage out there. It's pretty hard to tell.
comment:5 Changed 10 years ago by
What about jQuery.fn.size? It's hard to check it due to being a more common name. On the other hand, size code is much larger than andSelf one considering the latter is just an alias to another function.
comment:6 Changed 10 years ago by
Do we have any other api changes in this next release? I would rather batch these up since they can be disruptive and discourage upgrading.
comment:7 Changed 10 years ago by
Well, it's not officially public API but the whole jQuery.support behaves now differently, e.g. some former properties are now functions.
That's actually why I thought about 1.11/2.1 timeframe because we're changing so much with the switch to AMD etc. but maybe you're right most of these changes are internal and have little effect on our public API.
comment:8 Changed 10 years ago by
Officially, we haven't made any changes to our public API (support is documented as changeable). Every API change is annoying to our users, even if it makes sense.
I'd say we should just keep them in. Deprecated can refer to our recommendations and doesn't have to mean removable. They are easily removable with custom builds, if so desired and they take up little space anyway.
comment:9 Changed 10 years ago by
| Component: | unfiled → core |
|---|---|
| Milestone: | 1.11/2.1 → 2.next |
Let's hold off on this one until the following major release at least.
comment:10 Changed 10 years ago by
| Milestone: | 2.next → 1.next/2.next |
|---|
comment:11 Changed 10 years ago by
Neither of these seem all that important to me. The first step would be to get them into Migrate ASAP.
comment:14 Changed 9 years ago by
Actually, why don't we log deprecation errors for them in Core as well? .andSelf is just an alias so we'd need a wrapper but in .size() it'd be quite easy.
comment:15 Changed 9 years ago by
| Resolution: | → migrated |
|---|---|
| Status: | assigned → closed |
Migrated to https://github.com/jquery/jquery/issues/1749
Pull request: https://github.com/jquery/jquery/pull/1357