fixed text underline/overline/linethrough not ignoring charSpacing at…

… the end of line (fabricjs#5006) (fabricjs#5007)
thiagocunha · May 30, 2018 · 5510149 · 5510149
1 parent 06ce3d0
commit 5510149
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/src/shapes/text.class.js b/src/shapes/text.class.js
@@ -1138,7 +1138,8 @@
           leftOffset = this._getLeftOffset(),
           topOffset = this._getTopOffset(), top,
           boxStart, boxWidth, charBox, currentDecoration,
-          maxHeight, currentFill, lastFill;
+          maxHeight, currentFill, lastFill,
+          charSpacing = this._getWidthOfCharSpacing();
 
       for (var i = 0, len = this._textLines.length; i < len; i++) {
         heightOfLine = this.getHeightOfLine(i);
@@ -1186,7 +1187,7 @@
         currentDecoration && currentFill && ctx.fillRect(
           leftOffset + lineLeftOffset + boxStart,
           top + this.offsets[type] * size + dy,
-          boxWidth,
+          boxWidth - charSpacing,
           this.fontSize / 15
         );
         topOffset += heightOfLine;