From 374284d0d8ac81aa1949c4fc5a52854a33f4898b Mon Sep 17 00:00:00 2001 From: Timothy Flynn Date: Tue, 16 May 2023 07:53:53 -0400 Subject: [PATCH] Browser: Move screenshot context menu actions above inspector actions Feels a bit more natural, at least to align with muscle memory from other browsers. --- Userland/Applications/Browser/Tab.cpp | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/Userland/Applications/Browser/Tab.cpp b/Userland/Applications/Browser/Tab.cpp index 43e5e2829c..f83e24ecb6 100644 --- a/Userland/Applications/Browser/Tab.cpp +++ b/Userland/Applications/Browser/Tab.cpp @@ -562,12 +562,12 @@ Tab::Tab(BrowserWindow& window) m_page_context_menu->add_action(window.copy_selection_action()); m_page_context_menu->add_action(window.select_all_action()); m_page_context_menu->add_separator(); + m_page_context_menu->add_action(window.take_visible_screenshot_action()); + m_page_context_menu->add_action(window.take_full_screenshot_action()); + m_page_context_menu->add_separator(); m_page_context_menu->add_action(window.view_source_action()); m_page_context_menu->add_action(window.inspect_dom_tree_action()); m_page_context_menu->add_action(window.inspect_dom_node_action()); - m_page_context_menu->add_separator(); - m_page_context_menu->add_action(window.take_visible_screenshot_action()); - m_page_context_menu->add_action(window.take_full_screenshot_action()); view().on_context_menu_request = [&](auto screen_position) { m_page_context_menu->popup(screen_position); };