initial (incomplete) framework for DiskIOPoint manipulation
[ardour.git] / gtk2_ardour / ardour_ui2.cc
index 4f9b48a2b3d570313b5de6fc85685006e8251efd..a7794faf783972727e1b0499a586d726f7b413bd 100644 (file)
 #include <cmath>
 
 #include <sigc++/bind.h>
+#include <gtkmm/settings.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/window_title.h"
 
 #include "ardour/profile.h"
 #include "main_clock.h"
 #include "mixer_ui.h"
 #include "utils.h"
-#include "theme_manager.h"
+#include "time_info_box.h"
 #include "midi_tracer.h"
-#include "mini_timeline.h"
-#include "shuttle_control.h"
 #include "global_port_matrix.h"
 #include "location_ui.h"
 #include "rc_option_editor.h"
@@ -66,11 +64,11 @@ using namespace std;
 using namespace ARDOUR;
 using namespace PBD;
 using namespace Gtkmm2ext;
+using namespace ArdourWidgets;
 using namespace Gtk;
 using namespace Glib;
 using namespace ARDOUR_UI_UTILS;
 
-
 void
 ARDOUR_UI::setup_tooltips ()
 {
@@ -85,9 +83,9 @@ 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 (auto_input_button, _("Be sensible about input monitoring"));
-       set_tip (click_button, _("Enable/Disable audio click"));
+       set_tip (follow_edits_button, _("Playhead follows Range tool clicks, and Range selections"));
+       set_tip (auto_input_button, _("Track Input Monitoring automatically follows transport state"));
+       parameter_changed("click-gain");
        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 (feedback_alert_button, _("When active, there is a feedback loop."));
@@ -179,69 +177,94 @@ bool drag_failed (const Glib::RefPtr<Gdk::DragContext>& context, DragResult resu
        return false;
 }
 
-bool
-ARDOUR_UI::spacer_expose (GdkEventExpose* ev)
-{
-       cairo_t *cr = gdk_cairo_create (ev->window);
-    
-       cairo_rectangle (cr, 1, 0, ev->area.width-2, ev->area.height);
-       cairo_set_source_rgb (cr, 0,0,0);
-       cairo_fill (cr);
-       
-       return true;
-}
-
-bool
-ARDOUR_UI::transport_expose (GdkEventExpose* ev)
+void
+ARDOUR_UI::repack_transport_hbox ()
 {
-return false;
-       int x0, y0;
-       Gtk::Widget* window_parent;
-       Glib::RefPtr<Gdk::Window> win = Gtkmm2ext::window_to_draw_on (transport_table, &window_parent);
-       Glib::RefPtr<Gtk::Style> style = transport_table.get_style();
-       if (!win || !style) {
-               return false;
+       if (time_info_box) {
+               if (time_info_box->get_parent()) {
+                       transport_hbox.remove (*time_info_box);
+               }
+               if (UIConfiguration::instance().get_show_toolbar_selclock ()) {
+                       transport_hbox.pack_start (*time_info_box, false, false);
+                       time_info_box->show();
+               }
        }
 
-       Cairo::RefPtr<Cairo::Context> cr = transport_table.get_window()->create_cairo_context ();
-
-       cr->rectangle (ev->area.x, ev->area.y, ev->area.width, ev->area.height);
-       cr->clip ();
-
-       transport_table.translate_coordinates (*window_parent, 0, 0, x0, y0);
-
-       cr->rectangle (x0, y0, transport_table.get_width(), transport_table.get_height());
-       Gdk::Color bg (style->get_bg (transport_table.get_state()));
-       cr->set_source_rgb (bg.get_red_p(), bg.get_green_p(), bg.get_blue_p());
-       cr->fill ();
-
-       static const int xmargin = 2;
-       static const int ymargin = 1;
+       if (mini_timeline.get_parent()) {
+               transport_hbox.remove (mini_timeline);
+       }
+       if (UIConfiguration::instance().get_show_mini_timeline ()) {
+               transport_hbox.pack_start (mini_timeline, true, true);
+               mini_timeline.show();
+       }
 
-       /* draw box around record-options */
-       int xx, ww, hh, uu;
+       if (editor_meter) {
+               if (meter_box.get_parent()) {
+                       transport_hbox.remove (meter_box);
+                       transport_hbox.remove (editor_meter_peak_display);
+               }
 
-       punch_label.translate_coordinates (transport_table, -xmargin, 0, xx, uu); // left
-       punch_out_button.translate_coordinates (transport_table, xmargin, 0, ww, uu); // right
-       ww += punch_out_button.get_width () - xx; // width
-       hh = transport_table.get_height() - 1;
+               if (UIConfiguration::instance().get_show_editor_meter()) {
+                       transport_hbox.pack_end (editor_meter_peak_display, false, false);
+                       transport_hbox.pack_end (meter_box, false, false);
+                       meter_box.show();
+                       editor_meter_peak_display.show();
+               }
+       }
 
-       Gtkmm2ext::rounded_rectangle (cr->cobj(), x0 + xx - 0.5, y0 + 0.5, ww + 1, hh, 6);
-       cr->set_source_rgb (0, 0, 0);
-       cr->set_line_width (1.0);
-       cr->stroke ();
+       bool show_mon = UIConfiguration::instance().get_show_toolbar_monitoring ();
+       if (show_mon) {
+               monitor_in_button.show ();
+               monitor_disk_button.show ();
+               auto_input_button.show ();
+               monitoring_spacer.show ();
+       } else {
+               monitor_in_button.hide ();
+               monitor_disk_button.hide ();
+               auto_input_button.hide ();
+               monitoring_spacer.hide ();
+       }
 
-       /* line to rec-enable */
-       int rx;
-       rec_button.translate_coordinates (transport_table, -xmargin, 0, rx, uu); // top
-       int dx = rx + rec_button.get_width() - xx;
+       bool show_rec = UIConfiguration::instance().get_show_toolbar_recpunch ();
+       if (show_rec) {
+               punch_label.show ();
+               layered_label.show ();
+               punch_in_button.show ();
+               punch_out_button.show ();
+               layered_button.show ();
+               recpunch_spacer.show ();
+       } else {
+               punch_label.hide ();
+               layered_label.hide ();
+               punch_in_button.hide ();
+               punch_out_button.hide ();
+               layered_button.hide ();
+               recpunch_spacer.hide ();
+       }
 
-       cr->move_to (x0 + xx, 1.5 + y0 + ymargin + round (punch_in_button.get_height () * .5));
-       cr->rel_line_to (dx, 0);
-       cr->set_line_width (2.0);
-       cr->stroke ();
+}
 
-       return false;
+void
+ARDOUR_UI::update_clock_visibility ()
+{
+       if (ARDOUR::Profile->get_small_screen()) {
+               return;
+       }
+       if (UIConfiguration::instance().get_show_secondary_clock ()) {
+               secondary_clock->show();
+               secondary_clock->left_btn()->show();
+               secondary_clock->right_btn()->show();
+               if (secondary_clock_spacer) {
+                       secondary_clock_spacer->show();
+               }
+       } else {
+               secondary_clock->hide();
+               secondary_clock->left_btn()->hide();
+               secondary_clock->right_btn()->hide();
+               if (secondary_clock_spacer) {
+                       secondary_clock_spacer->hide();
+               }
+       }
 }
 
 void
@@ -253,6 +276,7 @@ ARDOUR_UI::setup_transport ()
        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);
+       click_button.signal_scroll_event().connect (sigc::mem_fun (*this, &ARDOUR_UI::click_button_scroll), false);
 
        act = ActionManager::get_action (X_("Transport"), X_("Stop"));
        stop_button.set_related_action (act);
@@ -271,9 +295,11 @@ ARDOUR_UI::setup_transport ()
        act = ActionManager::get_action (X_("MIDI"), X_("panic"));
        midi_panic_button.set_related_action (act);
        act = ActionManager::get_action (X_("Transport"), X_("ToggleExternalSync"));
+
        sync_button.set_related_action (act);
+       sync_button.signal_button_press_event().connect (sigc::mem_fun (*this, &ARDOUR_UI::sync_button_clicked), false);
 
-       sync_button.set_sizing_text (_("Internal")); // longest of sync_source_to_string()
+       sync_button.set_sizing_text (S_("LogestSync|M-Clk"));
 
        /* CANNOT sigc::bind these to clicked or toggled, must use pressed or released */
        act = ActionManager::get_action (X_("Main"), X_("cancel-solo"));
@@ -302,6 +328,11 @@ ARDOUR_UI::setup_transport ()
        act = ActionManager::get_action ("Transport", "TogglePunchOut");
        punch_out_button.set_related_action (act);
 
+       act = ActionManager::get_action ("Transport", "SessionMonitorIn");
+       monitor_in_button.set_related_action (act);
+       act = ActionManager::get_action ("Transport", "SessionMonitorDisk");
+       monitor_disk_button.set_related_action (act);
+
        /* connect signals */
        ARDOUR_UI::Clock.connect (sigc::mem_fun (primary_clock, &AudioClock::set));
        ARDOUR_UI::Clock.connect (sigc::mem_fun (secondary_clock, &AudioClock::set));
@@ -324,7 +355,6 @@ ARDOUR_UI::setup_transport ()
 
        auto_return_button.set_name ("transport option button");
        follow_edits_button.set_name ("transport option button");
-       auto_input_button.set_name ("transport option button");
 
        solo_alert_button.set_name ("rude solo");
        auditioning_alert_button.set_name ("rude audition");
@@ -347,17 +377,24 @@ ARDOUR_UI::setup_transport ()
        punch_out_button.set_name ("punch button");
        layered_button.set_name (("layered button"));
 
+       monitor_in_button.set_name ("monitor button");
+       monitor_disk_button.set_name ("monitor button");
+       auto_input_button.set_name ("transport option button");
+
        click_button.set_name ("transport button");
        sync_button.set_name ("transport active option button");
 
        /* and widget text */
        auto_return_button.set_text(_("Auto Return"));
-       follow_edits_button.set_text(_("Follow Edits"));
-       //auto_input_button.set_text (_("Auto Input"));
+       follow_edits_button.set_text(_("Follow Range"));
        punch_in_button.set_text (_("In"));
        punch_out_button.set_text (_("Out"));
        layered_button.set_text (_("Non-Layered"));
 
+       monitor_in_button.set_text (_("All In"));
+       monitor_disk_button.set_text (_("All Disk"));
+       auto_input_button.set_text (_("Auto-Input"));
+
        punch_label.set_text (_("Punch:"));
        layered_label.set_text (_("Rec:"));
 
@@ -365,19 +402,22 @@ ARDOUR_UI::setup_transport ()
 
        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()));
+                                                             "To re-attach the window, 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()));
+                                                             "To re-attach the window, 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()));
+                                                             "To re-attach the window, use the Window > %1 > Attach menu action"), rc_option_editor->name()));
 
        Gtkmm2ext::UI::instance()->set_tip (punch_in_button, _("Start recording at auto-punch start"));
        Gtkmm2ext::UI::instance()->set_tip (punch_out_button, _("Stop recording at auto-punch end"));
 
+       Gtkmm2ext::UI::instance()->set_tip (monitor_in_button, _("Force all tracks to monitor Input, unless they are explicitly set to monitor Disk"));
+       Gtkmm2ext::UI::instance()->set_tip (monitor_disk_button, _("Force all tracks to monitor Disk playback, unless they are explicitly set to Input"));
+
        /* setup icons */
 
        click_button.set_icon (ArdourIcon::TransportMetronom);
@@ -405,8 +445,9 @@ ARDOUR_UI::setup_transport ()
        punch_button_size_group->add_widget (punch_in_button);
        punch_button_size_group->add_widget (punch_out_button);
 
-       shuttle_box = manage (new ShuttleControl);
-       mini_timeline = manage (new MiniTimeline);
+       Glib::RefPtr<SizeGroup> monitor_button_size_group = SizeGroup::create (Gtk::SIZE_GROUP_HORIZONTAL);
+       monitor_button_size_group->add_widget (monitor_in_button);
+       monitor_button_size_group->add_widget (monitor_disk_button);
 
        /* and now the layout... */
 
@@ -414,11 +455,21 @@ ARDOUR_UI::setup_transport ()
        transport_table.set_spacings (0);
        transport_table.set_row_spacings (4);
        transport_table.set_border_width (2);
-       transport_frame.add (transport_table);
-       transport_frame.set_name ("BaseFrame");
+
+       transport_frame.set_name ("TransportFrame");
        transport_frame.set_shadow_type (Gtk::SHADOW_NONE);
 
-       transport_table.signal_expose_event().connect (sigc::mem_fun (*this, &ARDOUR_UI::transport_expose), false);
+       /* An event box to hold the table. We use this because we want specific
+          control over the background color, and without this event box,
+          nothing inside the transport_frame actually draws a background. We
+          would therefore end up seeing the background of the parent widget,
+          which is probably some default color. Adding the EventBox adds a
+          widget that will draw the background, using a style based on
+          the parent, "TransportFrame".
+       */
+       Gtk::EventBox* ebox = manage (new Gtk::EventBox);
+       transport_frame.add (*ebox);
+       ebox->add (transport_table);
 
        /* transport controls sub-group */
        click_button.set_size_request (PX_SCALE(20), PX_SCALE(20));
@@ -442,9 +493,9 @@ ARDOUR_UI::setup_transport ()
        alert_box->set_homogeneous (true);
        alert_box->set_spacing (1);
        alert_box->set_border_width (0);
-       alert_box->pack_start (solo_alert_button, true, false, 0);
-       alert_box->pack_start (auditioning_alert_button, true, false, 0);
-       alert_box->pack_start (feedback_alert_button, true, false, 0);
+       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);
 
        /* clock button size groups */
        Glib::RefPtr<SizeGroup> button_height_size_group = SizeGroup::create (Gtk::SIZE_GROUP_VERTICAL);
@@ -456,90 +507,123 @@ ARDOUR_UI::setup_transport ()
 
        button_height_size_group->add_widget (stop_button);
 //     button_height_size_group->add_widget (sync_button);
-//     button_height_size_group->add_widget (layered_button);
        button_height_size_group->add_widget (auto_return_button);
+
+       //tab selections
        button_height_size_group->add_widget (editor_visibility_button);
        button_height_size_group->add_widget (mixer_visibility_button);
 
-       Glib::RefPtr<SizeGroup> clock1_size_group = SizeGroup::create (SIZE_GROUP_BOTH);
+       //punch section
+       button_height_size_group->add_widget (punch_in_button);
+       button_height_size_group->add_widget (punch_out_button);
+       button_height_size_group->add_widget (layered_button);
+
+       //input monitoring section
+       button_height_size_group->add_widget (monitor_in_button);
+       button_height_size_group->add_widget (monitor_disk_button);
+       button_height_size_group->add_widget (auto_input_button);
+
+       Glib::RefPtr<SizeGroup> clock1_size_group = SizeGroup::create (SIZE_GROUP_HORIZONTAL);
        clock1_size_group->add_widget (*primary_clock->left_btn());
        clock1_size_group->add_widget (*primary_clock->right_btn());
 
-       Glib::RefPtr<SizeGroup> clock2_size_group = SizeGroup::create (SIZE_GROUP_BOTH);
+       Glib::RefPtr<SizeGroup> clock2_size_group = SizeGroup::create (SIZE_GROUP_HORIZONTAL);
        clock2_size_group->add_widget (*secondary_clock->left_btn());
        clock2_size_group->add_widget (*secondary_clock->right_btn());
 
+       /* sub-layout for Sync | Shuttle (grow) */
+       HBox* ssbox = manage (new HBox);
+       ssbox->set_spacing (PX_SCALE(2));
+       ssbox->pack_start (sync_button, false, false, 0);
+       ssbox->pack_start (shuttle_box, true, true, 0);
+       ssbox->pack_start (*shuttle_box.info_button(), false, false, 0);
+
+
        /* and the main table layout */
+       int vpadding = 1;
+       int hpadding = 2;
+       int col = 0;
+#define TCOL col, col + 1
 
-       transport_table.attach (*tbox, 0, 2, 0, 1 , SHRINK, SHRINK, 0, 0);
-       transport_table.attach (sync_button, 0, 1, 1, 2 , FILL, SHRINK, 0, 0);
-       transport_table.attach (*shuttle_box, 1, 2, 1, 2 , FILL, SHRINK, 3, 0);
+       transport_table.attach (*tbox, TCOL, 0, 1 , SHRINK, SHRINK, 0, 0);
+       transport_table.attach (*ssbox, TCOL, 1, 2 , FILL, SHRINK, 0, 0);
+       ++col;
 
-       //spacer
-       EventBox *spacer = manage (new EventBox ()); spacer->set_size_request(3, 42);
-       spacer->signal_expose_event().connect (sigc::mem_fun (*this, &ARDOUR_UI::spacer_expose), false);  spacer->show();
-       transport_table.attach (*spacer, 2, 3, 0, 2 , FILL, SHRINK, 3, 0);
-       
-       transport_table.attach (punch_label, 3, 4, 0, 1 , FILL, SHRINK, 3, 0);
-       transport_table.attach (layered_label, 3, 4, 1, 2 , FILL, SHRINK, 3, 0);
+       transport_table.attach (*(manage (new ArdourVSpacer ())), TCOL, 0, 2 , SHRINK, EXPAND|FILL, 3, 0);
+       ++col;
 
-       transport_table.attach (punch_in_button, 4, 5, 0, 1 , FILL, SHRINK, 0, 2);
-       transport_table.attach (punch_out_button, 6, 7, 0, 1 , FILL, SHRINK, 0, 2);
-       transport_table.attach (layered_button, 4, 7, 1, 2 , FILL, SHRINK, 0, 2);
+       transport_table.attach (punch_label, TCOL, 0, 1 , FILL, SHRINK, 3, 0);
+       transport_table.attach (layered_label, TCOL, 1, 2 , FILL, SHRINK, 3, 0);
+       ++col;
 
-       //spacer
-       spacer = manage (new EventBox ()); spacer->set_size_request(3, 42);
-       spacer->signal_expose_event().connect (sigc::mem_fun (*this, &ARDOUR_UI::spacer_expose), false);  spacer->show();
-       transport_table.attach (*spacer, 7, 8, 0, 2 , FILL, SHRINK, 3, 0);
+       transport_table.attach (punch_in_button,  col,      col + 1, 0, 1 , FILL, SHRINK, hpadding, vpadding);
+       transport_table.attach (punch_space,      col + 1,  col + 2, 0, 1 , FILL, SHRINK, 0, vpadding);
+       transport_table.attach (punch_out_button, col + 2,  col + 3, 0, 1 , FILL, SHRINK, hpadding, vpadding);
+       transport_table.attach (layered_button,   col,      col + 3, 1, 2 , FILL, SHRINK, hpadding, vpadding);
+       col += 3;
 
-       transport_table.attach (follow_edits_button, 8, 9, 0, 1 , FILL, SHRINK, 2, 0);
-       transport_table.attach (auto_return_button, 8, 9, 1, 2 , FILL, SHRINK, 2, 0);
+       transport_table.attach (recpunch_spacer, TCOL, 0, 2 , SHRINK, EXPAND|FILL, 3, 0);
+       ++col;
 
-       //spacer
-       spacer = manage (new EventBox ()); spacer->set_size_request(3, 42);
-       spacer->signal_expose_event().connect (sigc::mem_fun (*this, &ARDOUR_UI::spacer_expose), false);  spacer->show();
-       transport_table.attach (*spacer, 9, 10, 0, 2 , FILL, SHRINK, 3, 0);
+       transport_table.attach (auto_input_button,   col,     col + 3, 0, 1 , FILL, SHRINK, hpadding, vpadding);
+       transport_table.attach (monitor_in_button,   col,     col + 1, 1, 2 , FILL, SHRINK, hpadding, vpadding);
+       transport_table.attach (mon_space,           col + 1, col + 2, 1, 2 , FILL, SHRINK, 2, vpadding);
+       transport_table.attach (monitor_disk_button, col + 2, col + 3, 1, 2 , FILL, SHRINK, hpadding, vpadding);
+       col += 3;
 
-       transport_table.attach (*primary_clock, 10, 12, 0, 1 , FILL, SHRINK, 2, 0);
-       transport_table.attach (*primary_clock->left_btn(), 10, 11, 1, 2 , FILL, SHRINK, 2, 0);
-       transport_table.attach (*primary_clock->right_btn(), 11, 12, 1, 2 , FILL, SHRINK, 2, 0);
+       transport_table.attach (monitoring_spacer, TCOL, 0, 2 , SHRINK, EXPAND|FILL, 3, 0);
+       ++col;
 
-       //spacer
-       spacer = manage (new EventBox ()); spacer->set_size_request(3, 42);
-       spacer->signal_expose_event().connect (sigc::mem_fun (*this, &ARDOUR_UI::spacer_expose), false);  spacer->show();
-       transport_table.attach (*spacer, 12, 13, 0, 2 , FILL, SHRINK, 3, 0);
+       transport_table.attach (follow_edits_button, TCOL, 0, 1 , FILL, SHRINK, hpadding, vpadding);
+       transport_table.attach (auto_return_button,  TCOL, 1, 2 , FILL, SHRINK, hpadding, vpadding);
+       ++col;
 
-       if (!ARDOUR::Profile->get_small_screen()) {
-               transport_table.attach (*secondary_clock, 13, 15, 0, 1 , FILL, SHRINK, 2, 0);
-               transport_table.attach (*secondary_clock->left_btn(), 13, 14, 1, 2 , FILL, SHRINK, 2, 0);
-               transport_table.attach (*secondary_clock->right_btn(), 14, 15, 1, 2 , FILL, SHRINK, 2, 0);
-       }
+       transport_table.attach (*(manage (new ArdourVSpacer ())), TCOL, 0, 2 , SHRINK, EXPAND|FILL, 3, 0);
+       ++col;
 
-       //spacer
-       spacer = manage (new EventBox ()); spacer->set_size_request(3, 42);
-       spacer->signal_expose_event().connect (sigc::mem_fun (*this, &ARDOUR_UI::spacer_expose), false);  spacer->show();
-       transport_table.attach (*spacer, 15, 16, 0, 2 , FILL, SHRINK, 3, 0);
+       transport_table.attach (*primary_clock,              col,     col + 2, 0, 1 , FILL, SHRINK, hpadding, 0);
+       transport_table.attach (*primary_clock->left_btn(),  col,     col + 1, 1, 2 , FILL, SHRINK, hpadding, 0);
+       transport_table.attach (*primary_clock->right_btn(), col + 1, col + 2, 1, 2 , FILL, SHRINK, hpadding, 0);
+       col += 2;
 
-       transport_table.attach (*alert_box, 16, 17, 0, 2, SHRINK, EXPAND|FILL, 2, 0);
+       transport_table.attach (*(manage (new ArdourVSpacer ())), TCOL, 0, 2 , SHRINK, EXPAND|FILL, 3, 0);
+       ++col;
 
-       //spacer
-       spacer = manage (new EventBox ()); spacer->set_size_request(3, 42);
-       spacer->signal_expose_event().connect (sigc::mem_fun (*this, &ARDOUR_UI::spacer_expose), false);  spacer->show();
-       transport_table.attach (*spacer, 17, 18, 0, 2 , FILL, SHRINK, 3, 0);
+       if (!ARDOUR::Profile->get_small_screen()) {
+               transport_table.attach (*secondary_clock,              col,     col + 2, 0, 1 , FILL, SHRINK, hpadding, 0);
+               transport_table.attach (*secondary_clock->left_btn(),  col,     col + 1, 1, 2 , FILL, SHRINK, hpadding, 0);
+               transport_table.attach (*secondary_clock->right_btn(), col + 1, col + 2, 1, 2 , FILL, SHRINK, hpadding, 0);
+               secondary_clock->set_no_show_all (true);
+               secondary_clock->left_btn()->set_no_show_all (true);
+               secondary_clock->right_btn()->set_no_show_all (true);
+               col += 2;
+
+               secondary_clock_spacer = manage (new ArdourVSpacer ());
+               transport_table.attach (*secondary_clock_spacer, TCOL, 0, 2 , SHRINK, EXPAND|FILL, 3, 0);
+               ++col;
+       }
 
+       transport_table.attach (*alert_box, TCOL, 0, 2, SHRINK, EXPAND|FILL, hpadding, 0);
+       ++col;
 
-       /* editor-meter is in transport_hbox */
-       transport_hbox.set_spacing (PX_SCALE(1));
-       transport_table.attach (transport_hbox, 18, 19, 0, 2, SHRINK, EXPAND|FILL, 2, 0);
+       transport_table.attach (*(manage (new ArdourVSpacer ())), TCOL, 0, 2 , SHRINK, EXPAND|FILL, 3, 0);
+       ++col;
 
-       transport_table.attach (*mini_timeline, 19, 20, 0, 2, EXPAND|FILL, EXPAND|FILL, 1, 0);
+       /* editor-meter, mini-timeline and selection clock are options in the transport_hbox */
+       transport_hbox.set_spacing (3);
+       transport_table.attach (transport_hbox, TCOL, 0, 2, EXPAND|FILL, EXPAND|FILL, hpadding, 0);
+       ++col;
 
        /* lua script action buttons */
-       transport_table.attach (action_script_table, 20, 21, 0, 2, SHRINK, EXPAND|FILL, 1, 0);
+       transport_table.attach (action_script_table, TCOL, 0, 2, SHRINK, EXPAND|FILL, 1, 0);
+       ++col;
 
-       transport_table.attach (editor_visibility_button, 21, 22, 0, 1 , FILL, SHRINK, 2, 0);
-       transport_table.attach (mixer_visibility_button, 21, 22, 1, 2 , FILL, SHRINK, 2, 0);
+       transport_table.attach (editor_visibility_button, TCOL, 0, 1 , FILL, SHRINK, hpadding, vpadding);
+       transport_table.attach (mixer_visibility_button,  TCOL, 1, 2 , FILL, SHRINK, hpadding, vpadding);
+       ++col;
 
+       repack_transport_hbox ();
+       update_clock_visibility ();
        /* desensitize */
 
        feedback_alert_button.set_sensitive (false);
@@ -551,6 +635,7 @@ ARDOUR_UI::setup_transport ()
        set_transport_sensitivity (false);
 }
 #undef PX_SCALE
+#undef TCOL
 
 void
 ARDOUR_UI::soloing_changed (bool onoff)
@@ -710,13 +795,21 @@ ARDOUR_UI::error_blink (bool onoff)
        }
 }
 
-
+void
+ARDOUR_UI::set_loop_sensitivity ()
+{
+       if (!_session || _session->config.get_external_sync()) {
+               auto_loop_button.set_sensitive (false);
+       } else {
+               auto_loop_button.set_sensitive (_session && _session->locations()->auto_loop_location());
+       }
+}
 
 void
 ARDOUR_UI::set_transport_sensitivity (bool yn)
 {
        ActionManager::set_sensitive (ActionManager::transport_sensitive_actions, yn);
-       shuttle_box->set_sensitive (yn);
+       shuttle_box.set_sensitive (yn);
 }
 
 void
@@ -749,7 +842,7 @@ ARDOUR_UI::show_ui_prefs ()
 {
        if (rc_option_editor) {
                show_tabbable (rc_option_editor);
-               rc_option_editor->set_current_page (_("GUI"));
+               rc_option_editor->set_current_page (_("Appearance"));
        }
 }
 
@@ -762,7 +855,43 @@ ARDOUR_UI::click_button_clicked (GdkEventButton* ev)
        }
 
        show_tabbable (rc_option_editor);
-       rc_option_editor->set_current_page (_("Misc/Click"));
+       rc_option_editor->set_current_page (_("Metronome"));
+       return true;
+}
+
+bool
+ARDOUR_UI::click_button_scroll (GdkEventScroll* ev)
+{
+       gain_t gain = Config->get_click_gain();
+       float gain_db = accurate_coefficient_to_dB (gain);
+
+       switch (ev->direction) {
+               case GDK_SCROLL_UP:
+               case GDK_SCROLL_LEFT:
+                       gain_db += 1;
+                       break;
+               case GDK_SCROLL_DOWN:
+               case GDK_SCROLL_RIGHT:
+                       gain_db -= 1;
+                       break;
+       }
+       gain_db = std::max (-60.f, gain_db);
+       gain = dB_to_coefficient (gain_db);
+       gain = std::min (gain, Config->get_max_gain());
+       Config->set_click_gain (gain);
+       return true;
+}
+
+bool
+ARDOUR_UI::sync_button_clicked (GdkEventButton* ev)
+{
+       if (ev->button != 3) {
+               /* this handler is just for button-3 clicks */
+               return false;
+       }
+
+       show_tabbable (rc_option_editor);
+       rc_option_editor->set_current_page (_("Sync"));
        return true;
 }