Changeset 257 for trunk/common

Timestamp:

12/19/07 20:16:39 (2 years ago)

Author:

ddavis

Message:

the max function returns undef if any values are undef. I wrapped all values passed to max with finiteInt. This fixes window scroll calculations. BugzID:64047

Files:

• trunk/common/DOM.js (modified) (2 diffs)

trunk/common/DOM.js

@@ -426 +426 @@
         
         return {
-            left: max( w.pageXOffset, de.scrollLeft, b.scrollLeft, w.scrollX ),
-            top: max( w.pageYOffset, de.scrollTop, b.scrollTop, w.scrollY )
+            left: max( finiteInt( w.pageXOffset ), finiteInt( de.scrollLeft ), finiteInt( b.scrollLeft ), finiteInt( w.scrollX ) ),
+            top: max( finiteInt( w.pageYOffset ), finiteInt( de.scrollTop ), finiteInt( b.scrollTop ), finiteInt( w.scrollY ) )
         };
     },
@@ -439 +439 @@
         };
         /* window scroll is fixed in safari 3; devicePixelRatio is a safari 3 only object */
-        if ( navigator.userAgent.toLowerCase().match(/webkit/) && !window.devicePixelRatio ) {
+        if ( window.devicePixelRatio || !navigator.userAgent.toLowerCase().match(/webkit/) ) {
             var s = DOM.getWindowScroll( window );
             pos.x += s.left;