From 24840a86738dccf66b25c1c2293b18d045b071a3 Mon Sep 17 00:00:00 2001 From: Weslen do Nascimento Date: Tue, 12 Nov 2019 20:01:24 -0300 Subject: [PATCH] [Autocomplete] Fix usage of Home/End keys (#18338) --- .../src/useAutocomplete/useAutocomplete.js | 16 ++++++++++------ 1 file changed, 10 insertions(+), 6 deletions(-) diff --git a/packages/material-ui-lab/src/useAutocomplete/useAutocomplete.js b/packages/material-ui-lab/src/useAutocomplete/useAutocomplete.js index feb69e2f2ddbfd..dda61b78c627c1 100644 --- a/packages/material-ui-lab/src/useAutocomplete/useAutocomplete.js +++ b/packages/material-ui-lab/src/useAutocomplete/useAutocomplete.js @@ -490,14 +490,18 @@ export default function useAutocomplete(props) { switch (event.key) { case 'Home': - // Prevent scroll of the page - event.preventDefault(); - changeHighlightedIndex('start', 'next'); + if (popupOpen) { + // Prevent scroll of the page + event.preventDefault(); + changeHighlightedIndex('start', 'next'); + } break; case 'End': - // Prevent scroll of the page - event.preventDefault(); - changeHighlightedIndex('end', 'previous'); + if (popupOpen) { + // Prevent scroll of the page + event.preventDefault(); + changeHighlightedIndex('end', 'previous'); + } break; case 'PageUp': // Prevent scroll of the page