@flowaccount/pdfmake 1.0.7-staging.1 → 1.0.7-staging.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@flowaccount/pdfmake",
-  "version": "1.0.7-staging.1",
+  "version": "1.0.7-staging.3",
   "description": "Client/server side PDF printing in pure JavaScript",
   "main": "src/printer.js",
   "browser": "build/pdfmake.js",

package/src/layoutBuilder.js

@@ -538,7 +538,8 @@ LayoutBuilder.prototype.processNode = function (node) {
 					startY: ctx.y,
 					startPage: ctx.page,
 					availableHeight: ctx.availableHeight,
-					itemCount: currentPage ? currentPage.items.length : 0
+					itemCount: currentPage ? currentPage.items.length : 0,
+					x: ctx.x
 				};
 			}
 
@@ -683,6 +684,129 @@ LayoutBuilder.prototype.processNode = function (node) {
 					// Intermediate pages (remark only) should not be repositioned
 					var pageIdx = ctx.page;
 					if (pageIdx > preRenderState.startPage) {
+						// Also reposition items on the START page (where summary might be)
+						var startPage = pages[preRenderState.startPage];
+						if (startPage && startPage.items && startPage.items.length > 0) {
+							var startPageHeight = startPage.pageSize.height;
+							var startPageBottom = startPageHeight - pageMargins.bottom;
+
+							// Find content bottom on start page
+							var startContentBottom = 0;
+							for (var si = 0; si < startPage.items.length; si++) {
+								var sItem = startPage.items[si];
+								var sItemBottom = 0;
+								if (sItem.item) {
+									if (typeof sItem.item.y === 'number') {
+										var sh = 0;
+										// Line items have getHeight() method
+										if (sItem.type === 'line' && typeof sItem.item.getHeight === 'function') {
+											sh = sItem.item.getHeight();
+										} else {
+											sh = sItem.item.h || sItem.item.height || 0;
+										}
+										sItemBottom = sItem.item.y + sh;
+									}
+									if (typeof sItem.item.y2 === 'number' && sItem.item.y2 > sItemBottom) {
+										sItemBottom = sItem.item.y2;
+									}
+								}
+								if (sItemBottom > startContentBottom) {
+									startContentBottom = sItemBottom;
+								}
+							}
+
+							// Calculate gap and move items on start page
+							var startGapHeight = startPageBottom - startContentBottom;
+							if (startGapHeight > 0) {
+								var startStackStartY = preRenderState.startY;
+								for (var si = 0; si < startPage.items.length; si++) {
+									var sItem = startPage.items[si];
+									var sItemY = null;
+									if (sItem.item && typeof sItem.item.y1 === 'number') {
+										sItemY = sItem.item.y1;
+									} else if (sItem.item && typeof sItem.item.y === 'number') {
+										sItemY = sItem.item.y;
+									}
+
+										// Skip items fully above footer start
+									if (sItemY === null || sItemY < startStackStartY) continue;
+
+									// Move item down
+									if (sItem.item && typeof sItem.item.y === 'number') {
+										sItem.item.y += startGapHeight;
+									}
+									if (sItem.item && typeof sItem.item.y1 === 'number') {
+										sItem.item.y1 += startGapHeight;
+									}
+									if (sItem.item && typeof sItem.item.y2 === 'number') {
+										sItem.item.y2 += startGapHeight;
+									}
+									// Handle polylines (points array)
+									if (sItem.item && sItem.item.points && Array.isArray(sItem.item.points)) {
+										for (var spi = 0; spi < sItem.item.points.length; spi++) {
+											if (typeof sItem.item.points[spi].y === 'number') {
+												sItem.item.points[spi].y += startGapHeight;
+											}
+										}
+									}
+								}
+
+								// MULTI-PAGE: Draw guide lines in the gap area (same as single-page)
+								var colSpec = node._footerGapOption && node._footerGapOption.columns ? node._footerGapOption.columns : (self._footerGapOption && self._footerGapOption.columns);
+								if (colSpec) {
+									var rawWidths = colSpec.content && colSpec.content.vLines ? colSpec.content.vLines : [];
+									if (rawWidths && rawWidths.length > 1) {
+										var style = (colSpec.style || {});
+										var lw = style.lineWidth != null ? style.lineWidth : 0.5;
+										var lc = style.color || '#000000';
+										var dashCfg = style.dash;
+										var includeOuter = colSpec.includeOuter !== false;
+										var startIndex = includeOuter ? 0 : 1;
+										var endIndex = includeOuter ? rawWidths.length : rawWidths.length - 1;
+										var pageX = preRenderState.x || pageMargins.left;
+
+										for (var ci = startIndex; ci < endIndex; ci++) {
+											var xGuide = pageX + rawWidths[ci] - 0.25;
+											// Extend first and last vertical lines to content bottom after moving
+											var isFirstOrLast = (ci === 0) || (ci === rawWidths.length - 1);
+											var vLineY2 = isFirstOrLast ? (startContentBottom + startGapHeight) : (startStackStartY + startGapHeight);
+											startPage.items.push({
+												type: 'vector',
+												item: {
+													type: 'line',
+													x1: xGuide,
+													y1: startStackStartY,
+													x2: xGuide,
+													y2: vLineY2,
+													lineWidth: lw,
+													lineColor: lc,
+													dash: dashCfg ? {
+														length: dashCfg.length,
+														space: dashCfg.space != null ? dashCfg.space : dashCfg.gap
+													} : undefined,
+													_footerGuideLine: true
+												}
+											});
+										}
+
+										// Remove existing horizontal line at startStackStartY (drawn by tableProcessor)
+										// It's at wrong position now - we'll draw new one at bottom of gap
+										for (var ri = startPage.items.length - 1; ri >= 0; ri--) {
+											var rItem = startPage.items[ri];
+											if (rItem.type === 'vector' && rItem.item && rItem.item.type === 'line') {
+												// Check if it's a horizontal line at startStackStartY
+												if (Math.abs(rItem.item.y1 - startStackStartY) < 1 &&
+													Math.abs(rItem.item.y2 - startStackStartY) < 1) {
+													startPage.items.splice(ri, 1);
+													break;
+												}
+											}
+										}
+									}
+								}
+							}
+						}
+
 						var page = pages[pageIdx];
 						if (page && page.items && page.items.length > 0) {
 							var pageHeight = page.pageSize.height;

package/src/tableProcessor.js

@@ -560,11 +560,13 @@ TableProcessor.prototype.endRow = function (rowIndex, writer, pageBreaks, nextRo
 		var ctx = writer.context();
 		var currentPage = ctx.getCurrentPage && ctx.getCurrentPage();
 		var pageHeight = currentPage ? (currentPage.pageSize.height - ctx.pageMargins.bottom) : 0;
-
+		var footerGapOptBreak = ctx._footerGapOption;
+		var hLineY = (footerGapOptBreak && footerGapOptBreak.enabled) ? pageHeight - 0.25 : y2;
 		if (willBreak && this.layout.hLineWhenBroken !== false) {
 			// Check if we're at the true page bottom
 			var isAtTruePageBottom = (pageHeight - y2) <= nearBottomThreshold;
-			this.drawHorizontalLine(rowIndex + 1, writer, y2, true, null, isAtTruePageBottom);
+			// Use pageHeight for footer gap tables, y2 for normal tables
+			this.drawHorizontalLine(rowIndex + 1, writer, hLineY, true, null, isAtTruePageBottom);
 		}
 		if (rowBreakWithoutHeader && this.layout.hLineWhenBroken !== false) {
 			// Check if previous segment ended at true page bottom
@@ -573,6 +575,10 @@ TableProcessor.prototype.endRow = function (rowIndex, writer, pageBreaks, nextRo
 			this.drawHorizontalLine(rowIndex, writer, y1, true, null, prevWasAtPageBottom);
 		}
 
+		// vLineY: ONLY use hLineY when BOTH willBreak AND footerGapOptBreak.enabled are true
+		// Otherwise use y2 (original behavior) to avoid NaN errors
+		var vLineY = (willBreak && footerGapOptBreak && footerGapOptBreak.enabled) ? hLineY : y2;
+
 		for (i = 0, l = xs.length; i < l; i++) {
 			var leftCellBorder = false;
 			var rightCellBorder = false;
@@ -599,7 +605,7 @@ TableProcessor.prototype.endRow = function (rowIndex, writer, pageBreaks, nextRo
 			}
 
 			if (leftCellBorder) {
-				this.drawVerticalLine(xs[i].x, y1 - hzLineOffset, y2 + this.bottomLineWidth, xs[i].index, writer, rowIndex, xs[i - 1] ? xs[i - 1].index : null);
+				this.drawVerticalLine(xs[i].x, y1 - hzLineOffset, vLineY + this.bottomLineWidth, xs[i].index, writer, rowIndex, xs[i - 1] ? xs[i - 1].index : null);
 			}
 
 			if (i < l - 1) {

package/src/textTools.js

@@ -317,7 +317,7 @@ function getStyleProperty(item, styleContextStack, property, defaultValue) {
 	}
 
 	if(property === 'bold' && (item.text === '' || item.text === '\u200B')) {
-		return false
+		return false;
 	}
 
 	if (value !== null && value !== undefined) {