fix: remove isMounted antipattern

Closes #71
ubilabs · Mar 11, 2016 · b347e9a · b347e9a
1 parent 611fb41
commit b347e9a
Showing 1 changed file with 7 additions and 9 deletions.
diff --git a/src/Geosuggest.jsx b/src/Geosuggest.jsx
@@ -25,11 +25,11 @@ class Geosuggest extends React.Component {
   constructor(props) {
     super(props);
     this.state = {
-      isMounted: false,
       isSuggestsHidden: true,
       userInput: this.props.initialValue,
       activeSuggest: null,
-      suggests: []
+      suggests: [],
+      timer: null
     };
   }
 
@@ -65,15 +65,13 @@ class Geosuggest extends React.Component {
 
     this.autocompleteService = new googleMaps.places.AutocompleteService();
     this.geocoder = new googleMaps.Geocoder();
-
-    this.setState({isMounted: true});
   }
 
   /**
    * When the component will unmount
    */
   componentWillUnmount() {
-    this.setState({isMounted: false});
+    clearTimeout(this.state.timer);
   }
 
   /**
@@ -205,11 +203,11 @@ class Geosuggest extends React.Component {
    */
   hideSuggests() {
     this.props.onBlur();
-    setTimeout(() => {
-      if (this.state && this.state.isMounted) {
-        this.setState({isSuggestsHidden: true});
-      }
+    const timer = setTimeout(() => {
+      this.setState({isSuggestsHidden: true});
     }, 100);
+
+    this.setState({timer});
   }
 
   /**