Merge pull request #11 from stryju/patch-3

TODO: don't add class dups
iconic · Jun 27, 2014 · f2b59b7 · f2b59b7
2 parents 1f09fd9 + 1814600
commit f2b59b7
Showing 1 changed file with 18 additions and 2 deletions.
diff --git a/svg-injector.js b/svg-injector.js
@@ -14,6 +14,23 @@
   var isLocal = window.location.protocol === 'file:';
   var hasSvgSupport = document.implementation.hasFeature('http://www.w3.org/TR/SVG11/feature#BasicStructure', '1.1');
 
+  function mergeClasses( list ) {
+    list = list.split(' ');
+
+    var hash = {};
+    var i    = list.length;
+    var out  = [];
+
+    while (i--) {
+      if (!hash.hasOwnProperty(list[i])) {
+        hash[list[i]] = 1;
+        out.unshift(list[i]);
+      }
+    }
+
+    return out.join(' ');
+  }
+
   // SVG Cache
   var svgCache = {};
 
@@ -201,9 +218,8 @@
 
       var imgClass = el.getAttribute('class');
       if (imgClass) {
-        // :TODO: don't add dups
         var classMerge = [svg.getAttribute('class'), 'iconic-injected-svg', imgClass].join(' ');
-        svg.setAttribute('class', classMerge);
+        svg.setAttribute('class', mergeClasses(classMerge));
       }
 
       var imgStyle = el.getAttribute('style');