From 59b4611cda84473f4173596fea78d292c38f87ae Mon Sep 17 00:00:00 2001 From: Paul Davis Date: Mon, 30 Jun 2014 19:18:27 -0400 Subject: [PATCH] (hopefully) fix smart-mode breakage of xfade operations We needed to set the selected regionview from a click when clicking on xfade items, but I also removed duplicate logic for some similar items --- gtk2_ardour/editor_mouse.cc | 9 +-------- 1 file changed, 1 insertion(+), 8 deletions(-) diff --git a/gtk2_ardour/editor_mouse.cc b/gtk2_ardour/editor_mouse.cc index 0fdd933832..fe311906b6 100644 --- a/gtk2_ardour/editor_mouse.cc +++ b/gtk2_ardour/editor_mouse.cc @@ -565,13 +565,6 @@ Editor::button_selection (ArdourCanvas::Item* /*item*/, GdkEvent* event, ItemTyp case RegionViewName: case LeftFrameHandle: case RightFrameHandle: - if (eff_mouse_mode != MouseRange) { - set_selected_regionview_from_click (press, op); - } else if (event->type == GDK_BUTTON_PRESS) { - set_selected_track_as_side_effect (op); - } - break; - case FadeInHandleItem: case FadeInTrimHandleItem: case FadeInItem: @@ -580,7 +573,7 @@ Editor::button_selection (ArdourCanvas::Item* /*item*/, GdkEvent* event, ItemTyp case FadeOutItem: case StartCrossFadeItem: case EndCrossFadeItem: - if (eff_mouse_mode != MouseRange) { + if (get_smart_mode() || eff_mouse_mode != MouseRange) { set_selected_regionview_from_click (press, op); } else if (event->type == GDK_BUTTON_PRESS) { set_selected_track_as_side_effect (op);