From 1ef1621bfc011ca7f702afdfef4b86984c9f5897 Mon Sep 17 00:00:00 2001 From: Andrew Leonard <16251412+ajyey@users.noreply.github.com> Date: Fri, 31 May 2024 12:39:19 -0400 Subject: [PATCH] Bugfix: Collapse the navbar on mobile when the user selects search (#2462) This PR addresses #2438 by programmatically collapsing the navbar if the user clicks on search on mobile. ## Behavior before ![ToggleBehaviorBefore](https://github.com/alshedivat/al-folio/assets/16251412/562765b2-57eb-4a3d-bf41-eee4fcfddd5f) ## Behavior after ![ToggleBehaviorAfter](https://github.com/alshedivat/al-folio/assets/16251412/78bb917b-d7b3-4e58-bd76-f422b8ab7fd5) --- _includes/scripts/search.liquid | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/_includes/scripts/search.liquid b/_includes/scripts/search.liquid index 63d2dd4..9c3dd9d 100644 --- a/_includes/scripts/search.liquid +++ b/_includes/scripts/search.liquid @@ -13,9 +13,9 @@ const openSearchModal = () => { // collapse navbarNav if expanded on mobile - const navbarNav = document.querySelector('.navbar-collapse'); - if (navbarNav.classList.contains('show')) { - navbarNav.classList.remove('show'); + const $navbarNav = $('#navbarNav'); + if ($navbarNav.hasClass('show')) { + $navbarNav.collapse('hide'); } ninjaKeys.open(); };