fabric-vectr 6.7.2 → 6.7.4

package/dist/index.js

@@ -366,7 +366,7 @@
   }
   const cache = new Cache();
 
-  var version = "6.7.2";
+  var version = "6.7.4";
 
   // use this syntax so babel plugin see this import here
   const VERSION = version;
@@ -5002,8 +5002,9 @@
         withShadow,
         additionalTransform
       } = arguments.length > 1 && arguments[1] !== undefined ? arguments[1] : {};
+      var skipShadow = true;
       let clipPath = this.clipPath;
-      const styleInfo = noStyle ? '' : `style="${this.getSvgStyles()}" `,
+      const styleInfo = noStyle ? '' : `style="${this.getSvgStyles(skipShadow)}" `,
         shadowInfo = withShadow ? `style="${this.getSvgFilter()}" ` : '',
         vectorEffect = this.strokeUniform ? 'vector-effect="non-scaling-stroke" ' : '',
         absoluteClipPath = clipPath && clipPath.absolutePositioned,
@@ -9548,6 +9549,47 @@
      */
     constructor(options) {
       super();
+      /**
+       * The object's controls' position in viewport coordinates
+       * Calculated by {@link Control#positionHandler} and {@link Control#calcCornerCoords}, depending on {@link padding}.
+       * `corner/touchCorner` describe the 4 points forming the interactive area of the corner.
+       * Used to draw and locate controls.
+       */
+      /**
+       * keeps the value of the last hovered corner during mouse move.
+       * 0 is no corner, or 'mt', 'ml', 'mtr' etc..
+       * It should be private, but there is no harm in using it as
+       * a read-only property.
+       * this isn't cleaned automatically. Non selected objects may have wrong values
+       * @type [string]
+       */
+      /**
+       * a map of control visibility for this object.
+       * this was left when controls were introduced to not break the api too much
+       * this takes priority over the generic control visibility
+       */
+      /**
+       * holds the controls for the object.
+       * controls are added by default_controls.js
+       */
+      /**
+       * internal boolean to signal the code that the object is
+       * part of the move action.
+       */
+      /**
+       * A boolean used from the gesture module to keep tracking of a scaling
+       * action when there is no scaling transform in place.
+       * This is an edge case and is used twice in all codebase.
+       * Probably added to keep track of some performance issues
+       * @TODO use git blame to investigate why it was added
+       * DON'T USE IT. WE WILL TRY TO REMOVE IT
+       */
+      /**
+       * When set to `true`, object's group should not cache
+       * @type Boolean
+       * @default false
+       */
+      _defineProperty(this, "preventGroupCache", void 0);
       Object.assign(this, this.constructor.createControls(), InteractiveFabricObject.ownDefaults);
       this.setOptions(options);
     }
@@ -10041,41 +10083,6 @@
       // for subclasses
     }
   }
-  /**
-   * The object's controls' position in viewport coordinates
-   * Calculated by {@link Control#positionHandler} and {@link Control#calcCornerCoords}, depending on {@link padding}.
-   * `corner/touchCorner` describe the 4 points forming the interactive area of the corner.
-   * Used to draw and locate controls.
-   */
-  /**
-   * keeps the value of the last hovered corner during mouse move.
-   * 0 is no corner, or 'mt', 'ml', 'mtr' etc..
-   * It should be private, but there is no harm in using it as
-   * a read-only property.
-   * this isn't cleaned automatically. Non selected objects may have wrong values
-   * @type [string]
-   */
-  /**
-   * a map of control visibility for this object.
-   * this was left when controls were introduced to not break the api too much
-   * this takes priority over the generic control visibility
-   */
-  /**
-   * holds the controls for the object.
-   * controls are added by default_controls.js
-   */
-  /**
-   * internal boolean to signal the code that the object is
-   * part of the move action.
-   */
-  /**
-   * A boolean used from the gesture module to keep tracking of a scaling
-   * action when there is no scaling transform in place.
-   * This is an edge case and is used twice in all codebase.
-   * Probably added to keep track of some performance issues
-   * @TODO use git blame to investigate why it was added
-   * DON'T USE IT. WE WILL TRY TO REMOVE IT
-   */
   _defineProperty(InteractiveFabricObject, "ownDefaults", interactiveObjectDefaultValues);
 
   /***
@@ -12073,7 +12080,7 @@
       const ownCache = FabricObject.prototype.shouldCache.call(this);
       if (ownCache) {
         for (let i = 0; i < this._objects.length; i++) {
-          if (this._objects[i].willDrawShadow()) {
+          if (this._objects[i].willDrawShadow() || this._objects[i].preventGroupCache) {
             this.ownCaching = false;
             return false;
           }
@@ -14144,6 +14151,8 @@
         const obj = this.searchPossibleTargets(objects, pointer);
         if (obj) {
           if (obj instanceof Group) {
+            // Let text in group can be edited after isolation
+            obj.interactive = true;
             this.isolatedObject = obj;
             this.discardActiveObject();
             // The newly created group object has its child node coords initialized, otherwise it cannot be selected.
@@ -19816,8 +19825,8 @@
     _getCacheCanvasDimensions() {
       const dims = super._getCacheCanvasDimensions();
       const fontSize = this.fontSize;
-      dims.width += fontSize * dims.zoomX;
-      dims.height += fontSize * dims.zoomY;
+      dims.width += fontSize * dims.zoomX * 2;
+      dims.height += fontSize * dims.zoomY * 2;
       return dims;
     }
 
@@ -23786,6 +23795,7 @@
         // clear cache and re-calculate height
         this.height = this.calcTextHeight();
       }
+      this.preventGroupCache = this.path ? true : false;
     }
 
     /**