(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
This commit is contained in:
Paul Davis 2014-06-30 19:18:27 -04:00
parent dff73d34d9
commit b47c20783d

View file

@ -565,13 +565,6 @@ Editor::button_selection (ArdourCanvas::Item* /*item*/, GdkEvent* event, ItemTyp
case RegionViewName: case RegionViewName:
case LeftFrameHandle: case LeftFrameHandle:
case RightFrameHandle: 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 FadeInHandleItem:
case FadeInTrimHandleItem: case FadeInTrimHandleItem:
case FadeInItem: case FadeInItem:
@ -580,7 +573,7 @@ Editor::button_selection (ArdourCanvas::Item* /*item*/, GdkEvent* event, ItemTyp
case FadeOutItem: case FadeOutItem:
case StartCrossFadeItem: case StartCrossFadeItem:
case EndCrossFadeItem: case EndCrossFadeItem:
if (eff_mouse_mode != MouseRange) { if (get_smart_mode() || eff_mouse_mode != MouseRange) {
set_selected_regionview_from_click (press, op); set_selected_regionview_from_click (press, op);
} else if (event->type == GDK_BUTTON_PRESS) { } else if (event->type == GDK_BUTTON_PRESS) {
set_selected_track_as_side_effect (op); set_selected_track_as_side_effect (op);