From 5ae1f64999c3562e011f1c666e306e5bb8f81e8e Mon Sep 17 00:00:00 2001
From: Andrea Bogazzi <andreabogazzi79@gmail.com>
Date: Tue, 17 Mar 2020 11:47:55 +0100
Subject: [PATCH 1/3] fix texbox changeWidth

---
 CHANGELOG.md            | 1 +
 src/controls.actions.js | 6 ++++--
 2 files changed, 5 insertions(+), 2 deletions(-)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 8fd8cc27808..1a01f035bcd 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -2,6 +2,7 @@
 
 ## [4.0.0-beta.8]
 
+- fix(controls): correctly handling the uniform scaling option [#6218](https://github.com/fabricjs/fabric.js/pull/6218)
 - fix(fabric.Object): fix activeSelection toDataURL canvas restore [#6216](https://github.com/fabricjs/fabric.js/pull/6216)
 - fix(svg_parsers): Add support for empty <style/> tags [#6169](https://github.com/fabricjs/fabric.js/pull/6169)
 - fix(SVG_export, text): Check font faces markup for objects within groups [#6195](https://github.com/fabricjs/fabric.js/pull/6195)
diff --git a/src/controls.actions.js b/src/controls.actions.js
index 4e9ad1d44f3..0571e424e45 100644
--- a/src/controls.actions.js
+++ b/src/controls.actions.js
@@ -443,8 +443,10 @@
 
   // currently unusued, needed for the textbox.
   function changeWidth(eventData, transform, x, y) {
-    var target = transform.target, localPoint = getLocalPoint(target, transform.originX, transform.originY, x, y);
-    transform.target.set('width', Math.abs(localPoint.x));
+    var target = transform.target, localPoint = getLocalPoint(target, transform.originX, transform.originY, x, y),
+        strokePadding = target.strokeWidth / (target.strokeUniform ? target.scaleX : 1),
+        newWidth = Math.abs(localPoint.x / target.scaleX) - strokePadding;
+    target.set('width', Math.max(newWidth, 0));
     return true;
   }
 

From ce69881140060d9830cc7f48c7e21c6ea2e09b15 Mon Sep 17 00:00:00 2001
From: Andrea Bogazzi <andreabogazzi79@gmail.com>
Date: Tue, 17 Mar 2020 11:48:05 +0100
Subject: [PATCH 2/3] fix texbox changeWidth

---
 test/unit/controls_handlers.js | 49 ++++++++++++++++++++++++++++++++++
 1 file changed, 49 insertions(+)
 create mode 100644 test/unit/controls_handlers.js

diff --git a/test/unit/controls_handlers.js b/test/unit/controls_handlers.js
new file mode 100644
index 00000000000..52cb1f131f3
--- /dev/null
+++ b/test/unit/controls_handlers.js
@@ -0,0 +1,49 @@
+(function(){
+  QUnit.module('fabric.controlHandlers', function(hooks) {
+    var eventData, transform;
+    var canvas = new fabric.Canvas(null);
+    hooks.beforeEach(function() {
+      var target = new fabric.Rect({ width: 100, height: 100 });
+      canvas.add(target);
+      eventData = {
+      };
+      transform = {
+        originX: 'left',
+        orginY: 'top',
+        target: target,
+      };
+    });
+    hooks.afterEach(function() {
+      canvas.clear();
+    });
+    QUnit.test('changeWidth changes the width', function(assert) {
+      assert.equal(transform.target.width, 100);
+      fabric.controlHandlers.changeWidth(eventData, transform, 200, 300);
+      assert.equal(transform.target.width, 199);
+    });
+    QUnit.test('changeWidth changes the width with big strokeWidth', function(assert) {
+      transform.target.strokeWidth = 15;
+      fabric.controlHandlers.changeWidth(eventData, transform, 200, 300);
+      assert.equal(transform.target.width, 185);
+    });
+    QUnit.test('changeWidth changes the width with big strokeWidth and strokeUniform', function(assert) {
+      transform.target.strokeWidth = 15;
+      transform.target.strokeUniform = true;
+      fabric.controlHandlers.changeWidth(eventData, transform, 200, 300);
+      assert.equal(transform.target.width, 185);
+    });
+    QUnit.test('changeWidth changes the width with big strokeWidth and strokeUniform + scaling', function(assert) {
+      transform.target.strokeWidth = 15;
+      transform.target.strokeUniform = true;
+      transform.target.scaleX = 3;
+      fabric.controlHandlers.changeWidth(eventData, transform, 200, 300);
+      assert.equal(Math.floor(transform.target.width), 61);
+    });
+    QUnit.test('changeWidth changes the width with big strokeWidth + scaling', function(assert) {
+      transform.target.strokeWidth = 15;
+      transform.target.scaleX = 3;
+      fabric.controlHandlers.changeWidth(eventData, transform, 200, 300);
+      assert.equal(Math.floor(transform.target.width), 51);
+    });
+  });
+})();

From 426e6094725b9a99bd397b713b0331dd4fe338ba Mon Sep 17 00:00:00 2001
From: Andrea Bogazzi <andreabogazzi79@gmail.com>
Date: Tue, 17 Mar 2020 11:50:06 +0100
Subject: [PATCH 3/3] added changelog

---
 CHANGELOG.md | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 1a01f035bcd..82236e48c4c 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -1,7 +1,7 @@
 # Changelog
 
 ## [4.0.0-beta.8]
-
+- fix(controls): Handle textbox width change properly [#6219](https://github.com/fabricjs/fabric.js/pull/6219)
 - fix(controls): correctly handling the uniform scaling option [#6218](https://github.com/fabricjs/fabric.js/pull/6218)
 - fix(fabric.Object): fix activeSelection toDataURL canvas restore [#6216](https://github.com/fabricjs/fabric.js/pull/6216)
 - fix(svg_parsers): Add support for empty <style/> tags [#6169](https://github.com/fabricjs/fabric.js/pull/6169)