Fixed #15240 - Menubar | The bug where the menuitems don't close when…

… clicking outside the menuitem in mobile mode
primefaces · May 21, 2024 · f33a998 · f33a998
1 parent c42b73b
commit f33a998
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/src/app/components/menubar/menubar.ts b/src/app/components/menubar/menubar.ts
@@ -734,6 +734,7 @@ export class Menubar implements AfterContentInit, OnDestroy, OnInit {
 
     hide(event?, isFocus?: boolean) {
         if (this.mobileActive) {
+            this.mobileActive = false;
             setTimeout(() => {
                 DomHandler.focus(this.menubutton.nativeElement);
             }, 0);
@@ -1098,8 +1099,8 @@ export class Menubar implements AfterContentInit, OnDestroy, OnInit {
                     const isOutsideContainer = this.rootmenu.el.nativeElement !== event.target && !this.rootmenu.el.nativeElement.contains(event.target);
                     const isOutsideMenuButton = this.mobileActive && this.menubutton.nativeElement !== event.target && !this.menubutton.nativeElement.contains(event.target);
 
-                    if (isOutsideContainer) {
-                        isOutsideMenuButton ? (this.mobileActive = false) : this.hide();
+                    if (isOutsideContainer && isOutsideMenuButton) {
+                        this.hide();
                     }
                 });
             }