From 2937d645cbf32e720d71f2f07f01e30fce0f9a88 Mon Sep 17 00:00:00 2001 From: Brandon Aaron Date: Mon, 30 Mar 2009 07:59:31 +0000 Subject: [PATCH] fix for #4439, offset.initialize can cause the page to jump --- src/offset.js | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/src/offset.js b/src/offset.js index 9ef5aa0..1ee1405 100644 --- a/src/offset.js +++ b/src/offset.js @@ -52,7 +52,7 @@ else jQuery.offset = { initialize: function() { - var body = document.body, container = document.createElement('div'), innerDiv, checkDiv, table, td, prop, bodyMarginTop = body.style.marginTop, + var body = document.body, container = document.createElement('div'), innerDiv, checkDiv, table, td, prop, bodyMarginTop = parseFloat(jQuery.curCSS(body, 'marginTop', true), 10) || 0, html = '
'; jQuery.extend( container.style, { position: 'absolute', top: 0, left: 0, margin: 0, border: 0, width: '1px', height: '1px', visibility: 'hidden' } ); @@ -71,9 +71,7 @@ jQuery.offset = { innerDiv.style.overflow = 'hidden', innerDiv.style.position = 'relative'; this.subtractsBorderForOverflowNotVisible = (checkDiv.offsetTop === -5); - body.style.marginTop = '1px'; - this.doesNotIncludeMarginInBodyOffset = (body.offsetTop === 0); - body.style.marginTop = bodyMarginTop; + this.doesNotIncludeMarginInBodyOffset = (body.offsetTop !== bodyMarginTop); body.removeChild(container); jQuery.offset.initialize = function(){}; -- 1.7.10.4