[IMP] List,Pre,Table: simplify renderChildren usages

By leveraging `append` and `flat`.

Author: dmo-odoo

packages/plugin-list/src/ListHtmlDomRenderer.ts

@@ -12,14 +12,8 @@ export class ListHtmlDomRenderer extends AbstractRenderer<Node[]> {
         const tag = node.listType === ListType.ORDERED ? 'OL' : 'UL';
         const domListNode = document.createElement(tag);
         this.engine.renderAttributes(Attributes, node, domListNode);
-
-        for (const child of node.childVNodes) {
-            const renderedChild = await this.engine.render(child);
-            for (const domChild of renderedChild) {
-                domListNode.appendChild(domChild);
-            }
-        }
-
+        const renderedChildren = await this.renderChildren(node);
+        domListNode.append(...renderedChildren.flat());
         return [domListNode];
     }
 }

packages/plugin-list/src/ListItemHtmlDomRenderer.ts

@@ -23,17 +23,11 @@ export class ListItemHtmlDomRenderer extends AbstractRenderer<Node[]> {
         // while other nodes will be rendered inside the "LI".
         if (node.is(VElement) && node.htmlTag === 'P') {
             const renderedChildren = await this.renderChildren(node);
-            for (const renderedChild of renderedChildren) {
-                for (const domChild of renderedChild) {
-                    domListItem.appendChild(domChild);
-                }
-            }
+            domListItem.append(...renderedChildren.flat());
         } else {
             // The node was wrapped in a "LI" but needs to be rendered as well.
             const renderedNode = await this.super.render(node);
-            for (const domNode of renderedNode) {
-                domListItem.appendChild(domNode);
-            }
+            domListItem.append(...renderedNode);
         }
         // Render the node's attributes that were stored on the technical key
         // that specifies those attributes belong on the list item.

packages/plugin-pre/src/PreHtmlDomRenderer.ts

@@ -14,10 +14,8 @@ export class PreHtmlDomRenderer extends AbstractRenderer<Node[]> {
     async render(node: PreNode): Promise<Node[]> {
         const pre = document.createElement('pre');
         this.engine.renderAttributes(Attributes, node, pre);
-        for (const child of node.children()) {
-            const domChildren = await this.renderChild(child);
-            pre.append(...domChildren);
-        }
+        const renderedChildren = await this.renderChildren(node);
+        pre.append(...renderedChildren.flat());
         return [pre];
     }
 }

packages/plugin-table/src/TableCellHtmlDomRenderer.ts

@@ -21,11 +21,7 @@ export class TableCellHtmlDomRenderer extends AbstractRenderer<Node[]> {
         // Render the cell and its contents.
         const td = document.createElement(cell.header ? 'th' : 'td');
         const renderedChildren = await this.renderChildren(cell);
-        for (const renderedChild of renderedChildren) {
-            for (const domChild of renderedChild) {
-                td.append(domChild);
-            }
-        }
+        td.append(...renderedChildren.flat());
 
         // Render attributes.
         // Colspan and rowspan are handled differently from other attributes:

packages/plugin-table/src/TableRowHtmlDomRenderer.ts

@@ -15,11 +15,7 @@ export class TableRowHtmlDomRenderer extends AbstractRenderer<Node[]> {
         const domRow = document.createElement('tr');
         this.engine.renderAttributes(Attributes, row, domRow);
         const renderedChildren = await this.renderChildren(row);
-        for (const renderedChild of renderedChildren) {
-            for (const domChild of renderedChild) {
-                domRow.appendChild(domChild);
-            }
-        }
+        domRow.append(...renderedChildren.flat());
         return [domRow];
     }
 }