Browser: Use CommonActions where possible and various fixes

This replaces some actions with CommonActions and also adds '...' to
menu items that require user input.

3 files changed, 9 insertions, 11 deletions

diff --git a/Userland/Applications/Browser/BookmarksBarWidget.cpp b/Userland/Applications/Browser/BookmarksBarWidget.cpp
index da12d7e753..60552fff6e 100644
--- a/Userland/Applications/Browser/BookmarksBarWidget.cpp
+++ b/Userland/Applications/Browser/BookmarksBarWidget.cpp
@@ -135,10 +135,10 @@ BookmarksBarWidget::BookmarksBarWidget(const String& bookmarks_file, bool enable
             on_bookmark_click(m_context_menu_url, Mod_Ctrl);
     }));
     m_context_menu->add_separator();
-    m_context_menu->add_action(GUI::Action::create("&Edit", [this](auto&) {
+    m_context_menu->add_action(GUI::Action::create("&Edit...", [this](auto&) {
         edit_bookmark(m_context_menu_url);
     }));
-    m_context_menu->add_action(GUI::Action::create("&Delete", [this](auto&) {
+    m_context_menu->add_action(GUI::CommonActions::make_delete_action([this](auto&) {
         remove_bookmark(m_context_menu_url);
     }));
 
diff --git a/Userland/Applications/Browser/BrowserWindow.cpp b/Userland/Applications/Browser/BrowserWindow.cpp
index 08c184ff63..6e3d463b70 100644
--- a/Userland/Applications/Browser/BrowserWindow.cpp
+++ b/Userland/Applications/Browser/BrowserWindow.cpp
@@ -125,12 +125,10 @@ void BrowserWindow::build_menus()
     auto& file_menu = add_menu("&File");
     file_menu.add_action(WindowActions::the().create_new_tab_action());
 
-    auto close_tab_action = GUI::Action::create(
-        "&Close Tab", { Mod_Ctrl, Key_W }, Gfx::Bitmap::try_load_from_file("/res/icons/16x16/close-tab.png"), [this](auto&) {
-            active_tab().on_tab_close_request(active_tab());
-        },
+    auto close_tab_action = GUI::CommonActions::make_close_tab_action([this](auto&) {
+        active_tab().on_tab_close_request(active_tab());
+    },
         this);
-    close_tab_action->set_status_tip("Close current tab");
     file_menu.add_action(close_tab_action);
 
     file_menu.add_separator();
@@ -220,7 +218,7 @@ void BrowserWindow::build_menus()
     auto& settings_menu = add_menu("&Settings");
 
     m_change_homepage_action = GUI::Action::create(
-        "Set Homepage URL", [this](auto&) {
+        "Set Homepage URL...", [this](auto&) {
             auto homepage_url = Config::read_string("Browser", "Preferences", "Home", "about:blank");
             if (GUI::InputBox::show(this, homepage_url, "Enter URL", "Change homepage URL") == GUI::InputBox::ExecOK) {
                 if (URL(homepage_url).is_valid()) {
@@ -237,7 +235,7 @@ void BrowserWindow::build_menus()
 
     m_search_engine_actions.set_exclusive(true);
     auto& search_engine_menu = settings_menu.add_submenu("&Search Engine");
-
+    search_engine_menu.set_icon(Gfx::Bitmap::try_load_from_file("/res/icons/16x16/find.png"));
     bool search_engine_set = false;
     auto add_search_engine = [&](auto& name, auto& url_format) {
         auto action = GUI::Action::create_checkable(
diff --git a/Userland/Applications/Browser/Tab.cpp b/Userland/Applications/Browser/Tab.cpp
index 02d7733f9b..46db749e8b 100644
--- a/Userland/Applications/Browser/Tab.cpp
+++ b/Userland/Applications/Browser/Tab.cpp
@@ -324,10 +324,10 @@ Tab::Tab(BrowserWindow& window)
     };
 
     m_tab_context_menu = GUI::Menu::construct();
-    m_tab_context_menu->add_action(GUI::Action::create("&Reload Tab", [this](auto&) {
+    m_tab_context_menu->add_action(GUI::CommonActions::make_reload_action([this](auto&) {
         this->window().reload_action().activate();
     }));
-    m_tab_context_menu->add_action(GUI::Action::create("&Close Tab", [this](auto&) {
+    m_tab_context_menu->add_action(GUI::CommonActions::make_close_tab_action([this](auto&) {
         on_tab_close_request(*this);
     }));
     m_tab_context_menu->add_action(GUI::Action::create("&Duplicate Tab", [this](auto&) {