Reinstate spacer between Punch In,Out.
[ardour.git] / gtk2_ardour / ardour_ui2.cc
index 100e0235a38bef1c7e377187c8928346e7af1273..0d5a52ba13478bc4b1f282191d3f033c1b3f756f 100644 (file)
@@ -29,6 +29,8 @@
 #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/window_title.h"
 
 #include "ardour/profile.h"
 #include "ardour/session.h"
 #include "ardour/types.h"
 
+#include "ardour_spacer.h"
 #include "ardour_ui.h"
 #include "keyboard.h"
 #include "public_editor.h"
 #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"
+#include "mini_timeline.h"
 #include "shuttle_control.h"
 #include "global_port_matrix.h"
 #include "location_ui.h"
 #include "rc_option_editor.h"
-#include "time_info_box.h"
 
-#include "i18n.h"
+#include "pbd/i18n.h"
 
 using namespace std;
 using namespace ARDOUR;
@@ -68,170 +71,11 @@ using namespace Gtk;
 using namespace Glib;
 using namespace ARDOUR_UI_UTILS;
 
-
-static GtkNotebook*
-tab_window_root_drop (GtkNotebook* src,
-                     GtkWidget* w,
-                     gint x,
-                     gint y,
-                     gpointer user_data)
-{
-       return ARDOUR_UI::instance()->tab_window_root_drop (src, w, x, y, user_data);
-}
-
-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
-       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_label.show ();
-       status_bar_event_box->show ();
-
-       status_bar_event_box->signal_button_press_event().connect (mem_fun (*this, &ARDOUR_UI::status_bar_button_press));
-
-       status_bar_hpacker.pack_start (*status_bar_event_box, true, true, 6);
-       status_bar_hpacker.pack_start (menu_bar_base, false, false, 2);
-#else
-       top_packer.pack_start (menu_bar_base, false, false);
-#endif
-
-       main_vpacker.pack_start (top_packer, false, false);
-
-       /* now add the transport frame to the top of main window */
-       
-       main_vpacker.pack_start (transport_frame, false, false);
-       main_vpacker.pack_start (_tabs, true, true);
-
-#ifdef TOP_MENUBAR
-       main_vpacker.pack_start (status_bar_hpacker, false, false);
-#endif
-
-       setup_transport();
-       build_menu_bar ();
-       setup_tooltips ();
-
-       /* pack the main vpacker into the main window and show everything
-        */
-       
-       _main_window.add (main_vpacker);
-       transport_frame.show_all ();
-
-       const XMLNode* mnode = main_window_settings ();
-
-       if (mnode) {
-               const XMLProperty* prop;
-               gint x = -1;
-               gint y = -1;
-               gint w = -1;
-               gint h = -1;
-
-               if ((prop = mnode->property (X_("x"))) != 0) {
-                       x = atoi (prop->value());
-               }
-
-               if ((prop = mnode->property (X_("y"))) != 0) {
-                       y = atoi (prop->value());
-               } 
-
-               if ((prop = mnode->property (X_("w"))) != 0) {
-                       w = atoi (prop->value());
-               } 
-               
-               if ((prop = mnode->property (X_("h"))) != 0) {
-                       h = atoi (prop->value());
-               }
-
-               if (x >= 0 && y >= 0 && w >= 0 && h >= 0) {
-                       _main_window.set_position (Gtk::WIN_POS_NONE);
-               }
-               
-               if (x >= 0 && y >= 0) {
-                       _main_window.move (x, y);
-               }
-               
-               if (w > 0 && h > 0) {
-                       _main_window.set_default_size (w, h);
-               }
-       }
-       
-       _main_window.show_all ();
-       setup_toplevel_window (_main_window, "", this);
-       
-       rc_option_editor = new RCOptionEditor;
-       rc_option_editor->add_to_notebook (_tabs, _("Preferences"));
-       rc_option_editor->contents().show_all ();
-       
-       _tabs.signal_switch_page().connect (sigc::mem_fun (*this, &ARDOUR_UI::tabs_switch));
-       _tabs.signal_page_removed().connect (sigc::mem_fun (*this, &ARDOUR_UI::tabs_page_removed));
-       _tabs.signal_page_added().connect (sigc::mem_fun (*this, &ARDOUR_UI::tabs_page_added));
-
-       /* It would be nice if Gtkmm had wrapped this rather than just
-        * deprecating the old set_window_creation_hook() method, but oh well...
-        */
-       g_signal_connect (_tabs.gobj(), "create-window", (GCallback) ::tab_window_root_drop, this);
-
-       return 0;
-}
-
-void
-ARDOUR_UI::tabs_page_removed (Gtk::Widget*, guint)
-{
-       if (_tabs.get_n_pages() == 1) {
-               _tabs.set_show_tabs (false);
-       } else {
-               _tabs.set_show_tabs (true);
-       }
-}
-
-void
-ARDOUR_UI::tabs_page_added (Gtk::Widget*, guint)
-{
-       if (_tabs.get_n_pages() == 1) {
-               _tabs.set_show_tabs (false);
-       } else {
-               _tabs.set_show_tabs (true);
-       }
-}
-
-void
-ARDOUR_UI::tabs_switch (GtkNotebookPage*, guint page_number)
-{
-       if (page_number == 2) {
-               if (!rc_option_editor) {
-                       rc_option_editor = new RCOptionEditor;
-                       rc_option_editor_placeholder.pack_start (*rc_option_editor, true, true);
-                       rc_option_editor_placeholder.show_all ();
-               }
-       }
-}
-
 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"));
@@ -325,71 +169,78 @@ ARDOUR_UI::tearoff_settings (const char* name) const
 
 #define PX_SCALE(px) std::max((float)px, rintf((float)px * UIConfiguration::instance().get_ui_scale()))
 
-void
-ARDOUR_UI::setup_transport ()
+static
+bool drag_failed (const Glib::RefPtr<Gdk::DragContext>& context, DragResult result, Tabbable* tab)
 {
-       RefPtr<Action> act;
-
-       transport_tearoff_hbox.set_border_width (PX_SCALE(3));
-       transport_tearoff_hbox.set_spacing (PX_SCALE(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() || Profile->get_mixbus()) {
-               transport_tearoff->set_can_be_torn_off (false);
+       if (result == Gtk::DRAG_RESULT_NO_TARGET) {
+               tab->detach ();
+               return true;
        }
+       return false;
+}
 
-       transport_hbox.pack_start (*transport_tearoff, true, false);
+bool
+ARDOUR_UI::transport_expose (GdkEventExpose* ev)
+{
+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;
+       }
 
-       transport_base.set_name ("TransportBase");
-       transport_base.add (transport_hbox);
+       Cairo::RefPtr<Cairo::Context> cr = transport_table.get_window()->create_cairo_context ();
 
-       transport_frame.set_shadow_type (SHADOW_OUT);
-       transport_frame.set_name ("BaseFrame");
-       transport_frame.add (transport_base);
+       cr->rectangle (ev->area.x, ev->area.y, ev->area.width, ev->area.height);
+       cr->clip ();
 
-       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));
+       transport_table.translate_coordinates (*window_parent, 0, 0, x0, y0);
 
-       auto_return_button.set_text(_("Auto Return"));
+       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 ();
 
-       follow_edits_button.set_text(_("Follow Edits"));
+       static const int xmargin = 2;
+       static const int ymargin = 1;
 
-//     auto_input_button.set_text (_("Auto Input"));
+       /* draw box around record-options */
+       int xx, ww, hh, uu;
 
-       click_button.set_icon (ArdourIcon::TransportMetronom);
+       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;
 
-       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);
+       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 ();
 
-       auto_return_button.set_name ("transport option button");
-       follow_edits_button.set_name ("transport option button");
-       auto_input_button.set_name ("transport option button");
+       /* 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;
 
-       /* these have to provide a clear indication of active state */
+       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 ();
 
-       click_button.set_name ("transport button");
-       sync_button.set_name ("transport active option button");
+       return false;
+}
 
-       stop_button.set_active (true);
+void
+ARDOUR_UI::setup_transport ()
+{
+       RefPtr<Action> act;
+       /* setup actions */
 
-       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);
+       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);
 
        act = ActionManager::get_action (X_("Transport"), X_("Stop"));
        stop_button.set_related_action (act);
@@ -410,14 +261,22 @@ ARDOUR_UI::setup_transport ()
        act = ActionManager::get_action (X_("Transport"), X_("ToggleExternalSync"));
        sync_button.set_related_action (act);
 
-       /* clocks, etc. */
+       sync_button.set_sizing_text (S_("LogestSync|M-Clk"));
 
-       ARDOUR_UI::Clock.connect (sigc::mem_fun (primary_clock, &AudioClock::set));
-       ARDOUR_UI::Clock.connect (sigc::mem_fun (secondary_clock, &AudioClock::set));
+       /* CANNOT sigc::bind these to clicked or toggled, must use pressed or released */
+       act = ActionManager::get_action (X_("Main"), X_("cancel-solo"));
+       solo_alert_button.set_related_action (act);
+       auditioning_alert_button.signal_clicked.connect (sigc::mem_fun(*this,&ARDOUR_UI::audition_alert_clicked));
+       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);
 
-       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));
+       layered_button.signal_clicked.connect (sigc::mem_fun(*this,&ARDOUR_UI::layered_button_clicked));
+
+       editor_visibility_button.set_related_action (ActionManager::get_action (X_("Common"), X_("change-editor-visibility")));
+       mixer_visibility_button.set_related_action (ActionManager::get_action (X_("Common"), X_("change-mixer-visibility")));
+       prefs_visibility_button.set_related_action (ActionManager::get_action (X_("Common"), X_("change-preferences-visibility")));
 
        act = ActionManager::get_action ("Transport", "ToggleAutoReturn");
        auto_return_button.set_related_action (act);
@@ -426,32 +285,100 @@ ARDOUR_UI::setup_transport ()
        act = ActionManager::get_action ("Transport", "ToggleAutoInput");
        auto_input_button.set_related_action (act);
 
-       /* alerts */
+       act = ActionManager::get_action ("Transport", "TogglePunchIn");
+       punch_in_button.set_related_action (act);
+       act = ActionManager::get_action ("Transport", "TogglePunchOut");
+       punch_out_button.set_related_action (act);
 
-       /* CANNOT sigc::bind these to clicked or toggled, must use pressed or released */
+       /* connect signals */
+       ARDOUR_UI::Clock.connect (sigc::mem_fun (primary_clock, &AudioClock::set));
+       ARDOUR_UI::Clock.connect (sigc::mem_fun (secondary_clock, &AudioClock::set));
+
+       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));
+
+       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));
+
+       /* catch context clicks so that we can show a menu on these buttons */
+
+       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);
+
+       /* setup widget style/name */
+
+       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");
-       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);
        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);
+       solo_alert_button.set_elements (ArdourButton::Element(ArdourButton::Body|ArdourButton::Text));
+       auditioning_alert_button.set_elements (ArdourButton::Element(ArdourButton::Body|ArdourButton::Text));
+       feedback_alert_button.set_elements (ArdourButton::Element(ArdourButton::Body|ArdourButton::Text));
+
+       solo_alert_button.set_layout_font (UIConfiguration::instance().get_SmallerFont());
+       auditioning_alert_button.set_layout_font (UIConfiguration::instance().get_SmallerFont());
+       feedback_alert_button.set_layout_font (UIConfiguration::instance().get_SmallerFont());
+
+       editor_visibility_button.set_name (X_("page switch button"));
+       mixer_visibility_button.set_name (X_("page switch button"));
+       prefs_visibility_button.set_name (X_("page switch button"));
 
-       /* all transport buttons should be the same size vertically and
-        * horizontally
-        */
+       punch_in_button.set_name ("punch button");
+       punch_out_button.set_name ("punch button");
+       layered_button.set_name (("layered 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"));
+       punch_in_button.set_text (_("In"));
+       punch_out_button.set_text (_("Out"));
+       layered_button.set_text (_("Non-Layered"));
+
+       punch_label.set_text (_("Punch:"));
+       layered_label.set_text (_("Rec:"));
+
+       /* and tooltips */
+
+       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()));
+
+       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"));
+
+       /* setup icons */
+
+       click_button.set_icon (ArdourIcon::TransportMetronom);
+       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);
+
+       /* transport control size-group */
 
        Glib::RefPtr<SizeGroup> transport_button_size_group = SizeGroup::create (SIZE_GROUP_BOTH);
        transport_button_size_group->add_widget (goto_start_button);
@@ -462,147 +389,165 @@ ARDOUR_UI::setup_transport ()
        transport_button_size_group->add_widget (roll_button);
        transport_button_size_group->add_widget (stop_button);
 
-       /* 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));
+       Glib::RefPtr<SizeGroup> punch_button_size_group = SizeGroup::create (Gtk::SIZE_GROUP_HORIZONTAL);
+       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);
 
-       HBox* tbox1 = manage (new HBox);
-       HBox* tbox2 = manage (new HBox);
-       HBox* tbox = manage (new HBox);
+       /* and now the layout... */
 
-       VBox* vbox1 = manage (new VBox);
-       VBox* vbox2 = manage (new VBox);
+       /* top level packing */
+       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_shadow_type (Gtk::SHADOW_NONE);
 
-       Alignment* a1 = manage (new Alignment);
-       Alignment* a2 = manage (new Alignment);
+       transport_table.signal_expose_event().connect (sigc::mem_fun (*this, &ARDOUR_UI::transport_expose), false);
 
-       tbox1->set_spacing (PX_SCALE(2));
-       tbox2->set_spacing (PX_SCALE(2));
+       /* transport controls sub-group */
+       click_button.set_size_request (PX_SCALE(20), PX_SCALE(20));
+
+       HBox* tbox = manage (new HBox);
        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);
-       }
+       tbox->pack_start (midi_panic_button, true, true, 0);
+       tbox->pack_start (click_button, true, true, 0);
+       tbox->pack_start (goto_start_button, true, true);
+       tbox->pack_start (goto_end_button, true, true);
+       tbox->pack_start (auto_loop_button, true, true);
+       tbox->pack_start (play_selection_button, true, true);
+
+       tbox->pack_start (roll_button, true, true);
+       tbox->pack_start (stop_button, true, true);
+       tbox->pack_start (rec_button, true, true, 3);
+
+       /* alert box sub-group */
+       VBox* alert_box = manage (new VBox);
+       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);
+
+       /* clock button size groups */
+       Glib::RefPtr<SizeGroup> button_height_size_group = SizeGroup::create (Gtk::SIZE_GROUP_VERTICAL);
+       button_height_size_group->add_widget (follow_edits_button);
+       button_height_size_group->add_widget (*primary_clock->left_btn());
+       button_height_size_group->add_widget (*primary_clock->right_btn());
+       button_height_size_group->add_widget (*secondary_clock->left_btn());
+       button_height_size_group->add_widget (*secondary_clock->right_btn());
 
-       tbox1->pack_start (goto_start_button, true, true);
-       tbox1->pack_start (goto_end_button, true, true);
-       tbox1->pack_start (auto_loop_button, true, true);
+       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);
+       button_height_size_group->add_widget (editor_visibility_button);
+       button_height_size_group->add_widget (mixer_visibility_button);
 
-       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);
+       Glib::RefPtr<SizeGroup> clock1_size_group = SizeGroup::create (SIZE_GROUP_BOTH);
+       clock1_size_group->add_widget (*primary_clock->left_btn());
+       clock1_size_group->add_widget (*primary_clock->right_btn());
 
-       vbox1->pack_start (*tbox1, true, true);
-       vbox2->pack_start (*tbox2, true, true);
+       Glib::RefPtr<SizeGroup> clock2_size_group = SizeGroup::create (SIZE_GROUP_BOTH);
+       clock2_size_group->add_widget (*secondary_clock->left_btn());
+       clock2_size_group->add_widget (*secondary_clock->right_btn());
 
-       a1->add (*vbox1);
-       a1->set (0.5, 0.5, 0.0, 1.0);
-       a2->add (*vbox2);
-       a2->set (0.5, 0.5, 0.0, 1.0);
+       /* 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, 3);
 
-       tbox->pack_start (*a1, false, false);
-       tbox->pack_start (*a2, false, false);
 
-       HBox* clock_box = manage (new HBox);
+       /* and the main table layout */
 
-       clock_box->pack_start (*primary_clock, false, false);
-       if (!ARDOUR::Profile->get_small_screen() && !ARDOUR::Profile->get_trx()) {
-               clock_box->pack_start (*secondary_clock, false, false);
-       }
-       clock_box->set_spacing (PX_SCALE(3));
+       int col = 0;
+#define TCOL col, col + 1
 
-       shuttle_box = manage (new ShuttleControl);
-       shuttle_box->show ();
+       transport_table.attach (*tbox, TCOL, 0, 1 , SHRINK, SHRINK, 0, 0);
+       transport_table.attach (*ssbox, TCOL, 1, 2 , FILL, SHRINK, 0, 0);
+       ++col;
 
-       VBox* transport_vbox = manage (new VBox);
-       transport_vbox->set_name ("TransportBase");
-       transport_vbox->set_border_width (0);
-       transport_vbox->set_spacing (PX_SCALE(3));
-       transport_vbox->pack_start (*tbox, true, true, 0);
+       transport_table.attach (*(manage (new ArdourVSpacer ())), TCOL, 0, 2 , SHRINK, EXPAND|FILL, 3, 0);
+       ++col;
 
-       if (!Profile->get_trx()) {
-               transport_vbox->pack_start (*shuttle_box, false, false, 0);
-       }
+       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;
 
-       time_info_box = manage (new TimeInfoBox);
+       transport_table.attach (punch_in_button,  col,     col + 1, 0, 1 , FILL, SHRINK, 0, 2);
+       transport_table.attach (*(manage (new Label (""))), col + 1, col + 2, 0, 1 , FILL, SHRINK, 2, 2);
+       transport_table.attach (punch_out_button, col + 2, col + 3, 0, 1 , FILL, SHRINK, 0, 2);
+       transport_table.attach (layered_button,   col,     col + 3, 1, 2 , FILL, SHRINK, 0, 2);
+       col += 3;
 
-       if (ARDOUR::Profile->get_trx()) {
-               transport_tearoff_hbox.pack_start (*time_info_box, false, false);
-       }
+       transport_table.attach (*(manage (new ArdourVSpacer ())), TCOL, 0, 2 , SHRINK, EXPAND|FILL, 3, 0);
+       ++col;
 
-       transport_tearoff_hbox.pack_start (*transport_vbox, false, false);
+       transport_table.attach (follow_edits_button, TCOL, 0, 1 , FILL, SHRINK, 2, 0);
+       transport_table.attach (auto_return_button,  TCOL, 1, 2 , FILL, SHRINK, 2, 0);
+       ++col;
 
-       /* transport related toggle controls */
+       transport_table.attach (*(manage (new ArdourVSpacer ())), TCOL, 0, 2 , SHRINK, EXPAND|FILL, 3, 0);
+       ++col;
 
-       VBox* auto_box = manage (new VBox);
-       auto_box->set_homogeneous (true);
-       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_table.attach (*primary_clock,              col,     col + 2, 0, 1 , FILL, SHRINK, 2, 0);
+       transport_table.attach (*primary_clock->left_btn(),  col,     col + 1, 1, 2 , FILL, SHRINK, 2, 0);
+       transport_table.attach (*primary_clock->right_btn(), col + 1, col + 2, 1, 2 , FILL, SHRINK, 2, 0);
+       col += 2;
 
-       if (!ARDOUR::Profile->get_trx()) {
-               transport_tearoff_hbox.pack_start (*auto_box, false, false);
-       }
-       transport_tearoff_hbox.pack_start (*clock_box, true, true);
+       transport_table.attach (*(manage (new ArdourVSpacer ())), TCOL, 0, 2 , SHRINK, EXPAND|FILL, 3, 0);
+       ++col;
 
-       if (ARDOUR::Profile->get_trx()) {
-               transport_tearoff_hbox.pack_start (*auto_box, false, false);
+       if (!ARDOUR::Profile->get_small_screen()) {
+               transport_table.attach (*secondary_clock,              col,     col + 2, 0, 1 , FILL, SHRINK, 2, 0);
+               transport_table.attach (*secondary_clock->left_btn(),  col,     col + 1, 1, 2 , FILL, SHRINK, 2, 0);
+               transport_table.attach (*secondary_clock->right_btn(), col + 1, col + 2, 1, 2 , FILL, SHRINK, 2, 0);
+               col += 2;
        }
 
-       if (!ARDOUR::Profile->get_trx()) {
-               transport_tearoff_hbox.pack_start (*time_info_box, false, false);
-       }
+       transport_table.attach (*(manage (new ArdourVSpacer ())), TCOL, 0, 2 , SHRINK, EXPAND|FILL, 3, 0);
+       ++col;
 
-       if (!ARDOUR::Profile->get_trx()) {
-               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);
-       }
+       transport_table.attach (*alert_box, TCOL, 0, 2, SHRINK, EXPAND|FILL, 2, 0);
+       ++col;
+
+       transport_table.attach (*(manage (new ArdourVSpacer ())), TCOL, 0, 2 , SHRINK, EXPAND|FILL, 3, 0);
+       ++col;
 
-       if (Profile->get_sae()) {
-               Image* img = manage (new Image ((::get_icon (X_("sae")))));
-               transport_tearoff_hbox.pack_end (*img, false, false);
-       }
+       /* editor-meter is in transport_hbox */
+       transport_hbox.set_spacing (PX_SCALE(1));
+       transport_table.attach (transport_hbox, TCOL, 0, 2, SHRINK, EXPAND|FILL, 2, 0);
+       ++col;
 
-       /* desensitize */
+       transport_table.attach (*mini_timeline, TCOL, 0, 2, EXPAND|FILL, EXPAND|FILL, 1, 0);
+       ++col;
 
-       set_transport_sensitivity (false);
+       /* lua script action buttons */
+       transport_table.attach (action_script_table, TCOL, 0, 2, SHRINK, EXPAND|FILL, 1, 0);
+       ++col;
 
-       XMLNode* tnode = tearoff_settings ("transport");
-       if (tnode) {
-               transport_tearoff->set_state (*tnode);
-       }
-}
-#undef PX_SCALE
+       transport_table.attach (editor_visibility_button, TCOL, 0, 1 , FILL, SHRINK, 2, 0);
+       transport_table.attach (mixer_visibility_button,  TCOL, 1, 2 , FILL, SHRINK, 2, 0);
+       ++col;
 
-void
-ARDOUR_UI::detach_tearoff (Box* b, Widget* w)
-{
-       b->remove (*w);
-}
+       /* desensitize */
 
-void
-ARDOUR_UI::reattach_tearoff (Box* b, Widget* w, int32_t n)
-{
-       b->pack_start (*w);
-       b->reorder_child (*w, n);
-}
+       feedback_alert_button.set_sensitive (false);
+       feedback_alert_button.set_visual_state (Gtkmm2ext::NoVisualState);
+       auditioning_alert_button.set_sensitive (false);
+       auditioning_alert_button.set_visual_state (Gtkmm2ext::NoVisualState);
 
-void
-ARDOUR_UI::reattach_all_tearoffs ()
-{
-       if (transport_tearoff) transport_tearoff->put_it_back();
-       if (editor) editor->reattach_all_tearoffs ();
+       stop_button.set_active (true);
+       set_transport_sensitivity (false);
 }
+#undef PX_SCALE
+#undef TCOL
 
 void
 ARDOUR_UI::soloing_changed (bool onoff)
@@ -616,6 +561,10 @@ void
 ARDOUR_UI::_auditioning_changed (bool onoff)
 {
        auditioning_alert_button.set_active (onoff);
+       auditioning_alert_button.set_sensitive (onoff);
+       if (!onoff) {
+               auditioning_alert_button.set_visual_state (Gtkmm2ext::NoVisualState);
+       }
        set_transport_sensitivity (!onoff);
 }
 
@@ -625,19 +574,12 @@ ARDOUR_UI::auditioning_changed (bool onoff)
        UI::instance()->call_slot (MISSING_INVALIDATOR, boost::bind (&ARDOUR_UI::_auditioning_changed, this, onoff));
 }
 
-bool
-ARDOUR_UI::audition_alert_press (GdkEventButton*)
+void
+ARDOUR_UI::audition_alert_clicked ()
 {
        if (_session) {
                _session->cancel_audition();
        }
-       return true;
-}
-
-bool
-ARDOUR_UI::feedback_alert_press (GdkEventButton *)
-{
-       return true;
 }
 
 bool
@@ -660,6 +602,14 @@ ARDOUR_UI::error_alert_press (GdkEventButton* ev)
        return !do_toggle;
 }
 
+void
+ARDOUR_UI::layered_button_clicked ()
+{
+       if (_session) {
+               _session->config.set_layered_record_mode (!_session->config.get_layered_record_mode ());
+       }
+}
+
 void
 ARDOUR_UI::solo_blink (bool onoff)
 {
@@ -775,14 +725,6 @@ ARDOUR_UI::editor_realized ()
        UIConfiguration::instance().reset_dpi ();
 }
 
-void
-ARDOUR_UI::update_tearoff_visibility ()
-{
-       if (editor) {
-               editor->update_tearoff_visibility ();
-       }
-}
-
 void
 ARDOUR_UI::maximise_editing_space ()
 {
@@ -802,11 +744,12 @@ ARDOUR_UI::restore_editing_space ()
 void
 ARDOUR_UI::show_ui_prefs ()
 {
-       tabs().set_current_page (2);
-       rc_option_editor->set_current_page (_("GUI"));
+       if (rc_option_editor) {
+               show_tabbable (rc_option_editor);
+               rc_option_editor->set_current_page (_("GUI"));
+       }
 }
 
-
 bool
 ARDOUR_UI::click_button_clicked (GdkEventButton* ev)
 {
@@ -815,8 +758,8 @@ ARDOUR_UI::click_button_clicked (GdkEventButton* ev)
                return false;
        }
 
-       tabs().set_current_page (2);
-       rc_option_editor->set_current_page (_("Misc"));
+       show_tabbable (rc_option_editor);
+       rc_option_editor->set_current_page (_("Misc/Click"));
        return true;
 }
 
@@ -857,4 +800,5 @@ ARDOUR_UI::update_title ()
                WindowTitle title (Glib::get_application_name());
                _main_window.set_title (title.get_string());
        }
+
 }