Discource-C
/
discourse
mirror of https://github.com/discourse/discourse.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

REVERT: "FIX: Properly close search menu on click/touch outside (#25000)" (#25005) 

This reverts commit c91d053dec.
This commit is contained in:
Mark VanLandingham 2023-12-21 15:54:17 -06:00 committed by GitHub
parent 025e40354c
commit fe8b1a1074
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 3 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
app/assets/javascripts/discourse/app/components/search-menu.js
View File

@ -67,17 +67,8 @@ export default class SearchMenu extends Component {
@bind @bind
onDocumentPress(event) { onDocumentPress(event) {
// If the search menu header button is clicked, we don't need to do anything and can if (!event.target.closest(".search-menu-container.menu-panel-results")) {
// let the header handle cleanup. Otherwise we mess with the toggling of the popup. this.menuPanelOpen = false;
if (event.target.closest(".header-dropdown-toggle.search-dropdown")) {
return;
}
if (
this.menuPanelOpen &&
!event.target.closest(".search-menu .search-menu-container")
) {
this.close();
} }
} }
@ -115,7 +106,7 @@ export default class SearchMenu extends Component {
// We want to blur the active element (search input) when in stand-alone mode // We want to blur the active element (search input) when in stand-alone mode
// so that when we focus on the search input again, the menu panel pops up // so that when we focus on the search input again, the menu panel pops up
document.getElementById(SEARCH_INPUT_ID)?.blur(); document.activeElement.blur();
this.menuPanelOpen = false; this.menuPanelOpen = false;
} }