Merge pull request tajo#47 from MrOrz/master

Avoid calling .bind() in render() and simplify openPortal()

lib/portal.js

@@ -12,7 +12,7 @@ export default class Portal extends React.Component {
   constructor() {
     super();
     this.state = {active: false};
-    this.openPortal = this.openPortal.bind(this);
+    this.handleWrapperClick = this.handleWrapperClick.bind(this);
     this.closePortal = this.closePortal.bind(this);
     this.handleOutsideMouseClick = this.handleOutsideMouseClick.bind(this);
     this.handleKeydown = this.handleKeydown.bind(this);
@@ -31,7 +31,7 @@ export default class Portal extends React.Component {
     }
 
     if (this.props.isOpened) {
-      this.openPortal(this.props);
+      this.openPortal();
     }
   }
 
@@ -91,17 +91,19 @@ export default class Portal extends React.Component {
 
   render() {
     if (this.props.openByClickOn) {
-      return <div className="openByClickOn" onClick={this.openPortal.bind(this, this.props)}>{this.props.openByClickOn}</div>;
+      return <div className="openByClickOn" onClick={this.handleWrapperClick}>{this.props.openByClickOn}</div>;
     } else {
       return null;
     }
   }
 
-  openPortal(props, e) {
-    if (e) {
-      e.preventDefault();
-      e.stopPropagation();
-    }
+  handleWrapperClick(e) {
+    e.preventDefault();
+    e.stopPropagation();
+    this.openPortal();
+  }
+
+  openPortal(props = this.props) {
     this.setState({active: true});
     this.renderPortal(props);