Ticket #11737 (closed feature: fixed)
Remove jQuery.sub
| Reported by: | dmethvin | Owned by: | dmethvin |
|---|---|---|---|
| Priority: | low | Milestone: | 1.9 |
| Component: | core | Version: | 1.7.2 |
| Keywords: | Cc: | ||
| Blocking: | Blocked by: |
Description
- It is quirky, per #11662.
- It is not commonly used.
- It shouldn't be commonly used.
- It is bytes we don't want to drag around.
- It can be a plugin.
Change History
comment:1 Changed 13 months ago by dmethvin
- Priority changed from undecided to low
- Status changed from new to open
- Component changed from unfiled to core
- Milestone changed from None to 1.8
comment:2 Changed 13 months ago by gibson042
I'm torn on this one. The main utility of jQuery.sub is for plugins, which makes converting it into a plugin of its own pointless. It's not much code to insert into a plugin, but on the other hand it dives pretty deep into partially undocumented internals, and asking plugin authors to copy in that block of code means not only redundancy but also missing out on things like a fix to #11662.
I was excited for the introduction of jQuery.sub, and think I'd like to try saving it.
comment:3 Changed 12 months ago by dmethvin
- Owner set to dmethvin
- Status changed from open to assigned
- Summary changed from Deprecate jQuery.sub to Remove jQuery.sub
This was actually deprecated in 1.7 so it can be removed in 1.8.
comment:4 Changed 11 months ago by dmethvin
- Milestone changed from 1.8 to 1.9
Pushed to 1.9 so we can do all the deprecations together.
comment:5 Changed 9 months ago by dmethvin
- Type changed from enhancement to feature
Bulk change from enhancement to feature.
comment:6 Changed 6 months ago by dmethvin
- Status changed from assigned to closed
- Resolution set to fixed
Moved to jquery-compat at https://github.com/jquery/jquery/commit/6b14cef7a85d22d979266a754b2a279ecd636b24
Please follow the bug reporting guidlines and use jsFiddle when providing test cases and demonstrations instead of pasting the code in the ticket.