Show control point context menu on automation lines

Right clicking an automation line will select both ends of it and make control point operations perfectly relevant.
Ardour · Oct 24, 2022 · a0d5fb3 · a0d5fb3
1 parent 8ce40c6
commit a0d5fb3
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/gtk2_ardour/editor.cc b/gtk2_ardour/editor.cc
@@ -6593,9 +6593,10 @@ Editor::popup_control_point_context_menu (ArdourCanvas::Item* item, GdkEvent* ev
 
 	items.push_back (MenuElem (_("Edit..."), sigc::bind (sigc::mem_fun (*this, &Editor::edit_control_point), item)));
 	items.push_back (MenuElem (_("Delete"), sigc::bind (sigc::mem_fun (*this, &Editor::remove_control_point), item)));
+	/* Don't insensitivate for AudioRegionGainLine and AutomationLine */
 	ControlPoint * control_point;
-	if (((control_point = reinterpret_cast<ControlPoint *> (item->get_data ("control_point"))) == 0)
-	    || !can_remove_control_point (*control_point)) {
+	if (((control_point = reinterpret_cast<ControlPoint *> (item->get_data ("control_point"))))
+	    && !can_remove_control_point (*control_point)) {
 		items.back().set_sensitive (false);
 	}
 

diff --git a/gtk2_ardour/editor_mouse.cc b/gtk2_ardour/editor_mouse.cc
@@ -1713,6 +1713,8 @@ Editor::button_release_handler (ArdourCanvas::Item* item, GdkEvent* event, ItemT
 				popup_track_context_menu (1, event->button.time, item_type, false);
 				break;
 
+			case GainLineItem:
+			case AutomationLineItem:
 			case ControlPointItem:
 				popup_control_point_context_menu (item, event);
 				break;