Mackie Control: Fix Flip mode getting confused if subview is not sends.
[ardour.git] / libs / surfaces / mackie / mcp_buttons.cc
index 60de07480e3cd130a133e782d892339d11a44f9e..78d9596a9fb7a4d1b791d7ab27a5b38b33474d33 100644 (file)
@@ -60,6 +60,7 @@ LedState
 MackieControlProtocol::option_press (Button &)
 {
        _modifier_state |= MODIFIER_OPTION;
+       access_action ("Editor/set-loop-from-edit-range");
        return on;
 }
 LedState
@@ -73,6 +74,7 @@ MackieControlProtocol::control_press (Button &)
 {
        _modifier_state |= MODIFIER_CONTROL;
        DEBUG_TRACE (DEBUG::MackieControl, string_compose ("CONTROL Press: modifier state now set to %1\n", _modifier_state));
+       access_action ("Editor/set-punch-from-edit-range");
        return on;
 }
 LedState
@@ -86,6 +88,7 @@ LedState
 MackieControlProtocol::cmd_alt_press (Button &)
 {
        _modifier_state |= MODIFIER_CMDALT;
+       access_action ("Editor/set-session-from-edit-range");
        return on;
 }
 LedState
@@ -108,9 +111,9 @@ MackieControlProtocol::left_press (Button &)
        DEBUG_TRACE (DEBUG::MackieControl, string_compose ("bank left with current initial = %1 nstrips = %2 tracks/busses = %3\n",
                                                           _current_initial_bank, strip_cnt, sorted.size()));
        if (_current_initial_bank > 0) {
-               switch_banks ((_current_initial_bank - 1) / strip_cnt * strip_cnt);
+               (void) switch_banks ((_current_initial_bank - 1) / strip_cnt * strip_cnt);
        } else {
-               switch_banks (0);
+               (void) switch_banks (0);
        }
 
 
@@ -120,7 +123,7 @@ MackieControlProtocol::left_press (Button &)
 LedState
 MackieControlProtocol::left_release (Button &)
 {
-       return off;
+       return none;
 }
 
 LedState
@@ -141,23 +144,16 @@ MackieControlProtocol::right_press (Button &)
 
        if (_current_initial_bank < max_bank) {
                uint32_t new_initial = (_current_initial_bank / strip_cnt * strip_cnt) + strip_cnt;
-
-               switch_banks (new_initial);
-       } else {
-               switch_banks (max_bank);
+               (void) switch_banks (new_initial);
        }
 
-       return on;
+       return none;
 }
 
 LedState
 MackieControlProtocol::right_release (Button &)
 {
-       if (zoom_mode()) {
-
-       }
-
-       return off;
+       return none;
 }
 
 LedState
@@ -350,44 +346,40 @@ MackieControlProtocol::scrub_release (Mackie::Button &)
 LedState
 MackieControlProtocol::undo_press (Button&)
 {
-       if (main_modifier_state() & MODIFIER_SHIFT) {
-               Redo(); /* EMIT SIGNAL */
-       } else {
-               Undo(); /* EMIT SIGNAL */
-       }
-       return off;
+       toggle_punch_out ();
+       return none;
 }
 
 LedState
 MackieControlProtocol::undo_release (Button&)
 {
-       return off;
+       return none;
 }
 
 LedState
 MackieControlProtocol::drop_press (Button &)
 {
-       session->remove_last_capture();
-       return on;
+       access_action ("Editor/start-range-from-playhead");
+       return none;
 }
 
 LedState
 MackieControlProtocol::drop_release (Button &)
 {
-       return off;
+       return none;
 }
 
 LedState
 MackieControlProtocol::save_press (Button &)
 {
-       session->save_state ("");
-       return on;
+       toggle_punch_in ();
+       return none;
 }
 
 LedState
 MackieControlProtocol::save_release (Button &)
 {
-       return off;
+       return none;
 }
 
 LedState
@@ -421,13 +413,29 @@ MackieControlProtocol::timecode_beats_release (Button &)
 LedState
 MackieControlProtocol::marker_press (Button &)
 {
+       _modifier_state |= MODIFIER_MARKER;
+       marker_modifier_consumed_by_button = false;
+       return on;
+}
+
+LedState
+MackieControlProtocol::marker_release (Button &)
+{
+       _modifier_state &= ~MODIFIER_MARKER;
+
+       if (marker_modifier_consumed_by_button) {
+               /* marker was used a modifier for some other button(s), so do
+                  nothing
+               */
+               return off;
+       }
+
        string markername;
 
        /* Don't add another mark if one exists within 1/100th of a second of
         * the current position and we're not rolling.
         */
 
-
        framepos_t where = session->audible_frame();
 
        if (session->transport_stopped() && session->locations()->mark_at (where, session->frame_rate() / 100.0)) {
@@ -437,12 +445,6 @@ MackieControlProtocol::marker_press (Button &)
        session->locations()->next_available_name (markername,"marker");
        add_marker (markername);
 
-       return on;
-}
-
-LedState
-MackieControlProtocol::marker_release (Button &)
-{
        return off;
 }
 
@@ -454,6 +456,11 @@ LedState
 MackieControlProtocol::stop_press (Button &)
 {
        transport_stop ();
+
+       if (main_modifier_state() == MODIFIER_SHIFT) {
+               session->midi_panic();
+       }
+
        return on;
 }
 
@@ -496,7 +503,11 @@ MackieControlProtocol::record_release (Button &)
 LedState
 MackieControlProtocol::rewind_press (Button &)
 {
-       if (main_modifier_state() == MODIFIER_CONTROL) {
+       if (modifier_state() & MODIFIER_MARKER) {
+               prev_marker ();
+       } else if (modifier_state() & MODIFIER_NUDGE) {
+               access_action ("Editor/nudge-playhead-backward");
+       } else if (main_modifier_state() & MODIFIER_SHIFT) {
                goto_start ();
        } else {
                rewind ();
@@ -513,7 +524,11 @@ MackieControlProtocol::rewind_release (Button &)
 LedState
 MackieControlProtocol::ffwd_press (Button &)
 {
-       if (main_modifier_state() == MODIFIER_CONTROL) {
+       if (modifier_state() & MODIFIER_MARKER) {
+               next_marker ();
+       } else if (modifier_state() & MODIFIER_NUDGE) {
+               access_action ("Editor/nudge-playhead-forward");
+       } else if (main_modifier_state() & MODIFIER_SHIFT) {
                goto_end();
        } else {
                ffwd ();
@@ -558,14 +573,14 @@ MackieControlProtocol::clicking_release (Button &)
 LedState
 MackieControlProtocol::enter_press (Button &)
 {
-       Enter(); /* EMIT SIGNAL */
-       return off;
+       access_action ("Transport/ToggleFollowEdits");
+       return none;
 }
 
 LedState
 MackieControlProtocol::enter_release (Button &)
 {
-       return off;
+       return none;
 }
 
 LedState
@@ -581,7 +596,7 @@ MackieControlProtocol::bank_release (Button& b, uint32_t basic_bank_num)
                bank_num = 8 + basic_bank_num;
        }
 
-       switch_banks (n_strips() * bank_num);
+       (void) switch_banks (n_strips() * bank_num);
 
        return on;
 }
@@ -673,7 +688,8 @@ MackieControlProtocol::F8_release (Button &)
 LedState
 MackieControlProtocol::pan_press (Button &)
 {
-       set_pot_mode (Pan);
+       /* XXX eventually pan may have its own subview mode */
+       set_subview_mode (MackieControlProtocol::None, boost::shared_ptr<Route>());
        return none;
 }
 LedState
@@ -722,12 +738,14 @@ MackieControlProtocol::dyn_release (Button &)
 LedState
 MackieControlProtocol::flip_press (Button &)
 {
-       if (_flip_mode != Normal) {
-               set_flip_mode (Normal);
-       } else {
-               set_flip_mode (Mirror);
+       if (subview_mode() == MackieControlProtocol::Sends) {
+               if (_flip_mode != Normal) {
+                       set_flip_mode (Normal);
+               } else {
+                       set_flip_mode (Mirror);
+               }
+               return ((_flip_mode != Normal) ? on : off);
        }
-       return ((_flip_mode != Normal) ? on : off);
 }
 LedState
 MackieControlProtocol::flip_release (Button &)
@@ -747,22 +765,24 @@ MackieControlProtocol::name_value_release (Button &)
 LedState
 MackieControlProtocol::touch_press (Button &)
 {
-       return off;
+       return none;
 }
 LedState
 MackieControlProtocol::touch_release (Button &)
 {
-       return off;
+       set_automation_state (ARDOUR::Touch);
+       return none;
 }
 LedState
 MackieControlProtocol::cancel_press (Button &)
 {
-       return off;
+       access_action ("Transport/ToggleExternalSync");
+       return none;
 }
 LedState
 MackieControlProtocol::cancel_release (Button &)
 {
-       return off;
+       return none;
 }
 LedState
 MackieControlProtocol::user_a_press (Button &)
@@ -817,14 +837,14 @@ MackieControlProtocol::master_fader_touch_release (Mackie::Button &)
 Mackie::LedState
 MackieControlProtocol::read_press (Mackie::Button&)
 {
-       _metering_active = !_metering_active;
-       notify_metering_state_changed ();
-       return _metering_active;
+       return none;
 }
+
 Mackie::LedState
 MackieControlProtocol::read_release (Mackie::Button&)
 {
-       return _metering_active;
+       set_automation_state (ARDOUR::Play);
+       return none;
 }
 Mackie::LedState
 MackieControlProtocol::write_press (Mackie::Button&)
@@ -834,6 +854,7 @@ MackieControlProtocol::write_press (Mackie::Button&)
 Mackie::LedState
 MackieControlProtocol::write_release (Mackie::Button&)
 {
+       set_automation_state (ARDOUR::Write);
        return none;
 }
 
@@ -841,8 +862,16 @@ Mackie::LedState
 MackieControlProtocol::clearsolo_press (Mackie::Button&)
 {
        // clears all solos and listens (pfl/afl)
-       session->set_solo (session->get_routes(), false);
-       session->set_listen (session->get_routes(), false);
+
+       if (session) {
+               if (session->soloing()) {
+                       session->set_solo (session->get_routes(), false);
+               } else if (session->listening()) {
+                       session->set_listen (session->get_routes(), false);
+               }
+
+               session->clear_all_solo_state (session->get_routes()); // safeguard, ideally this won't do anything, check the log-window
+       }
        return none;
 }
 
@@ -856,7 +885,7 @@ MackieControlProtocol::clearsolo_release (Mackie::Button&)
 Mackie::LedState
 MackieControlProtocol::track_press (Mackie::Button&)
 {
-       set_pot_mode (Trim);
+       set_subview_mode (TrackView, first_selected_route());
        return none;
 }
 Mackie::LedState
@@ -911,13 +940,13 @@ MackieControlProtocol::audiotracks_release (Mackie::Button&)
 Mackie::LedState
 MackieControlProtocol::audioinstruments_press (Mackie::Button& b)
 {
-       return dyn_press (b);
+       return none;
 }
 
 Mackie::LedState
 MackieControlProtocol::audioinstruments_release (Mackie::Button& b)
 {
-       return dyn_release (b);
+       return none;
 
 }
 Mackie::LedState
@@ -992,22 +1021,41 @@ MackieControlProtocol::grp_press (Mackie::Button&)
 Mackie::LedState
 MackieControlProtocol::grp_release (Mackie::Button&)
 {
+       /* There is no "Off" button for automation,
+          so we use Group for this purpose.
+       */
+       set_automation_state (Off);
        return none;
 }
 Mackie::LedState
 MackieControlProtocol::nudge_press (Mackie::Button&)
 {
-       return none;
+       _modifier_state |= MODIFIER_NUDGE;
+       nudge_modifier_consumed_by_button = false;
+       return on;
 }
 Mackie::LedState
 MackieControlProtocol::nudge_release (Mackie::Button&)
 {
-       return none;
+       _modifier_state &= ~MODIFIER_NUDGE;
+
+       /* XXX these action names are stupid, because the action can affect
+        * regions, markers or the playhead depending on selection state. 
+        */
+
+       if (main_modifier_state() & MODIFIER_SHIFT) {
+               access_action ("Region/nudge-backward");
+       } else {
+               access_action ("Region/nudge-forward");
+       }
+
+       return off;
 }
 Mackie::LedState
 MackieControlProtocol::replace_press (Mackie::Button&)
 {
-       return none;
+       access_action ("Editor/finish-range-from-playhead");
+       return off;
 }
 Mackie::LedState
 MackieControlProtocol::replace_release (Mackie::Button&)