diff options
author | adam j. sontag <ajpiano@ajpiano.com> | 2010-06-21 13:50:02 -0400 |
---|---|---|
committer | adam j. sontag <ajpiano@ajpiano.com> | 2010-06-21 14:18:06 -0400 |
commit | a2bd8a53f3a750606abea9bbb6ee2302437d42f3 (patch) | |
tree | 531d315f0abfc28d599154088267e3a042fc97f4 /src/traversing.js | |
parent | 6a0942c9d5bbcc7eb6b953b8d7191b7bbd1e669f (diff) | |
download | jquery-a2bd8a53f3a750606abea9bbb6ee2302437d42f3.tar.gz jquery-a2bd8a53f3a750606abea9bbb6ee2302437d42f3.zip |
.closest() should return a unique set of elements, not duplicates of the same ancestor. Fixes #6700
Diffstat (limited to 'src/traversing.js')
-rw-r--r-- | src/traversing.js | 14 |
1 files changed, 9 insertions, 5 deletions
diff --git a/src/traversing.js b/src/traversing.js index fde7219bb..583726b10 100644 --- a/src/traversing.js +++ b/src/traversing.js @@ -53,9 +53,10 @@ jQuery.fn.extend({ }, closest: function( selectors, context ) { + var ret; if ( jQuery.isArray( selectors ) ) { - var ret = [], cur = this[0], match, matches = {}, selector, level = 1; - + var cur = this[0], match, matches = {}, selector, level = 1; + ret = []; if ( cur && selectors.length ) { for ( var i = 0, l = selectors.length; i < l; i++ ) { selector = selectors[i]; @@ -80,13 +81,12 @@ jQuery.fn.extend({ } } - return ret; + return ret.length > 1 ? jQuery.unique(ret) : ret; } var pos = jQuery.expr.match.POS.test( selectors ) ? jQuery( selectors, context || this.context ) : null; - - return this.map(function( i, cur ) { + ret = jQuery.map(this.get(),function( cur,i ) { while ( cur && cur.ownerDocument && cur !== context ) { if ( pos ? pos.index(cur) > -1 : jQuery(cur).is(selectors) ) { return cur; @@ -95,6 +95,10 @@ jQuery.fn.extend({ } return null; }); + + ret = ret.length > 1 ? jQuery.unique(ret) : ret; + + return this.pushStack( ret, "closest", selectors ); }, // Determine the position of an element within |