fix: fix keyboard option navigation skipping all items (#1165)

Fixes #1161. Navigating with up or down arrows would always go to first and last option, skipping everything in between. Caused by missing break statements, probably deleted by accident. #1161 Co-authored-by: Dino Pejaković <[email protected]>
sagalbot · Apr 17, 2020 · f47180c · f47180c
1 parent 538df2c
commit f47180c
Showing 1 changed file with 2 additions and 0 deletions.
diff --git a/src/mixins/typeAheadPointer.js b/src/mixins/typeAheadPointer.js
@@ -26,6 +26,7 @@ export default {
       for (let i = this.typeAheadPointer - 1; i >= 0; i--) {
         if (this.selectable(this.filteredOptions[i])) {
           this.typeAheadPointer = i;
+          break;
         }
       }
     },
@@ -39,6 +40,7 @@ export default {
       for (let i = this.typeAheadPointer + 1; i < this.filteredOptions.length; i++) {
         if (this.selectable(this.filteredOptions[i])) {
           this.typeAheadPointer = i;
+          break;
         }
       }
     },