Disable removeUselessStrokeAndFill if there is a <style> or <script>

cmp-twitt · Mar 29, 2015 · cf37722 · cf37722
1 parent 00df0b3
commit cf37722
Show file tree

Hide file tree

Showing 2 changed files with 29 additions and 2 deletions.
diff --git a/plugins/removeUselessStrokeAndFill.js b/plugins/removeUselessStrokeAndFill.js
@@ -11,7 +11,9 @@ exports.params = {
 
 var shape = require('./_collections').elemsGroups.shape,
     regStrokeProps = /^stroke/,
-    regFillProps = /^fill-/;
+    regFillProps = /^fill-/,
+    styleOrScript = ['style', 'script'],
+    hasStyleOrScript = false;
 
 /**
  * Remove useless stroke and fill attrs.
@@ -24,7 +26,11 @@ var shape = require('./_collections').elemsGroups.shape,
  */
 exports.fn = function(item, params) {
 
-    if (item.isElem(shape) && !item.computedAttr('id')) {
+    if (item.isElem(styleOrScript)) {
+        hasStyleOrScript = true;
+    }
+
+    if (!hasStyleOrScript && item.isElem(shape) && !item.computedAttr('id')) {
 
         var stroke = params.stroke && item.computedAttr('stroke'),
             fill = params.fill && !item.computedAttr('fill', 'none');

diff --git a/test/plugins/removeUselessStrokeAndFill.03.svg b/test/plugins/removeUselessStrokeAndFill.03.svg