Merge branch 'blank-login'

src/components/Actions.js

@@ -5,7 +5,7 @@ export default class Actions extends React.Component {
   render() {
     return (
       <div className="Actions">
-        <CreateLogin createLogin={this.props.createLogin}/>
+        <CreateLogin handleCreateLogin={this.props.handleCreateLogin}/>
       </div>
     )
   }

src/components/CreateLogin.js

@@ -37,7 +37,16 @@ export default class CreateLogin extends React.Component {
     e.preventDefault()
     const username = e.target.elements.username.value.trim()
     const password = e.target.elements.password.value.trim()
-    const success = this.props.createLogin(username, password)
+
+    if (!username || !password) {
+      this.setState(() => ({
+        createLoginModalErrorMessage: 'Fields cannot be empty!'
+      }))
+      return false
+    }
+
+
+    const success = this.props.handleCreateLogin(username, password)
 
     if (success) {
       this.setState(() => ({

src/components/LoginManager.js

@@ -50,13 +50,13 @@ export default class LoginManager extends React.Component {
           handleDelete={this.handleDeleteLogin}
         />
         <Actions
-          createLogin={this.createLogin}
+          handleCreateLogin={this.handleCreateLogin}
         />
       </div>
     )
   }
 
-  createLogin = (username, password) => {
+  handleCreateLogin = (username, password) => {
     const matchedLogin = this._findLogin({ username, password })
 
     if (matchedLogin) {

src/components/Logins.js

@@ -60,6 +60,13 @@ export default class Logins extends React.Component {
     const newUsername = e.target.elements.newUsername.value.trim()
     const newPassword = e.target.elements.newPassword.value.trim()
 
+    if (!newUsername || !newPassword) {
+      this.setState(() => ({
+        editLoginModalErrorMessage: 'Fields cannot be empty!'
+      }))
+      return false;
+    }
+
     if ((newUsername === this.state.editingLogin.username)
       && newPassword === this.state.editingLogin.password) {
       // didn't change at all.