Sorry, I posted an old version of code. Here the new one:
jQuery.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 )
)));
};
On Oct 30, 6:55 am, Robert Katić <[email protected]> wrote:
> Using $.fn.add(selector) the context property remains the same.
> It's ok, but the given selector will be applied always with the
> default context (document).
>
> This is not correct for me if we are using jQuery with xml documents
> for example.
>
> $("user", xmlDoc).add("lusers");
>
> There is no way to add lusers of the xmlDoc document!
>
> Here an optional context argument would be useful:
>
> $("user", xmlDoc).add("lusers", xmlDoc);
>
> But even this is not ideal for me. If the context argument is not
> given (first example), which document would be used?
> I suppose the obvious answer is xmlDoc.
>
> An corrected implementation of add() would be something like this:
>
> jQuery.fn.add = function( selector ) {
> return this.pushStack( jQuery.unique( jQuery.merge(
> this.get(),
> typeof selector === "string" ?
> jQuery( selector, (this.context || this[0] ||
> 0).ownerDocument ) :
> jQuery.makeArray( selector )
> )));
>
> };
--
You received this message because you are subscribed to the Google Groups
"jQuery Development" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to
[email protected].
For more options, visit this group at
http://groups.google.com/group/jquery-dev?hl=en.
