fix: render empty list entries (#2893)

Author: liborm85

CHANGELOG.md

@@ -3,6 +3,7 @@
 ## Unreleased
 
 - Update Roboto font (version 3.014)
+- Fixed render empty list entries
 
 ## 0.2.20 - 2025-05-09
 

examples/lists.js

@@ -395,6 +395,23 @@ var docDefinition = {
 				{ text: 'item 3', listType: 'circle' }
 			]
 		},
+		{ text: '\n\nOrdered list with an empty item:', style: 'header' },
+		{
+			ol: [
+				'First item',
+				'Second item',
+				'',
+				'Fourth item'
+			]
+		},
+		{ text: '\n\nUnordered list with an empty item:', style: 'header' },
+		{
+			ul: [
+				'First bullet',
+				'',
+				'Third bullet'
+			]
+		},
 	],
 	styles: {
 		header: {

src/layoutBuilder.js

@@ -66,9 +66,17 @@ LayoutBuilder.prototype.layoutDocument = function (docStructure, fontProvider, s
 			return false;
 		}
 
-		linearNodeList = linearNodeList.filter(function (node) {
-			return node.positions.length > 0;
-		});
+		const hasRenderableContent = node => {
+			if (!node || node.positions.length === 0) {
+				return false;
+			}
+			if (node.text === '' && !node.listMarker) {
+				return false;
+			}
+			return true;
+		};
+
+		linearNodeList = linearNodeList.filter(hasRenderableContent);
 
 		linearNodeList.forEach(function (node) {
 			var nodeInfo = {};

src/textTools.js

@@ -120,13 +120,25 @@ TextTools.prototype.widthOfString = function (text, font, fontSize, characterSpa
 
 function splitWords(text, noWrap) {
 	var results = [];
+
+	if (text === undefined || text === null) {
+		text = '';
+	} else {
+		text = String(text);
+	}
+
 	text = text.replace(/\t/g, '    ');
 
 	if (noWrap) {
 		results.push({ text: text });
 		return results;
 	}
 
+	if (text.length === 0) {
+	 	results.push({ text: '' });
+	 	return results;
+	}
+
 	var breaker = new LineBreaker(text);
 	var last = 0;
 	var bk;

tests/textTools.js

@@ -241,13 +241,17 @@ describe('TextTools', function () {
 		it('should support cast to text', function () {
 			var node = docPreprocessor.preprocessNode(mixedTextArrayWithVariousTypes);
 			var result = TextTools.__get__('normalizeTextArray')(node.stack, styleStack);
-			assert.equal(result.length, 6);
-			assert.equal(result[0].text, '2016');
-			assert.equal(result[1].text, 'true');
-			assert.equal(result[2].text, 'false');
-			assert.equal(result[3].text, '2016');
-			assert.equal(result[4].text, 'true');
-			assert.equal(result[5].text, 'false');
+			assert.equal(result.length, 10);
+			assert.equal(result[0].text, '');
+			assert.equal(result[1].text, '');
+			assert.equal(result[2].text, '2016');
+			assert.equal(result[3].text, 'true');
+			assert.equal(result[4].text, 'false');
+			assert.equal(result[5].text, '');
+			assert.equal(result[6].text, '');
+			assert.equal(result[7].text, '2016');
+			assert.equal(result[8].text, 'true');
+			assert.equal(result[9].text, 'false');
 		});
 
 		it('should support keep noWrap from style', function () {