clear none text font styles instead of setting to none

Author: archmoj

Diff for: src/components/drawing/index.js

@@ -46,11 +46,15 @@ drawing.font = function(s, font) {
     if(style) s.style('font-style', style);
     if(variant) s.style('font-variant', variant);
 
-    if(textcase) s.style('text-transform', textcase2transform(textcase));
-    if(shadow) s.style('text-shadow', shadow === 'auto' ? svgTextUtils.makeTextShadow(Color.contrast(color)) : shadow);
-    if(lineposition) s.style('text-decoration-line', lineposition2decorationLine(lineposition));
+    if(textcase) s.style('text-transform', dropNone(textcase2transform(textcase)));
+    if(shadow) s.style('text-shadow', shadow === 'auto' ? svgTextUtils.makeTextShadow(Color.contrast(color)) : dropNone(shadow));
+    if(lineposition) s.style('text-decoration-line', dropNone(lineposition2decorationLine(lineposition)));
 };
 
+function dropNone(a) {
+    return a === 'none' ? undefined : a;
+}
+
 var textcase2transformOptions = {
     normal: 'none',
     lower: 'lowercase',

Diff for: src/snapshot/tosvg.js

@@ -118,20 +118,6 @@ module.exports = function toSVG(gd, format, scale) {
             if(fv && fv === 'normal') {
                 txt.style('font-variant', undefined);
             }
-
-            // Drop none text-shadow, text-transform and text-decoration-line to reduce the size
-            var ts = this.style.textShadow;
-            if(ts && ts === 'none') {
-                txt.style('text-shadow', undefined);
-            }
-            var tt = this.style.textTransform;
-            if(tt && tt === 'none') {
-                txt.style('text-transform', undefined);
-            }
-            var td = this.style.textDecorationLine;
-            if(td && td === 'none') {
-                txt.style('text-decoration-line', undefined);
-            }
         });
 
     svg.selectAll('.gradient_filled,.pattern_filled').each(function() {