Merge pull request svg#546 from gjjones/master

adds addAttributes plugin

lib/svgo/js2svg.js

@@ -316,11 +316,18 @@ JS2SVG.prototype.createAttrs = function(elem) {
 
     elem.eachAttr(function(attr) {
 
-        attrs +=    ' ' +
-                    attr.name +
-                    this.config.attrStart +
-                    String(attr.value).replace(this.config.regValEntities, this.config.encodeEntity) +
-                    this.config.attrEnd;
+        if (attr.value !== undefined) {
+            attrs +=    ' ' +
+                        attr.name +
+                        this.config.attrStart +
+                        String(attr.value).replace(this.config.regValEntities, this.config.encodeEntity) +
+                        this.config.attrEnd;
+        }
+        else {
+            attrs +=    ' ' +
+                        attr.name;
+        }
+
 
     }, this);
 

lib/svgo/jsAPI.js

@@ -248,7 +248,6 @@ JSAPI.prototype.renameElem = function(name) {
     attr = attr || {};
 
     if (attr.name === undefined ||
-        attr.value === undefined ||
         attr.prefix === undefined ||
         attr.local === undefined
     ) return false;

plugins/addAttributesToSVGElement.js

@@ -0,0 +1,57 @@
+'use strict';
+
+exports.type = 'full';
+
+exports.active = false;
+
+exports.description = 'adds attributes to an outer <svg> element';
+
+var ENOCLS = 'Error in plugin "addAttributesToSVGElement": absent parameters.\n\
+It should have a list of classes in "attributes" or one "attribute".\n\
+Config example:\n\n\
+\
+plugins:\n\
+- addAttributesToSVGElement:\n\
+    attribute: "mySvg"\n\n\
+\
+plugins:\n\
+- addAttributesToSVGElement:\n\
+    attributes: ["mySvg", "size-big"]\n';
+
+/**
+ * Add attributes to an outer <svg> element. Example config:
+ *
+ * plugins:
+ * - addAttributesToSVGElement:
+ *     attribute: 'data-icon'
+ *
+ * plugins:
+ * - addAttributesToSVGElement:
+ *     attributes: ['data-icon', 'data-disabled']
+ *
+ * @author April Arcus
+ */
+exports.fn = function(data, params) {
+    if (!params || !(Array.isArray(params.attributes) && params.attributes.some(String) || params.attribute)) {
+        console.error(ENOCLS);
+        return data;
+    }
+
+    var attributes = params.attributes || [ params.attribute ],
+        svg = data.content[0];
+
+    if (svg.isElem('svg')) {
+        attributes.forEach(function (attribute) {
+            if (!svg.hasAttr(attribute)) {
+                svg.addAttr({
+                    name: attribute,
+                    prefix: '',
+                    local: attribute
+                });
+            }
+        });
+    }
+
+    return data;
+
+};