@nasser-sw/fabric 7.0.1-beta29 → 7.0.1-beta30

package/.history/package_20251227153856.json

@@ -0,0 +1,164 @@
+{
+  "name": "@nasser-sw/fabric",
+  "description": "Object model for HTML5 canvas, and SVG-to-canvas parser. Backed by jsdom and node-canvas.",
+  "homepage": "http://fabricjs.com/",
+  "version": "7.0.1-beta30",
+  "author": "Juriy Zaytsev <kangax@gmail.com>",
+  "contributors": [
+    {
+      "name": "Andrea Bogazzi",
+      "email": "andreabogazzi79@gmail.com",
+      "url": "https://github.com/asturur"
+    },
+    {
+      "name": "Shachar Nencel",
+      "email": "shacharnen@gmail.com",
+      "url": "https://github.com/ShaMan123"
+    },
+    {
+      "name": "Steve Eberhardt",
+      "email": "melchiar2@gmail.com",
+      "url": "https://github.com/melchiar"
+    }
+  ],
+  "keywords": [
+    "canvas",
+    "graphic",
+    "graphics",
+    "SVG",
+    "node-canvas",
+    "parser",
+    "HTML5",
+    "object model"
+  ],
+  "publishConfig": {
+    "access": "public"
+  },
+  "repository": {
+    "type": "git",
+    "url": "https://github.com/fabricjs/fabric.js"
+  },
+  "bugs": {
+    "url": "https://github.com/fabricjs/fabric.js/issues"
+  },
+  "license": "MIT",
+  "scripts": {
+    "docs": "typedoc",
+    "cli": "node ./scripts/index.mjs",
+    "sandboxscript": "node ./scripts/sandbox.mjs",
+    "build": "npm run cli -- build",
+    "build:fast": "npm run build -- -f",
+    "dev": "npm run cli -- dev",
+    "start": "npm run sandboxscript -- start",
+    "export": "npm run cli -- website export",
+    "test:vitest": "vitest --run --project unit-node",
+    "test:vitest:chromium": "vitest --run --project unit-chromium",
+    "test:vitest:firefox": "vitest --run --project unit-firefox",
+    "test:vitest:all": "vitest --run",
+    "test:vitest:coverage": "vitest --run --coverage --project unit-node",
+    "test:vitest:coverage:watch": "npm run test:vitest --coverage=true",
+    "coverage:merge": "nyc merge coveragefiles .nyc_output/merged-coverage.json",
+    "coverage:report": "nyc report --skip-full=true --reporter=lcov --reporter=text --reporter=text-summary",
+    "coverage:report:ci": "nyc report --reporter=text-summary",
+    "test:e2e": "npm run playwright:typecheck && playwright test",
+    "playwright:typecheck": "tsc -p ./e2e/tsconfig.json --noEmit",
+    "sandbox": "npm run sandboxscript -- sandbox",
+    "local-server": "serve ./ -l tcp://localhost:8080",
+    "lint": "eslint src extensions",
+    "prettier:check": "prettier --check .",
+    "prettier:write": "prettier --write ."
+  },
+  "devDependencies": {
+    "@babel/cli": "^7.28.3",
+    "@babel/core": "^7.28.3",
+    "@babel/preset-env": "^7.28.3",
+    "@babel/preset-typescript": "^7.27.1",
+    "@eslint/js": "^9.31.0",
+    "@playwright/test": "^1.55.0",
+    "@rollup/plugin-babel": "^6.0.4",
+    "@rollup/plugin-json": "^6.1.0",
+    "@rollup/plugin-terser": "^0.4.4",
+    "@rollup/plugin-typescript": "^12.1.4",
+    "@types/fs-extra": "^11.0.4",
+    "@types/jsdom": "^21.1.7",
+    "@types/micromatch": "^4.0.9",
+    "@types/node": "^24.3.0",
+    "@vitest/browser": "^3.2.4",
+    "@vitest/coverage-v8": "^3.2.4",
+    "@vitest/ui": "^3.2.4",
+    "babel-plugin-transform-imports": "git+https://git@github.com/fabricjs/babel-plugin-transform-imports.git",
+    "chalk": "^5.6.0",
+    "commander": "^14.0.0",
+    "es-toolkit": "1.39.7",
+    "eslint-config-prettier": "^10.1.8",
+    "fs-extra": "^11.3.1",
+    "inquirer": "^12.9.4",
+    "micromatch": "^4.0.8",
+    "moment": "^2.30.1",
+    "nyc": "^17.1.0",
+    "prettier": "^3.6.2",
+    "ps-list": "^8.1.1",
+    "rollup": "^4.48.0",
+    "semver": "^7.7.2",
+    "serve": "^14.2.4",
+    "tslib": "^2.8.1",
+    "typescript": "^5.9.2",
+    "typescript-eslint": "^8.40.0",
+    "v8-to-istanbul": "^9.3.0",
+    "vite": "^6.3.5",
+    "vitest": "^3.2.4"
+  },
+  "engines": {
+    "node": ">=18.0.0"
+  },
+  "overrides": {
+    "canvas": {
+      "canvas": "3.2.0"
+    }
+  },
+  "module": "./dist/index.mjs",
+  "types": "./dist/index.d.ts",
+  "typesVersions": {
+    ">=4.2": {
+      "*": [
+        "dist/index.d.ts"
+      ],
+      "node": [
+        "dist/index.node.d.ts"
+      ]
+    }
+  },
+  "sideEffects": false,
+  "exports": {
+    ".": {
+      "types": "./dist/index.d.ts",
+      "import": "./dist/index.min.mjs",
+      "require": "./dist/index.min.js",
+      "default": "./dist/index.min.js"
+    },
+    "./es": {
+      "types": "./dist/index.d.ts",
+      "import": "./dist/fabric.min.mjs",
+      "require": null,
+      "default": null
+    },
+    "./node": {
+      "node": "./dist/index.node.cjs",
+      "types": "./dist/index.node.d.ts",
+      "import": "./dist/index.node.mjs",
+      "require": "./dist/index.node.cjs",
+      "default": "./dist/index.node.cjs"
+    },
+    "./extensions": {
+      "node": "./dist-extensions/index.mjs",
+      "types": "./dist-extensions/extensions/index.d.ts",
+      "import": "./dist-extensions/index.mjs",
+      "require": null,
+      "default": "./dist-extensions/fabric-extensions.min.js"
+    }
+  },
+  "optionalDependencies": {
+    "canvas": "^3.2.0",
+    "jsdom": "^26.1.0"
+  }
+}

package/dist/index.js

@@ -360,7 +360,7 @@
   }
   const cache = new Cache();
 
-  var version = "7.0.1-beta28";
+  var version = "7.0.1-beta29";
 
   // use this syntax so babel plugin see this import here
   const VERSION = version;
@@ -9337,7 +9337,7 @@
     /**
      * Position handler for expand controls
      * Positions controls at the expansion boundary, not the object boundary
-     * Note: expansion values are stored in local object coordinates
+     * Note: expansion values are stored in UNSCALED local object coordinates
      */
     positionHandler(dim, finalMatrix, fabricObject, currentControl) {
       const expansion = getExpansion(fabricObject);
@@ -9348,23 +9348,15 @@
         expandBottom
       } = expansion;
 
-      // dim already includes object's own scale
+      // dim already includes object's own scale (dim = width * scaleX, height * scaleY)
       const halfWidth = dim.x / 2;
       const halfHeight = dim.y / 2;
 
-      // When object is in a group (like Frame), we need to scale expansion values
-      // to match the coordinate space of dim
-      let scaleFactorX = 1;
-      let scaleFactorY = 1;
-      if (fabricObject.group) {
-        // dim is in object's scaled local space
-        // expansion values are also in local space
-        // But we need to account for the object's scale since dim includes it
-        const objectScaleX = fabricObject.scaleX || 1;
-        const objectScaleY = fabricObject.scaleY || 1;
-        scaleFactorX = objectScaleX;
-        scaleFactorY = objectScaleY;
-      }
+      // Expansion values are in UNSCALED coordinates.
+      // dim is in SCALED coordinates.
+      // Scale expansion by object's scale to match dim.
+      const scaleFactorX = fabricObject.scaleX || 1;
+      const scaleFactorY = fabricObject.scaleY || 1;
 
       // Scale expansion values to match dim's coordinate space
       const scaledExpandLeft = expandLeft * scaleFactorX;
@@ -9460,15 +9452,15 @@
 
     // Use Fabric's getLocalPoint with center origin (returns scaled coordinates)
     const localPoint = getLocalPoint(transform, CENTER, CENTER, x, y);
-
-    // Use SCALED half width since localPoint is in scaled coords
     const scaleX = target.scaleX || 1;
     const scaledHalfWidth = target.width * scaleX / 2;
 
-    // Calculate new expansion (how far left of the object's left edge)
-    // localPoint.x is negative when left of center
-    // Expansion starts when localPoint.x < -scaledHalfWidth
-    const newExpandLeft = Math.max(0, -scaledHalfWidth - localPoint.x);
+    // Calculate expansion in scaled coordinates first
+    const scaledExpansion = Math.max(0, -scaledHalfWidth - localPoint.x);
+
+    // Store in UNSCALED coordinates (divide by scale)
+    // This ensures positionHandler can scale it back correctly
+    const newExpandLeft = scaledExpansion / scaleX;
     if (Math.abs(newExpandLeft - expansion.expandLeft) > 0.5) {
       var _target$canvas;
       expansion.expandLeft = newExpandLeft;
@@ -9490,15 +9482,14 @@
 
     // Use Fabric's getLocalPoint with center origin (returns scaled coordinates)
     const localPoint = getLocalPoint(transform, CENTER, CENTER, x, y);
-
-    // Use SCALED half width since localPoint is in scaled coords
     const scaleX = target.scaleX || 1;
     const scaledHalfWidth = target.width * scaleX / 2;
 
-    // Calculate new expansion (how far right of the object's right edge)
-    // localPoint.x is positive when right of center
-    // Expansion starts when localPoint.x > scaledHalfWidth
-    const newExpandRight = Math.max(0, localPoint.x - scaledHalfWidth);
+    // Calculate expansion in scaled coordinates first
+    const scaledExpansion = Math.max(0, localPoint.x - scaledHalfWidth);
+
+    // Store in UNSCALED coordinates (divide by scale)
+    const newExpandRight = scaledExpansion / scaleX;
     if (Math.abs(newExpandRight - expansion.expandRight) > 0.5) {
       var _target$canvas2;
       expansion.expandRight = newExpandRight;
@@ -9520,13 +9511,14 @@
 
     // Use Fabric's getLocalPoint with center origin (returns scaled coordinates)
     const localPoint = getLocalPoint(transform, CENTER, CENTER, x, y);
-
-    // Use SCALED half height since localPoint is in scaled coords
     const scaleY = target.scaleY || 1;
     const scaledHalfHeight = target.height * scaleY / 2;
 
-    // Calculate new expansion (how far above the object's top edge)
-    const newExpandTop = Math.max(0, -scaledHalfHeight - localPoint.y);
+    // Calculate expansion in scaled coordinates first
+    const scaledExpansion = Math.max(0, -scaledHalfHeight - localPoint.y);
+
+    // Store in UNSCALED coordinates (divide by scale)
+    const newExpandTop = scaledExpansion / scaleY;
     if (Math.abs(newExpandTop - expansion.expandTop) > 0.5) {
       var _target$canvas3;
       expansion.expandTop = newExpandTop;
@@ -9548,13 +9540,14 @@
 
     // Use Fabric's getLocalPoint with center origin (returns scaled coordinates)
     const localPoint = getLocalPoint(transform, CENTER, CENTER, x, y);
-
-    // Use SCALED half height since localPoint is in scaled coords
     const scaleY = target.scaleY || 1;
     const scaledHalfHeight = target.height * scaleY / 2;
 
-    // Calculate new expansion (how far below the object's bottom edge)
-    const newExpandBottom = Math.max(0, localPoint.y - scaledHalfHeight);
+    // Calculate expansion in scaled coordinates first
+    const scaledExpansion = Math.max(0, localPoint.y - scaledHalfHeight);
+
+    // Store in UNSCALED coordinates (divide by scale)
+    const newExpandBottom = scaledExpansion / scaleY;
     if (Math.abs(newExpandBottom - expansion.expandBottom) > 0.5) {
       var _target$canvas4;
       expansion.expandBottom = newExpandBottom;
@@ -9990,8 +9983,8 @@
         size = this._calculateCurrentDimensions().scalarAdd(this.borderScaleFactor);
       }
 
-      // Draw expansion preview if in expand mode or has expansion
-      if (this.expandMode || this.hasExpansion()) {
+      // Draw expansion preview only when actively in expand mode
+      if (this.expandMode) {
         this.drawExpandPreview(ctx, size, options);
       }
       this._drawBorders(ctx, size, styleOverride);
@@ -10405,7 +10398,7 @@
      * Note: expansion values are stored in local object coordinates
      */
     drawExpandPreview(ctx, size, options) {
-      if (!this.expandMode && !this.hasExpansion()) return;
+      if (!this.expandMode) return;
       const {
         expandLeft,
         expandRight,
@@ -10413,23 +10406,14 @@
         expandBottom
       } = this.expansion;
 
-      // When object is in a group (like Frame), expansion values are in local coords
-      // but context is transformed. We need to scale expansion by the transform.
-      // For standalone objects, options.scaleX/Y will be just the object's own scale.
-      // For objects in groups, it includes parent scale too.
-      let scaleFactorX = 1;
-      let scaleFactorY = 1;
-      if (this.group && options) {
-        // Get the group's contribution to the scale
-        // options.scaleX includes both object scale and parent scale
-        // We want just the parent's scale contribution
-        const objectScaleX = this.scaleX || 1;
-        const objectScaleY = this.scaleY || 1;
-        scaleFactorX = (options.scaleX || 1) / objectScaleX;
-        scaleFactorY = (options.scaleY || 1) / objectScaleY;
-      }
-
-      // Scale expansion values by the parent's scale factor
+      // Expansion values are stored in UNSCALED object coordinates.
+      // size is in screen coordinates (includes full transform).
+      // We need to scale expansion by the FULL transform to match size.
+      // options.scaleX/Y contains the full scale (object * parent * viewport).
+      const scaleFactorX = (options === null || options === void 0 ? void 0 : options.scaleX) || this.scaleX || 1;
+      const scaleFactorY = (options === null || options === void 0 ? void 0 : options.scaleY) || this.scaleY || 1;
+
+      // Scale expansion values to screen coordinates
       const scaledExpandLeft = expandLeft * scaleFactorX;
       const scaledExpandRight = expandRight * scaleFactorX;
       const scaledExpandTop = expandTop * scaleFactorY;