Preserve whitespace in elements containing text

lib/svgo/js2svg.js

@@ -1,7 +1,7 @@
 'use strict';
 
 var EOL = require('os').EOL,
-    textElem = require('../../plugins/_collections.js').elemsGroups.textContent.concat('title');
+    textElems = require('../../plugins/_collections.js').textElems;
 
 var defaults = {
     doctypeStart: '<!DOCTYPE',
@@ -258,7 +258,7 @@ JS2SVG.prototype.createElem = function(data) {
             tagCloseStart = this.config.tagCloseStart,
             tagCloseEnd = this.config.tagCloseEnd,
             openIndent = this.createIndent(),
-            textIndent = '',
+            closeIndent = this.createIndent(),
             processedData = '',
             dataEnd = '';
 
@@ -268,29 +268,27 @@ JS2SVG.prototype.createElem = function(data) {
             tagCloseStart = defaults.tagCloseStart;
             tagCloseEnd = defaults.tagCloseEnd;
             openIndent = '';
-        } else if (data.isElem(textElem)) {
-            if (this.config.pretty) {
-                textIndent += openIndent + this.config.indent;
-            }
+        } else if (data.isElem(textElems)) {
+            tagOpenEnd = defaults.tagOpenEnd;
+            tagCloseStart = defaults.tagCloseStart;
+            closeIndent = '';
             this.textContext = data;
         }
 
         processedData += this.convert(data).data;
 
         if (this.textContext == data) {
             this.textContext = null;
-            if (this.config.pretty) dataEnd = EOL;
         }
 
         return  openIndent +
                 tagOpenStart +
                 data.elem +
                 this.createAttrs(data) +
                 tagOpenEnd +
-                textIndent +
                 processedData +
                 dataEnd +
-                this.createIndent() +
+                closeIndent +
                 tagCloseStart +
                 data.elem +
                 tagCloseEnd;

lib/svgo/svg2js.js

@@ -4,12 +4,13 @@ var SAX = require('sax'),
     JSAPI = require('./jsAPI.js'),
     CSSClassList = require('./css-class-list'),
     CSSStyleDeclaration = require('./css-style-declaration'),
-    entityDeclaration = /<!ENTITY\s+(\S+)\s+(?:'([^\']+)'|"([^\"]+)")\s*>/g;
+    entityDeclaration = /<!ENTITY\s+(\S+)\s+(?:'([^\']+)'|"([^\"]+)")\s*>/g,
+    textElems = require('../../plugins/_collections.js').textElems;
 
 var config = {
     strict: true,
     trim: false,
-    normalize: true,
+    normalize: false,
     lowercase: true,
     xmlns: true,
     position: true
@@ -117,8 +118,8 @@ module.exports = function(data, callback) {
         elem = pushToContent(elem);
         current = elem;
 
-        // Save info about <text> tag to prevent trimming of meaningful whitespace
-        if (data.name == 'text' && !data.prefix) {
+        // Save info about tags containing text to prevent trimming of meaningful whitespace
+        if (textElems.indexOf(data.name) !== -1 && !data.prefix) {
             textContext = current;
         }
 
@@ -145,9 +146,7 @@ module.exports = function(data, callback) {
 
         var last = stack.pop();
 
-        // Trim text inside <text> tag.
         if (last == textContext) {
-            trim(textContext);
             textContext = null;
         }
         current = stack[stack.length - 1];

plugins/_collections.js

@@ -15,6 +15,8 @@ exports.elemsGroups = {
     filterPrimitive: ['feBlend', 'feColorMatrix', 'feComponentTransfer', 'feComposite', 'feConvolveMatrix', 'feDiffuseLighting', 'feDisplacementMap', 'feFlood', 'feGaussianBlur', 'feImage', 'feMerge', 'feMorphology', 'feOffset', 'feSpecularLighting', 'feTile', 'feTurbulence']
 };
 
+exports.textElems = exports.elemsGroups.textContent.concat('title');
+
 exports.pathElems = ['path', 'glyph', 'missing-glyph'];
 
 // http://www.w3.org/TR/SVG11/intro.html#Definitions

test/svg2js/_index.js

@@ -186,6 +186,15 @@ describe('svg2js', function() {
 
         });
 
+        describe('text nodes', function() {
+
+            it('should contain preserved whitespace', function() {
+                const textNode = root.content[3].content[1].content[0].content[1]
+                return textNode.content[0].text.should.equal('  test  ')
+            });
+
+        });
+
         describe('API', function() {
 
             describe('clone()', function() {