Merge pull request #122 from gaearon/prevent

Prevent default on all key navigations
facebook · Apr 10, 2019 · 88e51ad · 88e51ad
2 parents 4b0424f + 5b480a8
commit 88e51ad
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/src/devtools/views/Components/Tree.js b/src/devtools/views/Components/Tree.js
@@ -90,10 +90,11 @@ export default function Tree(props: Props) {
       // eslint-disable-next-line default-case
       switch (event.key) {
         case 'ArrowDown':
-          selectNextElementInTree();
           event.preventDefault();
+          selectNextElementInTree();
           break;
         case 'ArrowLeft':
+          event.preventDefault();
           element =
             selectedElementID !== null
               ? store.getElementByID(selectedElementID)
@@ -109,6 +110,7 @@ export default function Tree(props: Props) {
           }
           break;
         case 'ArrowRight':
+          event.preventDefault();
           element =
             selectedElementID !== null
               ? store.getElementByID(selectedElementID)
@@ -122,11 +124,10 @@ export default function Tree(props: Props) {
           } else {
             selectNextElementInTree();
           }
-          event.preventDefault();
           break;
         case 'ArrowUp':
-          selectPreviousElementInTree();
           event.preventDefault();
+          selectPreviousElementInTree();
           break;
       }
     };