From 5b480a85cb17b7455c6896e4d9b2f28abd080fc8 Mon Sep 17 00:00:00 2001 From: Dan Abramov Date: Wed, 10 Apr 2019 17:46:34 +0100 Subject: [PATCH] Prevent default on all key navigations --- src/devtools/views/Components/Tree.js | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/src/devtools/views/Components/Tree.js b/src/devtools/views/Components/Tree.js index d5a2b3182659a..533f314a7953d 100644 --- a/src/devtools/views/Components/Tree.js +++ b/src/devtools/views/Components/Tree.js @@ -84,10 +84,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) @@ -103,6 +104,7 @@ export default function Tree(props: Props) { } break; case 'ArrowRight': + event.preventDefault(); element = selectedElementID !== null ? store.getElementByID(selectedElementID) @@ -116,11 +118,10 @@ export default function Tree(props: Props) { } else { selectNextElementInTree(); } - event.preventDefault(); break; case 'ArrowUp': - selectPreviousElementInTree(); event.preventDefault(); + selectPreviousElementInTree(); break; } };