deac/jquery
1
0
Fork 0
Code Issues Pull requests Releases Wiki Activity

Switch to using createRange for element comparision instead of Array indexOf checks - thanks for the tip, Ioseb.

Browse source
This commit is contained in:
John Resig 2009-02-17 17:51:50 +00:00
parent 4b7e1c906f
commit d8b3bd700f
1 changed files with 9 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

16
src/selector.js
View file

@ -692,20 +692,22 @@ if ( document.documentElement.compareDocumentPosition ) {
} }
return ret; return ret;
}; };
} else if ( Array.prototype.indexOf ) { } else if ( "sourceIndex" in document.documentElement ) {
var indexOf = Array.prototype.indexOf,
allSort = document.getElementsByTagName("*");
sortOrder = function( a, b ) { sortOrder = function( a, b ) {
var ret = indexOf.call( allSort, a ) - indexOf.call( allSort, b ); var ret = a.sourceIndex - b.sourceIndex;
if ( ret === 0 ) { if ( ret === 0 ) {
hasDuplicate = true; hasDuplicate = true;
} }
return ret; return ret;
}; };
} else if ( "sourceIndex" in document.documentElement ) { } else if ( document.createRange ) {
sortOrder = function( a, b ) { sortOrder = function( a, b ) {
var ret = a.sourceIndex - b.sourceIndex; var aRange = a.ownerDocument.createRange(), bRange = b.ownerDocument.createRange();
aRange.selectNode(a);
aRange.collapse(true);
bRange.selectNode(b);
bRange.collapse(true);
var ret = aRange.compareBoundaryPoints(Range.START_TO_END, bRange);
if ( ret === 0 ) { if ( ret === 0 ) {
hasDuplicate = true; hasDuplicate = true;
} }