Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
Merge branch 'bug11720-nth-argument' of https://github.com/sindresorh…
…us/jquery into 1.8pre

* 'bug11720-nth-argument' of https://github.com/sindresorhus/jquery:
  Remove unnecessary arguments from .nth(). Fixes #11720
  • Loading branch information
rwaldron committed May 7, 2012
2 parents 0add0b3 + 4de7b54 commit fbee36b
Showing 1 changed file with 4 additions and 5 deletions.
9 changes: 4 additions & 5 deletions src/traversing.js
Expand Up @@ -186,10 +186,10 @@ jQuery.each({
return jQuery.dir( elem, "parentNode", until );
},
next: function( elem ) {
return jQuery.nth( elem, 2, "nextSibling" );
return jQuery.nth( elem, "nextSibling" );
},
prev: function( elem ) {
return jQuery.nth( elem, 2, "previousSibling" );
return jQuery.nth( elem, "previousSibling" );
},
nextAll: function( elem ) {
return jQuery.dir( elem, "nextSibling" );
Expand Down Expand Up @@ -260,12 +260,11 @@ jQuery.extend({
return matched;
},

nth: function( cur, result, dir, elem ) {
result = result || 1;
nth: function( cur, dir ) {
var num = 0;

for ( ; cur; cur = cur[dir] ) {
if ( cur.nodeType === 1 && ++num === result ) {
if ( cur.nodeType === 1 && num++ === 1 ) {
break;
}
}
Expand Down

0 comments on commit fbee36b

Please sign in to comment.