fabric 6.0.0 → 6.0.2

package/src/controls/scale.ts

@@ -17,6 +17,7 @@ import {
 } from './util';
 import { wrapWithFireEvent } from './wrapWithFireEvent';
 import { wrapWithFixedAnchor } from './wrapWithFixedAnchor';
+import { SCALE_X, SCALE_Y, SCALING } from '../constants';
 
 type ScaleTransform = Transform & {
   gestureScale?: number;
@@ -206,12 +207,12 @@ function scaleObject(
   const oldScaleX = target.scaleX,
     oldScaleY = target.scaleY;
   if (!by) {
-    !isLocked(target, 'lockScalingX') && target.set('scaleX', scaleX);
-    !isLocked(target, 'lockScalingY') && target.set('scaleY', scaleY);
+    !isLocked(target, 'lockScalingX') && target.set(SCALE_X, scaleX);
+    !isLocked(target, 'lockScalingY') && target.set(SCALE_Y, scaleY);
   } else {
     // forbidden cases already handled on top here.
-    by === 'x' && target.set('scaleX', scaleX);
-    by === 'y' && target.set('scaleY', scaleY);
+    by === 'x' && target.set(SCALE_X, scaleX);
+    by === 'y' && target.set(SCALE_Y, scaleY);
   }
   return oldScaleX !== target.scaleX || oldScaleY !== target.scaleY;
 }
@@ -271,16 +272,16 @@ const scaleObjectY: TransformActionHandler<ScaleTransform> = (
 };
 
 export const scalingEqually = wrapWithFireEvent(
-  'scaling',
+  SCALING,
   wrapWithFixedAnchor(scaleObjectFromCorner)
 );
 
 export const scalingX = wrapWithFireEvent(
-  'scaling',
+  SCALING,
   wrapWithFixedAnchor(scaleObjectX)
 );
 
 export const scalingY = wrapWithFireEvent(
-  'scaling',
+  SCALING,
   wrapWithFixedAnchor(scaleObjectY)
 );

package/src/controls/scaleSkew.ts

@@ -4,6 +4,7 @@ import type {
   TPointerEvent,
   TransformActionHandler,
 } from '../EventTypeDefs';
+import { SCALE_X, SCALE_Y, SKEW_X, SKEW_Y } from '../constants';
 import type { FabricObject } from '../shapes/Object/FabricObject';
 import type { TAxisKey } from '../typedefs';
 import { scaleCursorStyleHandler, scalingX, scalingY } from './scale';
@@ -26,11 +27,11 @@ export const scaleOrSkewActionName: ControlCallback<
   const isAlternative = isAltAction(eventData, fabricObject);
   if (control.x === 0) {
     // then is scaleY or skewX
-    return isAlternative ? 'skewX' : 'scaleY';
+    return isAlternative ? SKEW_X : SCALE_Y;
   }
   if (control.y === 0) {
     // then is scaleY or skewX
-    return isAlternative ? 'skewY' : 'scaleX';
+    return isAlternative ? SKEW_Y : SCALE_X;
   }
   return '';
 };

package/src/controls/skew.ts

@@ -19,7 +19,14 @@ import {
 } from './util';
 import { wrapWithFireEvent } from './wrapWithFireEvent';
 import { wrapWithFixedAnchor } from './wrapWithFixedAnchor';
-import { CENTER } from '../constants';
+import {
+  CENTER,
+  SCALE_X,
+  SCALE_Y,
+  SKEWING,
+  SKEW_X,
+  SKEW_Y,
+} from '../constants';
 
 export type SkewTransform = Transform & { skewingSide: -1 | 1 };
 
@@ -36,16 +43,16 @@ const AXIS_KEYS: Record<
 > = {
   x: {
     counterAxis: 'y',
-    scale: 'scaleX',
-    skew: 'skewX',
+    scale: SCALE_X,
+    skew: SKEW_X,
     lockSkewing: 'lockSkewingX',
     origin: 'originX',
     flip: 'flipX',
   },
   y: {
     counterAxis: 'x',
-    scale: 'scaleY',
-    skew: 'skewY',
+    scale: SCALE_Y,
+    skew: SKEW_Y,
     lockSkewing: 'lockSkewingY',
     origin: 'originY',
     flip: 'flipY',
@@ -130,7 +137,7 @@ function skewObject(
       dimAfter = target._getTransformedDimensions(),
       compensationFactor = skewX !== 0 ? dimBefore.x / dimAfter.x : 1;
     compensationFactor !== 1 &&
-      target.set('scaleX', compensationFactor * scaleX);
+      target.set(SCALE_X, compensationFactor * scaleX);
   }
 
   return changed;
@@ -188,7 +195,7 @@ function skewHandler(
     origin = -skewingDirection * 0.5 + 0.5;
 
   const finalHandler = wrapWithFireEvent<SkewTransform>(
-    'skewing',
+    SKEWING,
     wrapWithFixedAnchor((eventData, transform, x, y) =>
       skewObject(axis, transform, new Point(x, y))
     )

package/src/filters/Blur.ts

@@ -136,7 +136,7 @@ export class Blur extends BaseFilter<'Blur', BlurOwnProps> {
     uniformLocations: TWebGLUniformLocationMap
   ) {
     const delta = this.chooseRightDelta();
-    gl.uniform2fv(uniformLocations.delta, delta);
+    gl.uniform2fv(uniformLocations.uDelta, delta);
   }
 
   isNeutralState() {

package/src/filters/Pixelate.ts

@@ -27,7 +27,7 @@ export class Pixelate extends BaseFilter<'Pixelate', PixelateOwnProps> {
 
   static defaults = pixelateDefaultValues;
 
-  static uniformLocations = ['uBlockSize'];
+  static uniformLocations = ['uBlocksize'];
 
   /**
    * Apply the Pixelate operation to a Uint8ClampedArray representing the pixels of an image.

package/src/parser/attributes.ts

@@ -1,3 +1,5 @@
+import { FILL, STROKE } from '../constants';
+
 /**
  * Attributes parsed from all SVG elements
  * @type array
@@ -5,11 +7,11 @@
 export const SHARED_ATTRIBUTES = [
   'display',
   'transform',
-  'fill',
+  FILL,
   'fill-opacity',
   'fill-rule',
   'opacity',
-  'stroke',
+  STROKE,
   'stroke-dasharray',
   'stroke-linecap',
   'stroke-dashoffset',

package/src/parser/elements_parser.ts

@@ -10,7 +10,7 @@ import {
 import { removeTransformMatrixForSvgParsing } from '../util/transform_matrix_removal';
 import type { FabricObject } from '../shapes/Object/FabricObject';
 import { Point } from '../Point';
-import { CENTER } from '../constants';
+import { CENTER, FILL, STROKE } from '../constants';
 import { getGradientDefs } from './getGradientDefs';
 import { getCSSRules } from './getCSSRules';
 import type { LoadImageOptions } from '../util';
@@ -77,8 +77,8 @@ export class ElementsParser {
         this.options,
         this.cssRules
       );
-      this.resolveGradient(obj, el, 'fill');
-      this.resolveGradient(obj, el, 'stroke');
+      this.resolveGradient(obj, el, FILL);
+      this.resolveGradient(obj, el, STROKE);
       if (obj instanceof FabricImage && obj._originalElement) {
         removeTransformMatrixForSvgParsing(
           obj,

package/src/parser/normalizeValue.ts

@@ -1,7 +1,7 @@
 import { multiplyTransformMatrices } from '../util/misc/matrix';
 import { parseUnit } from '../util/misc/svgParsing';
 import { parseTransformAttribute } from './parseTransformAttribute';
-import { CENTER, LEFT, RIGHT, NONE } from '../constants';
+import { CENTER, LEFT, RIGHT, NONE, FILL, STROKE } from '../constants';
 
 export function normalizeValue(
   attr: string,
@@ -12,7 +12,7 @@ export function normalizeValue(
   const isArray = Array.isArray(value);
   let parsed: number | number[];
   let ouputValue: string | null | boolean | number[] | number = value;
-  if ((attr === 'fill' || attr === 'stroke') && value === NONE) {
+  if ((attr === FILL || attr === STROKE) && value === NONE) {
     ouputValue = '';
   } else if (attr === 'strokeUniform') {
     return value === 'non-scaling-stroke';
@@ -48,13 +48,13 @@ export function normalizeValue(
     // parseUnit returns px and we convert it to em
     parsed = (parseUnit(value, fontSize) / fontSize) * 1000;
   } else if (attr === 'paintFirst') {
-    const fillIndex = value.indexOf('fill');
-    const strokeIndex = value.indexOf('stroke');
-    ouputValue = 'fill';
+    const fillIndex = value.indexOf(FILL);
+    const strokeIndex = value.indexOf(STROKE);
+    ouputValue = FILL;
     if (fillIndex > -1 && strokeIndex > -1 && strokeIndex < fillIndex) {
-      ouputValue = 'stroke';
+      ouputValue = STROKE;
     } else if (fillIndex === -1 && strokeIndex > -1) {
-      ouputValue = 'stroke';
+      ouputValue = STROKE;
     }
   } else if (attr === 'href' || attr === 'xlink:href' || attr === 'font') {
     return value;

package/src/parser/parseTransformAttribute.ts

@@ -1,4 +1,4 @@
-import { iMatrix } from '../constants';
+import { ROTATE, SCALE, SKEW_X, SKEW_Y, iMatrix } from '../constants';
 import { reNum } from './constants';
 import type { TMat2D } from '../typedefs';
 import { cleanupSvgAttribute } from '../util/internals/cleanupSvgAttribute';
@@ -70,16 +70,16 @@ export function parseTransformAttribute(attributeValue: string): TMat2D {
       case 'translate':
         matrix = createTranslateMatrix(arg0, arg1);
         break;
-      case 'rotate':
+      case ROTATE:
         matrix = createRotateMatrix({ angle: arg0 }, { x: arg1, y: arg2 });
         break;
-      case 'scale':
+      case SCALE:
         matrix = createScaleMatrix(arg0, arg1);
         break;
-      case 'skewX':
+      case SKEW_X:
         matrix = createSkewXMatrix(arg0);
         break;
-      case 'skewY':
+      case SKEW_Y:
         matrix = createSkewYMatrix(arg0);
         break;
       case 'matrix':

package/src/shapes/Circle.ts

@@ -9,6 +9,7 @@ import { FabricObject, cacheProperties } from './Object/FabricObject';
 import type { Abortable, TClassProperties, TOptions } from '../typedefs';
 import type { FabricObjectProps, SerializedObjectProps } from './Object/types';
 import type { CSSRules } from '../parser/typedefs';
+import { SCALE_X, SCALE_Y } from '../constants';
 
 interface UniqueCircleProps {
   /**
@@ -134,7 +135,7 @@ export class Circle<
    * @return {Number}
    */
   getRadiusX(): number {
-    return this.get('radius') * this.get('scaleX');
+    return this.get('radius') * this.get(SCALE_X);
   }
 
   /**
@@ -142,7 +143,7 @@ export class Circle<
    * @return {Number}
    */
   getRadiusY(): number {
-    return this.get('radius') * this.get('scaleY');
+    return this.get('radius') * this.get(SCALE_Y);
   }
 
   /**

package/src/shapes/Ellipse.ts

@@ -1,4 +1,4 @@
-import { twoMathPi } from '../constants';
+import { SCALE_X, SCALE_Y, twoMathPi } from '../constants';
 import { SHARED_ATTRIBUTES } from '../parser/attributes';
 import { parseAttributes } from '../parser/parseAttributes';
 import type { Abortable, TClassProperties, TOptions } from '../typedefs';
@@ -98,7 +98,7 @@ export class Ellipse<
    * @return {Number}
    */
   getRx() {
-    return this.get('rx') * this.get('scaleX');
+    return this.get('rx') * this.get(SCALE_X);
   }
 
   /**
@@ -106,7 +106,7 @@ export class Ellipse<
    * @return {Number}
    */
   getRy() {
-    return this.get('ry') * this.get('scaleY');
+    return this.get('ry') * this.get(SCALE_Y);
   }
 
   /**

package/src/shapes/IText/DraggableTextDelegate.ts

@@ -10,7 +10,7 @@ import { setStyle } from '../../util/dom_style';
 import { cloneDeep } from '../../util/internals/cloneDeep';
 import type { TextStyleDeclaration } from '../Text/StyledText';
 import { getDocumentFromElement } from '../../util/dom_misc';
-import { NONE } from '../../constants';
+import { CHANGED, NONE } from '../../constants';
 
 /**
  * #### Dragging IText/Textbox Lifecycle
@@ -326,7 +326,7 @@ export class DraggableTextDelegate {
       target.hiddenTextarea!.value = target.text;
       target._updateTextarea();
       target.hiddenTextarea!.focus();
-      target.fire('changed', {
+      target.fire(CHANGED, {
         index: insertAt + selectionStartOffset,
         action: 'drop',
       });
@@ -364,7 +364,7 @@ export class DraggableTextDelegate {
             target.hiddenTextarea &&
               (target.hiddenTextarea.value = target.text);
             target._updateTextarea();
-            target.fire('changed', {
+            target.fire(CHANGED, {
               index: selectionStart,
               action: 'dragend',
             });

package/src/shapes/IText/IText.ts

@@ -16,7 +16,7 @@ import {
   JUSTIFY_LEFT,
   JUSTIFY_RIGHT,
 } from '../Text/constants';
-import { CENTER, LEFT, RIGHT } from '../../constants';
+import { CENTER, FILL, LEFT, RIGHT } from '../../constants';
 import type { ObjectToCanvasElementOptions } from '../Object/Object';
 
 type CursorBoundaries = {
@@ -521,7 +521,7 @@ export class IText<
     }
     ctx.fillStyle =
       this.cursorColor ||
-      (this.getValueOfPropertyAt(lineIndex, charIndex, 'fill') as string);
+      (this.getValueOfPropertyAt(lineIndex, charIndex, FILL) as string);
     ctx.globalAlpha = this._currentCursorOpacity;
     ctx.fillRect(
       boundaries.left + boundaries.leftOffset - cursorWidth / 2,
@@ -677,7 +677,7 @@ export class IText<
    */
   getCurrentCharColor(): string | TFiller | null {
     const cp = this._getCurrentCharIndex();
-    return this.getValueOfPropertyAt(cp.l, cp.c, 'fill');
+    return this.getValueOfPropertyAt(cp.l, cp.c, FILL);
   }
 
   /**

package/src/shapes/IText/ITextBehavior.ts

@@ -13,7 +13,7 @@ import type { TextStyleDeclaration } from '../Text/StyledText';
 import type { SerializedTextProps, TextProps } from '../Text/Text';
 import type { TOptions } from '../../typedefs';
 import { getDocumentFromElement } from '../../util/dom_misc';
-import { LEFT, RIGHT, reNewline } from '../../constants';
+import { LEFT, MODIFIED, RIGHT, reNewline } from '../../constants';
 import type { IText } from './IText';
 
 /**
@@ -696,7 +696,7 @@ export abstract class ITextBehavior<
       this.setCoords();
     }
     this.fire('editing:exited');
-    isTextChanged && this.fire('modified');
+    isTextChanged && this.fire(MODIFIED);
     if (this.canvas) {
       this.canvas.fire('text:editing:exited', {
         target: this as unknown as IText,

package/src/shapes/IText/ITextKeyBehavior.ts

@@ -7,7 +7,7 @@ import type { TKeyMapIText } from './constants';
 import type { TOptions } from '../../typedefs';
 import type { TextProps, SerializedTextProps } from '../Text/Text';
 import { getDocumentFromElement } from '../../util/dom_misc';
-import { LEFT, RIGHT } from '../../constants';
+import { CHANGED, LEFT, RIGHT } from '../../constants';
 import type { IText } from './IText';
 import type { TextStyleDeclaration } from '../Text/StyledText';
 
@@ -173,7 +173,7 @@ export abstract class ITextKeyBehavior<
     }
     const updateAndFire = () => {
       this.updateFromTextArea();
-      this.fire('changed');
+      this.fire(CHANGED);
       if (this.canvas) {
         this.canvas.fire('text:changed', { target: this as unknown as IText });
         this.canvas.requestRenderAll();

package/src/shapes/Image.ts

@@ -25,7 +25,7 @@ import { FabricObject, cacheProperties } from './Object/FabricObject';
 import type { FabricObjectProps, SerializedObjectProps } from './Object/types';
 import type { ObjectEvents } from '../EventTypeDefs';
 import { WebGLFilterBackend } from '../filters/WebGLFilterBackend';
-import { NONE } from '../constants';
+import { FILL, NONE } from '../constants';
 import { getDocumentFromElement } from '../util/dom_misc';
 import type { CSSRules } from '../parser/typedefs';
 import type { Resize } from '../filters/Resize';
@@ -419,7 +419,7 @@ export class FabricImage<
       ];
       this.fill = origFill;
     }
-    if (this.paintFirst !== 'fill') {
+    if (this.paintFirst !== FILL) {
       svgString = svgString.concat(strokeSvg, imageMarkup);
     } else {
       svgString = svgString.concat(imageMarkup, strokeSvg);

package/src/shapes/Object/AnimatableObject.ts

@@ -1,4 +1,5 @@
 import type { TColorArg } from '../../color/typedefs';
+import { FILL, STROKE } from '../../constants';
 import type { ObjectEvents } from '../../EventTypeDefs';
 import type { TAnimation } from '../../util/animation/animate';
 import { animate, animateColor } from '../../util/animation/animate';
@@ -17,7 +18,7 @@ export abstract class AnimatableObject<
    * List of properties to consider for animating colors.
    * @type String[]
    */
-  static colorProperties: string[] = ['fill', 'stroke', 'backgroundColor'];
+  static colorProperties: string[] = [FILL, STROKE, 'backgroundColor'];
 
   /**
    * Animates object's properties

package/src/shapes/Object/FabricObjectSVGExportMixin.ts

@@ -1,7 +1,7 @@
 import type { TSVGReviver } from '../../typedefs';
 import { uid } from '../../util/internals/uid';
 import { colorPropToSVG, matrixToSVG } from '../../util/misc/svgParsing';
-import { NONE } from '../../constants';
+import { FILL, NONE, STROKE } from '../../constants';
 import type { FabricObject } from './FabricObject';
 import { isFiller } from '../../util/typeAssertions';
 
@@ -34,8 +34,8 @@ export class FabricObjectSVGExportMixin {
       opacity = typeof this.opacity !== 'undefined' ? this.opacity : '1',
       visibility = this.visible ? '' : ' visibility: hidden;',
       filter = skipShadow ? '' : this.getSvgFilter(),
-      fill = colorPropToSVG('fill', this.fill),
-      stroke = colorPropToSVG('stroke', this.stroke);
+      fill = colorPropToSVG(FILL, this.fill),
+      stroke = colorPropToSVG(STROKE, this.stroke);
 
     return [
       stroke,
@@ -247,8 +247,6 @@ export class FabricObjectSVGExportMixin {
   }
 
   addPaintOrder(this: FabricObjectSVGExportMixin & FabricObject) {
-    return this.paintFirst !== 'fill'
-      ? ` paint-order="${this.paintFirst}" `
-      : '';
+    return this.paintFirst !== FILL ? ` paint-order="${this.paintFirst}" ` : '';
   }
 }

package/src/shapes/Object/InteractiveObject.ts

@@ -19,6 +19,7 @@ import type { FabricObjectProps } from './types/FabricObjectProps';
 import type { TFabricObjectProps, SerializedObjectProps } from './types';
 import { createObjectDefaultControls } from '../../controls/commonControls';
 import { interactiveObjectDefaultValues } from './defaultValues';
+import { SCALE } from '../../constants';
 
 export type TOCoord = Point & {
   corner: TCornerPoint;
@@ -182,7 +183,7 @@ export class InteractiveFabricObject<
       if (
         this === (target as unknown as this) &&
         action &&
-        action.startsWith('scale')
+        action.startsWith(SCALE)
       ) {
         return false;
       }

package/src/shapes/Object/Object.ts

@@ -5,6 +5,9 @@ import {
   CENTER,
   iMatrix,
   LEFT,
+  SCALE_X,
+  SCALE_Y,
+  STROKE,
   TOP,
   VERSION,
 } from '../../constants';
@@ -282,19 +285,21 @@ export class FabricObject<
   }
 
   /**
-   * The class type. Used to identify which class this is.
-   * This is used for serialization purposes and internally it can be used
-   * to identify classes. As a developer you could use `instance of Class`
-   * but to avoid importing all the code and blocking tree shaking we try
-   * to avoid doing that.
+   * The class type.
+   * This is used for serialization and deserialization purposes and internally it can be used
+   * to identify classes.
+   * When we transform a class in a plain JS object we need a way to recognize which class it was,
+   * and the type is the way we do that. It has no other purposes and you should not give one.
+   * Hard to reach on instances and please do not use to drive instance's logic (this.constructor.type).
+   * To idenfity a class use instanceof class ( instanceof Rect ).
+   * We do not do that in fabricJS code because we want to try to have code splitting possible.
    */
   static type = 'FabricObject';
 
   /**
    * Legacy identifier of the class. Prefer using utils like isType or instanceOf
    * Will be removed in fabric 7 or 8.
-   * The setter exists because is very hard to catch all the ways in which a type value
-   * could be set in the instance
+   * The setter exists to avoid type errors in old code and possibly current deserialization code.
    * @TODO add sustainable warning message
    * @type string
    * @deprecated
@@ -698,10 +703,10 @@ export class FabricObject<
    * @param {*} value
    */
   _set(key: string, value: any) {
-    if (key === 'scaleX' || key === 'scaleY') {
+    if (key === SCALE_X || key === SCALE_Y) {
       value = this._constrainScale(value);
     }
-    if (key === 'scaleX' && value < 0) {
+    if (key === SCALE_X && value < 0) {
       this.flipX = !this.flipX;
       value *= -1;
     } else if (key === 'scaleY' && value < 0) {
@@ -847,7 +852,7 @@ export class FabricObject<
    */
   needsItsOwnCache() {
     if (
-      this.paintFirst === 'stroke' &&
+      this.paintFirst === STROKE &&
       this.hasFill() &&
       this.hasStroke() &&
       !!this.shadow
@@ -1181,7 +1186,7 @@ export class FabricObject<
    * @param {CanvasRenderingContext2D} ctx Context to render on
    */
   _renderPaintInOrder(ctx: CanvasRenderingContext2D) {
-    if (this.paintFirst === 'stroke') {
+    if (this.paintFirst === STROKE) {
       this._renderStroke(ctx);
       this._renderFill(ctx);
     } else {

package/src/shapes/Object/ObjectGeometry.ts

@@ -6,7 +6,7 @@ import type {
   TOriginX,
   TOriginY,
 } from '../../typedefs';
-import { iMatrix } from '../../constants';
+import { SCALE_X, SCALE_Y, iMatrix } from '../../constants';
 import { Intersection } from '../../Intersection';
 import { Point } from '../../Point';
 import { makeBoundingBoxFromPoints } from '../../util/misc/boundingBoxFromPoints';
@@ -356,8 +356,8 @@ export class ObjectGeometry<EventSpec extends ObjectEvents = ObjectEvents>
    * @return {void}
    */
   scale(value: number): void {
-    this._set('scaleX', value);
-    this._set('scaleY', value);
+    this._set(SCALE_X, value);
+    this._set(SCALE_Y, value);
     this.setCoords();
   }
 

package/src/shapes/Object/defaultValues.ts

@@ -1,4 +1,13 @@
-import { TOP, LEFT } from '../../constants';
+import {
+  TOP,
+  LEFT,
+  SCALE_Y,
+  SCALE_X,
+  SKEW_X,
+  SKEW_Y,
+  FILL,
+  STROKE,
+} from '../../constants';
 import type { TClassProperties } from '../../typedefs';
 import type { InteractiveFabricObject } from './InteractiveObject';
 import type { FabricObject } from './Object';
@@ -6,8 +15,8 @@ import type { FabricObject } from './Object';
 export const stateProperties = [
   TOP,
   LEFT,
-  'scaleX',
-  'scaleY',
+  SCALE_X,
+  SCALE_Y,
   'flipX',
   'flipY',
   'originX',
@@ -17,13 +26,13 @@ export const stateProperties = [
   'globalCompositeOperation',
   'shadow',
   'visible',
-  'skewX',
-  'skewY',
+  SKEW_X,
+  SKEW_Y,
 ];
 
 export const cacheProperties = [
-  'fill',
-  'stroke',
+  FILL,
+  STROKE,
   'strokeWidth',
   'strokeDashArray',
   'width',
@@ -60,7 +69,7 @@ export const fabricObjectDefaultValues: Partial<
   strokeUniform: false,
   padding: 0,
   opacity: 1,
-  paintFirst: 'fill',
+  paintFirst: FILL,
   fill: 'rgb(0,0,0)',
   fillRule: 'nonzero',
   stroke: null,

package/src/shapes/Object/types/ControlProps.ts

@@ -31,7 +31,7 @@ export interface ControlProps {
    * Color of controlling corners of an object (when it's active and transparentCorners false)
    * @since 1.6.2
    * @type String
-   * @default null
+   * @default ''
    */
   cornerStrokeColor: string;
 
@@ -41,7 +41,7 @@ export interface ControlProps {
    * And you can swap it with one of the alternative proposed with the control api
    * @since 1.6.2
    * @type 'rect' | 'circle'
-   * @default rect
+   * @default 'rect'
    * @deprecated
    */
   cornerStyle: 'rect' | 'circle';
@@ -50,6 +50,7 @@ export interface ControlProps {
    * Array specifying dash pattern of an object's control (hasBorder must be true)
    * @since 1.6.2
    * @type Array | null
+   * @default null
    */
   cornerDashArray: number[] | null;
 
@@ -63,7 +64,7 @@ export interface ControlProps {
   /**
    * When set to `false`, object's controls are not displayed and can not be used to manipulate object
    * @type Boolean
-   * @default
+   * @default true
    */
   hasControls: boolean;
 }

package/src/shapes/Polyline.spec.ts

@@ -106,7 +106,6 @@ describe('Polyline', () => {
       });
     });
   });
-
   it('should safeguard passing points in options', () => {
     expect(new Polyline(points, { points: [{ x: 1, y: 1 }] })).toEqual(
       expect.objectContaining({