Modify ↓
Ticket #5434 (closed enhancement: fixed)
$.fn.add( selector, context )
| Reported by: | rkatic | Owned by: | |
|---|---|---|---|
| Priority: | major | Milestone: | 1.4 |
| Component: | core | Version: | 1.3.2 |
| Keywords: | add, context | Cc: | |
| Blocking: | Blocked by: |
Description
- The optional context will be applied on evolving the selector.
- If the context argument is not given, then at lest use the same document.
Here the expected implementation:
Query.fn.add = function( selector, context ) {
return this.pushStack( jQuery.unique( jQuery.merge(
this.get(),
typeof selector === "string" ?
jQuery( selector, context
|| this.context && this.context.ownerDocument || this.context)
|| this[0] && (this[0].ownerDocument || this[0]) ) :
jQuery.makeArray( selector )
)));
};
Change History
Please follow the bug reporting guidlines and use jsFiddle when providing test cases and demonstrations instead of pasting the code in the ticket.
Note: See
TracTickets for help on using
tickets.

I've implemented this change along with test cases. Thanks for the preliminary patch! http://github.com/jquery/jquery/commit/b0fe380cf89564305646bbd55d1fd7bd210fd591