From: jeresig Date: Mon, 1 Mar 2010 21:26:57 +0000 (-0500) Subject: Fixed some cases where expression, expression was used and shouldn't be. X-Git-Url: http://git.asbjorn.it/?a=commitdiff_plain;h=fc08d0ea8dc881ed56102eb14e3ad9c54cccc34e;p=jquery.git Fixed some cases where expression, expression was used and shouldn't be. --- diff --git a/src/offset.js b/src/offset.js index fbefd7a..365b35a 100644 --- a/src/offset.js +++ b/src/offset.js @@ -75,7 +75,8 @@ if ( "getBoundingClientRect" in document.documentElement ) { left += parseFloat( computedStyle.borderLeftWidth ) || 0; } - prevOffsetParent = offsetParent, offsetParent = elem.offsetParent; + prevOffsetParent = offsetParent; + offsetParent = elem.offsetParent; } if ( jQuery.offset.subtractsBorderForOverflowNotVisible && computedStyle.overflow !== "visible" ) { @@ -116,12 +117,16 @@ jQuery.offset = { this.doesNotAddBorder = (checkDiv.offsetTop !== 5); this.doesAddBorderForTableAndCells = (td.offsetTop === 5); - checkDiv.style.position = "fixed", checkDiv.style.top = "20px"; + checkDiv.style.position = "fixed"; + checkDiv.style.top = "20px"; + // safari subtracts parent border width here which is 5px this.supportsFixedPosition = (checkDiv.offsetTop === 20 || checkDiv.offsetTop === 15); checkDiv.style.position = checkDiv.style.top = ""; - innerDiv.style.overflow = "hidden", innerDiv.style.position = "relative"; + innerDiv.style.overflow = "hidden"; + innerDiv.style.position = "relative"; + this.subtractsBorderForOverflowNotVisible = (checkDiv.offsetTop === -5); this.doesNotIncludeMarginInBodyOffset = (body.offsetTop !== bodyMarginTop);