@bsuite/page-builder 1.0.4 → 1.0.5

package/dist/usePageGridLayout.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"usePageGridLayout.d.ts","sourceRoot":"","sources":["../src/usePageGridLayout.ts"],"names":[],"mappings":"AAiBA,OAAO,KAAK,EAGV,wBAAwB,EACxB,uBAAuB,EACxB,MAAM,YAAY,CAAC;AAEpB,eAAO,MAAM,0BAA0B,2IAM7B,CAAC;AAEX;;;;;;;;GAQG;AACH,eAAO,MAAM,uBAAuB,6BAA6B,CAAC;AAElE;;;;;;;;;;;;;;;;;;;GAmBG;AACH,eAAO,MAAM,oBAAoB,OAAO,CAAC;AAiBzC,MAAM,WAAW,0BAA0B;IACzC,OAAO,EAAE,MAAM,CAAC;IAChB,OAAO,EAAE,OAAO,CAAC;CAClB;AAED,wBAAgB,iBAAiB,CAAC,EAChC,OAAO,EACP,cAAc,EACd,WAAgB,EAChB,aAAiB,EACjB,WAAkB,EAClB,gBAA6C,EAC7C,iBAA4C,EAC5C,iBAAiB,GAClB,EAAE,wBAAwB,GAAG,uBAAuB,CA4pBpD"}
+{"version":3,"file":"usePageGridLayout.d.ts","sourceRoot":"","sources":["../src/usePageGridLayout.ts"],"names":[],"mappings":"AAiBA,OAAO,KAAK,EAGV,wBAAwB,EACxB,uBAAuB,EACxB,MAAM,YAAY,CAAC;AAEpB,eAAO,MAAM,0BAA0B,2IAM7B,CAAC;AAEX;;;;;;;;GAQG;AACH,eAAO,MAAM,uBAAuB,6BAA6B,CAAC;AAElE;;;;;;;;;;;;;;;;;;;GAmBG;AACH,eAAO,MAAM,oBAAoB,OAAO,CAAC;AAiBzC,MAAM,WAAW,0BAA0B;IACzC,OAAO,EAAE,MAAM,CAAC;IAChB,OAAO,EAAE,OAAO,CAAC;CAClB;AA6BD,wBAAgB,iBAAiB,CAAC,EAChC,OAAO,EACP,cAAc,EACd,WAAgB,EAChB,aAAiB,EACjB,WAAkB,EAClB,gBAA6C,EAC7C,iBAA4C,EAC5C,iBAAiB,GAClB,EAAE,wBAAwB,GAAG,uBAAuB,CAirBpD"}

package/dist/usePageGridLayout.js

@@ -55,6 +55,33 @@ export const PACKAGE_LAYOUT_EPOCH = 1000;
  * opts out with an explicit `autoHeight: false`.
  */
 const DEFAULT_ITEM_AUTO_HEIGHT = true;
+/**
+ * Do two layout SETS describe the same arrangement, at every breakpoint?
+ *
+ * Compares only the five fields that place an item. Anything else react-grid-layout
+ * attaches (`moved`, `static`, `isDraggable`, measured heights) is noise here, and
+ * comparing it would make every reflow look like a change.
+ *
+ * EVERY breakpoint, not just `lg`. A gesture made while the canvas is at `md`
+ * leaves `lg` untouched, so an `lg`-only comparison reads that real gesture as a
+ * reflow and silently drops it. That is not hypothetical — it broke the D-75
+ * "folds a gesture made at md back onto lg" test the moment the comparison was
+ * introduced.
+ */
+function sameArrangement(a, b) {
+    if (!a || !b)
+        return false;
+    const keys = new Set([...Object.keys(a), ...Object.keys(b)]);
+    const fingerprint = (items) => (items ?? [])
+        .map((i) => `${i.i}:${i.x}:${i.y}:${i.w}:${i.h}`)
+        .sort()
+        .join('|');
+    for (const k of keys) {
+        if (fingerprint(a[k]) !== fingerprint(b[k]))
+            return false;
+    }
+    return true;
+}
 export function usePageGridLayout({ pageKey, defaultLayouts, defaultCols = 12, layoutVersion = 1, canEditPage = true, editorEventNames = DEFAULT_EDITOR_EVENT_NAMES, preferenceAdapter = defaultPreferenceAdapter, defaultAutoHeight, }) {
     const effectiveLayoutVersion = layoutVersion + PACKAGE_LAYOUT_EPOCH;
     const containerRef = useRef(null);
@@ -443,6 +470,26 @@ export function usePageGridLayout({ pageKey, defaultLayouts, defaultCols = 12, l
     const onLayoutChange = useCallback((_layout, layouts) => {
         if (!isEditing)
             return;
+        /*
+         * A COLUMN-COUNT CHANGE MAKES REACT-GRID-LAYOUT REFLOW AND EMIT HERE, and
+         * that emission is not a user gesture. Committing it copies the rescaled
+         * DISPLAY back over the authored layout — which is the data loss this
+         * whole path exists to prevent, arriving through the back door.
+         *
+         * Removing the writes from `handleColumnChange` was not enough, and the
+         * unit tests did not catch it because they call the handler directly with
+         * no grid attached. Driving the deployed app is what found it: the
+         * persisted widths still marched 2,2,2 -> 4,4,4 -> 1,1,1 with the
+         * handler already inert.
+         *
+         * A reflow emits exactly `currentLayouts`, because that is what was just
+         * rendered. Any real gesture differs from it — and a gesture that does
+         * NOT differ has nothing to commit anyway, so this cannot swallow one.
+         * Identified by comparison rather than by a flag, so it needs no event
+         * ordering to be correct.
+         */
+        if (sameArrangement(layouts, currentLayouts))
+            return;
         pendingLayoutRef.current = layouts;
         if (layoutCommitFrameRef.current !== null)
             return;
@@ -459,7 +506,7 @@ export function usePageGridLayout({ pageKey, defaultLayouts, defaultCols = 12, l
                 commitLayout(pending);
             }
         });
-    }, [commitLayout, isEditing]);
+    }, [commitLayout, currentLayouts, isEditing]);
     useEffect(() => {
         return () => {
             if (layoutCommitFrameRef.current !== null &&

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bsuite/page-builder",
-  "version": "1.0.4",
+  "version": "1.0.5",
   "description": "Shared BSuite responsive page-builder grid and layout persistence primitives",
   "type": "module",
   "license": "UNLICENSED",