EuiComboBox - only call copyInputStyles when still mounted (#813)

* only call copyInputStyles when still mounted * update reademe * test if autoSizeInput exists
elastic · May 10, 2018 · 96ddf15 · 96ddf15
1 parent 895f2ca
commit 96ddf15
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -9,6 +9,7 @@
 - Added aria labeling requirements for `EuiBadge` , as well as a generic prop_type function `requiresAriaLabel` in `utils` to check for it. ([#777](https://github.com/elastic/eui/pull/777)) ([#802](https://github.com/elastic/eui/pull/802))
 - Ensure switches’ inputs are still hidden when `[disabled]` ([#778](https://github.com/elastic/eui/pull/778))
 - Made boolean matching in `EuiSearchBar` more exact so it doesn't match words starting with booleans, like "truest" or "offer" ([#776](https://github.com/elastic/eui/pull/776))
+- `EuiComboBox` do not setState or call refs once component is unmounted ([807](https://github.com/elastic/eui/pull/807) and [#813](https://github.com/elastic/eui/pull/813))
 
 ## [`0.0.46`](https://github.com/elastic/eui/tree/v0.0.46)
 

diff --git a/src/components/combo_box/combo_box.js b/src/components/combo_box/combo_box.js
@@ -454,7 +454,9 @@ export class EuiComboBox extends Component {
 
     // TODO: This will need to be called once the actual stylesheet loads.
     setTimeout(() => {
-      this.autoSizeInput.copyInputStyles();
+      if (this.autoSizeInput) {
+        this.autoSizeInput.copyInputStyles();
+      }
     }, 100);
   }