Fix astate setting for combo parameters when setting all astate
[ardour.git] / gtk2_ardour / ardour_ui2.cc
index 5a64e5314091bc9494b7fbc13a73fc11c52ebfcb..c4404b824358f7030b86d6d7d93dfd3c09ae838b 100644 (file)
 #include <cmath>
 
 #include <sigc++/bind.h>
+#include "canvas/canvas.h"
+
 #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 "gtkmm2ext/cairocell.h"
+#include "gtkmm2ext/utils.h"
+#include "gtkmm2ext/click_box.h"
+#include "gtkmm2ext/window_title.h"
 
 #include "ardour/profile.h"
 #include "ardour/session.h"
@@ -47,6 +50,7 @@
 #include "audio_clock.h"
 #include "actions.h"
 #include "main_clock.h"
+#include "mixer_ui.h"
 #include "utils.h"
 #include "theme_manager.h"
 #include "midi_tracer.h"
@@ -56,7 +60,7 @@
 #include "rc_option_editor.h"
 #include "time_info_box.h"
 
-#include "i18n.h"
+#include "pbd/i18n.h"
 
 using namespace std;
 using namespace ARDOUR;
@@ -64,69 +68,14 @@ using namespace PBD;
 using namespace Gtkmm2ext;
 using namespace Gtk;
 using namespace Glib;
+using namespace ARDOUR_UI_UTILS;
 
-int
-ARDOUR_UI::setup_windows ()
-{
-       if (create_editor ()) {
-               error << _("UI: cannot setup editor") << endmsg;
-               return -1;
-       }
-
-       if (create_mixer ()) {
-               error << _("UI: cannot setup mixer") << endmsg;
-               return -1;
-       }
-
-       if (create_meterbridge ()) {
-               error << _("UI: cannot setup meterbridge") << endmsg;
-               return -1;
-       }
-
-       /* all other dialogs are created conditionally */
-
-       we_have_dependents ();
-
-#ifdef TOP_MENUBAR
-       HBox* status_bar_packer = manage (new HBox);
-       EventBox* status_bar_event_box = manage (new EventBox);
-
-       status_bar_event_box->add (status_bar_label);
-       status_bar_event_box->add_events (Gdk::BUTTON_PRESS_MASK|Gdk::BUTTON_RELEASE_MASK);
-       status_bar_label.set_size_request (300, -1);
-       status_bar_packer->pack_start (*status_bar_event_box, true, true, 6);
-       status_bar_packer->pack_start (error_log_button, false, false);
-
-       status_bar_label.show ();
-       status_bar_event_box->show ();
-       status_bar_packer->show ();
-       error_log_button.show ();
-
-       error_log_button.signal_clicked().connect (mem_fun (*this, &UI::toggle_errors));
-       status_bar_event_box->signal_button_press_event().connect (mem_fun (*this, &ARDOUR_UI::status_bar_button_press));
-
-       editor->get_status_bar_packer().pack_start (*status_bar_packer, true, true);
-       editor->get_status_bar_packer().pack_start (menu_bar_base, false, false, 6);
-#else
-       top_packer.pack_start (menu_bar_base, false, false);
-#endif
-
-       top_packer.pack_start (transport_frame, false, false);
-
-       editor->add_toplevel_controls (top_packer);
-
-       setup_transport();
-
-       build_menu_bar ();
-
-       setup_tooltips ();
-
-       return 0;
-}
 
 void
 ARDOUR_UI::setup_tooltips ()
 {
+       ArdourCanvas::Canvas::set_tooltip_timeout (Gtk::Settings::get_default()->property_gtk_tooltip_timeout ());
+
        set_tip (roll_button, _("Play from playhead"));
        set_tip (stop_button, _("Stop playback"));
        set_tip (rec_button, _("Toggle record"));
@@ -136,14 +85,16 @@ ARDOUR_UI::setup_tooltips ()
        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 (follow_edits_button, _("Playhead follows Range Selections and Edits"));
+       set_tip (follow_edits_button, _("Playhead follows range selections and edits"));
        set_tip (auto_input_button, _("Be sensible about input monitoring"));
        set_tip (click_button, _("Enable/Disable audio click"));
        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 (auditioning_alert_button, _("When active, auditioning is taking place.\nClick to stop the audition"));
        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 All Peak Indicators"));
+       set_tip (error_alert_button, _("Show Error Log and acknowledge warnings"));
 
        synchronize_sync_source_and_video_pullup ();
 
@@ -173,18 +124,25 @@ ARDOUR_UI::display_message (const char *prefix, gint prefix_len, RefPtr<TextBuff
        string text;
 
        UI::display_message (prefix, prefix_len, ptag, mtag, msg);
-#ifdef TOP_MENUBAR
+
+       ArdourLogLevel ll = LogLevelNone;
 
        if (strcmp (prefix, _("[ERROR]: ")) == 0) {
                text = "<span color=\"red\" weight=\"bold\">";
+               ll = LogLevelError;
        } else if (strcmp (prefix, _("[WARNING]: ")) == 0) {
                text = "<span color=\"yellow\" weight=\"bold\">";
+               ll = LogLevelWarning;
        } else if (strcmp (prefix, _("[INFO]: ")) == 0) {
                text = "<span color=\"green\" weight=\"bold\">";
+               ll = LogLevelInfo;
        } else {
                text = "<span color=\"white\" weight=\"bold\">???";
        }
 
+       _log_not_acknowledged = std::max(_log_not_acknowledged, ll);
+
+#ifdef TOP_MENUBAR
        text += prefix;
        text += "</span>";
        text += msg;
@@ -209,23 +167,25 @@ ARDOUR_UI::tearoff_settings (const char* name) const
        return 0;
 }
 
+#define PX_SCALE(px) std::max((float)px, rintf((float)px * UIConfiguration::instance().get_ui_scale()))
+
+static
+bool drag_failed (const Glib::RefPtr<Gdk::DragContext>& context, DragResult result, Tabbable* tab)
+{
+       if (result == Gtk::DRAG_RESULT_NO_TARGET) {
+               tab->detach ();
+               return true;
+       }
+       return false;
+}
+
 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);
-
-       if (Profile->get_sae()) {
-               transport_tearoff->set_can_be_torn_off (false);
-       }
-
-       transport_hbox.pack_start (*transport_tearoff, true, false);
+       transport_hbox.set_border_width (PX_SCALE(3));
+       transport_hbox.set_spacing (PX_SCALE(3));
 
        transport_base.set_name ("TransportBase");
        transport_base.add (transport_hbox);
@@ -234,22 +194,14 @@ ARDOUR_UI::setup_transport ()
        transport_frame.set_name ("BaseFrame");
        transport_frame.add (transport_base);
 
-       transport_tearoff->Detach.connect (sigc::bind (sigc::mem_fun(*this, &ARDOUR_UI::detach_tearoff), static_cast<Box*>(&top_packer),
-                                                static_cast<Widget*>(&transport_frame)));
-       transport_tearoff->Attach.connect (sigc::bind (sigc::mem_fun(*this, &ARDOUR_UI::reattach_tearoff), static_cast<Box*> (&top_packer),
-                                                static_cast<Widget*> (&transport_frame), 1));
-       transport_tearoff->Hidden.connect (sigc::bind (sigc::mem_fun(*this, &ARDOUR_UI::detach_tearoff), static_cast<Box*>(&top_packer),
-                                                static_cast<Widget*>(&transport_frame)));
-       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));
-
        auto_return_button.set_text(_("Auto Return"));
 
        follow_edits_button.set_text(_("Follow Edits"));
 
 //     auto_input_button.set_text (_("Auto Input"));
 
-       click_button.set_image (get_icon (X_("metronome")));
+       click_button.set_icon (ArdourIcon::TransportMetronom);
+
        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);
@@ -265,18 +217,15 @@ ARDOUR_UI::setup_transport ()
 
        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")));
+       goto_start_button.set_icon (ArdourIcon::TransportStart);
+       goto_end_button.set_icon (ArdourIcon::TransportEnd);
+       roll_button.set_icon (ArdourIcon::TransportPlay);
+       stop_button.set_icon (ArdourIcon::TransportStop);
+       play_selection_button.set_icon (ArdourIcon::TransportRange);
+       auto_loop_button.set_icon (ArdourIcon::TransportLoop);
+       rec_button.set_icon (ArdourIcon::RecButton);
+       midi_panic_button.set_icon (ArdourIcon::TransportPanic);
 
-       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"));
        stop_button.set_related_action (act);
        act = ActionManager::get_action (X_("Transport"), X_("Roll"));
@@ -317,18 +266,26 @@ ARDOUR_UI::setup_transport ()
        /* CANNOT sigc::bind these to clicked or toggled, must use pressed or released */
 
        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);
+       act = ActionManager::get_action (X_("Main"), X_("cancel-solo"));
+       solo_alert_button.set_related_action (act);
        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, true, false);
-       alert_box.pack_start (auditioning_alert_button, true, false);
-       alert_box.pack_start (feedback_alert_button, true, false);
+       error_alert_button.set_name ("error alert");
+       error_alert_button.signal_button_release_event().connect (sigc::mem_fun(*this,&ARDOUR_UI::error_alert_press), false);
+       act = ActionManager::get_action (X_("Editor"), X_("toggle-log-window"));
+       error_alert_button.set_related_action(act);
+       error_alert_button.set_fallthrough_to_parent(true);
+
+       alert_box.set_homogeneous (true);
+       alert_box.set_spacing (PX_SCALE(2));
+       alert_box.pack_start (solo_alert_button, true, true);
+       alert_box.pack_start (auditioning_alert_button, true, true);
+       alert_box.pack_start (feedback_alert_button, true, true);
 
        /* all transport buttons should be the same size vertically and
-        * horizontally 
+        * horizontally
         */
 
        Glib::RefPtr<SizeGroup> transport_button_size_group = SizeGroup::create (SIZE_GROUP_BOTH);
@@ -340,7 +297,11 @@ ARDOUR_UI::setup_transport ()
        transport_button_size_group->add_widget (roll_button);
        transport_button_size_group->add_widget (stop_button);
 
-       goto_start_button.set_size_request (-1, 40);
+       /* the icon for this has an odd aspect ratio, so fatten up the button */
+       midi_panic_button.set_size_request (PX_SCALE(25), -1);
+       goto_start_button.set_size_request (PX_SCALE(28), PX_SCALE(44));
+       click_button.set_size_request (PX_SCALE(32), PX_SCALE(44));
+
 
        HBox* tbox1 = manage (new HBox);
        HBox* tbox2 = manage (new HBox);
@@ -352,28 +313,33 @@ ARDOUR_UI::setup_transport ()
        Alignment* a1 = manage (new Alignment);
        Alignment* a2 = manage (new Alignment);
 
-       tbox1->set_spacing (2);
-       tbox2->set_spacing (2);
-       tbox->set_spacing (2);
+       tbox1->set_spacing (PX_SCALE(2));
+       tbox2->set_spacing (PX_SCALE(2));
+       tbox->set_spacing (PX_SCALE(2));
+
+       if (!Profile->get_trx()) {
+               tbox1->pack_start (midi_panic_button, true, true, 5);
+               tbox1->pack_start (click_button, true, true, 5);
+       }
 
-       tbox1->pack_start (midi_panic_button, false, false, 5);
-       tbox1->pack_start (click_button, false, false, 5);
-       tbox1->pack_start (goto_start_button, false, false);
-       tbox1->pack_start (goto_end_button, false, false);
-       tbox1->pack_start (auto_loop_button, false, false);
+       tbox1->pack_start (goto_start_button, true, true);
+       tbox1->pack_start (goto_end_button, true, true);
+       tbox1->pack_start (auto_loop_button, true, true);
 
-       tbox2->pack_start (play_selection_button, false, false);
-       tbox2->pack_start (roll_button, false, false);
-       tbox2->pack_start (stop_button, false, false);
-       tbox2->pack_start (rec_button, false, false, 5);
+       if (!Profile->get_trx()) {
+               tbox2->pack_start (play_selection_button, true, true);
+       }
+       tbox2->pack_start (roll_button, true, true);
+       tbox2->pack_start (stop_button, true, true);
+       tbox2->pack_start (rec_button, true, true, 5);
 
-       vbox1->pack_start (*tbox1, false, false);
-       vbox2->pack_start (*tbox2, false, false);
+       vbox1->pack_start (*tbox1, true, true);
+       vbox2->pack_start (*tbox2, true, true);
 
        a1->add (*vbox1);
-       a1->set (0.5, 1.0, 0.0, 0.0);
+       a1->set (0.5, 0.5, 0.0, 1.0);
        a2->add (*vbox2);
-       a2->set (0.5, 1.0, 0.0, 0.0);
+       a2->set (0.5, 0.5, 0.0, 1.0);
 
        tbox->pack_start (*a1, false, false);
        tbox->pack_start (*a2, false, false);
@@ -381,71 +347,101 @@ ARDOUR_UI::setup_transport ()
        HBox* clock_box = manage (new HBox);
 
        clock_box->pack_start (*primary_clock, false, false);
-       if (!ARDOUR::Profile->get_small_screen()) {
+       if (!ARDOUR::Profile->get_small_screen() && !ARDOUR::Profile->get_trx()) {
                clock_box->pack_start (*secondary_clock, false, false);
        }
-       clock_box->set_spacing (3);
+       clock_box->set_spacing (PX_SCALE(3));
 
-       shuttle_box = new ShuttleControl;
+       shuttle_box = manage (new ShuttleControl);
        shuttle_box->show ();
 
        VBox* transport_vbox = manage (new VBox);
        transport_vbox->set_name ("TransportBase");
        transport_vbox->set_border_width (0);
-       transport_vbox->set_spacing (3);
+       transport_vbox->set_spacing (PX_SCALE(3));
        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);
+       if (!Profile->get_trx()) {
+               transport_vbox->pack_start (*shuttle_box, false, false, 0);
+       }
+
+       time_info_box = manage (new TimeInfoBox);
+
+       transport_hbox.pack_start (*transport_vbox, false, true);
 
        /* transport related toggle controls */
 
        VBox* auto_box = manage (new VBox);
        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);
+       auto_box->set_spacing (PX_SCALE(2));
+       auto_box->pack_start (sync_button, true, true);
+       if (!ARDOUR::Profile->get_trx()) {
+               auto_box->pack_start (follow_edits_button, true, true);
+               auto_box->pack_start (auto_return_button, true, true);
+       }
 
-       transport_tearoff_hbox.pack_start (*auto_box, false, false);
-       transport_tearoff_hbox.pack_start (*clock_box, true, true);
+       if (!ARDOUR::Profile->get_trx()) {
+               transport_hbox.pack_start (*auto_box, false, false);
+       }
+       transport_hbox.pack_start (*clock_box, true, true);
 
-       time_info_box = manage (new TimeInfoBox);
-       transport_tearoff_hbox.pack_start (*time_info_box, false, false);
+       if (ARDOUR::Profile->get_trx()) {
+               transport_hbox.pack_start (*auto_box, false, false);
+       }
 
-        if (Profile->get_small_screen()) {
-                transport_tearoff_hbox.pack_start (_editor_transport_box, false, false);
-        }
-       transport_tearoff_hbox.pack_start (alert_box, false, false);
+       if (!ARDOUR::Profile->get_trx()) {
+               transport_hbox.pack_start (*time_info_box, false, false);
+       }
 
-       if (Profile->get_sae()) {
-               Image* img = manage (new Image ((::get_icon (X_("sae")))));
-               transport_tearoff_hbox.pack_end (*img, false, false);
+       if (!ARDOUR::Profile->get_trx()) {
+               transport_hbox.pack_start (alert_box, false, false);
+               transport_hbox.pack_start (meter_box, false, false);
+               transport_hbox.pack_start (editor_meter_peak_display, false, false);
        }
 
-       /* desensitize */
+       Gtk::VBox*   window_button_box = manage (new Gtk::VBox);
 
-       set_transport_sensitivity (false);
+       editor_visibility_button.signal_drag_failed().connect (sigc::bind (sigc::ptr_fun (drag_failed), editor));
+       mixer_visibility_button.signal_drag_failed().connect (sigc::bind (sigc::ptr_fun (drag_failed), mixer));
+       prefs_visibility_button.signal_drag_failed().connect (sigc::bind (sigc::ptr_fun (drag_failed), rc_option_editor));
 
-       XMLNode* tnode = tearoff_settings ("transport");
-       if (tnode) {
-               transport_tearoff->set_state (*tnode);
-       }
-}
+       /* catch context clicks so that we can show a menu on these buttons */
 
-void
-ARDOUR_UI::detach_tearoff (Box* b, Widget* w)
-{
-//     editor->ensure_float (transport_tearoff->tearoff_window());
-       b->remove (*w);
-}
+       editor_visibility_button.signal_button_press_event().connect (sigc::bind (sigc::mem_fun (*this, &ARDOUR_UI::tabbable_visibility_button_press), X_("editor")), false);
+       mixer_visibility_button.signal_button_press_event().connect (sigc::bind (sigc::mem_fun (*this, &ARDOUR_UI::tabbable_visibility_button_press), X_("mixer")), false);
+       prefs_visibility_button.signal_button_press_event().connect (sigc::bind (sigc::mem_fun (*this, &ARDOUR_UI::tabbable_visibility_button_press), X_("preferences")), false);
 
-void
-ARDOUR_UI::reattach_tearoff (Box* b, Widget* w, int32_t n)
-{
-       b->pack_start (*w);
-       b->reorder_child (*w, n);
+       editor_visibility_button.set_related_action (ActionManager::get_action (X_("Common"), X_("change-editor-visibility")));
+       editor_visibility_button.set_name (X_("page switch button"));
+       mixer_visibility_button.set_related_action (ActionManager::get_action (X_("Common"), X_("change-mixer-visibility")));
+       mixer_visibility_button.set_name (X_("page switch button"));
+       prefs_visibility_button.set_related_action (ActionManager::get_action (X_("Common"), X_("change-preferences-visibility")));
+       prefs_visibility_button.set_name (X_("page switch button"));
+
+       Gtkmm2ext::UI::instance()->set_tip (editor_visibility_button,
+                                           string_compose (_("Drag this tab to the desktop to show %1 in its own window\n\n"
+                                                             "To put the window back, use the Window > %1 > Attach menu action"), editor->name()));
+
+       Gtkmm2ext::UI::instance()->set_tip (mixer_visibility_button,
+                                           string_compose (_("Drag this tab to the desktop to show %1 in its own window\n\n"
+                                                             "To put the window back, use the Window > %1 > Attach menu action"), mixer->name()));
+
+       Gtkmm2ext::UI::instance()->set_tip (prefs_visibility_button,
+                                           string_compose (_("Drag this tab to the desktop to show %1 in its own window\n\n"
+                                                             "To put the window back, use the Window > %1 > Attach menu action"), rc_option_editor->name()));
+
+       window_button_box->pack_start (editor_visibility_button, true, false);
+       window_button_box->pack_start (mixer_visibility_button, true, false);
+       window_button_box->pack_start (prefs_visibility_button, true, false);
+
+       transport_hbox.pack_end (*window_button_box, false, false);
+       transport_hbox.pack_end (action_script_table, false, false);
+
+       /* desensitize */
+
+       set_transport_sensitivity (false);
 }
+#undef PX_SCALE
 
 void
 ARDOUR_UI::soloing_changed (bool onoff)
@@ -478,22 +474,29 @@ ARDOUR_UI::audition_alert_press (GdkEventButton*)
 }
 
 bool
-ARDOUR_UI::solo_alert_press (GdkEventButton*)
+ARDOUR_UI::feedback_alert_press (GdkEventButton *)
 {
-       if (_session) {
-               if (_session->soloing()) {
-                       _session->set_solo (_session->get_routes(), false);
-               } else if (_session->listening()) {
-                       _session->set_listen (_session->get_routes(), false);
-               }
-       }
        return true;
 }
 
 bool
-ARDOUR_UI::feedback_alert_press (GdkEventButton *)
+ARDOUR_UI::error_alert_press (GdkEventButton* ev)
 {
-       return true;
+       bool do_toggle = true;
+       if (ev->button == 1) {
+               if (_log_not_acknowledged == LogLevelError) {
+                       // just acknowledge the error, don't hide the log if it's already visible
+                       RefPtr<Action> act = ActionManager::get_action (X_("Editor"), X_("toggle-log-window"));
+                       Glib::RefPtr<ToggleAction> tact = Glib::RefPtr<ToggleAction>::cast_dynamic (act);
+                       if (tact && tact->get_active()) {
+                               do_toggle = false;
+                       }
+               }
+               _log_not_acknowledged = LogLevelNone;
+               error_blink (false); // immediate acknowledge
+       }
+       // maybe fall through to to button toggle
+       return !do_toggle;
 }
 
 void
@@ -569,6 +572,32 @@ ARDOUR_UI::feedback_blink (bool onoff)
        }
 }
 
+void
+ARDOUR_UI::error_blink (bool onoff)
+{
+       switch (_log_not_acknowledged) {
+               case LogLevelError:
+                       // blink
+                       if (onoff) {
+                               error_alert_button.set_custom_led_color(0xff0000ff); // bright red
+                       } else {
+                               error_alert_button.set_custom_led_color(0x880000ff); // dark red
+                       }
+                       break;
+               case LogLevelWarning:
+                       error_alert_button.set_custom_led_color(0xccaa00ff); // yellow
+                       break;
+               case LogLevelInfo:
+                       error_alert_button.set_custom_led_color(0x88cc00ff); // lime green
+                       break;
+               default:
+                       error_alert_button.set_custom_led_color(0x333333ff); // gray
+                       break;
+       }
+}
+
+
+
 void
 ARDOUR_UI::set_transport_sensitivity (bool yn)
 {
@@ -582,30 +611,31 @@ ARDOUR_UI::editor_realized ()
        boost::function<void (string)> pc (boost::bind (&ARDOUR_UI::parameter_changed, this, _1));
        Config->map_parameters (pc);
 
-       reset_dpi ();
+       UIConfiguration::instance().reset_dpi ();
 }
 
 void
-ARDOUR_UI::update_tearoff_visibility ()
+ARDOUR_UI::maximise_editing_space ()
 {
        if (editor) {
-               editor->update_tearoff_visibility ();
+               editor->maximise_editing_space ();
        }
 }
 
 void
-ARDOUR_UI::maximise_editing_space ()
+ARDOUR_UI::restore_editing_space ()
 {
        if (editor) {
-               editor->maximise_editing_space ();
+               editor->restore_editing_space ();
        }
 }
 
 void
-ARDOUR_UI::restore_editing_space ()
+ARDOUR_UI::show_ui_prefs ()
 {
-       if (editor) {
-               editor->restore_editing_space ();
+       if (rc_option_editor) {
+               show_tabbable (rc_option_editor);
+               rc_option_editor->set_current_page (_("GUI"));
        }
 }
 
@@ -617,18 +647,13 @@ ARDOUR_UI::click_button_clicked (GdkEventButton* ev)
                return false;
        }
 
-       RefPtr<Action> act = ActionManager::get_action (X_("Common"), X_("ToggleRCOptionsEditor"));
-       assert (act);
-
-       RefPtr<ToggleAction> tact = RefPtr<ToggleAction>::cast_dynamic (act);
-       tact->set_active ();
-
-       rc_option_editor->set_current_page (_("Misc"));
+       show_tabbable (rc_option_editor);
+       rc_option_editor->set_current_page (_("Misc/Click"));
        return true;
 }
 
 void
-ARDOUR_UI::toggle_always_play_range ()
+ARDOUR_UI::toggle_follow_edits ()
 {
        RefPtr<Action> act = ActionManager::get_action (X_("Transport"), X_("ToggleFollowEdits"));
        assert (act);
@@ -636,7 +661,33 @@ ARDOUR_UI::toggle_always_play_range ()
        RefPtr<ToggleAction> tact = RefPtr<ToggleAction>::cast_dynamic (act);
        assert (tact);
 
-       Config->set_always_play_range (tact->get_active ());
+       UIConfiguration::instance().set_follow_edits (tact->get_active ());
 }
 
-       
+void
+ARDOUR_UI::update_title ()
+{
+       if (_session) {
+               bool dirty = _session->dirty();
+
+               string session_name;
+
+               if (_session->snap_name() != _session->name()) {
+                       session_name = _session->snap_name();
+               } else {
+                       session_name = _session->name();
+               }
+
+               if (dirty) {
+                       session_name = "*" + session_name;
+               }
+
+               WindowTitle title (session_name);
+               title += Glib::get_application_name();
+               _main_window.set_title (title.get_string());
+       } else {
+               WindowTitle title (Glib::get_application_name());
+               _main_window.set_title (title.get_string());
+       }
+
+}