mirrors/serenity
1
0
Fork 0
mirror of https://github.com/SerenityOS/serenity.git synced 2025-01-24 02:12:09 -05:00
Code Issues Projects Releases Packages Wiki Activity

Browser: Scope bookmark actions so that they don't block the delete key

Browse source 
This fixes #10940.

Previously, all presses of the Delete key without a modifier in Browser
were uselessly consumed by the "Delete" action in the bookmark context
menu.
This commit is contained in:
Sam Atkins 2021-11-22 21:48:48 +00:00 • committed by Andreas Kling
parent fb5ea617f1
commit 97eb662e62
1 changed files with 22 additions and 14 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

24
Userland/Applications/Browser/BookmarksBarWidget.cpp
View file

@ -124,23 +124,31 @@ BookmarksBarWidget::BookmarksBarWidget(const String& bookmarks_file, bool enable
m_separator = GUI::Widget::construct();
m_context_menu = GUI::Menu::construct();
auto default_action = GUI::Action::create("&Open", [this](auto&) {
auto default_action = GUI::Action::create(
"&Open", [this](auto&) {
if (on_bookmark_click)
on_bookmark_click(m_context_menu_url, Mod_None);
});
},
this);
m_context_menu_default_action = default_action;
m_context_menu->add_action(default_action);
m_context_menu->add_action(GUI::Action::create("Open in New &Tab", [this](auto&) {
m_context_menu->add_action(GUI::Action::create(
"Open in New &Tab", [this](auto&) {
if (on_bookmark_click)
on_bookmark_click(m_context_menu_url, Mod_Ctrl);
}));
},
this));
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::CommonActions::make_delete_action([this](auto&) {
},
this));
m_context_menu->add_action(GUI::CommonActions::make_delete_action(
[this](auto&) {
remove_bookmark(m_context_menu_url);
}));
},
this));
Vector<GUI::JsonArrayModel::FieldSpec> fields;
fields.empend("title", "Title", Gfx::TextAlignment::CenterLeft);