@contrail/documents 2.4.0-alpha.a01d866 → 2.4.0-alpha.efe1930

package/CHANGELOG.md

@@ -64,16 +64,8 @@ Also shipped on the 1.x line as 1.21.0.
 ### Changed
 
 - Container membership measures a scaled element from its scaled box, through the new
-  `getBoundingClientRectScaled` above. Only a sticky note, a regular iframe, a text-tool text or a
-  table carries `scale` at all; for this work the one that matters is a text-tool dynamic text,
-  which previously measured at its unscaled size and could resolve to the wrong container.
-- `DocumentContainerUtil.anchorFor` measures through the same factory call the index uses, so a
-  caller holding an element gets the answer the index would give it. Reading `size` directly
-  disagreed for the types whose box is not position plus size: a component's box is built from the
-  rows it draws, and a line's comes from its `lineDefinition`.
-- Container depth counts nesting levels rather than enclosing boxes. Two containers that merely
-  cross both enclose anything inside the overlap, so counting ancestors put that inner container a
-  level too deep and made `maxDepth` report a level no path could reach.
+  `getBoundingClientRectScaled` above. A scaled element previously measured at its unscaled size
+  and could resolve to the wrong container.
 
 ## [2.3.0] - 2026-09-21
 

package/lib/containers/document-container.d.ts

@@ -19,7 +19,7 @@ export declare class DocumentContainerUtil {
     static centerOf(box: CoordinateBox): PositionDefinition;
     static anchorOf(documentElement: DocumentElement, box: CoordinateBox): PositionDefinition;
     static positionKeepingAnchor(documentElement: DocumentElement, newSize: SizeDefinition): PositionDefinition;
-    static anchorFor(documentElement: DocumentElement, position?: PositionDefinition, startElement?: DocumentElement, endElement?: DocumentElement): PositionDefinition;
+    static anchorFor(documentElement: DocumentElement, position?: PositionDefinition): PositionDefinition;
     static containsPoint(box: CoordinateBox, point: PositionDefinition): boolean;
     static encloses(outer: CoordinateBox, inner: CoordinateBox): boolean;
     static isWiderAndTallerThan(box: CoordinateBox, other: CoordinateBox): boolean;

package/lib/containers/document-container.js

@@ -3,7 +3,6 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.DocumentContainerUtil = void 0;
 const rotation_math_1 = require("../util/mask/rotation-math");
 const dynamic_text_util_1 = require("../util/dynamic-text/dynamic-text-util");
-const document_element_factory_1 = require("../document-element-factory");
 class DocumentContainerUtil {
     static toCoordinateBox(x, y, width, height) {
         return { x, y, width, height, left: x, right: x + width, top: y, bottom: y + height };
@@ -38,23 +37,14 @@ class DocumentContainerUtil {
         const after = cornerOffset(newSize);
         return { x: position.x + before.x - after.x, y: position.y + before.y - after.y };
     }
-    static anchorFor(documentElement, position, startElement, endElement) {
+    static anchorFor(documentElement, position) {
         var _a, _b, _c, _d, _e;
         const at = position !== null && position !== void 0 ? position : documentElement === null || documentElement === void 0 ? void 0 : documentElement.position;
         if (!at) {
             return null;
         }
         const size = (_a = documentElement === null || documentElement === void 0 ? void 0 : documentElement.size) !== null && _a !== void 0 ? _a : { width: 0, height: 0 };
-        let width = size.width * ((_c = (_b = documentElement === null || documentElement === void 0 ? void 0 : documentElement.scale) === null || _b === void 0 ? void 0 : _b.x) !== null && _c !== void 0 ? _c : 1);
-        let height = size.height * ((_e = (_d = documentElement === null || documentElement === void 0 ? void 0 : documentElement.scale) === null || _d === void 0 ? void 0 : _d.y) !== null && _e !== void 0 ? _e : 1);
-        try {
-            const measured = document_element_factory_1.DocumentElementFactory.getBoundingClientRectScaled(Object.assign(Object.assign({}, documentElement), { position: at }), startElement, endElement);
-            width = measured.width;
-            height = measured.height;
-        }
-        catch (_f) {
-        }
-        const box = DocumentContainerUtil.toCoordinateBox(at.x, at.y, width, height);
+        const box = DocumentContainerUtil.toCoordinateBox(at.x, at.y, size.width * ((_c = (_b = documentElement === null || documentElement === void 0 ? void 0 : documentElement.scale) === null || _b === void 0 ? void 0 : _b.x) !== null && _c !== void 0 ? _c : 1), size.height * ((_e = (_d = documentElement === null || documentElement === void 0 ? void 0 : documentElement.scale) === null || _d === void 0 ? void 0 : _d.y) !== null && _e !== void 0 ? _e : 1));
         return DocumentContainerUtil.anchorOf(documentElement, box);
     }
     static containsPoint(box, point) {

package/lib/containers/document-container.service.js

@@ -89,21 +89,7 @@ class DocumentContainerService {
                 ? containers.filter((other) => other !== container && document_container_1.DocumentContainerUtil.encloses(other.box, container.box))
                 : [];
             ancestorsById.set(container.id, ancestors);
-        }
-        const depths = new Map();
-        const depthOf = (container) => {
-            var _a;
-            const cached = depths.get(container.id);
-            if (cached !== undefined) {
-                return cached;
-            }
-            const ancestors = (_a = ancestorsById.get(container.id)) !== null && _a !== void 0 ? _a : [];
-            const depth = ancestors.length ? 1 + Math.max(...ancestors.map(depthOf)) : 0;
-            depths.set(container.id, depth);
-            return depth;
-        };
-        for (const container of containers) {
-            container.depth = depthOf(container);
+            container.depth = ancestors.length;
         }
         return { containers, ancestorsById };
     }

package/lib/util/dynamic-text/dynamic-text-util.d.ts

@@ -48,7 +48,6 @@ export declare const HARD_CODED_TEXT: {
     'section.itemOptionCount': string;
     'section.colorCount': string;
 };
-export declare function isContainerBinding(bindingProperty: string | null | undefined): boolean;
 export declare class DynamicTextUtil {
     static isDynamicText(documentElement: DocumentElement): boolean;
     static takesValueFromContainer(bindingProperty: string | null | undefined): boolean;

package/lib/util/dynamic-text/dynamic-text-util.js

@@ -1,7 +1,6 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.DynamicTextUtil = exports.HARD_CODED_TEXT = exports.CLICK_TO_CONFIGURE = exports.MOVE_TO_SECTION_TEXT = exports.MOVE_TO_FRAME_TEXT = exports.DynamicTextDisplayFunction = exports.DYNAMIC_TEXT_DATA_SOURCE = void 0;
-exports.isContainerBinding = isContainerBinding;
 const types_1 = require("@contrail/types");
 const util_1 = require("@contrail/util");
 const document_util_1 = require("@contrail/document-util");
@@ -51,9 +50,6 @@ exports.HARD_CODED_TEXT = {
 };
 const CONTAINER_PROPERTY_SLUGS = ['name', 'count', 'itemFamilyCount', 'itemOptionCount', 'colorCount'];
 const CONTAINER_SOURCES = [DYNAMIC_TEXT_DATA_SOURCE.FRAME, DYNAMIC_TEXT_DATA_SOURCE.SECTION];
-function isContainerBinding(bindingProperty) {
-    return DynamicTextUtil.takesValueFromContainer(bindingProperty);
-}
 class DynamicTextUtil {
     static isDynamicText(documentElement) {
         return ((documentElement === null || documentElement === void 0 ? void 0 : documentElement.type) === 'text' &&

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@contrail/documents",
-  "version": "2.4.0-alpha.a01d866",
+  "version": "2.4.0-alpha.efe1930",
   "description": "Documents library for contrail platform",
   "main": "lib/index.js",
   "types": "lib/index.d.ts",