plugins/removeNonInheritableGroupAttrs: more attrs groups to exclude (fix #118 fix #116)

plugins/removeNonInheritableGroupAttrs.js

@@ -5,7 +5,7 @@ exports.type = 'perItem';
 exports.active = true;
 
 var inheritableAttrs = require('./_collections').inheritableAttrs,
-    presentationAttrs = require('./_collections').attrsGroups.presentation,
+    attrsGroups = require('./_collections').attrsGroups,
     excludedAttrs = ['display', 'opacity'];
 
 /**
@@ -22,9 +22,12 @@ exports.fn = function(item) {
 
         item.eachAttr(function(attr) {
             if (
-                presentationAttrs.indexOf(attr.name) !== -1 &&
-                excludedAttrs.indexOf(attr.name) === -1 &&
-                inheritableAttrs.indexOf(attr.name) === -1
+                ~attrsGroups.presentation.indexOf(attr.name) &&
+                ~attrsGroups.graphicalEvent.indexOf(attr.name) &&
+                ~attrsGroups.core.indexOf(attr.name) &&
+                ~attrsGroups.conditionalProcessing.indexOf(attr.name) &&
+                !~excludedAttrs.indexOf(attr.name) &&
+                !~inheritableAttrs.indexOf(attr.name)
             ) {
                 item.removeAttr(attr.name);
             }

test/plugins/removeNonInheritableGroupAttrs.01.svg

@@ -7,7 +7,7 @@
 @@@
 
 <svg xmlns="http://www.w3.org/2000/svg">
-    <g class="test" transform="rotate(45)" display="none" opacity="0.5" visibility="visible">
+    <g class="test" clip-path="url(#clip1)" transform="rotate(45)" display="none" opacity="0.5" visibility="visible">
         <path d="M0 0 L 10 20"/>
     </g>
 </svg>