fix edge case bug

elastic · Oct 16, 2018 · 91e83d1 · 91e83d1
1 parent 779d360
commit 91e83d1
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/src/ui/public/query_bar/components/query_bar.tsx b/src/ui/public/query_bar/components/query_bar.tsx
@@ -147,9 +147,10 @@ export class QueryBar extends Component<Props, State> {
   public decrementIndex = (currentIndex: number) => {
     const previousIndex = currentIndex - 1;
     if (previousIndex < 0) {
-      this.setState({ index: 0 });
+      this.setState({ index: this.state.suggestions.length - 1 });
+    } else {
+      this.setState({ index: previousIndex });
     }
-    this.setState({ index: previousIndex });
   };
 
   public getSuggestions = async () => {
@@ -338,7 +339,7 @@ export class QueryBar extends Component<Props, State> {
           if (isSuggestionsVisible && this.state.suggestions[index]) {
             this.selectSuggestion(this.state.suggestions[index]);
           } else {
-            this.onSubmit(event.preventDefault);
+            this.onSubmit(() => event.preventDefault());
           }
           break;
         case KEY_CODES.ESC:

diff --git a/src/ui/public/query_bar/components/typeahead/suggestion_component.tsx b/src/ui/public/query_bar/components/typeahead/suggestion_component.tsx
@@ -71,6 +71,7 @@ export const SuggestionComponent: SFC<Props> = props => {
           // Description currently always comes from us and we escape any potential user input
           // at the time we're generating the description text
           // eslint-disable-next-line react/no-danger
+          // @ts-ignore
           dangerouslySetInnerHTML={{ __html: props.suggestion.description }}
         />
       </div>