Prevented collapsing group with passing animated attribute

Fixes #534
svg · Apr 25, 2016 · 2bbabee · 2bbabee
1 parent 7d556d0
commit 2bbabee
Show file tree

Hide file tree

Showing 2 changed files with 19 additions and 0 deletions.
diff --git a/plugins/collapseGroups.js b/plugins/collapseGroups.js
@@ -8,6 +8,11 @@ exports.description = 'collapses useless groups';
 
 var animationElems = require('./_collections').elemsGroups.animation;
 
+function hasAnimatedAttr(item) {
+    return item.isElem(animationElems) && item.hasAttr('attributeName', this) ||
+        !item.isEmpty() && item.content.some(hasAnimatedAttr, this);
+}
+
 /*
  * Collapse useless groups.
  *
@@ -50,6 +55,8 @@ exports.fn = function(item) {
                         inner.isElem('g') && !g.hasAttr('transform') && !inner.hasAttr('transform')
                     )) {
                         g.eachAttr(function(attr) {
+                            if (g.content.some(hasAnimatedAttr, attr.name)) return;
+
                             if (!inner.hasAttr(attr.name)) {
                                 inner.addAttr(attr);
                             } else if (attr.name == 'transform' || attr.name == 'class') {

diff --git a/test/plugins/collapseGroups.11.svg b/test/plugins/collapseGroups.11.svg