From df5c5d338a7e3375fb1d6d56b5da59f5d1cc527f Mon Sep 17 00:00:00 2001 From: Paul Davis Date: Tue, 12 Jan 2016 08:28:54 -0500 Subject: [PATCH] space bar action (Transport/ToggleRoll) should still start transport if synced to Engine (JACK) --- gtk2_ardour/ardour_ui.cc | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/gtk2_ardour/ardour_ui.cc b/gtk2_ardour/ardour_ui.cc index 0d6d9e8499..cfc1197afd 100644 --- a/gtk2_ardour/ardour_ui.cc +++ b/gtk2_ardour/ardour_ui.cc @@ -2110,7 +2110,12 @@ ARDOUR_UI::toggle_roll (bool with_abort, bool roll_out_of_bounded_mode) if (affect_transport) { if (rolling) { _session->request_stop (with_abort, true); - } else if (!_session->config.get_external_sync()) { + } else { + /* the only external sync condition we can be in here + * would be Engine (JACK) sync, in which case we still + * want to do this. + */ + if (UIConfiguration::instance().get_follow_edits() && ( editor->get_selection().time.front().start == _session->transport_frame() ) ) { //if playhead is exactly at the start of a range, we can assume it was placed there by follow_edits _session->request_play_range (&editor->get_selection().time, true); _session->set_requested_return_frame( editor->get_selection().time.front().start ); //force an auto-return here