fix table break to new page

Author: rushabh-atharva

packages/layout/src/node/getWrap.ts

@@ -31,6 +31,10 @@ const getWrap = (
 
   if (!node.props) return true;
 
+  // Check wrap prop first - if explicitly set to false, respect that
+  // This allows elements to move to the next page instead of splitting
+  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;

packages/layout/src/node/shouldBreak.ts

@@ -37,7 +37,12 @@ const shouldBreak = (
   if ('fixed' in child.props) return false;
 
   const shouldSplit = height < child.box.top + child.box.height;
-  const canWrap = getWrap(child, child, []);
+  // Use consistent parameters with splitNodes for wrap calculation
+  const prevNode =
+    previousElements.length > 0
+      ? previousElements[previousElements.length - 1]
+      : undefined;
+  const canWrap = getWrap(child, prevNode, previousElements);
 
   // Calculate the y coordinate where the desired presence of the child ends
   const endOfPresence = getEndOfPresence(child, futureElements);

packages/layout/src/steps/resolvePagination.ts

@@ -63,7 +63,7 @@ const breakableViewChild = (children, height, path = '', currentChildren) => {
     if (
       shouldNodeBreak(
         children[i],
-        children.slice(i + 1, height),
+        children.slice(i + 1),
         height,
         currentChildren,
       )
@@ -136,7 +136,8 @@ const splitNodes = (height: number, contentArea: number, nodes: SafeNode[]) => {
         warnUnavailableSpace(child);
         break;
       } else {
-        // We don't want to break non wrapable nodes, so we just let them be.
+        // Element is larger than a page and has wrap=false
+        // Move to next page and enable wrapping so it can flow across pages
         const box = Object.assign({}, child.box, {
           top: child.box.top - height,
         });
@@ -152,6 +153,21 @@ const splitNodes = (height: number, contentArea: number, nodes: SafeNode[]) => {
       }
     }
 
+    // Element fits on a full page but not in remaining space - move to next page
+    // This keeps sections together when they can fit on a fresh 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;
+    }
+
     if (shouldBreak) {
       const box = Object.assign({}, child.box, { top: child.box.top - height });
       const props = Object.assign({}, child.props, {

packages/renderer/.gitignore

@@ -1 +0,0 @@
-lib