From 63b8d041008aefdedf01898de7049a204c32ec99 Mon Sep 17 00:00:00 2001 From: thankyouverycool <66646555+thankyouverycool@users.noreply.github.com> Date: Tue, 16 Aug 2022 07:04:03 -0400 Subject: [PATCH] WindowServer: Walk the open menu stack in reverse on MouseMove events Fixes lower level menus stealing focus from higher submenus. --- Userland/Services/WindowServer/MenuManager.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Userland/Services/WindowServer/MenuManager.cpp b/Userland/Services/WindowServer/MenuManager.cpp index ad0e8543aa..07d8cab2fe 100644 --- a/Userland/Services/WindowServer/MenuManager.cpp +++ b/Userland/Services/WindowServer/MenuManager.cpp @@ -197,7 +197,7 @@ void MenuManager::handle_mouse_event(MouseEvent& mouse_event) } if (mouse_event.type() == Event::MouseMove) { - for (auto& menu : m_open_menu_stack) { + for (auto& menu : m_open_menu_stack.in_reverse()) { if (!menu) continue; if (!menu->menu_window()->rect().contains(mouse_event.position()))