Make click handlers in EmailView more semantic

Author: benmvp

05-email-view/src/components/EmailView.js

@@ -7,7 +7,7 @@ export default class EmailView extends React.Component {
         onClose: React.PropTypes.func.isRequired
     }
 
-    _handleCloseClick(e) {
+    _handleClose(e) {
         e.stopPropagation();
         this.props.onClose();
     }
@@ -24,7 +24,7 @@ export default class EmailView extends React.Component {
                 <h2>From: <a href={`mailto:${from}`}>{from}</a></h2>
                 <h3>{date}</h3>
                 <div dangerouslySetInnerHTML={rawMessage} />
-                <button onClick={this._handleCloseClick.bind(this)}>Close</button>
+                <button onClick={this._handleClose.bind(this)}>Close</button>
             </div>
         );
     }

06-email-form/src/components/EmailView.js

@@ -7,7 +7,7 @@ export default class EmailView extends React.Component {
         onClose: React.PropTypes.func.isRequired
     }
 
-    _handleCloseClick(e) {
+    _handleClose(e) {
         e.stopPropagation();
         this.props.onClose();
     }
@@ -24,7 +24,7 @@ export default class EmailView extends React.Component {
                 <h2>From: <a href={`mailto:${from}`}>{from}</a></h2>
                 <h3>{date}</h3>
                 <div dangerouslySetInnerHTML={rawMessage} />
-                <button onClick={this._handleCloseClick.bind(this)}>Close</button>
+                <button onClick={this._handleClose.bind(this)}>Close</button>
             </div>
         );
     }

07-submit-email-form/src/components/EmailView.js

@@ -7,7 +7,7 @@ export default class EmailView extends React.Component {
         onClose: React.PropTypes.func.isRequired
     }
 
-    _handleCloseClick(e) {
+    _handleClose(e) {
         e.stopPropagation();
         this.props.onClose();
     }
@@ -24,7 +24,7 @@ export default class EmailView extends React.Component {
                 <h2>From: <a href={`mailto:${from}`}>{from}</a></h2>
                 <h3>{date}</h3>
                 <div dangerouslySetInnerHTML={rawMessage} />
-                <button onClick={this._handleCloseClick.bind(this)}>Close</button>
+                <button onClick={this._handleClose.bind(this)}>Close</button>
             </div>
         );
     }

08-optimistic-updating/src/components/EmailView.js

@@ -7,7 +7,7 @@ export default class EmailView extends React.Component {
         onClose: React.PropTypes.func.isRequired
     }
 
-    _handleCloseClick(e) {
+    _handleClose(e) {
         e.stopPropagation();
         this.props.onClose();
     }
@@ -24,7 +24,7 @@ export default class EmailView extends React.Component {
                 <h2>From: <a href={`mailto:${from}`}>{from}</a></h2>
                 <h3>{date}</h3>
                 <div dangerouslySetInnerHTML={rawMessage} />
-                <button onClick={this._handleCloseClick.bind(this)}>Close</button>
+                <button onClick={this._handleClose.bind(this)}>Close</button>
             </div>
         );
     }

09-delete-email/src/components/EmailView.js

@@ -8,12 +8,12 @@ export default class EmailView extends React.Component {
         onDelete: React.PropTypes.func.isRequired
     }
 
-    _handleCloseClick(e) {
+    _handleClose(e) {
         e.stopPropagation();
         this.props.onClose();
     }
 
-    _handleDeleteClick(e) {
+    _handleDelete(e) {
         e.stopPropagation();
         this.props.onDelete();
     }
@@ -30,8 +30,8 @@ export default class EmailView extends React.Component {
                 <h2>From: <a href={`mailto:${from}`}>{from}</a></h2>
                 <h3>{date}</h3>
                 <div dangerouslySetInnerHTML={rawMessage} />
-                <button onClick={this._handleDeleteClick.bind(this)}>Delete</button>
-                <button onClick={this._handleCloseClick.bind(this)}>Close</button>
+                <button onClick={this._handleDelete.bind(this)}>Delete</button>
+                <button onClick={this._handleClose.bind(this)}>Close</button>
             </div>
         );
     }

10-mark-unread/src/components/EmailView.js

@@ -11,25 +11,25 @@ export default class EmailView extends React.Component {
         onMarkRead: React.PropTypes.func
     }
 
-    _handleCloseClick(e) {
+    _handleClose(e) {
         e.stopPropagation();
         this.props.onClose();
     }
 
-    _handleDeleteClick(e) {
+    _handleDelete(e) {
         e.stopPropagation();
         this.props.onDelete();
     }
 
-    _handleMarkUnreadClick(e) {
+    _handleMarkUnread(e) {
         e.stopPropagation();
 
         if (this.props.onMarkUnread) {
             this.props.onMarkUnread();
         }
     }
 
-    _handleMarkReadClick(e) {
+    _handleMarkRead(e) {
         e.stopPropagation();
 
         if (this.props.onMarkRead) {
@@ -43,8 +43,8 @@ export default class EmailView extends React.Component {
         } = this.props;
         let rawMessage = {__html: message};
         let markUnreadReadButton = unread
-            ? (<button onClick={this._handleMarkReadClick.bind(this)}>Mark Read</button>)
-            : (<button onClick={this._handleMarkUnreadClick.bind(this)}>Mark Unread</button>);
+            ? (<button onClick={this._handleMarkRead.bind(this)}>Mark Read</button>)
+            : (<button onClick={this._handleMarkUnread.bind(this)}>Mark Unread</button>);
 
         return (
             <div>
@@ -53,8 +53,8 @@ export default class EmailView extends React.Component {
                 <h3>{date}</h3>
                 <div dangerouslySetInnerHTML={rawMessage} />
                 {markUnreadReadButton}
-                <button onClick={this._handleDeleteClick.bind(this)}>Delete</button>
-                <button onClick={this._handleCloseClick.bind(this)}>Close</button>
+                <button onClick={this._handleDelete.bind(this)}>Delete</button>
+                <button onClick={this._handleClose.bind(this)}>Close</button>
             </div>
         );
     }