_shouldSetNestedCoords

fabricjs · Apr 4, 2022 · c58a274 · c58a274
1 parent d8f8251
commit c58a274
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 2 deletions.
diff --git a/src/shapes/active_selection.class.js b/src/shapes/active_selection.class.js
@@ -52,6 +52,12 @@
       this.setCoords();
     },
 
+    /**
+     * @private
+     */
+    _shouldSetNestedCoords: function () {
+      return true;
+    },
 
     /**
      * @private

diff --git a/src/shapes/group.class.js b/src/shapes/group.class.js
@@ -129,6 +129,13 @@
       return this;
     },
 
+    /**
+     * @private
+     */
+    _shouldSetNestedCoords: function () {
+      return this.subTargetCheck;
+    },
+
     /**
      * Add objects
      * @param {...fabric.Object} objects
@@ -268,7 +275,7 @@
           )
         );
       }
-      object.setCoords();
+      this._shouldSetNestedCoords() && object.setCoords();
       object._set('group', this);
       object._set('canvas', this.canvas);
       this.interactive && this._watchObject(true, object);
@@ -439,7 +446,7 @@
      */
     setCoords: function () {
       this.callSuper('setCoords');
-      (this.subTargetCheck || this.type === 'activeSelection') && this.forEachObject(function (object) {
+      this._shouldSetNestedCoords() && this.forEachObject(function (object) {
         object.setCoords();
       });
     },