Merge pull request HubSpot#59 from HubSpot/is-open-prop-change-should-trigger-close-menu

Trigger closeMenu when isOpen changes from true to false

Author: TrevorBurnham

dist/react-select-plus.js

@@ -1161,6 +1161,10 @@ var Select = _react2['default'].createClass({
 
 var valueArray = this.getValueArray(nextProps.value, nextProps);
 
+if (!nextProps.isOpen && this.props.isOpen) {
+this.closeMenu();
+}
+
 if (nextProps.required) {
 this.setState({
 required: this.handleRequired(valueArray[0], nextProps.multi)

dist/react-select-plus.min.js

@@ -1461,6 +1461,10 @@ var Select = _react2['default'].createClass({
 
 var valueArray = this.getValueArray(nextProps.value, nextProps);
 
+if (!nextProps.isOpen && this.props.isOpen) {
+this.closeMenu();
+}
+
 if (nextProps.required) {
 this.setState({
 required: this.handleRequired(valueArray[0], nextProps.multi)

examples/dist/bundle.js

@@ -1161,6 +1161,10 @@ var Select = _react2['default'].createClass({
 
 var valueArray = this.getValueArray(nextProps.value, nextProps);
 
+if (!nextProps.isOpen && this.props.isOpen) {
+this.closeMenu();
+}
+
 if (nextProps.required) {
 this.setState({
 required: this.handleRequired(valueArray[0], nextProps.multi)

examples/dist/standalone.js

@@ -275,6 +275,10 @@ var Select = _react2['default'].createClass({
 
 var valueArray = this.getValueArray(nextProps.value, nextProps);
 
+if (!nextProps.isOpen && this.props.isOpen) {
+this.closeMenu();
+}
+
 if (nextProps.required) {
 this.setState({
 required: this.handleRequired(valueArray[0], nextProps.multi)

lib/Select.js

@@ -222,6 +222,10 @@ const Select = React.createClass({
 
 const valueArray = this.getValueArray(nextProps.value, nextProps);
 
+ if (!nextProps.isOpen && this.props.isOpen) {
+ this.closeMenu();
+ }
+
 if (nextProps.required) {
 this.setState({
 required: this.handleRequired(valueArray[0], nextProps.multi),