core: fixed offset method for FF3 - FF3 seems to have problems reporting the correct...
[jquery.git] / src / offset.js
index 23923c3..f0adae0 100644 (file)
@@ -3,24 +3,24 @@
 // http://jquery.com/plugins/project/dimensions
 jQuery.fn.offset = function() {
        var left = 0, top = 0, elem = this[0], results;
-       
+
        if ( elem ) with ( jQuery.browser ) {
-               var parent       = elem.parentNode, 
+               var parent       = elem.parentNode,
                    offsetChild  = elem,
-                   offsetParent = elem.offsetParent, 
+                   offsetParent = elem.offsetParent,
                    doc          = elem.ownerDocument,
                    safari2      = safari && parseInt(version) < 522 && !/adobeair/i.test(userAgent),
                    css          = jQuery.curCSS,
                    fixed        = css(elem, "position") == "fixed";
-       
+
                // Use getBoundingClientRect if available
-               if ( elem.getBoundingClientRect ) {
+               if ( !(mozilla && elem == document.body) && elem.getBoundingClientRect ) {
                        var box = elem.getBoundingClientRect();
-               
+
                        // Add the document scroll offsets
                        add(box.left + Math.max(doc.documentElement.scrollLeft, doc.body.scrollLeft),
                                box.top  + Math.max(doc.documentElement.scrollTop,  doc.body.scrollTop));
-               
+
                        // IE adds the HTML element's border, by default it is medium which is 2px
                        // IE 6 and 7 quirks mode the border width is overwritable by the following css html { border: 0; }
                        // IE 7 standards mode, the border is always 2px
@@ -28,54 +28,54 @@ jQuery.fn.offset = function() {
                        // However, in IE6 and 7 quirks mode the clientLeft and clientTop properties are not updated when overwriting it via CSS
                        // Therefore this method will be off by 2px in IE while in quirksmode
                        add( -doc.documentElement.clientLeft, -doc.documentElement.clientTop );
-       
+
                // Otherwise loop through the offsetParents and parentNodes
                } else {
-               
+
                        // Initial element offsets
                        add( elem.offsetLeft, elem.offsetTop );
-                       
+
                        // Get parent offsets
                        while ( offsetParent ) {
                                // Add offsetParent offsets
                                add( offsetParent.offsetLeft, offsetParent.offsetTop );
-                       
+
                                // Mozilla and Safari > 2 does not include the border on offset parents
                                // However Mozilla adds the border for table or table cells
                                if ( mozilla && !/^t(able|d|h)$/i.test(offsetParent.tagName) || safari && !safari2 )
                                        border( offsetParent );
-                                       
+
                                // Add the document scroll offsets if position is fixed on any offsetParent
                                if ( !fixed && css(offsetParent, "position") == "fixed" )
                                        fixed = true;
-                       
+
                                // Set offsetChild to previous offsetParent unless it is the body element
                                offsetChild  = /^body$/i.test(offsetParent.tagName) ? offsetChild : offsetParent;
                                // Get next offsetParent
                                offsetParent = offsetParent.offsetParent;
                        }
-               
+
                        // Get parent scroll offsets
                        while ( parent && parent.tagName && !/^body|html$/i.test(parent.tagName) ) {
                                // Remove parent scroll UNLESS that parent is inline or a table to work around Opera inline/table scrollLeft/Top bug
                                if ( !/^inline|table.*$/i.test(css(parent, "display")) )
                                        // Subtract parent scroll offsets
                                        add( -parent.scrollLeft, -parent.scrollTop );
-                       
+
                                // Mozilla does not add the border for a parent that has overflow != visible
                                if ( mozilla && css(parent, "overflow") != "visible" )
                                        border( parent );
-                       
+
                                // Get next parent
                                parent = parent.parentNode;
                        }
-               
+
                        // Safari <= 2 doubles body offsets with a fixed position element/offsetParent or absolutely positioned offsetChild
                        // Mozilla doubles body offsets with a non-absolutely positioned offsetChild
-                       if ( (safari2 && (fixed || css(offsetChild, "position") == "absolute")) || 
+                       if ( (safari2 && (fixed || css(offsetChild, "position") == "absolute")) ||
                                (mozilla && css(offsetChild, "position") != "absolute") )
                                        add( -doc.body.offsetLeft, -doc.body.offsetTop );
-                       
+
                        // Add the document scroll offsets if position is fixed
                        if ( fixed )
                                add(Math.max(doc.documentElement.scrollLeft, doc.body.scrollLeft),
@@ -91,8 +91,8 @@ jQuery.fn.offset = function() {
        }
 
        function add(l, t) {
-               left += parseInt(l) || 0;
-               top += parseInt(t) || 0;
+               left += parseInt(l, 10) || 0;
+               top += parseInt(t, 10) || 0;
        }
 
        return results;
@@ -101,34 +101,36 @@ jQuery.fn.offset = function() {
 
 jQuery.fn.extend({
        position: function() {
-               var left = 0, top = 0, elem = this[0], offset, parentOffset, offsetParent, results;
-               
-               if (elem) {
+               var left = 0, top = 0, results;
+
+               if ( this[0] ) {
                        // Get *real* offsetParent
-                       offsetParent = this.offsetParent();
-                       
+                       var offsetParent = this.offsetParent(),
+
                        // Get correct offsets
-                       offset       = this.offset();
-                       parentOffset = offsetParent.offset();
-                       
+                       offset       = this.offset(),
+                       parentOffset = /^body|html$/i.test(offsetParent[0].tagName) ? { top: 0, left: 0 } : offsetParent.offset();
+
                        // Subtract element margins
-                       offset.top  -= parseInt( jQuery.curCSS(elem, 'marginTop', true) ) || 0;
-                       offset.left -= parseInt( jQuery.curCSS(elem, 'marginLeft', true) ) || 0;
-                       
+                       // note: when an element has margin: auto the offsetLeft and marginLeft 
+                       // are the same in Safari causing offset.left to incorrectly be 0
+                       offset.top  -= num( this, 'marginTop' );
+                       offset.left -= num( this, 'marginLeft' );
+
                        // Add offsetParent borders
-                       parentOffset.top  += parseInt( jQuery.curCSS(offsetParent[0], 'borderTopWidth', true) ) || 0;
-                       parentOffset.left += parseInt( jQuery.curCSS(offsetParent[0], 'borderLeftWidth', true) ) || 0;
-                       
+                       parentOffset.top  += num( offsetParent, 'borderTopWidth' );
+                       parentOffset.left += num( offsetParent, 'borderLeftWidth' );
+
                        // Subtract the two offsets
                        results = {
                                top:  offset.top  - parentOffset.top,
                                left: offset.left - parentOffset.left
                        };
                }
-               
+
                return results;
        },
-       
+
        offsetParent: function() {
                var offsetParent = this[0].offsetParent;
                while ( offsetParent && (!/^body|html$/i.test(offsetParent.tagName) && jQuery.css(offsetParent, 'position') == 'static') )
@@ -140,26 +142,28 @@ jQuery.fn.extend({
 
 // Create scrollLeft and scrollTop methods
 jQuery.each( ['Left', 'Top'], function(i, name) {
-       jQuery.fn[ 'scroll' + name ] = function(val) {
+       var method = 'scroll' + name;
+       
+       jQuery.fn[ method ] = function(val) {
                if (!this[0]) return;
-               
+
                return val != undefined ?
-               
+
                        // Set the scroll offset
                        this.each(function() {
                                this == window || this == document ?
-                                       window.scrollTo( 
-                                               name == 'Left' ? val : jQuery(window)[ 'scrollLeft' ](),
-                                               name == 'Top'  ? val : jQuery(window)[ 'scrollTop'  ]()
+                                       window.scrollTo(
+                                               !i ? val : jQuery(window).scrollLeft(),
+                                                i ? val : jQuery(window).scrollTop()
                                        ) :
-                                       this[ 'scroll' + name ] = val;
+                                       this[ method ] = val;
                        }) :
-                       
+
                        // Return the scroll offset
                        this[0] == window || this[0] == document ?
-                               self[ (name == 'Left' ? 'pageXOffset' : 'pageYOffset') ] ||
-                                       jQuery.boxModel && document.documentElement[ 'scroll' + name ] ||
-                                       document.body[ 'scroll' + name ] :
-                               this[0][ 'scroll' + name ];
+                               self[ i ? 'pageYOffset' : 'pageXOffset' ] ||
+                                       jQuery.boxModel && document.documentElement[ method ] ||
+                                       document.body[ method ] :
+                               this[0][ method ];
        };
-});
\ No newline at end of file
+});