Fix the deprecated Position.withinIncludingScrollOffsets. [#196 state…

…:resolved]
prototypejs · Jul 16, 2008 · c84acf0 · c84acf0
1 parent a134431
commit c84acf0
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/CHANGELOG b/CHANGELOG
@@ -1,3 +1,5 @@
+* Fix the deprecated Position.withinIncludingScrollOffsets. [jddalton]
+
 * Consolidate document.body getOffsetParent checks into a private utility method. [staaky, jddalton]
 
 * Fix issues with Element#cumulativeScrollOffset and Opera <9.5. [staaky, jddalton]

diff --git a/src/deprecated.js b/src/deprecated.js
@@ -67,8 +67,8 @@ var Position = {
     var offsetcache = Element.cumulativeScrollOffset(element),
     dimensions = Element.getDimensions(element);
 
-    this.xcomp = x + offsetcache[0];
-    this.ycomp = y + offsetcache[1];
+    this.xcomp = x + offsetcache[0] - this.deltaX;
+    this.ycomp = y + offsetcache[1] - this.deltaY;
     this.offset = Element.cumulativeOffset(element);
 
     return (this.ycomp >= this.offset[1] &&