code improvemnts: to please master teawaterwire

components/App.jsx

@@ -81,16 +81,12 @@ let App = React.createClass({
 
   // return recalculated balance
   getNewBalance(transactions, currency) {
-    let balance = 0;
-
-    transactions.forEach((t) => {
-      balance += fx.convert(
-        t.amount,
-        {from: t.currency, to: currency}
+    return transactions.reduce((previous, current) => {
+      return previous + fx.convert(
+        current.amount,
+        {from: current.currency, to: currency}
       );
-    });
-
-    return balance;
+    }, 0);
   },
 
   render() {

components/CurrencySelection.jsx

@@ -2,9 +2,7 @@ let React = require('react');
 
 let classNames = require('classnames');
 
-let ReactBootstrap = require('react-bootstrap');
-let ButtonGroup = ReactBootstrap.ButtonGroup;
-let Button = ReactBootstrap.Button;
+let {ButtonGroup, Button} = require('react-bootstrap');
 
 // currency selection component
 let CurrencySelection = React.createClass({

components/MainMenu.jsx

@@ -1,9 +1,6 @@
 let React = require('react');
 
-let ReactBootstrap = require('react-bootstrap');
-let Nav = ReactBootstrap.Nav;
-let NavItem = ReactBootstrap.NavItem;
-let Navbar = ReactBootstrap.Navbar;
+let {Navbar, Nav, NavItem} = require('react-bootstrap');
 
 // top menu
 let MainMenu = React.createClass({

components/TransactionAction.jsx

@@ -1,15 +1,10 @@
 let React = require('react');
 
+let {Grid, Row, Col, Input, Button} = require('react-bootstrap');
+
 let classNames = require('classnames');
 let accounting = require('accounting');
 
-let ReactBootstrap = require('react-bootstrap');
-let Grid = ReactBootstrap.Grid;
-let Row = ReactBootstrap.Row;
-let Col = ReactBootstrap.Col;
-let Input = ReactBootstrap.Input;
-let Button = ReactBootstrap.Button;
-
 const DEPOSIT = 'deposit';
 const WITHDRAWAL = 'withdrawal';
 

components/TransactionHistory.jsx

@@ -1,7 +1,6 @@
 let React = require('react');
 
-let ReactBootstrap = require('react-bootstrap');
-let Table = ReactBootstrap.Table;
+let {Table} = require('react-bootstrap');
 
 let TransactionItem = require('./TransactionItem');
 
@@ -12,7 +11,7 @@ let TransactionHistory = React.createClass({
     let transactions = this.props.transactions.map((t, i) => {
       return (
         <TransactionItem
-          key={this.props.transactions.size - i - 1}
+          key={i}
           number={this.props.transactions.size - i}
           amount={t.amount}
           currency={t.currency}

components/TransactionItem.jsx

@@ -12,14 +12,8 @@ let TransactionItem = React.createClass({
     let faCurrency = 'fa-' + this.props.currency.toLowerCase();
     let classesCurrency = classNames('fa', faCurrency);
 
-    if (this.props.amount > 0) {
-      typeClass = "success";
-    } else {
-      typeClass = "danger";
-    }
-
     return (
-      <tr className={typeClass}>
+      <tr className={this.props.amount > 0 ? 'success' : 'danger'}>
         <td>{this.props.number}</td>
         <td>
           <i className={classesCurrency}></i> {accounting.formatMoney(this.props.amount, '')}