Revert "WindowServer: Remove redundant Window::set_visible()..."

This reverts commit 239520ae54. 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.
Author: https://github.com/swoertz 🔰 Commit: https://github.com/SerenityOS/serenity/commit/727cdcf82ce Pull-request: https://github.com/SerenityOS/serenity/pull/10829 Issue: https://github.com/SerenityOS/serenity/issues/10828
2021-11-06 22:31:10 +01:00 · 2021-11-06 22:31:10 +01:00 · 727cdcf82c · 2024-07-18 01:27:15 +09:00
commit 727cdcf82c
parent 1d777073fd
1 changed files with 1 additions and 0 deletions
--- 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({});
 }