fix(): account for padding

closes #6989 https://jsfiddle.net/p7xzwg4o/4/
fabricjs · Mar 8, 2022 · 47f018d · 47f018d
1 parent f70019b
commit 47f018d
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/src/mixins/object_interactivity.mixin.js b/src/mixins/object_interactivity.mixin.js
@@ -156,7 +156,7 @@
       var bbox = fabric.util.sizeAfterTransform(this.width, this.height, options),
           strokeWidth = this.strokeWidth,
           strokeUniform = this.strokeUniform,
-          borderScaleFactor = this.borderScaleFactor,
+          borderScaleFactor = this.borderScaleFactor + this.padding * 2,
           width =
             bbox.x + strokeWidth * (strokeUniform ? this.canvas.getZoom() : options.scaleX) + borderScaleFactor,
           height =

diff --git a/src/shapes/group.class.js b/src/shapes/group.class.js
@@ -727,7 +727,7 @@
         var objCenter, sizeVector, min, max, a, b;
         objects.forEach(function (object, i) {
           objCenter = object.getRelativeCenterPoint();
-          sizeVector = object._getTransformedDimensions().scalarDivideEquals(2);
+          sizeVector = object._getTransformedDimensions().scalarDivideEquals(2).scalarAddEquals(object.padding);
           if (object.angle) {
             var rad = degreesToRadians(object.angle),
                 sin = Math.abs(fabric.util.sin(rad)),