From bf74e3aa23e8cc308d86ca0187a7cd924f98c3bf Mon Sep 17 00:00:00 2001 From: eikek Date: Wed, 9 Jun 2021 22:18:18 +0200 Subject: [PATCH] Don't toggle dropdown on space Closing the dropdown menu is now possible with ESC. Space will only open the dropdown, but not close it. So now it's possible to type a space into the search field. Fixes: #863 --- modules/webapp/src/main/elm/Comp/Dropdown.elm | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/modules/webapp/src/main/elm/Comp/Dropdown.elm b/modules/webapp/src/main/elm/Comp/Dropdown.elm index cd82f6b9..966236ea 100644 --- a/modules/webapp/src/main/elm/Comp/Dropdown.elm +++ b/modules/webapp/src/main/elm/Comp/Dropdown.elm @@ -359,7 +359,7 @@ update msg model = Just Util.Html.ESC -> if model.menuOpen then - ( model, Cmd.none ) + update ToggleMenu model else case model.selected of @@ -374,7 +374,11 @@ update msg model = ( model, Cmd.none ) Just Util.Html.Space -> - update ToggleMenu model + if model.menuOpen then + ( model, Cmd.none ) + + else + update ToggleMenu model Just Util.Html.Enter -> let