Remove unnecessary parts

+ handling of props is taken care by {...this.props}. {this.props.x ?
this.props.x : "default"} becomes unnecessary.
+ Correct README regarding this
+ Add to README - class to className change under internals

README.md

@@ -168,12 +168,24 @@ The props passed to the output component is passed on to the root SVG node and t
 is transformed to
 
 ```html
-<svg width={this.props.width ? this.props.width : '50'} {...this.props}>
+<svg width="50" {...this.props}>
   ...
 </svg>
 ```
 
-#### 4. export React.Component
+#### 4. class to className
+
+```html
+<svg class="hello"/>
+```
+
+is transformed to
+
+```html
+<svg className="hello"/>
+```
+
+#### 5. export React.Component
 
 The loader should now export the svg component. And this is done by wrapping it in a Component Class.
 

src/plugin.js

@@ -52,50 +52,18 @@ export default function (babel) {
     }
   };
 
-  // Flow props from Output component to root SVG
-  // converts
-  // <svg width="50">
-  // to
-  // <svg width={this.props.width ? this.props.width : "50"}>
-  const attrVisitor = {
-    JSXAttribute(path) {
-      if (!t.isJSXIdentifier(path.node.name)) return;
-      // don't handle style attr. It needs to be an object
-      if (path.node.name.name === 'style') return;
-
-      // else
-      const valueExpression = t.memberExpression(
-        t.memberExpression(
-          t.thisExpression(),
-          t.identifier('props')
-        ),
-        t.identifier(hyphenToCamel(path.node.name.name))
-      );
-
-      path.node.value = t.jSXExpressionContainer(
-        t.conditionalExpression(
-          valueExpression,
-          valueExpression,
-          t.stringLiteral(path.node.value.value)
-        )
-      );
-
-    }
-  };
-
   // converts
   // <svg>
   // to
   // <svg {...this.props}>
   // after passing through attributes visitors
   const svgVisitor = {
     JSXOpeningElement(path) {
-      if (t.isJSXIdentifier(path.node.name) && path.node.name.name.toLowerCase() === 'svg') {
-        path.traverse(classNameVisitor);
-        path.traverse(camelizeVisitor);
-        path.traverse(attrVisitor);
-        path.traverse(styleAttrVisitor);
+      path.traverse(classNameVisitor);
+      path.traverse(camelizeVisitor);
+      path.traverse(styleAttrVisitor);
 
+      if (path.node.name.name.toLowerCase() === 'svg') {
         // add spread props
         path.node.attributes.push(
           t.jSXSpreadAttribute(
@@ -105,11 +73,6 @@ export default function (babel) {
             )
           )
         );
-      } else {
-        // don't ignore style attr transformations for other nodes
-        path.traverse(classNameVisitor);
-        path.traverse(camelizeVisitor);
-        path.traverse(styleAttrVisitor);
       }
     }
   };