Merge pull request #24 from jimutt/enable_keyboard_nav

Enable usage of tab to focus next element after selection
tristen · Mar 7, 2019 · 5cbd4f4 · 5cbd4f4
2 parents bc1c7c5 + a693344
commit 5cbd4f4
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 3 deletions.
diff --git a/dist/suggestions.js b/dist/suggestions.js
diff --git a/src/list.js b/src/list.js
@@ -37,6 +37,10 @@ List.prototype.isEmpty = function() {
   return !this.items.length;
 };
 
+List.prototype.isVisible = function() {
+  return this.element.style.display === 'block';
+};
+
 List.prototype.draw = function() {
   this.element.innerHTML = '';
 

diff --git a/src/suggestions.js b/src/suggestions.js
@@ -64,9 +64,11 @@ Suggestions.prototype.handleKeyUp = function(keyCode) {
 Suggestions.prototype.handleKeyDown = function(e) {
   switch (e.keyCode) {
     case 13: // ENTER
-    case 9:  // TAB
-      e.preventDefault();
+    case 9: // TAB
       if (!this.list.isEmpty()) {
+        if (this.list.isVisible()) {
+          e.preventDefault();
+        }
         this.value(this.list.items[this.list.active].original);
         this.list.hide();
       }