fix: remove excess div wrappers (#4528)

* Removing superfluous div wrapping It seems like standalone text is already wrapped in paragraph elements, so this code seems superfluous. Additionally, it adds div wrappers at every line break as described in #4524 * Fix for newlines and returns This change skips newlines and returns to focus on unbounded text only. * misc: fix indentation * misc: fix indentation (2) Co-authored-by: Nicolas Giard <github@ngpixel.com>

fix: remove excess div wrappers (#4528)
12aef93c · broxen · GitHub · 525f7581 · 12aef93c
Unverified Commit 12aef93c authored Oct 09, 2021 by broxen Committed by GitHub Oct 09, 2021
Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 2 deletions

renderer.js server/modules/rendering/html-core/renderer.js +2 -2

No files found.
--- a/server/modules/rendering/html-core/renderer.js
+++ b/server/modules/rendering/html-core/renderer.js
@@ -234,11 +234,11 @@ module.exports = {
    })

    // --------------------------------
-    // Wrap root text nodes
+    // Wrap non-empty root text nodes
    // --------------------------------

    $('body').contents().toArray().forEach(item => {
-      if (item && item.type === 'text' && item.parent.name === 'body') {
+      if (item && item.type === 'text' && item.parent.name === 'body' && item.data !== `\n` && item.data !== `\r`) {
        $(item).wrap('<div></div>')
      }
    })