Re-ordering the args to setOffset() in case people were relying upon the old order (doubtful).

This commit is contained in:
jeresig 2010-01-07 14:08:32 -05:00
parent 4fa00369f0
commit 0e5370b89c

View file

@ -8,7 +8,7 @@ if ( "getBoundingClientRect" in document.documentElement ) {
if ( options ) { if ( options ) {
return this.each(function( i ) { return this.each(function( i ) {
jQuery.offset.setOffset( this, i, options ); jQuery.offset.setOffset( this, options, i );
}); });
} }
@ -34,7 +34,7 @@ if ( "getBoundingClientRect" in document.documentElement ) {
if ( options ) { if ( options ) {
return this.each(function( i ) { return this.each(function( i ) {
jQuery.offset.setOffset( this, i, options ); jQuery.offset.setOffset( this, options, i );
}); });
} }
@ -137,7 +137,7 @@ jQuery.offset = {
return { top: top, left: left }; return { top: top, left: left };
}, },
setOffset: function( elem, i, options ) { setOffset: function( elem, options, i ) {
// set position first, in-case top/left are set even on static elem // set position first, in-case top/left are set even on static elem
if ( /static/.test( jQuery.curCSS( elem, "position" ) ) ) { if ( /static/.test( jQuery.curCSS( elem, "position" ) ) ) {
elem.style.position = "relative"; elem.style.position = "relative";