fix: consider scroll position for the coordinates

0x04 · Sep 25, 2024 · f835147 · f835147
1 parent ecec447
commit f835147
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/src/classes/mouse-grid.mjs b/src/classes/mouse-grid.mjs
@@ -30,9 +30,11 @@ export class MouseGrid {
   }
 
   getPointFromEvent(event) {
+    const { scrollTop, scrollLeft } = event.target
+
     return this.getPointFromCoordinate(
-      event.offsetX - this.gridRect.x,
-      event.offsetY - this.gridRect.y
+      scrollLeft + event.offsetX - this.gridRect.x,
+      scrollTop + event.offsetY - this.gridRect.y
     )
   }
 }
diff --git a/src/components/canvas.mjs b/src/components/canvas.mjs
@@ -34,7 +34,7 @@ export class Canvas {
 
     element.innerText = data.toString()
 
-    this.rectangle = new Rectangle(0, 0, element.clientWidth, element.clientHeight)
+    this.rectangle = new Rectangle(0, 0, element.scrollWidth, element.scrollHeight)
     this.mouseGrid = new MouseGrid(this.rectangle, this.metrics.rectangle)
   }