Bug 1210632 - Move the pointer to (0,0) with the New Measure Tool draft
authorShounak <shounak.bhandekar@gmail.com>
Sat, 30 Dec 2017 21:41:53 +0530
changeset 719015 ab3821e6b58514d5b8148bab2e37af1e52a0583a
parent 719014 2f05b6f751ccbf502af4587f8ea06d41ec3f773a
child 745683 eec97f0c284855f1839a5ea25bb6b39c195c2607
push id95136
push userbmo:pbrosset@mozilla.com
push dateThu, 11 Jan 2018 10:42:56 +0000
bugs1210632
milestone59.0a1
Bug 1210632 - Move the pointer to (0,0) with the New Measure Tool MozReview-Commit-ID: 7XhSGD1QbiA
devtools/server/actors/highlighters/measuring-tool.js
--- a/devtools/server/actors/highlighters/measuring-tool.js
+++ b/devtools/server/actors/highlighters/measuring-tool.js
@@ -514,18 +514,18 @@ MeasuringToolHighlighter.prototype = {
         break;
       case "mousemove":
         ({ scrollX, scrollY, innerWidth, innerHeight } = this.env.window);
         x = event.clientX + scrollX;
         y = event.clientY + scrollY;
 
         let { coords } = this;
 
-        x = Math.min(innerWidth + scrollX - 1, Math.max(0 + scrollX, x));
-        y = Math.min(innerHeight + scrollY, Math.max(1 + scrollY, y));
+        x = Math.min(innerWidth + scrollX, Math.max(scrollX, x));
+        y = Math.min(innerHeight + scrollY, Math.max(scrollY, y));
 
         this.setSize(x - coords.x, y - coords.y);
 
         let type = this._isDragging ? "size" : "position";
 
         this.showLabel(type);
         break;
       case "mouseleave":