fix flow and lint errors

codehag · codehag · commit f7dc5870fbd6 · 2017-10-13T17:05:45.000+02:00
diff --git a/src/actions/tests/ui.spec.js b/src/actions/tests/ui.spec.js
@@ -2,13 +2,10 @@ import { createStore, selectors, actions } from "../../utils/test-head";
 
 const {
   getActiveSearch,
-  getFileSearchQueryState,
-  getFileSearchModifierState,
   getFrameworkGroupingState,
   getPaneCollapse,
   getSymbolSearchType,
-  getHighlightedLineRange,
-  getSearchResults
+  getHighlightedLineRange
 } = selectors;
 
 describe("ui", () => {
diff --git a/src/components/Editor/SearchBar.js b/src/components/Editor/SearchBar.js
@@ -45,6 +45,7 @@ function getShortcuts() {
 }
 
 type SearchBarState = {
+  query: string,
   selectedResultIndex: number,
   count: number,
   index: number
@@ -97,8 +98,10 @@ class SearchBar extends Component {
   }
 
   componentDidMount() {
-    const shortcuts = this.context.shortcuts;
+    // overwrite this.doSearch with debounced version to
+    // reduce frequency of queries
     this.doSearch = debounce(this.doSearch, 100);
+    const shortcuts = this.context.shortcuts;
     const {
       searchShortcut,
       searchAgainShortcut,
@@ -161,23 +164,14 @@ class SearchBar extends Component {
 
     if (this.props.searchOn && editor) {
       const selection = editor.codeMirror.getSelection();
-      this.setSearchValue(selection);
+      this.setState({ query: selection });
       if (selection !== "") {
         this.doSearch(selection);
       }
       this.selectSearchInput();
     }
   };
 
-  setSearchValue = (value: string) => {
-    const searchInput = this.searchInput();
-    if (value == "" || !searchInput) {
-      return;
-    }
-
-    searchInput.value = value;
-  };
-
   selectSearchInput = () => {
     const searchInput = this.searchInput();
     if (searchInput) {
