plugins/moveElemsAttrsToGroup: merge split-level transforms instead o…

…f replacing (fix #10)
svg · Oct 20, 2012 · 50b825d · 50b825d
1 parent b4141bd
commit 50b825d
Show file tree

Hide file tree

Showing 3 changed files with 26 additions and 1 deletion.
diff --git a/plugins/moveElemsAttrsToGroup.js b/plugins/moveElemsAttrsToGroup.js
@@ -29,6 +29,7 @@ exports.moveElemsAttrsToGroup = function(item, params) {
     if (item.isElem('g') && !item.isEmpty() && item.content.length > 1) {
 
         var intersection = {},
+            hasTransform = false,
             every = item.content.every(function(g) {
                 if (g.elem && g.attrs) {
                     if (!Object.keys(intersection).length) {
@@ -48,7 +49,15 @@ exports.moveElemsAttrsToGroup = function(item, params) {
             item.content.forEach(function(g) {
                 for (var name in intersection) {
                     g.removeAttr(name);
-                    item.addAttr(intersection[name]);
+
+                    if (name === 'transform') {
+                        if (!hasTransform) {
+                            item.attr('transform').value += ' ' + intersection[name].value;
+                            hasTransform = true;
+                        }
+                    } else {
+                        item.addAttr(intersection[name]);
+                    }
                 }
             });
         }

diff --git a/test/plugins/moveElemsAttrsToGroup.04.orig.svg b/test/plugins/moveElemsAttrsToGroup.04.orig.svg
diff --git a/test/plugins/moveElemsAttrsToGroup.04.should.svg b/test/plugins/moveElemsAttrsToGroup.04.should.svg