Fixed attributes resolving during the defs regrouping.

Closes #138

CHANGELOG.md

@@ -5,6 +5,8 @@ The format is based on [Keep a Changelog](http:https://keepachangelog.com/)
 and this project adheres to [Semantic Versioning](http:https://semver.org/).
 
 ## [Unreleased]
+### Fixed
+- Attributes resolving during the `defs` regrouping.
 
 ## [0.9.4] - 2018-03-22
 ### Fixed

src/task/group_defs.rs

@@ -105,8 +105,8 @@ fn resolve_attrs(node: &Node) {
  | EId::Mask
  | EId::Pattern
  | EId::Symbol => {
- let mut parent = node.clone();
- while let Some(p) = parent.parent() {
+ let mut parent = Some(node.clone());
+ while let Some(p) = parent {
  let attrs = p.attributes();
  for (aid, attr) in attrs.iter_svg().filter(|&(_, a)| a.is_inheritable()) {
  for mut child in node.children() {
@@ -118,7 +118,7 @@ fn resolve_attrs(node: &Node) {
  }
  }
 
- parent = p.clone();
+ parent = p.parent();
  }
  }
  _ => {}
@@ -318,6 +318,24 @@ mod tests {
  </defs>
  <g fill='url(#lg1)'/>
 </svg>
+");
+
+ test!(move_attrs_3,
+"<svg>
+ <g fill='none'>
+ <mask fill='#ff0000'>
+ <path/>
+ </mask>
+ </g>
+</svg>",
+"<svg>
+ <defs>
+ <mask fill='#ff0000'>
+ <path fill='#ff0000'/>
+ </mask>
+ </defs>
+ <g fill='none'/>
+</svg>
 ");
 
 }