fabric - npm Package Compare versions

Comparing version 6.0.0-beta14 to 6.0.0-beta15

dist/fabric.d.ts

@@ -33,3 +33,14 @@ export { getEnv, getFabricDocument, getFabricWindow, setEnv } from './src/env';
 export { PatternBrush } from './src/brushes/PatternBrush';
-export { FabricObject, _Object as Object, } from './src/shapes/Object/FabricObject';
+export { FabricObject, 
+/**
+ * @deprecated Due to a naming conflict with the
+ * {@link https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Object JS API},
+ * `fabric.Object` has been renamed to `FabricObject`
+ *
+ * @example
+ * import { Object } from 'fabric'; // deprecated
+ * import { FabricObject } from 'fabric'; // migration path
+ *
+ */
+FabricObject as Object, } from './src/shapes/Object/FabricObject';
 export type { TFabricObjectProps, FabricObjectProps, SerializedObjectProps, } from './src/shapes/Object/types';
@@ -51,3 +62,14 @@ export type { SerializedLineProps } from './src/shapes/Line';
 export type { GraphemeBBox, SerializedTextProps, TPathAlign, TPathSide, TextProps, } from './src/shapes/Text/Text';
-export { Text, FabricText } from './src/shapes/Text/Text';
+export { FabricText, 
+/**
+ * @deprecated Due to a naming conflict with the
+ * {@link https://developer.mozilla.org/en-US/docs/Web/API/Text/Text Web API},
+ * `fabric.Text` has been renamed to `FabricText`
+ *
+ * @example
+ * import { Text } from 'fabric'; // deprecated
+ * import { FabricText } from 'fabric'; // migration path
+ *
+ */
+FabricText as Text, } from './src/shapes/Text/Text';
 export type { ITextProps, SerializedITextProps, } from './src/shapes/IText/IText';
@@ -62,3 +84,14 @@ export { IText } from './src/shapes/IText/IText';
 export { ActiveSelection } from './src/shapes/ActiveSelection';
-export { Image, FabricImage } from './src/shapes/Image';
+export { FabricImage, 
+/**
+ * @deprecated Due to a naming conflict with the
+ * {@link https://developer.mozilla.org/en-US/docs/Web/API/HTMLImageElement/Image Web API},
+ * `fabric.Image` has been renamed to `FabricImage`
+ *
+ * @example
+ * import { Image } from 'fabric'; // deprecated
+ * import { FabricImage } from 'fabric'; // migration path
+ *
+ */
+FabricImage as Image, } from './src/shapes/Image';
 export type { ImageSource, SerializedImageProps, ImageProps, } from './src/shapes/Image';
@@ -65,0 +98,0 @@ export { createCollectionMixin } from './src/Collection';

dist/src/shapes/Image.d.ts

@@ -313,11 +313,3 @@ import type { BaseFilter } from '../filters/BaseFilter';
 }
-/**
- * @deprecated The old fabric.Image class can't be imported as Image because of conflict with Web API.
- * https://developer.mozilla.org/en-US/docs/Web/API/HTMLImageElement/Image
- * For this reason it has been renamed to FabricImage.
- * Please use `import { FabricImage }` in place of `import { Image as FabricImage }`
- */
-export declare class Image extends FabricImage {
-}
 export {};
 //# sourceMappingURL=Image.d.ts.map

dist/src/shapes/Object/FabricObject.d.ts

@@ -11,10 +11,2 @@ import type { ObjectEvents } from '../../EventTypeDefs';
 export { cacheProperties } from './defaultValues';
-/**
- * @deprecated The old fabric.Object class can't be imported as Object because of conflict with the JS api
- * https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Object
- * For this reason it has been renamed to FabricObject.
- * Please use `import { FabricObject }` in place of `import { Object as FabricObject }`
- */
-export declare class _Object extends FabricObject {
-}
 //# sourceMappingURL=FabricObject.d.ts.map

dist/src/shapes/Text/Text.d.ts

@@ -698,11 +698,3 @@ import type { ObjectEvents } from '../../EventTypeDefs';
 }
-/**
- * @deprecated The old fabric.Text class can't be imported as Text because of conflict with Web API.
- * https://developer.mozilla.org/en-US/docs/Web/API/Text/Text
- * For this reason it has been renamed to FabricText.
- * Please use `import { FabricText }` in place of `import { Text as FabricText }`
- */
-export declare class Text extends FabricText {
-}
 export {};
 //# sourceMappingURL=Text.d.ts.map

fabric.ts

@@ -46,3 +46,14 @@ export { getEnv, getFabricDocument, getFabricWindow, setEnv } from './src/env';
   FabricObject,
-  _Object as Object,
+
+  /**
+   * @deprecated Due to a naming conflict with the
+   * {@link https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Object JS API},
+   * `fabric.Object` has been renamed to `FabricObject`
+   *
+   * @example
+   * import { Object } from 'fabric'; // deprecated
+   * import { FabricObject } from 'fabric'; // migration path
+   *
+   */
+  FabricObject as Object,
 } from './src/shapes/Object/FabricObject';
@@ -79,3 +90,16 @@
 } from './src/shapes/Text/Text';
-export { Text, FabricText } from './src/shapes/Text/Text';
+export {
+  FabricText,
+  /**
+   * @deprecated Due to a naming conflict with the
+   * {@link https://developer.mozilla.org/en-US/docs/Web/API/Text/Text Web API},
+   * `fabric.Text` has been renamed to `FabricText`
+   *
+   * @example
+   * import { Text } from 'fabric'; // deprecated
+   * import { FabricText } from 'fabric'; // migration path
+   *
+   */
+  FabricText as Text,
+} from './src/shapes/Text/Text';
 export type {
@@ -113,3 +137,17 @@ ITextProps,
 export { ActiveSelection } from './src/shapes/ActiveSelection';
-export { Image, FabricImage } from './src/shapes/Image';
+export {
+  FabricImage,
+
+  /**
+   * @deprecated Due to a naming conflict with the
+   * {@link https://developer.mozilla.org/en-US/docs/Web/API/HTMLImageElement/Image Web API},
+   * `fabric.Image` has been renamed to `FabricImage`
+   *
+   * @example
+   * import { Image } from 'fabric'; // deprecated
+   * import { FabricImage } from 'fabric'; // migration path
+   *
+   */
+  FabricImage as Image,
+} from './src/shapes/Image';
 export type {
@@ -116,0 +154,0 @@ ImageSource,

package.json

@@ -5,3 +5,3 @@ {
   "homepage": "http://fabricjs.com/",
-  "version": "6.0.0-beta14",
+  "version": "6.0.0-beta15",
   "author": "Juriy Zaytsev <[e-mail]>",
@@ -8,0 +8,0 @@ "contributors": [

src/parser/parseUseDirectives.ts

 import { svgNS } from './constants';
-import { elementById } from './elementById';
 import { getMultipleNodes } from './getMultipleNodes';
@@ -20,3 +19,8 @@ import { applyViewboxTransform } from './applyViewboxTransform';
     const y = el.getAttribute('y') || 0;
-    let el2 = elementById(doc, xlink)!.cloneNode(true) as Element;
+    const el2Orig = doc.getElementById(xlink);
+    if (el2Orig === null) {
+      // if we can't find the target of the xlink, consider this use tag bad, similar to no xlink
+      return;
+    }
+    let el2 = el2Orig.cloneNode(true) as Element;
     let currentTrans =
@@ -23,0 +27,0 @@ (el2.getAttribute('transform') || '') +

src/parser/recursivelyParseGradientsXlink.ts

@@ -1,3 +0,1 @@
-import { elementById } from './elementById';
-
 const gradientsAttrs = [
@@ -23,3 +21,3 @@ 'gradientTransform',
   const xLink = gradient.getAttribute(xlinkAttr)?.slice(1) || '',
-    referencedGradient = elementById(doc, xLink);
+    referencedGradient = doc.getElementById(xLink);
   if (referencedGradient && referencedGradient.getAttribute(xlinkAttr)) {
@@ -26,0 +24,0 @@ recursivelyParseGradientsXlink(doc, referencedGradient as Element);

src/shapes/Image.ts

@@ -862,9 +862,1 @@ import { getFabricDocument, getEnv } from '../env';
 classRegistry.setSVGClass(FabricImage);
-
-/**
- * @deprecated The old fabric.Image class can't be imported as Image because of conflict with Web API.
- * https://developer.mozilla.org/en-US/docs/Web/API/HTMLImageElement/Image
- * For this reason it has been renamed to FabricImage.
- * Please use `import { FabricImage }` in place of `import { Image as FabricImage }`
- */
-export class Image extends FabricImage {}

src/shapes/Object/FabricObject.ts

@@ -30,9 +30,1 @@ import type { ObjectEvents } from '../../EventTypeDefs';
 export { cacheProperties } from './defaultValues';
-
-/**
- * @deprecated The old fabric.Object class can't be imported as Object because of conflict with the JS api
- * https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Object
- * For this reason it has been renamed to FabricObject.
- * Please use `import { FabricObject }` in place of `import { Object as FabricObject }`
- */
-export class _Object extends FabricObject {}

src/shapes/Text/Text.ts

@@ -1918,9 +1918,1 @@ import { cache } from '../../cache';
 classRegistry.setSVGClass(FabricText);
-
-/**
- * @deprecated The old fabric.Text class can't be imported as Text because of conflict with Web API.
- * https://developer.mozilla.org/en-US/docs/Web/API/Text/Text
- * For this reason it has been renamed to FabricText.
- * Please use `import { FabricText }` in place of `import { Text as FabricText }`
- */
-export class Text extends FabricText {}

No alert changes

Worsened metrics

Total package byte prevSize

14392933

decreased by-0.1%

Number of package files

573

decreased by-0.17%

Lines of code

163433

decreased by-0.06%

No dependency changes