@contrail/documents 2.4.0-alpha.bce8caa → 2.4.0-alpha.cb23d8e

package/CHANGELOG.md

@@ -13,15 +13,12 @@ Also shipped on the 1.x line as 1.21.0.
 
 - `DocumentContainerService.buildIndex` and `DocumentContainerIndex` — which containers hold which
   elements, for one document, resolved once and read back by lookup. A container is a frame or a
-  section, described by a `ContainerKind` carrying `elementType` and `nests`. An element belongs to
-  the container whose box covers its anchor point and which is drawn below it, unless the element's
-  own box encloses that container or the element is a group holding it. `nests` decides the shape of
-  the answer: sections nest, so an element in a section inside another is in both, and the chain
-  requires the inner container to be drawn above the outer; frames do not, so an element covered by
-  two frames is in the topmost. A container of another kind is placed by the same rules as any
-  element, so a frame can be in a section and a section drawn above a frame is in that frame. The
-  rule is written out in `src/containers/DOCUMENT_CONTAINERS_SPEC.md`, which the code is measured
-  against.
+  section, described by a `ContainerKind` carrying `elementType`, `nests` and `layer`. An element
+  belongs to the container whose box covers its anchor point and which is drawn below it. Nesting
+  gives a path with one container per depth, the later-drawn one winning where two cross. A
+  container is only ever content of a kind layered behind it, so a frame belongs to a section and a
+  section never to a frame. Shapes wider and taller than the container, and groups holding it, are
+  excluded.
 - `SECTION_ELEMENT_TYPE`, `FRAME_ELEMENT_TYPE` and `SHAPE_ELEMENT_TYPES` in
   `document-element-constants`, so the rule above has one definition rather than one per app.
 - `DocumentContainerUtil` — the container geometry as statics, with `anchorFor` for a caller
@@ -54,17 +51,10 @@ Also shipped on the 1.x line as 1.21.0.
 - The corner is the rotated one (VIBE-9567). A 200x20 text turned on its side draws as a 20x200
   column, and its unrotated `position` can sit well clear of the frame it is visibly on, which
   reported "Move to a frame" for a vertical text the user could see was placed correctly.
-- An element covering a container is not inside it, whatever its type. This replaces a rule that
-  excluded only shapes wider and taller than the container, which let a background image join the
-  frame it covered and needed an exemption for dynamic text, whose size the container decides. An
-  unrotated dynamic text cannot be excluded by enclosure, since its anchor is the corner of the box
-  it would have to cover with.
-- `isDynamicText` requires a `propertyBindings.text` binding, which is the one the value is read
-  from. The text tool creates an element with empty `propertyBindings` and `propertyBindingsMetaData`
-  and both are truthy, so an unconfigured text was anchored at its corner while the canvas centred
-  it.
-- A line or callout is measured from the elements it connects even with no stored `position` of its
-  own, where it was previously left out of the index entirely.
+- A dynamic text is also exempt from the oversized-shape exclusion, which keeps a shape wider and
+  taller than a container off it. That rule reads size too, so a dynamic text larger than its frame
+  in both directions had two self-consistent answers and kept whichever it started from. The rule
+  exists to keep a backdrop shape off a frame it covers, which is not what a dynamic text is doing.
 - `min` and `max` rendered an infinity sign when every member value was empty. The early return
   checked the list length before the empty values were filtered out, so `Math.min` of nothing
   reached the formatter. Both now fall through to `n/a`.
@@ -81,12 +71,9 @@ Also shipped on the 1.x line as 1.21.0.
   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`.
-- A container path is a containment chain, walked outward from the container the element sits on.
-  Two containers that merely cross are not a chain, so the later-drawn one takes the element rather
-  than both claiming it. `containerPathAtPoint` walks the same chain, so a drop preview agrees with
-  the drop; it applies only the anchor rule, having no element to compare against a container.
-- `buildIndex` scans element types before measuring anything, so a document holding no containers
-  of the kinds asked for returns an empty index without a geometry pass.
+- 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.
 
 ## [2.3.0] - 2026-09-21
 

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

@@ -69,9 +69,12 @@ class DocumentContainerService {
     }
     static measure(documentElement, documentElementsById) {
         var _a, _b;
+        if (!(documentElement === null || documentElement === void 0 ? void 0 : documentElement.position)) {
+            return null;
+        }
         try {
-            const start = ((_a = documentElement === null || documentElement === void 0 ? void 0 : documentElement.start) === null || _a === void 0 ? void 0 : _a.id) ? documentElementsById.get(documentElement.start.id) : undefined;
-            const end = ((_b = documentElement === null || documentElement === void 0 ? void 0 : documentElement.end) === null || _b === void 0 ? void 0 : _b.id) ? documentElementsById.get(documentElement.end.id) : undefined;
+            const start = ((_a = documentElement.start) === null || _a === void 0 ? void 0 : _a.id) ? documentElementsById.get(documentElement.start.id) : undefined;
+            const end = ((_b = documentElement.end) === null || _b === void 0 ? void 0 : _b.id) ? documentElementsById.get(documentElement.end.id) : undefined;
             const { x, y, width, height } = document_element_factory_1.DocumentElementFactory.getBoundingClientRectScaled(documentElement, start, end);
             return document_container_1.DocumentContainerUtil.toCoordinateBox(x, y, width, height);
         }

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

@@ -56,9 +56,8 @@ function isContainerBinding(bindingProperty) {
 }
 class DynamicTextUtil {
     static isDynamicText(documentElement) {
-        var _a;
         return ((documentElement === null || documentElement === void 0 ? void 0 : documentElement.type) === 'text' &&
-            !!((_a = documentElement.propertyBindings) === null || _a === void 0 ? void 0 : _a.text) &&
+            !!documentElement.propertyBindings &&
             !!documentElement.propertyBindingsMetaData);
     }
     static takesValueFromContainer(bindingProperty) {
@@ -209,7 +208,7 @@ class DynamicTextUtil {
         }
         else if (displayFunction === DynamicTextDisplayFunction.UNIQUE_ARRAY) {
             text = Array.from(new Set(memberValues
-                .filter((e) => e !== null && e !== undefined && e !== '')
+                .filter((e) => e !== null && e !== undefined)
                 .flat()
                 .map((e) => formatter.formatValueForProperty(e, property))))
                 .sort()

package/package.json

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