Merge pull request #3 from ricosmall/patch-1

Fix long list problem
kaiostech · Oct 7, 2019 · f1b3d9b · f1b3d9b
2 parents f642ee1 + f960909
commit f1b3d9b
Showing 1 changed file with 8 additions and 3 deletions.
diff --git a/src/hooks/useNavigation.js b/src/hooks/useNavigation.js
@@ -47,8 +47,13 @@ export const useNavigation = () => {
         const selectThisElement = element === selectElement;
         element.setAttribute("nav-selected", selectThisElement);
         element.setAttribute("nav-index", index);
-        if (element.nodeName === 'INPUT') {
-          selectThisElement ? element.focus() : element.blur();
+        if (selectThisElement) {
+          selectThisElement.scrollIntoView(true);
+          if (element.nodeName === 'INPUT') {
+            element.focus();
+          } else {
+            element.blur();
+          }
         }
       });
       setCurrent({ type: selectElement.tagName, index: setIndex });
@@ -58,4 +63,4 @@ export const useNavigation = () => {
   }
 
   return [current, setNavigation];
-};
+};