Merge pull request #1 from NateWr/convention_feedback

Fixes in response to Daniel's comments

src/App.js

@@ -1,7 +1,7 @@
 import React, { Component } from 'react';
 import Balance from './components/Balance';
 import MakePayment from './components/MakePayment';
-import Payments from './containers/Payments';
+import Payments from './components/Payments';
 import currencies from './data/currencies';
 import './App.css';
 

src/components/Button.js

@@ -3,7 +3,7 @@ import './Button.css';
 
 function Button(props) {
 	return (
-		<button className="Button">{props.label}</button>
+		<button {...props} className="Button" />
 	)
 }
 

src/components/MakePayment.js

@@ -12,7 +12,7 @@ class Payment extends Component {
   }
 
 	selectCurrency = (event) => {
-		let currency = event.target.value;
+		const currency = event.target.value;
 		this.setState({
 			selectedCurrency: currency
 		});
@@ -28,7 +28,7 @@ class Payment extends Component {
 					</select>
           <input className="MakePayment-amount" type="text" defaultValue="0.00" />
 					<input className="MakePayment-description" type="text" placeholder="What is this payment for?" />
-					<Button label="Pay" />
+					<Button>Pay</Button>
 				</div>
 			</div>
 		)

src/containers/Payments.js → src/components/Payments.js

@@ -23,7 +23,7 @@ function Payments(props) {
 					<td>Dinner with friends at a local restaurant</td>
 					<td>Pending</td>
 					<td>
-						<Button label="Cancel"></Button>
+						<Button>Cancel</Button>
 					</td>
 				</tr>
 				<tr>