From 58d7189e9c69dae0682975797f68cf4509f43ee1 Mon Sep 17 00:00:00 2001
From: Robin Gareus <robin@gareus.org>
Date: Wed, 9 Apr 2025 20:11:32 +0200
Subject: [PATCH] Fix "Delete" processor-box context menu

For some reason the action prefix before the slash is
irrelevant. The "RegionFxMenu/delete" action overrides
the earlier registered "ProcessorMenu/delete".

Processor Box context menu called `RegionFxBox::static_delete`.
---
 gtk2_ardour/region_editor.cc      | 4 ++--
 gtk2_ardour/regionfx_box.bindings | 4 ++--
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/gtk2_ardour/region_editor.cc b/gtk2_ardour/region_editor.cc
index 4a16207d00..cfaab16338 100644
--- a/gtk2_ardour/region_editor.cc
+++ b/gtk2_ardour/region_editor.cc
@@ -546,8 +546,8 @@ RegionEditor::RegionFxBox::register_actions ()
 
 	rfx_box_actions = ActionManager::create_action_group (bindings, X_("RegionFxMenu"));
 
-	ActionManager::register_action (rfx_box_actions, X_("delete"), _("Delete"), sigc::ptr_fun (RegionEditor::RegionFxBox::static_delete));
-	ActionManager::register_action (rfx_box_actions, X_("backspace"), _("Delete"), sigc::ptr_fun (RegionEditor::RegionFxBox::static_delete));
+	ActionManager::register_action (rfx_box_actions, X_("rfx-delete"), _("Delete"), sigc::ptr_fun (RegionEditor::RegionFxBox::static_delete));
+	ActionManager::register_action (rfx_box_actions, X_("rfx-backspace"), _("Delete"), sigc::ptr_fun (RegionEditor::RegionFxBox::static_delete));
 }
 
 void
diff --git a/gtk2_ardour/regionfx_box.bindings b/gtk2_ardour/regionfx_box.bindings
index 8cb6038369..a6a5ee412a 100644
--- a/gtk2_ardour/regionfx_box.bindings
+++ b/gtk2_ardour/regionfx_box.bindings
@@ -1,6 +1,6 @@
 <Bindings name="RegionFx Box">
   <Press>
-     <Binding key="Delete" action="RegionFxMenu/delete"/>
-     <Binding key="BackSpace" action="RegionFxMenu/backspace"/>
+     <Binding key="Delete" action="RegionFxMenu/rfx-delete"/>
+     <Binding key="BackSpace" action="RegionFxMenu/rfx-backspace"/>
   </Press>
 </Bindings>