From 727cdcf82ce65720f4c4ed184cfb241a5a3ce121 Mon Sep 17 00:00:00 2001 From: Simon Woertz Date: Sat, 6 Nov 2021 22:31:10 +0100 Subject: [PATCH] Revert "WindowServer: Remove redundant Window::set_visible()..." This reverts commit 239520ae549b21f1700e966d45eb0daaac939ae1. The call to set_visible() is not redundant. Removing the call leads to the "start" button in the taskbar not being painted as "pressed" even when it is. --- Userland/Services/WindowServer/Menu.cpp | 1 + 1 file changed, 1 insertion(+) diff --git a/Userland/Services/WindowServer/Menu.cpp b/Userland/Services/WindowServer/Menu.cpp index f69b8629b5..e6c404d8ff 100644 --- a/Userland/Services/WindowServer/Menu.cpp +++ b/Userland/Services/WindowServer/Menu.cpp @@ -638,6 +638,7 @@ void Menu::do_popup(const Gfx::IntPoint& position, bool make_input, bool as_subm } window.move_to(adjusted_pos); + set_visible(true); MenuManager::the().open_menu(*this, make_input); WindowManager::the().did_popup_a_menu({}); }