implemented removeEmptyTextNodes boolean (defaults to true to prevent…

… breaking changes); fixes GrapesJS#989
nopost · Apr 5, 2018 · 935ad30 · 935ad30
1 parent 376a608
commit 935ad30
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 3 deletions.
diff --git a/src/editor/config/config.js b/src/editor/config/config.js
@@ -62,6 +62,9 @@ module.exports = {
   // Ending tag for variable inside scripts in Components
   tagVarEnd: ' ]}',
 
+  // Remove empty text nodes when parsed, unless they contain a space
+  removeEmptyTextNodes: true,
+
   // Return JS of components inside HTML from 'editor.getHtml()'
   jsInHtml: true,
 

diff --git a/src/editor/index.js b/src/editor/index.js
@@ -80,6 +80,7 @@
  * @param {Object} [config.domComponents={}] Components configuration, see the relative documentation
  * @param {Object} [config.panels={}] Panels configuration, see the relative documentation
  * @param {Object} [config.showDevices=true] If true render a select of available devices inside style manager panel
+ * @param {Boolean} [config.removeEmptyTextNodes=true] If true, removes empty text nodes when parsed, unless they contain a space
  * @param {string} [config.defaultCommand='select-comp'] Command to execute when no other command is running
  * @param {Array} [config.plugins=[]] Array of plugins to execute on start
  * @param {Object} [config.pluginsOpts={}] Custom options for plugins

diff --git a/src/parser/model/ParserHtml.js b/src/parser/model/ParserHtml.js
@@ -57,6 +57,7 @@ module.exports = config => {
      * @return {Array<Object>}
      */
     parseNode(el) {
+      const config = (c.em && c.em.get('Config')) || {};
       const result = [];
       const nodes = el.childNodes;
 
@@ -152,9 +153,11 @@ module.exports = config => {
           }
 
           // Throw away empty nodes (keep spaces)
-          const content = node.nodeValue;
-          if (content != ' ' && !content.trim()) {
-            continue;
+          if (config.removeEmptyTextNodes) {
+            const content = node.nodeValue;
+            if (content != ' ' && !content.trim()) {
+              continue;
+            }
           }
         }