@@ -707,21 +707,23 @@ * Returns the X/Y coordinates of element relative to the viewport. **/ function viewportOffset(forElement) { - var valueT = 0, valueL = 0; + var valueT = 0, valueL = 0, docBody = document.body; var element = forElement; do { valueT += element.offsetTop || 0; valueL += element.offsetLeft || 0; // Safari fix - if (element.offsetParent == document.body && + if (element.offsetParent == docBody && Element.getStyle(element, 'position') == 'absolute') break; } while (element = element.offsetParent); - element = forElement; - var tagName = element.tagName, O = Prototype.Browser.Opera; + element = forElement; do { - if (!O || tagName && tagName.toUpperCase() === 'BODY') { + // Opera < 9.5 sets scrollTop/Left on both HTML and BODY elements, + // other browsers set it only on the HTML element. The BODY element + // can be skipped since it's scrollTop/Left should always be 0. + if (element != docBody) { valueT -= element.scrollTop || 0; valueL -= element.scrollLeft || 0; } src/dom/layout.js effb34e743271fe0e1e06a40a21a90b514405185 Nick Stakenburg nick@nickstakenburg.com http://github.com/staaky/prototype/commit/2bfe5891f95413d4751a7f0b53c31046aa0e96b1 2bfe5891f95413d4751a7f0b53c31046aa0e96b1 2009-11-03T10:57:02-08:00 2009-11-03T10:57:02-08:00 Remove browser sniff from Element.viewportOffset, fixed the offset on nested elements in Opera < 9.5. 798326a8fa759a48b37dee2a253a4ccb0506e5a8 Nick Stakenburg nick@nickstakenburg.com