use TRX Profile to alter transport button layout
[ardour.git] / gtk2_ardour / ardour_ui2.cc
index 3e3f8a15069074b93ca43476ae871fdadc3e0b42..2a794827e07ea56a29a0ff8351d9c72e388be134 100644 (file)
 #include "pbd/error.h"
 #include "pbd/basename.h"
 #include "pbd/fastlog.h"
+#include <gtkmm2ext/cairocell.h>
 #include <gtkmm2ext/utils.h>
 #include <gtkmm2ext/click_box.h>
 #include <gtkmm2ext/tearoff.h>
 
-#include "ardour/session.h"
-#include "ardour/audioengine.h"
-#include "ardour/ardour.h"
 #include "ardour/profile.h"
-#include "ardour/route.h"
+#include "ardour/session.h"
+#include "ardour/types.h"
 
 #include "ardour_ui.h"
 #include "keyboard.h"
 #include "public_editor.h"
 #include "audio_clock.h"
 #include "actions.h"
+#include "main_clock.h"
 #include "utils.h"
 #include "theme_manager.h"
 #include "midi_tracer.h"
@@ -54,6 +54,7 @@
 #include "global_port_matrix.h"
 #include "location_ui.h"
 #include "rc_option_editor.h"
+#include "time_info_box.h"
 
 #include "i18n.h"
 
@@ -77,12 +78,15 @@ ARDOUR_UI::setup_windows ()
                return -1;
        }
 
+       if (create_meterbridge ()) {
+               error << _("UI: cannot setup meterbridge") << endmsg;
+               return -1;
+       }
+
        /* all other dialogs are created conditionally */
 
        we_have_dependents ();
 
-       theme_manager->signal_unmap().connect (sigc::bind (sigc::ptr_fun(&ActionManager::uncheck_toggleaction), X_("<Actions>/Common/ToggleThemeManager")));
-
 #ifdef TOP_MENUBAR
        HBox* status_bar_packer = manage (new HBox);
        EventBox* status_bar_event_box = manage (new EventBox);
@@ -112,6 +116,7 @@ ARDOUR_UI::setup_windows ()
        editor->add_toplevel_controls (top_packer);
 
        setup_transport();
+
        build_menu_bar ();
 
        setup_tooltips ();
@@ -126,22 +131,20 @@ ARDOUR_UI::setup_tooltips ()
        set_tip (stop_button, _("Stop playback"));
        set_tip (rec_button, _("Toggle record"));
        set_tip (play_selection_button, _("Play range/selection"));
-       set_tip (join_play_range_button, _("Always play range/selection"));
        set_tip (goto_start_button, _("Go to start of session"));
        set_tip (goto_end_button, _("Go to end of session"));
        set_tip (auto_loop_button, _("Play loop range"));
-
+       set_tip (midi_panic_button, _("MIDI Panic\nSend note off and reset controller messages on all MIDI channels"));
        set_tip (auto_return_button, _("Return to last playback start when stopped"));
-       set_tip (auto_play_button, _("Start playback after any locate"));
+       set_tip (follow_edits_button, _("Playhead follows Range Selections and Edits"));
        set_tip (auto_input_button, _("Be sensible about input monitoring"));
-       set_tip (punch_in_button, _("Start recording at auto-punch start"));
-       set_tip (punch_out_button, _("Stop recording at auto-punch end"));
        set_tip (click_button, _("Enable/Disable audio click"));
-       set_tip (time_master_button, string_compose (_("Does %1 control the time?"), PROGRAM_NAME));
        set_tip (solo_alert_button, _("When active, something is soloed.\nClick to de-solo everything"));
        set_tip (auditioning_alert_button, _("When active, auditioning is taking place\nClick to stop the audition"));
-       set_tip (primary_clock, _("Primary Clock"));
-       set_tip (secondary_clock, _("Secondary Clock"));
+       set_tip (feedback_alert_button, _("When active, there is a feedback loop."));
+       set_tip (primary_clock, _("<b>Primary Clock</b> right-click to set display mode. Click to edit, click+drag a digit or mouse-over+scroll wheel to modify.\nText edits: right-to-left overwrite <tt>Esc</tt>: cancel; <tt>Enter</tt>: confirm; postfix the edit with '+' or '-' to enter delta times.\n"));
+       set_tip (secondary_clock, _("<b>Secondary Clock</b> right-click to set display mode. Click to edit, click+drag a digit or mouse-over+scroll wheel to modify.\nText edits: right-to-left overwrite <tt>Esc</tt>: cancel; <tt>Enter</tt>: confirm; postfix the edit with '+' or '-' to enter delta times.\n"));
+       set_tip (editor_meter_peak_display, _("Reset Level Meter"));
 
        synchronize_sync_source_and_video_pullup ();
 
@@ -195,7 +198,7 @@ XMLNode*
 ARDOUR_UI::tearoff_settings (const char* name) const
 {
        XMLNode* ui_node = Config->extra_xml(X_("UI"));
-        
+
        if (ui_node) {
                XMLNode* tearoff_node = ui_node->child (X_("Tearoffs"));
                if (tearoff_node) {
@@ -210,6 +213,11 @@ ARDOUR_UI::tearoff_settings (const char* name) const
 void
 ARDOUR_UI::setup_transport ()
 {
+       RefPtr<Action> act;
+
+       transport_tearoff_hbox.set_border_width (3);
+       transport_tearoff_hbox.set_spacing (3);
+
        transport_tearoff = manage (new TearOff (transport_tearoff_hbox));
        transport_tearoff->set_name ("TransportBase");
        transport_tearoff->tearoff_window().signal_key_press_event().connect (sigc::bind (sigc::ptr_fun (relay_key_press), &transport_tearoff->tearoff_window()), false);
@@ -236,236 +244,203 @@ ARDOUR_UI::setup_transport ()
        transport_tearoff->Visible.connect (sigc::bind (sigc::mem_fun(*this, &ARDOUR_UI::reattach_tearoff), static_cast<Box*> (&top_packer),
                                                  static_cast<Widget*> (&transport_frame), 1));
 
-       goto_start_button.set_name ("TransportButton");
-       goto_end_button.set_name ("TransportButton");
-       roll_button.set_name ("TransportButton");
-       stop_button.set_name ("TransportButton");
-       play_selection_button.set_name ("TransportButton");
-       rec_button.set_name ("TransportRecButton");
-       auto_loop_button.set_name ("TransportButton");
-       join_play_range_button.set_name ("TransportButton");
-
-       auto_return_button.set_name ("TransportButton");
-       auto_play_button.set_name ("TransportButton");
-       auto_input_button.set_name ("TransportButton");
-       punch_in_button.set_name ("TransportButton");
-       punch_out_button.set_name ("TransportButton");
-       click_button.set_name ("TransportButton");
-       time_master_button.set_name ("TransportButton");
-       sync_button.set_name ("TransportSyncButton");
-
-       stop_button.set_size_request(29, -1);
-       roll_button.set_size_request(29, -1);
-       auto_loop_button.set_size_request(29, -1);
-       play_selection_button.set_size_request(29, -1);
-       goto_start_button.set_size_request(29, -1);
-       goto_end_button.set_size_request(29, -1);
-       rec_button.set_size_request(29, -1);
-
-       Widget* w;
-
-       stop_button.set_visual_state (1);
-
-       w = manage (new Image (get_icon (X_("transport_start"))));
-       w->show();
-       goto_start_button.add (*w);
-       w = manage (new Image (get_icon (X_("transport_end"))));
-       w->show();
-       goto_end_button.add (*w);
-       w = manage (new Image (get_icon (X_("transport_play"))));
-       w->show();
-       roll_button.add (*w);
-       w = manage (new Image (get_icon (X_("transport_stop"))));
-       w->show();
-       stop_button.add (*w);
-       w = manage (new Image (get_icon (X_("transport_range"))));
-       w->show();
-       play_selection_button.add (*w);
-       w = manage (new Image (get_icon (X_("transport_record"))));
-       w->show();
-       rec_button.add (*w);
-       w = manage (new Image (get_icon (X_("transport_loop"))));
-       w->show();
-       auto_loop_button.add (*w);
-       w = manage (new Image (get_icon (X_("tool_object_range"))));
-       w->show ();
-       join_play_range_button.add (*w);
+       auto_return_button.set_text(_("Auto Return"));
 
-       RefPtr<Action> act;
+       follow_edits_button.set_text(_("Follow Edits"));
+
+//     auto_input_button.set_text (_("Auto Input"));
+
+       click_button.set_image (get_icon (X_("metronome")));
+       act = ActionManager::get_action ("Transport", "ToggleClick");
+       click_button.set_related_action (act);
+       click_button.signal_button_press_event().connect (sigc::mem_fun (*this, &ARDOUR_UI::click_button_clicked), false);
+
+       auto_return_button.set_name ("transport option button");
+       follow_edits_button.set_name ("transport option button");
+       auto_input_button.set_name ("transport option button");
+
+       /* these have to provide a clear indication of active state */
+
+       click_button.set_name ("transport button");
+       sync_button.set_name ("transport active option button");
+
+       stop_button.set_active (true);
+
+       goto_start_button.set_image (get_icon (X_("transport_start")));
+       goto_end_button.set_image (get_icon (X_("transport_end")));
+       roll_button.set_image (get_icon (X_("transport_play")));
+       stop_button.set_image (get_icon (X_("transport_stop")));
+       play_selection_button.set_image (get_icon (X_("transport_range")));
+       rec_button.set_image (get_icon (X_("transport_record")));
+       auto_loop_button.set_image (get_icon (X_("transport_loop")));
 
+       midi_panic_button.set_image (get_icon (X_("midi_panic")));
+       /* the icon for this has an odd aspect ratio, so fatten up the button */
+       midi_panic_button.set_size_request (25, -1);
+       
        act = ActionManager::get_action (X_("Transport"), X_("Stop"));
-       act->connect_proxy (stop_button);
+       stop_button.set_related_action (act);
        act = ActionManager::get_action (X_("Transport"), X_("Roll"));
-       act->connect_proxy (roll_button);
+       roll_button.set_related_action (act);
        act = ActionManager::get_action (X_("Transport"), X_("Record"));
-       act->connect_proxy (rec_button);
+       rec_button.set_related_action (act);
        act = ActionManager::get_action (X_("Transport"), X_("GotoStart"));
-       act->connect_proxy (goto_start_button);
+       goto_start_button.set_related_action (act);
        act = ActionManager::get_action (X_("Transport"), X_("GotoEnd"));
-       act->connect_proxy (goto_end_button);
+       goto_end_button.set_related_action (act);
        act = ActionManager::get_action (X_("Transport"), X_("Loop"));
-       act->connect_proxy (auto_loop_button);
+       auto_loop_button.set_related_action (act);
        act = ActionManager::get_action (X_("Transport"), X_("PlaySelection"));
-       act->connect_proxy (play_selection_button);
-       act = ActionManager::get_action (X_("Transport"), X_("ToggleTimeMaster"));
-       act->connect_proxy (time_master_button);
+       play_selection_button.set_related_action (act);
+       act = ActionManager::get_action (X_("MIDI"), X_("panic"));
+       midi_panic_button.set_related_action (act);
        act = ActionManager::get_action (X_("Transport"), X_("ToggleExternalSync"));
-       act->connect_proxy (sync_button);
-
+       sync_button.set_related_action (act);
 
        /* clocks, etc. */
 
-       ARDOUR_UI::Clock.connect (sigc::bind (sigc::mem_fun (primary_clock, &AudioClock::set), 'p'));
-       ARDOUR_UI::Clock.connect (sigc::bind (sigc::mem_fun (secondary_clock, &AudioClock::set), 's'));
-
-       primary_clock.ValueChanged.connect (sigc::mem_fun(*this, &ARDOUR_UI::primary_clock_value_changed));
-       secondary_clock.ValueChanged.connect (sigc::mem_fun(*this, &ARDOUR_UI::secondary_clock_value_changed));
-       big_clock.ValueChanged.connect (sigc::mem_fun(*this, &ARDOUR_UI::big_clock_value_changed));
-
-       ActionManager::get_action ("Transport", "ToggleAutoReturn")->connect_proxy (auto_return_button);
-       ActionManager::get_action ("Transport", "ToggleAutoPlay")->connect_proxy (auto_play_button);
-       ActionManager::get_action ("Transport", "ToggleAutoInput")->connect_proxy (auto_input_button);
-       ActionManager::get_action ("Transport", "ToggleClick")->connect_proxy (click_button);
-       ActionManager::get_action ("Transport", "TogglePunchIn")->connect_proxy (punch_in_button);
-       ActionManager::get_action ("Transport", "TogglePunchOut")->connect_proxy (punch_out_button);
+       ARDOUR_UI::Clock.connect (sigc::mem_fun (primary_clock, &AudioClock::set));
+       ARDOUR_UI::Clock.connect (sigc::mem_fun (secondary_clock, &AudioClock::set));
 
-       click_button.signal_button_press_event().connect (sigc::mem_fun (*this, &ARDOUR_UI::click_button_clicked), false);
-
-       preroll_button.set_name ("TransportButton");
-       postroll_button.set_name ("TransportButton");
+       primary_clock->ValueChanged.connect (sigc::mem_fun(*this, &ARDOUR_UI::primary_clock_value_changed));
+       secondary_clock->ValueChanged.connect (sigc::mem_fun(*this, &ARDOUR_UI::secondary_clock_value_changed));
+       big_clock->ValueChanged.connect (sigc::mem_fun(*this, &ARDOUR_UI::big_clock_value_changed));
 
-       preroll_clock.set_mode (AudioClock::MinSec);
-       preroll_clock.set_name ("TransportClockDisplay");
-       postroll_clock.set_mode (AudioClock::MinSec);
-       postroll_clock.set_name ("TransportClockDisplay");
+       act = ActionManager::get_action ("Transport", "ToggleAutoReturn");
+       auto_return_button.set_related_action (act);
+       act = ActionManager::get_action (X_("Transport"), X_("ToggleFollowEdits"));
+       follow_edits_button.set_related_action (act);
+       act = ActionManager::get_action ("Transport", "ToggleAutoInput");
+       auto_input_button.set_related_action (act);
 
        /* alerts */
 
        /* CANNOT sigc::bind these to clicked or toggled, must use pressed or released */
 
-       solo_alert_button.set_name ("TransportSoloAlert");
+       solo_alert_button.set_name ("rude solo");
        solo_alert_button.signal_button_press_event().connect (sigc::mem_fun(*this,&ARDOUR_UI::solo_alert_press), false);
-       auditioning_alert_button.set_name ("TransportAuditioningAlert");
+       auditioning_alert_button.set_name ("rude audition");
        auditioning_alert_button.signal_button_press_event().connect (sigc::mem_fun(*this,&ARDOUR_UI::audition_alert_press), false);
+       feedback_alert_button.set_name ("feedback alert");
+       feedback_alert_button.signal_button_press_event().connect (sigc::mem_fun (*this, &ARDOUR_UI::feedback_alert_press), false);
 
-       alert_box.pack_start (solo_alert_button, false, false);
-       alert_box.pack_start (auditioning_alert_button, false, false);
+       alert_box.pack_start (solo_alert_button, true, false);
+       alert_box.pack_start (auditioning_alert_button, true, false);
+       alert_box.pack_start (feedback_alert_button, true, false);
 
-       HBox* transport_hbox = manage (new HBox);
+       /* all transport buttons should be the same size vertically and
+        * horizontally 
+        */
 
-       transport_tearoff_hbox.set_border_width (3);
+       Glib::RefPtr<SizeGroup> transport_button_size_group = SizeGroup::create (SIZE_GROUP_BOTH);
+       transport_button_size_group->add_widget (goto_start_button);
+       transport_button_size_group->add_widget (goto_end_button);
+       transport_button_size_group->add_widget (auto_loop_button);
+       transport_button_size_group->add_widget (rec_button);
+       transport_button_size_group->add_widget (play_selection_button);
+       transport_button_size_group->add_widget (roll_button);
+       transport_button_size_group->add_widget (stop_button);
 
-       transport_hbox->pack_start (goto_start_button, false, false);
-       transport_hbox->pack_start (goto_end_button, false, false);
+       goto_start_button.set_size_request (-1, 40);
 
-       /* translators: Egternal is "External" with a descender character */
-       set_size_request_to_display_given_text (sync_button, X_("Egternal"), 4, 10);
+       HBox* tbox1 = manage (new HBox);
+       HBox* tbox2 = manage (new HBox);
+       HBox* tbox = manage (new HBox);
 
-       // transport_tearoff_hbox.pack_start (*svbox, false, false, 3);
+       VBox* vbox1 = manage (new VBox);
+       VBox* vbox2 = manage (new VBox);
 
-       if (Profile->get_sae()) {
-               transport_hbox->pack_start (auto_loop_button);
-               transport_hbox->pack_start (roll_button);
-       } else {
-               transport_hbox->pack_start (auto_loop_button, false, false);
+       Alignment* a1 = manage (new Alignment);
+       Alignment* a2 = manage (new Alignment);
 
-               Frame* jpframe = manage (new Frame);
-               HBox* jpbox = manage (new HBox);
+       tbox1->set_spacing (2);
+       tbox2->set_spacing (2);
+       tbox->set_spacing (2);
 
-               jpframe->add (*jpbox);
-               jpframe->set_shadow_type (SHADOW_NONE);
+       if (!Profile->get_trx()) {
+               tbox1->pack_start (midi_panic_button, false, false, 5);
+               tbox1->pack_start (click_button, false, false, 5);
+       }
 
-               jpbox->pack_start (play_selection_button, false, false);
-               jpbox->pack_start (join_play_range_button, false, false);
-               jpbox->pack_start (roll_button, false, false);
+       tbox1->pack_start (goto_start_button, false, false);
+       tbox1->pack_start (goto_end_button, false, false);
+       tbox1->pack_start (auto_loop_button, false, false);
 
-               transport_hbox->pack_start (*jpframe, false, false);
+       if (!Profile->get_trx()) {
+               tbox2->pack_start (play_selection_button, false, false);
        }
-       transport_hbox->pack_start (stop_button, false, false);
-       transport_hbox->pack_start (rec_button, false, false, 6);
+       tbox2->pack_start (roll_button, false, false);
+       tbox2->pack_start (stop_button, false, false);
+       tbox2->pack_start (rec_button, false, false, 5);
+
+       vbox1->pack_start (*tbox1, false, false);
+       vbox2->pack_start (*tbox2, false, false);
+
+       a1->add (*vbox1);
+       a1->set (0.5, 1.0, 0.0, 0.0);
+       a2->add (*vbox2);
+       a2->set (0.5, 1.0, 0.0, 0.0);
+
+       tbox->pack_start (*a1, false, false);
+       tbox->pack_start (*a2, false, false);
 
        HBox* clock_box = manage (new HBox);
-       clock_box->pack_start (primary_clock, false, false);
-       if (!ARDOUR::Profile->get_small_screen()) {
-               clock_box->pack_start (secondary_clock, false, false);
-       }
 
-       if (!Profile->get_sae()) {
-               VBox* time_controls_box = manage (new VBox);
-               time_controls_box->pack_start (sync_button, false, false);
-               time_controls_box->pack_start (time_master_button, false, false);
-               clock_box->pack_start (*time_controls_box, false, false, 1);
+       clock_box->pack_start (*primary_clock, false, false);
+       if (!ARDOUR::Profile->get_small_screen()) {
+               clock_box->pack_start (*secondary_clock, false, false);
        }
+       clock_box->set_spacing (3);
 
        shuttle_box = new ShuttleControl;
        shuttle_box->show ();
 
        VBox* transport_vbox = manage (new VBox);
        transport_vbox->set_name ("TransportBase");
-       transport_vbox->set_border_width (3);
+       transport_vbox->set_border_width (0);
        transport_vbox->set_spacing (3);
-       transport_vbox->pack_start (*transport_hbox, true, true, 0);
+       transport_vbox->pack_start (*tbox, true, true, 0);
        transport_vbox->pack_start (*shuttle_box, false, false, 0);
 
-       transport_tearoff_hbox.pack_start (*transport_vbox, false, false, 0);
-       transport_tearoff_hbox.pack_start (*clock_box, false, false, 0);
-
-       HBox* toggle_box = manage(new HBox);
+       transport_tearoff_hbox.pack_start (*transport_vbox, false, false);
 
-       VBox* punch_box = manage (new VBox);
-       punch_box->pack_start (punch_in_button, false, false);
-       punch_box->pack_start (punch_out_button, false, false);
-       toggle_box->pack_start (*punch_box, false, false);
+       /* transport related toggle controls */
 
        VBox* auto_box = manage (new VBox);
-       auto_box->pack_start (auto_play_button, false, false);
+       auto_box->set_homogeneous (true);
+       auto_box->set_spacing (2);
+       auto_box->pack_start (sync_button, false, false);
+       auto_box->pack_start (follow_edits_button, false, false);
        auto_box->pack_start (auto_return_button, false, false);
-        if (!Profile->get_small_screen()) {
-                toggle_box->pack_start (*auto_box, false, false);
-        }
-
-       VBox* io_box = manage (new VBox);
-       io_box->pack_start (auto_input_button, false, false);
-       io_box->pack_start (click_button, false, false);
-        if (!Profile->get_small_screen()) {
-                toggle_box->pack_start (*io_box, false, false);
-        }
-
-       /* desensitize */
 
-       set_transport_sensitivity (false);
-
-//     toggle_box->pack_start (preroll_button, false, false);
-//     toggle_box->pack_start (preroll_clock, false, false);
+       transport_tearoff_hbox.pack_start (*auto_box, false, false);
+       transport_tearoff_hbox.pack_start (*clock_box, true, true);
 
-//     toggle_box->pack_start (postroll_button, false, false);
-//     toggle_box->pack_start (postroll_clock, false, false);
+       time_info_box = manage (new TimeInfoBox);
+       transport_tearoff_hbox.pack_start (*time_info_box, false, false);
 
-       transport_tearoff_hbox.pack_start (*toggle_box, false, false, 4);
         if (Profile->get_small_screen()) {
-                transport_tearoff_hbox.pack_start (_editor_transport_box, false, false, 4);
+                transport_tearoff_hbox.pack_start (_editor_transport_box, false, false);
         }
        transport_tearoff_hbox.pack_start (alert_box, false, false);
+       transport_tearoff_hbox.pack_start (meter_box, false, false);
+       transport_tearoff_hbox.pack_start (editor_meter_peak_display, false, false);
 
        if (Profile->get_sae()) {
                Image* img = manage (new Image ((::get_icon (X_("sae")))));
-               transport_tearoff_hbox.pack_end (*img, false, false, 6);
+               transport_tearoff_hbox.pack_end (*img, false, false);
        }
 
+       /* desensitize */
+
+       set_transport_sensitivity (false);
+
        XMLNode* tnode = tearoff_settings ("transport");
        if (tnode) {
                transport_tearoff->set_state (*tnode);
        }
 }
 
-void
-ARDOUR_UI::manage_window (Window& win)
-{
-       win.signal_delete_event().connect (sigc::bind (sigc::ptr_fun (just_hide_it), &win));
-       win.signal_enter_notify_event().connect (sigc::bind (sigc::mem_fun (Keyboard::the_keyboard(), &Keyboard::enter_window), &win));
-       win.signal_leave_notify_event().connect (sigc::bind (sigc::mem_fun (Keyboard::the_keyboard(), &Keyboard::leave_window), &win));
-}
-
 void
 ARDOUR_UI::detach_tearoff (Box* b, Widget* w)
 {
@@ -491,10 +466,8 @@ ARDOUR_UI::soloing_changed (bool onoff)
 void
 ARDOUR_UI::_auditioning_changed (bool onoff)
 {
-       if (auditioning_alert_button.get_active() != onoff) {
-               auditioning_alert_button.set_active (onoff);
-               set_transport_sensitivity (!onoff);
-       }
+       auditioning_alert_button.set_active (onoff);
+       set_transport_sensitivity (!onoff);
 }
 
 void
@@ -525,6 +498,12 @@ ARDOUR_UI::solo_alert_press (GdkEventButton*)
        return true;
 }
 
+bool
+ARDOUR_UI::feedback_alert_press (GdkEventButton *)
+{
+       return true;
+}
+
 void
 ARDOUR_UI::solo_blink (bool onoff)
 {
@@ -534,13 +513,12 @@ ARDOUR_UI::solo_blink (bool onoff)
 
        if (_session->soloing() || _session->listening()) {
                if (onoff) {
-                       solo_alert_button.set_state (STATE_ACTIVE);
+                       solo_alert_button.set_active (true);
                } else {
-                       solo_alert_button.set_state (STATE_NORMAL);
+                       solo_alert_button.set_active (false);
                }
        } else {
                solo_alert_button.set_active (false);
-               solo_alert_button.set_state (STATE_NORMAL);
        }
 }
 
@@ -549,7 +527,7 @@ ARDOUR_UI::sync_blink (bool onoff)
 {
        if (_session == 0 || !_session->config.get_external_sync()) {
                /* internal sync */
-               sync_button.set_visual_state (0);
+               sync_button.set_active (false);
                return;
        }
 
@@ -557,13 +535,13 @@ ARDOUR_UI::sync_blink (bool onoff)
                /* not locked, so blink on and off according to the onoff argument */
 
                if (onoff) {
-                       sync_button.set_visual_state (1); // "-active"
+                       sync_button.set_active (true);
                } else {
-                       sync_button.set_visual_state (0); // normal
+                       sync_button.set_active (false);
                }
        } else {
                /* locked */
-               sync_button.set_visual_state (1); // "-active"
+               sync_button.set_active (true);
        }
 }
 
@@ -576,13 +554,26 @@ ARDOUR_UI::audition_blink (bool onoff)
 
        if (_session->is_auditioning()) {
                if (onoff) {
-                       auditioning_alert_button.set_state (STATE_ACTIVE);
+                       auditioning_alert_button.set_active (true);
                } else {
-                       auditioning_alert_button.set_state (STATE_NORMAL);
+                       auditioning_alert_button.set_active (false);
                }
        } else {
                auditioning_alert_button.set_active (false);
-               auditioning_alert_button.set_state (STATE_NORMAL);
+       }
+}
+
+void
+ARDOUR_UI::feedback_blink (bool onoff)
+{
+       if (_feedback_exists) {
+               if (onoff) {
+                       feedback_alert_button.set_active (true);
+               } else {
+                       feedback_alert_button.set_active (false);
+               }
+       } else {
+               feedback_alert_button.set_active (false);
        }
 }
 
@@ -603,28 +594,27 @@ ARDOUR_UI::editor_realized ()
 }
 
 void
-ARDOUR_UI::maximise_editing_space ()
+ARDOUR_UI::update_tearoff_visibility ()
 {
-       if (!editor) {
-               return;
+       if (editor) {
+               editor->update_tearoff_visibility ();
        }
+}
 
-       transport_tearoff->set_visible (false);
-       editor->maximise_editing_space ();
-       if (Config->get_keep_tearoffs()) {
-               transport_tearoff->set_visible (true);
+void
+ARDOUR_UI::maximise_editing_space ()
+{
+       if (editor) {
+               editor->maximise_editing_space ();
        }
 }
 
 void
 ARDOUR_UI::restore_editing_space ()
 {
-       if (!editor) {
-               return;
+       if (editor) {
+               editor->restore_editing_space ();
        }
-
-       transport_tearoff->set_visible (true);
-       editor->restore_editing_space ();
 }
 
 bool
@@ -635,12 +625,25 @@ ARDOUR_UI::click_button_clicked (GdkEventButton* ev)
                return false;
        }
 
-       RefPtr<Action> act = ActionManager::get_action (X_("Common"), X_("ToggleRCOptionsEditor"));
+       RefPtr<Action> act = ActionManager::get_action (X_("Window"), X_("toggle-rc-options-editor"));
        assert (act);
 
-       RefPtr<ToggleAction> tact = RefPtr<ToggleAction>::cast_dynamic (act);
-       tact->set_active ();
+       act->activate();
 
        rc_option_editor->set_current_page (_("Misc"));
        return true;
 }
+
+void
+ARDOUR_UI::toggle_always_play_range ()
+{
+       RefPtr<Action> act = ActionManager::get_action (X_("Transport"), X_("ToggleFollowEdits"));
+       assert (act);
+
+       RefPtr<ToggleAction> tact = RefPtr<ToggleAction>::cast_dynamic (act);
+       assert (tact);
+
+       Config->set_always_play_range (tact->get_active ());
+}
+
+