stop searching target on mousemove on transform (fabricjs#4442)

thiagocunha · Nov 8, 2017 · 8d59634 · 8d59634
1 parent a700bd1
commit 8d59634
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/src/mixins/canvas_events.mixin.js b/src/mixins/canvas_events.mixin.js
@@ -658,7 +658,7 @@
       else {
         this._transformObject(e);
       }
-      this._handleEvent(e, 'move', target);
+      this._handleEvent(e, 'move', this._currentTransform ? null : target);
     },
 
     /**

diff --git a/src/shapes/text.class.js b/src/shapes/text.class.js
@@ -853,8 +853,11 @@
         charBox = this.__charBounds[lineIndex][i];
         if (boxWidth === 0) {
           left += charBox.kernedWidth - charBox.width;
+          boxWidth += charBox.width;
+        }
+        else {
+          boxWidth += charBox.kernedWidth;
         }
-        boxWidth += charBox.kernedWidth;
         if (this.textAlign === 'justify' && !timeToRender) {
           if (this._reSpaceAndTab.test(line[i])) {
             timeToRender = true;
-Original file line number
+Diff line change
@@ Expand Up / @@ -658,7 +658,7 @@ @@
           else {
             this._transformObject(e);
           }
-          this._handleEvent(e, 'move', target);
+          this._handleEvent(e, 'move', this._currentTransform ? null : target);
         },
         /**
@@ Expand Down @@