From 5fe3cc3b855cb641741ab93eab626b71e231bf37 Mon Sep 17 00:00:00 2001 From: LuK1337 Date: Wed, 28 Jul 2021 15:37:45 +0200 Subject: [PATCH] WindowServer: Don't draw separator between pin window & close for modals Modal windows cannot be pinned and thus we end up drawing 2 separators. --- Userland/Services/WindowServer/Window.cpp | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/Userland/Services/WindowServer/Window.cpp b/Userland/Services/WindowServer/Window.cpp index b4db2feb0c..4e0d7f52f6 100644 --- a/Userland/Services/WindowServer/Window.cpp +++ b/Userland/Services/WindowServer/Window.cpp @@ -803,10 +803,9 @@ void Window::ensure_window_menu() m_window_menu_pin_item->set_icon(&pin_icon()); m_window_menu_pin_item->set_checkable(true); m_window_menu->add_item(move(pin_item)); + m_window_menu->add_item(make(*m_window_menu, MenuItem::Type::Separator)); } - m_window_menu->add_item(make(*m_window_menu, MenuItem::Type::Separator)); - auto close_item = make(*m_window_menu, (unsigned)WindowMenuAction::Close, "&Close"); m_window_menu_close_item = close_item.ptr(); m_window_menu_close_item->set_icon(&close_icon());