feat(pr/fixLineWidth) Fix problem on fabric.Line controls and line.ge…

…tWidth()
fabricjs · Feb 24, 2017 · 6f2cbdc · 6f2cbdc
1 parent 53c0109
commit 6f2cbdc
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 11 deletions.
diff --git a/src/mixins/object_geometry.mixin.js b/src/mixins/object_geometry.mixin.js
@@ -482,11 +482,8 @@
      */
     _getNonTransformedDimensions: function() {
       var strokeWidth = this.strokeWidth,
-          w = this.width,
+          w = this.width + strokeWidth,
           h = this.height + strokeWidth;
-      if (this.type !== 'line') {
-        w = w + strokeWidth;
-      }
       return { x: w, y: h };
     },
 

diff --git a/src/shapes/line.class.js b/src/shapes/line.class.js
@@ -225,10 +225,8 @@
         if (dim.x === 0) {
           dim.y -= this.strokeWidth;
         }
-        if (dim.y === 0) {
-          dim.x -= this.strokeWidth;
-        }
       }
+      dim.x = this.width;
       return dim;
     },
 
@@ -247,10 +245,6 @@
         x1 = xMult * (this.width - this.strokeWidth) * 0.5;
         x2 = xMult * (this.width - this.strokeWidth) * -0.5;
       }
-      else {
-        x1 = xMult * this.width * 0.5;
-        x2 = xMult * this.width * -0.5;
-      }
       return {
         x1: x1,
         x2: x2,