ats-form-react-pdf-layout 4.4.7 → 4.4.9

package/lib/index.js

@@ -1210,6 +1210,9 @@ const NON_WRAP_TYPES = [P.Svg, P.Note, P.Image, P.Canvas];
 const canCauseBlankSpace = (node, prevNode, currentChildren) => {
     if (!('preventBlankSpace' in node.props))
         return false;
+    // Don't override explicit wrap: false - respect user's intent to move to next page
+    if ('wrap' in node.props && node.props.wrap === false)
+        return false;
     const prevNodeHasHeightOne = prevNode?.box?.height === 1;
     const childrenIsEmpty = currentChildren?.length === 0;
     // padding/margin case
@@ -1227,6 +1230,9 @@ const getWrap = (node, prevNode, currentChildren) => {
         return false;
     if (!node.props)
         return true;
+    // Check wrap prop first - if explicitly set to false, respect that
+    if ('wrap' in node.props && node.props.wrap === false)
+        return false;
     if (canCauseBlankSpace(node, prevNode, currentChildren))
         return true;
     return 'wrap' in node.props ? node.props.wrap : true;
@@ -2650,6 +2656,30 @@ const splitNodes = (height, contentArea, nodes) => {
                 break;
             }
         }
+        // Element fits on a full page but not in remaining space and cannot wrap - move to next page
+        if (fitsInsidePage && shouldSplit && !canWrap) {
+            const box = Object.assign({}, child.box, { top: child.box.top - height });
+            const props = Object.assign({}, child.props, {
+                wrap: true,
+                break: false,
+            });
+            const next = Object.assign({}, child, { box, props });
+            currentChildren.push(...futureFixedNodes);
+            nextChildren.push(next, ...futureNodes);
+            break;
+        }
+        // Element fits on a full page but not in remaining space - move to next page to keep together
+        // This handles the case where wrap is true (default) but we still want to avoid splitting
+        // if the element would fit entirely on the next page
+        const remainingSpace = height - nodeTop;
+        const wouldBarelyFitOnCurrentPage = remainingSpace < nodeHeight * 0.3; // Less than 30% fits
+        if (fitsInsidePage && shouldSplit && wouldBarelyFitOnCurrentPage && currentChildren.length > 0) {
+            const box = Object.assign({}, child.box, { top: child.box.top - height });
+            const next = Object.assign({}, child, { box });
+            currentChildren.push(...futureFixedNodes);
+            nextChildren.push(next, ...futureNodes);
+            break;
+        }
         if (shouldBreak$1) {
             const box = Object.assign({}, child.box, { top: child.box.top - height });
             const props = Object.assign({}, child.props, {
@@ -2663,20 +2693,21 @@ const splitNodes = (height, contentArea, nodes) => {
         }
         if (shouldSplit || firstBreakableViewChild) {
             const [currentChild, nextChild] = split(child, height, contentArea);
-            // All children are moved to the next page, it doesn't make sense to show the parent on the current page
-            if (child.children.length > 0 && currentChild.children.length === 0) {
-                // But if the current page is empty then we can just include the parent on the current page
-                // if (currentChildren.length === 0) {
-                //   currentChildren.push(child, ...futureFixedNodes);
-                //   nextChildren.push(...futureNodes);
-                // } else {
-                const box = Object.assign({}, nextChild.box, {
-                    top: nextChild.box.top - height,
+            // Check if all non-fixed children are moved to the next page
+            const currentNonFixedChildren = currentChild.children
+                ? currentChild.children.filter((c) => !isFixed(c))
+                : [];
+            const hasNonFixedChildren = child.children
+                ? child.children.filter((c) => !isFixed(c)).length > 0
+                : false;
+            // All meaningful children are moved to the next page, move the parent too
+            if (hasNonFixedChildren && currentNonFixedChildren.length === 0) {
+                const box = Object.assign({}, child.box, {
+                    top: child.box.top - height,
                 });
-                const next = Object.assign({}, nextChild, { box });
+                const next = Object.assign({}, child, { box });
                 currentChildren.push(...futureFixedNodes);
                 nextChildren.push(next, ...futureNodes);
-                // }
                 break;
             }
             if (currentChild)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ats-form-react-pdf-layout",
-  "version": "4.4.7",
+  "version": "4.4.9",
   "license": "MIT",
   "description": "Resolve document component's layout",
   "author": "Atharva System",