From 945ce377aa4460b7e89b14d8fd1a6909fa038b65 Mon Sep 17 00:00:00 2001 From: Paul Davis Date: Tue, 3 Oct 2023 16:00:35 -0600 Subject: [PATCH] tempo mapping (like all other drags) must use button1 only Without this, Editor::button_release_handler() will handle a button3 press without checking for an active drag, resulting in two drags, and two reversible commands, and an abort ... --- gtk2_ardour/editor_canvas_events.cc | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/gtk2_ardour/editor_canvas_events.cc b/gtk2_ardour/editor_canvas_events.cc index b6129af5e9..b65af5ac09 100644 --- a/gtk2_ardour/editor_canvas_events.cc +++ b/gtk2_ardour/editor_canvas_events.cc @@ -1293,10 +1293,13 @@ Editor::canvas_grid_zone_event (GdkEvent* event) GdkEventScroll scroll; ArdourCanvas::Duple winpos; + switch (event->type) { case GDK_BUTTON_PRESS: - choose_mapping_drag (_canvas_grid_zone, event); + if (event->button.button == 1) { + choose_mapping_drag (_canvas_grid_zone, event); + } break; case GDK_BUTTON_RELEASE: