Skip to content
Permalink
Browse files

Further reduce historical cruft. Removes the untested and unnec. jQue…

…ry.nth()

Signed-off-by: Rick Waldron <waldron.rick@gmail.com>
  • Loading branch information...
rwaldron committed May 7, 2012
1 parent fbee36b commit 2af3642cd5f968d64c40b83250e00e077b48c086
Showing with 13 additions and 14 deletions.
  1. +13 −14 src/traversing.js
@@ -174,6 +174,17 @@ function isDisconnected( node ) {
return !node || !node.parentNode || node.parentNode.nodeType === 11;
}

function nth( cur, dir ) {
var num = 0;

This comment has been minimized.

Copy link
@Raynos

Raynos May 7, 2012

num variable is useless. nth is a silly name compared to findElementNode or something that's actually descriptive


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

jQuery.each({
parent: function( elem ) {
var parent = elem.parentNode;
@@ -186,10 +197,10 @@ jQuery.each({
return jQuery.dir( elem, "parentNode", until );
},
next: function( elem ) {
return jQuery.nth( elem, "nextSibling" );
return nth( elem, "nextSibling" );
},
prev: function( elem ) {
return jQuery.nth( elem, "previousSibling" );
return nth( elem, "previousSibling" );
},
nextAll: function( elem ) {
return jQuery.dir( elem, "nextSibling" );
@@ -260,18 +271,6 @@ jQuery.extend({
return matched;
},

nth: function( cur, dir ) {
var num = 0;

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

return cur;
},

sibling: function( n, elem ) {
var r = [];

0 comments on commit 2af3642

Please sign in to comment.
You can’t perform that action at this time.