Bug Tracker

Opened 5 years ago

Closed 5 years ago

#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:
Blocked by: Blocking:

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 (6)

comment:1 Changed 5 years ago by dmethvin

Component: unfiledcore
Milestone: None1.8
Priority: undecidedlow
Status: newopen

comment:2 Changed 5 years 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 5 years ago by dmethvin

Owner: set to dmethvin
Status: openassigned
Summary: Deprecate jQuery.subRemove jQuery.sub

This was actually deprecated in 1.7 so it can be removed in 1.8.

comment:4 Changed 5 years ago by dmethvin

Milestone: 1.81.9

Pushed to 1.9 so we can do all the deprecations together.

comment:5 Changed 5 years ago by dmethvin

Type: enhancementfeature

Bulk change from enhancement to feature.

comment:6 Changed 5 years ago by dmethvin

Resolution: fixed
Status: assignedclosed
Note: See TracTickets for help on using tickets.