Remove internal edit mode and add "content" tool.
[ardour.git] / gtk2_ardour / mixer_strip.cc
index a8ec7d5c79d971dfa12d132770d621f47596223f..831c5158b83a6aa7d74a62d37ebb82f98085a719 100644 (file)
 #include "ardour/audio_track.h"
 #include "ardour/audioengine.h"
 #include "ardour/internal_send.h"
+#include "ardour/meter.h"
 #include "ardour/midi_track.h"
 #include "ardour/pannable.h"
 #include "ardour/panner.h"
 #include "ardour/panner_shell.h"
+#include "ardour/panner_manager.h"
 #include "ardour/port.h"
 #include "ardour/profile.h"
 #include "ardour/route.h"
 #include "i18n.h"
 
 using namespace ARDOUR;
+using namespace ARDOUR_UI_UTILS;
 using namespace PBD;
 using namespace Gtk;
 using namespace Gtkmm2ext;
 using namespace std;
 using namespace ArdourMeter;
 
-int MixerStrip::scrollbar_height = 0;
+MixerStrip* MixerStrip::_entered_mixer_strip;
+
 PBD::Signal1<void,MixerStrip*> MixerStrip::CatchDeletion;
 
 MixerStrip::MixerStrip (Mixer_UI& mx, Session* sess, bool in_mixer)
@@ -85,15 +89,14 @@ MixerStrip::MixerStrip (Mixer_UI& mx, Session* sess, bool in_mixer)
        , gpm (sess, 250)
        , panners (sess)
        , button_size_group (Gtk::SizeGroup::create (Gtk::SIZE_GROUP_HORIZONTAL))
-       , button_table (3, 1)
-       , rec_solo_table (2, 2)
-       , top_button_table (1, 2)
-       , middle_button_table (1, 2)
-       , bottom_button_table (1, 2)
+       , rec_mon_table (2, 2)
+       , solo_iso_table (1, 2)
+       , mute_solo_table (1, 2)
+       , bottom_button_table (1, 3)
        , meter_point_button (_("pre"))
        , midi_input_enable_button (0)
        , _comment_button (_("Comments"))
-       , _visibility (X_("mixer-strip-visibility"))
+       , _visibility (X_("mixer-element-visibility"))
 {
        init ();
 
@@ -115,13 +118,14 @@ MixerStrip::MixerStrip (Mixer_UI& mx, Session* sess, boost::shared_ptr<Route> rt
        , gpm (sess, 250)
        , panners (sess)
        , button_size_group (Gtk::SizeGroup::create (Gtk::SIZE_GROUP_HORIZONTAL))
-       , button_table (3, 1)
-       , middle_button_table (1, 2)
-       , bottom_button_table (1, 2)
+       , rec_mon_table (2, 2)
+       , solo_iso_table (1, 2)
+       , mute_solo_table (1, 2)
+       , bottom_button_table (1, 3)
        , meter_point_button (_("pre"))
        , midi_input_enable_button (0)
        , _comment_button (_("Comments"))
-       , _visibility (X_("mixer-strip-visibility"))
+       , _visibility (X_("mixer-element-visibility"))
 {
        init ();
        set_route (rt);
@@ -130,13 +134,11 @@ MixerStrip::MixerStrip (Mixer_UI& mx, Session* sess, boost::shared_ptr<Route> rt
 void
 MixerStrip::init ()
 {
-       input_selector = 0;
-       output_selector = 0;
+       _entered_mixer_strip= 0;
        group_menu = 0;
        route_ops_menu = 0;
        ignore_comment_edit = false;
        ignore_toggle = false;
-       comment_window = 0;
        comment_area = 0;
        _width_owner = 0;
        spacer = 0;
@@ -155,69 +157,64 @@ MixerStrip::init ()
        hide_button.set_image(::get_icon("hide"));
        ARDOUR_UI::instance()->set_tip (&hide_button, _("Hide this mixer strip"));
 
+       input_button_box.set_spacing(2);
+       
        input_button.set_text (_("Input"));
        input_button.set_name ("mixer strip button");
-       input_button.set_size_request (-1, 20);
        input_button_box.pack_start (input_button, true, true);
 
        output_button.set_text (_("Output"));
        output_button.set_name ("mixer strip button");
-       Gtkmm2ext::set_size_request_to_display_given_text (output_button, longest_label.c_str(), 4, 4);
 
        ARDOUR_UI::instance()->set_tip (&meter_point_button, _("Click to select metering point"), "");
        meter_point_button.set_name ("mixer strip button");
 
-       /* TRANSLATORS: this string should be longest of the strings
-          used to describe meter points. In english, it's "input".
-       */
-       set_size_request_to_display_given_text (meter_point_button, _("tupni"), 5, 5);
-
-       bottom_button_table.attach (meter_point_button, 1, 2, 0, 1);
+       bottom_button_table.attach (meter_point_button, 2, 3, 0, 1);
 
        meter_point_button.signal_button_press_event().connect (sigc::mem_fun (gpm, &GainMeter::meter_press), false);
        meter_point_button.signal_button_release_event().connect (sigc::mem_fun (gpm, &GainMeter::meter_release), false);
-
+       
        hide_button.set_events (hide_button.get_events() & ~(Gdk::ENTER_NOTIFY_MASK|Gdk::LEAVE_NOTIFY_MASK));
 
-       monitor_input_button->set_diameter (3);
-       monitor_disk_button->set_diameter (3);
-
-        solo_isolated_led = manage (new ArdourButton (ArdourButton::led_default_elements));
-        solo_isolated_led->show ();
-        solo_isolated_led->set_diameter (3);
-        solo_isolated_led->set_no_show_all (true);
-        solo_isolated_led->set_name (X_("solo isolate"));
-        solo_isolated_led->add_events (Gdk::BUTTON_PRESS_MASK|Gdk::BUTTON_RELEASE_MASK);
-        solo_isolated_led->signal_button_release_event().connect (sigc::mem_fun (*this, &RouteUI::solo_isolate_button_release));
+       solo_isolated_led = manage (new ArdourButton (ArdourButton::led_default_elements));
+       solo_isolated_led->show ();
+       solo_isolated_led->set_no_show_all (true);
+       solo_isolated_led->set_name (X_("solo isolate"));
+       solo_isolated_led->add_events (Gdk::BUTTON_PRESS_MASK|Gdk::BUTTON_RELEASE_MASK);
+       solo_isolated_led->signal_button_release_event().connect (sigc::mem_fun (*this, &RouteUI::solo_isolate_button_release), false);
        UI::instance()->set_tip (solo_isolated_led, _("Isolate Solo"), "");
 
-        solo_safe_led = manage (new ArdourButton (ArdourButton::led_default_elements));
-        solo_safe_led->show ();
-        solo_safe_led->set_diameter (3);
-        solo_safe_led->set_no_show_all (true);
-        solo_safe_led->set_name (X_("solo safe"));
-        solo_safe_led->add_events (Gdk::BUTTON_PRESS_MASK|Gdk::BUTTON_RELEASE_MASK);
-        solo_safe_led->signal_button_release_event().connect (sigc::mem_fun (*this, &RouteUI::solo_safe_button_release));
+       solo_safe_led = manage (new ArdourButton (ArdourButton::led_default_elements));
+       solo_safe_led->show ();
+       solo_safe_led->set_no_show_all (true);
+       solo_safe_led->set_name (X_("solo safe"));
+       solo_safe_led->add_events (Gdk::BUTTON_PRESS_MASK|Gdk::BUTTON_RELEASE_MASK);
+       solo_safe_led->signal_button_release_event().connect (sigc::mem_fun (*this, &RouteUI::solo_safe_button_release), false);
        UI::instance()->set_tip (solo_safe_led, _("Lock Solo Status"), "");
 
-       solo_safe_led->set_text (_("lock"));
-       solo_isolated_led->set_text (_("iso"));
+       solo_safe_led->set_text (_("Lock"));
+       solo_isolated_led->set_text (_("Iso"));
 
-       top_button_table.set_homogeneous (true);
-       top_button_table.set_spacings (2);
-       top_button_table.attach (*monitor_input_button, 0, 1, 0, 1);
-        top_button_table.attach (*monitor_disk_button, 1, 2, 0, 1);
-       top_button_table.show ();
-
-       rec_solo_table.set_homogeneous (true);
-       rec_solo_table.set_row_spacings (2);
-       rec_solo_table.set_col_spacings (2);
-        rec_solo_table.attach (*solo_isolated_led, 1, 2, 0, 1);
-        rec_solo_table.attach (*solo_safe_led, 1, 2, 1, 2);
-        rec_solo_table.show ();
+       solo_iso_table.set_homogeneous (true);
+       solo_iso_table.set_spacings (2);
+       if (!ARDOUR::Profile->get_trx()) {
+               solo_iso_table.attach (*solo_isolated_led, 0, 1, 0, 1);
+               solo_iso_table.attach (*solo_safe_led, 1, 2, 0, 1);
+       }
+       solo_iso_table.show ();
 
-       button_table.set_homogeneous (false);
-       button_table.set_spacings (2);
+       rec_mon_table.set_homogeneous (true);
+       rec_mon_table.set_row_spacings (2);
+       rec_mon_table.set_col_spacings (2);
+       if (ARDOUR::Profile->get_mixbus()) {
+               rec_mon_table.resize (1, 3);
+               rec_mon_table.attach (*monitor_input_button, 1, 2, 0, 1);
+               rec_mon_table.attach (*monitor_disk_button, 2, 3, 0, 1);
+       } else if (!ARDOUR::Profile->get_trx()) {
+               rec_mon_table.attach (*monitor_input_button, 1, 2, 0, 1);
+               rec_mon_table.attach (*monitor_disk_button, 1, 2, 1, 2);
+       }
+       rec_mon_table.show ();
 
        if (solo_isolated_led) {
                button_size_group->add_widget (*solo_isolated_led);
@@ -225,71 +222,80 @@ MixerStrip::init ()
        if (solo_safe_led) {
                button_size_group->add_widget (*solo_safe_led);
        }
-       if (rec_enable_button) {
-               button_size_group->add_widget (*rec_enable_button);
-       }
-       if (monitor_disk_button) {
-               button_size_group->add_widget (*monitor_disk_button);
-       }
-       if (monitor_input_button) {
-               button_size_group->add_widget (*monitor_input_button);
-       }
-
-       button_table.attach (name_button, 0, 1, 0, 1);
-       button_table.attach (input_button_box, 0, 1, 1, 2);
-       button_table.attach (_invert_button_box, 0, 1, 2, 3);
 
-       middle_button_table.set_homogeneous (true);
-       middle_button_table.set_spacings (2);
+       if (!ARDOUR::Profile->get_mixbus()) {
+               if (rec_enable_button) {
+                       button_size_group->add_widget (*rec_enable_button);
+               }
+               if (monitor_disk_button) {
+                       button_size_group->add_widget (*monitor_disk_button);
+               }
+               if (monitor_input_button) {
+                       button_size_group->add_widget (*monitor_input_button);
+               }
+       }
+       
+       mute_solo_table.set_homogeneous (true);
+       mute_solo_table.set_spacings (2);
 
        bottom_button_table.set_spacings (2);
        bottom_button_table.set_homogeneous (true);
-//     bottom_button_table.attach (group_button, 0, 1, 0, 1);
+       bottom_button_table.attach (group_button, 1, 2, 0, 1);
        bottom_button_table.attach (gpm.gain_automation_state_button, 0, 1, 0, 1);
 
        name_button.set_name ("mixer strip button");
-       name_button.set_text (" "); /* non empty text, forces creation of the layout */
-       name_button.set_text (""); /* back to empty */
-       name_button.layout()->set_ellipsize (Pango::ELLIPSIZE_END);
+       name_button.set_text_ellipsize (Pango::ELLIPSIZE_END);
        name_button.signal_size_allocate().connect (sigc::mem_fun (*this, &MixerStrip::name_button_resized));
-       Gtkmm2ext::set_size_request_to_display_given_text (name_button, longest_label.c_str(), 2, 2);
-       name_button.set_size_request (-1, 20);
 
        ARDOUR_UI::instance()->set_tip (&group_button, _("Mix group"), "");
        group_button.set_name ("mixer strip button");
-       Gtkmm2ext::set_size_request_to_display_given_text (group_button, "Group", 2, 2);
 
        _comment_button.set_name (X_("mixer strip button"));
-       _comment_button.signal_clicked.connect (sigc::mem_fun (*this, &MixerStrip::toggle_comment_editor));
+       _comment_button.signal_clicked.connect (sigc::mem_fun (*this, &RouteUI::toggle_comment_editor));
 
-       global_vpacker.set_border_width (0);
+       global_vpacker.set_border_width (1);
        global_vpacker.set_spacing (0);
 
        width_button.set_name ("mixer strip button");
        hide_button.set_name ("mixer strip button");
-       top_event_box.set_name ("mixer strip button");
 
        width_button.signal_button_press_event().connect (sigc::mem_fun(*this, &MixerStrip::width_button_pressed), false);
        hide_button.signal_clicked.connect (sigc::mem_fun(*this, &MixerStrip::hide_clicked));
 
+//     width_hide_box.set_border_width (1);
+       width_hide_box.set_spacing (2);
        width_hide_box.pack_start (width_button, false, true);
-       width_hide_box.pack_start (top_event_box, true, true);
+       width_hide_box.pack_start (number_label, true, true);
        width_hide_box.pack_end (hide_button, false, true);
 
-       whvbox.pack_start (width_hide_box, true, true);
+       number_label.set_text ("-");
+       number_label.set_elements((ArdourButton::Element)(ArdourButton::Edge|ArdourButton::Body|ArdourButton::Text|ArdourButton::Inactive));
+       number_label.set_no_show_all ();
+       number_label.set_name ("tracknumber label");
+       number_label.set_fixed_colors (0x80808080, 0x80808080);
+       number_label.set_alignment (.5, .5);
+       number_label.set_fallthrough_to_parent (true);
 
        global_vpacker.set_spacing (2);
-       global_vpacker.pack_start (whvbox, Gtk::PACK_SHRINK);
-       global_vpacker.pack_start (button_table, Gtk::PACK_SHRINK);
-       global_vpacker.pack_start (processor_box, true, true);
+       if (!ARDOUR::Profile->get_trx()) {
+               global_vpacker.pack_start (width_hide_box, Gtk::PACK_SHRINK);
+               global_vpacker.pack_start (name_button, Gtk::PACK_SHRINK);
+               global_vpacker.pack_start (input_button_box, Gtk::PACK_SHRINK);
+               global_vpacker.pack_start (_invert_button_box, Gtk::PACK_SHRINK);
+               global_vpacker.pack_start (processor_box, true, true);
+       }
        global_vpacker.pack_start (panners, Gtk::PACK_SHRINK);
-       global_vpacker.pack_start (top_button_table, Gtk::PACK_SHRINK);
-       global_vpacker.pack_start (rec_solo_table, Gtk::PACK_SHRINK);
-       global_vpacker.pack_start (middle_button_table, Gtk::PACK_SHRINK);
+       global_vpacker.pack_start (rec_mon_table, Gtk::PACK_SHRINK);
+       global_vpacker.pack_start (solo_iso_table, Gtk::PACK_SHRINK);
+       global_vpacker.pack_start (mute_solo_table, Gtk::PACK_SHRINK);
        global_vpacker.pack_start (gpm, Gtk::PACK_SHRINK);
        global_vpacker.pack_start (bottom_button_table, Gtk::PACK_SHRINK);
-       global_vpacker.pack_start (output_button, Gtk::PACK_SHRINK);
-       global_vpacker.pack_start (_comment_button, Gtk::PACK_SHRINK);
+       if (!ARDOUR::Profile->get_trx()) {
+               global_vpacker.pack_start (output_button, Gtk::PACK_SHRINK);
+               global_vpacker.pack_start (_comment_button, Gtk::PACK_SHRINK);
+       } else {
+               global_vpacker.pack_start (name_button, Gtk::PACK_SHRINK);
+       }
 
        global_frame.add (global_vpacker);
        global_frame.set_shadow_type (Gtk::SHADOW_IN);
@@ -309,11 +315,16 @@ MixerStrip::init ()
        _session->engine().Running.connect (*this, invalidator (*this), boost::bind (&MixerStrip::engine_running, this), gui_context());
 
        input_button.signal_button_press_event().connect (sigc::mem_fun(*this, &MixerStrip::input_press), false);
+       input_button.signal_button_release_event().connect (sigc::mem_fun(*this, &MixerStrip::input_release), false);
+
        output_button.signal_button_press_event().connect (sigc::mem_fun(*this, &MixerStrip::output_press), false);
+       output_button.signal_button_release_event().connect (sigc::mem_fun(*this, &MixerStrip::output_release), false);
 
-       /* ditto for this button and busses */
+       number_label.signal_button_press_event().connect (sigc::mem_fun(*this, &MixerStrip::number_button_button_press), false);
 
        name_button.signal_button_press_event().connect (sigc::mem_fun(*this, &MixerStrip::name_button_button_press), false);
+       name_button.signal_button_release_event().connect (sigc::mem_fun(*this, &MixerStrip::name_button_button_release), false);
+
        group_button.signal_button_press_event().connect (sigc::mem_fun(*this, &MixerStrip::select_route_group), false);
 
        _width = (Width) -1;
@@ -348,16 +359,21 @@ MixerStrip::init ()
           must be the same as those used in RCOptionEditor so that the configuration changes
           are recognised when they occur.
        */
-       _visibility.add (&_invert_button_box, X_("PhaseInvert"), _("Phase Invert"));
-       _visibility.add (solo_safe_led, X_("SoloSafe"), _("Solo Safe"), true, boost::bind (&MixerStrip::override_solo_visibility, this));
-       _visibility.add (solo_isolated_led, X_("SoloIsolated"), _("Solo Isolated"), true, boost::bind (&MixerStrip::override_solo_visibility, this));
-       _visibility.add (&_comment_button, X_("Comments"), _("Comments"));
-       _visibility.add (&group_button, X_("Group"), _("Group"));
-       _visibility.add (&meter_point_button, X_("MeterPoint"), _("Meter Point"));
+       _visibility.add (&input_button_box, X_("Input"), _("Input"), false);
+       _visibility.add (&_invert_button_box, X_("PhaseInvert"), _("Phase Invert"), false);
+       _visibility.add (&rec_mon_table, X_("RecMon"), _("Record & Monitor"), false);
+       _visibility.add (&solo_iso_table, X_("SoloIsoLock"), _("Solo Iso / Lock"), false);
+       _visibility.add (&output_button, X_("Output"), _("Output"), false);
+       _visibility.add (&_comment_button, X_("Comments"), _("Comments"), false);
 
-       parameter_changed (X_("mixer-strip-visibility"));
+       parameter_changed (X_("mixer-element-visibility"));
 
        Config->ParameterChanged.connect (_config_connection, MISSING_INVALIDATOR, boost::bind (&MixerStrip::parameter_changed, this, _1), gui_context());
+       _session->config.ParameterChanged.connect (_config_connection, MISSING_INVALIDATOR, boost::bind (&MixerStrip::parameter_changed, this, _1), gui_context());
+
+       //watch for mouse enter/exit so we can do some stuff
+       signal_enter_notify_event().connect (sigc::mem_fun(*this, &MixerStrip::mixer_strip_enter_event ));
+       signal_leave_notify_event().connect (sigc::mem_fun(*this, &MixerStrip::mixer_strip_leave_event ));
 
        gpm.LevelMeterButtonPress.connect_same_thread (_level_meter_connection, boost::bind (&MixerStrip::level_meter_button_press, this, _1));
 }
@@ -366,22 +382,63 @@ MixerStrip::~MixerStrip ()
 {
        CatchDeletion (this);
 
-       delete input_selector;
-       delete output_selector;
-       delete comment_window;
+       if (this ==_entered_mixer_strip)
+               _entered_mixer_strip = NULL;
+}
+
+bool
+MixerStrip::mixer_strip_enter_event (GdkEventCrossing* /*ev*/)
+{
+       _entered_mixer_strip = this;
+       
+       //although we are triggering on the "enter", to the user it will appear that it is happenin on the "leave"
+       //because the mixerstrip control is a parent that encompasses the strip
+       deselect_all_processors();
+
+       return false;
+}
+
+bool
+MixerStrip::mixer_strip_leave_event (GdkEventCrossing *ev)
+{
+       //if we have moved outside our strip, but not into a child view, then deselect ourselves
+       if ( !(ev->detail == GDK_NOTIFY_INFERIOR) ) {
+               _entered_mixer_strip= 0;
+
+               //clear keyboard focus in the gain display.  this is cheesy but fixes a longstanding "bug" where the user starts typing in the gain entry, and leaves it active, thereby prohibiting other keybindings from working
+               gpm.gain_display.set_sensitive(false);
+               gpm.show_gain();
+               gpm.gain_display.set_sensitive(true);
+
+               //if we leave this mixer strip we need to clear out any selections
+               //processor_box.processor_display.select_none();  //but this doesn't work, because it gets triggered when (for example) you open the menu or start a drag
+       }
+       
+       return false;
 }
 
 void
 MixerStrip::set_route (boost::shared_ptr<Route> rt)
 {
+       //the rec/monitor stuff only shows up for tracks.
+       //the show_sends only shows up for buses.
+       //remove them all here, and we may add them back later
+       if (show_sends_button->get_parent()) {
+               rec_mon_table.remove (*show_sends_button);
+       }
        if (rec_enable_button->get_parent()) {
-               rec_solo_table.remove (*rec_enable_button);
+               rec_mon_table.remove (*rec_enable_button);
        }
-
-       if (show_sends_button->get_parent()) {
-               rec_solo_table.remove (*show_sends_button);
+       if (monitor_input_button->get_parent()) {
+               rec_mon_table.remove (*monitor_input_button);
        }
-
+       if (monitor_disk_button->get_parent()) {
+               rec_mon_table.remove (*monitor_disk_button);
+       }
+       if (group_button.get_parent()) {
+               bottom_button_table.remove (group_button);
+       }
+       
        RouteUI::set_route (rt);
 
        /* ProcessorBox needs access to _route so that it can read
@@ -394,12 +451,6 @@ MixerStrip::set_route (boost::shared_ptr<Route> rt)
        mute_changed (0);
        update_solo_display ();
 
-       delete input_selector;
-       input_selector = 0;
-
-       delete output_selector;
-       output_selector = 0;
-
        revert_to_default_display ();
 
        /* unpack these from the parent and stuff them into our own
@@ -415,41 +466,44 @@ MixerStrip::set_route (boost::shared_ptr<Route> rt)
 
        gpm.set_type (rt->meter_type());
        
-       middle_button_table.attach (gpm.gain_display,0,1,1,2, EXPAND|FILL, EXPAND);
-       middle_button_table.attach (gpm.peak_display,1,2,1,2);
+       mute_solo_table.attach (gpm.gain_display,0,1,1,2, EXPAND|FILL, EXPAND);
+       mute_solo_table.attach (gpm.peak_display,1,2,1,2, EXPAND|FILL, EXPAND);
 
        if (solo_button->get_parent()) {
-               middle_button_table.remove (*solo_button);
+               mute_solo_table.remove (*solo_button);
        }
 
        if (mute_button->get_parent()) {
-               middle_button_table.remove (*mute_button);
+               mute_solo_table.remove (*mute_button);
        }
 
        if (route()->is_master()) {
-               middle_button_table.attach (*mute_button, 0, 2, 0, 1);
+               mute_solo_table.attach (*mute_button, 0, 2, 0, 1);
                solo_button->hide ();
                mute_button->show ();
-               rec_solo_table.hide ();
+               rec_mon_table.hide ();
+               if (solo_iso_table.get_parent()) {
+                       solo_iso_table.get_parent()->remove(solo_iso_table);
+               }
        } else {
-               middle_button_table.attach (*mute_button, 0, 1, 0, 1);
-               middle_button_table.attach (*solo_button, 1, 2, 0, 1);
+               bottom_button_table.attach (group_button, 1, 2, 0, 1);
+               mute_solo_table.attach (*mute_button, 0, 1, 0, 1);
+               mute_solo_table.attach (*solo_button, 1, 2, 0, 1);
                mute_button->show ();
                solo_button->show ();
-               rec_solo_table.show ();
+               rec_mon_table.show ();
        }
 
-       if (_mixer_owned && (route()->is_master() || route()->is_monitor())) {
+       if (_mixer_owned && route()->is_master() ) {
 
-               if (scrollbar_height == 0) {
-                       HScrollbar scrollbar;
-                       Gtk::Requisition requisition(scrollbar.size_request ());
-                       scrollbar_height = requisition.height;
-               }
+               HScrollbar scrollbar;
+               Gtk::Requisition requisition(scrollbar.size_request ());
+               int scrollbar_height = requisition.height;
 
                spacer = manage (new EventBox);
-               spacer->set_size_request (-1, scrollbar_height);
+               spacer->set_size_request (-1, scrollbar_height+2);
                global_vpacker.pack_start (*spacer, false, false);
+               spacer->show();
        }
 
        if (is_track()) {
@@ -491,16 +545,26 @@ MixerStrip::set_route (boost::shared_ptr<Route> rt)
 
        if (is_track ()) {
 
-               rec_solo_table.attach (*rec_enable_button, 0, 1, 0, 2);
+               rec_mon_table.attach (*rec_enable_button, 0, 1, 0, ARDOUR::Profile->get_mixbus() ? 1 : 2);
                rec_enable_button->set_sensitive (_session->writable());
                rec_enable_button->show();
 
+               if (ARDOUR::Profile->get_mixbus()) {
+                       rec_mon_table.attach (*monitor_input_button, 1, 2, 0, 1);
+                       rec_mon_table.attach (*monitor_disk_button, 2, 3, 0, 1);
+               } else if (ARDOUR::Profile->get_trx()) {
+                       rec_mon_table.attach (*monitor_input_button, 1, 2, 0, 2);
+               } else {
+                       rec_mon_table.attach (*monitor_input_button, 1, 2, 0, 1);
+                       rec_mon_table.attach (*monitor_disk_button, 1, 2, 1, 2);
+               }
+
        } else {
 
                /* non-master bus */
 
                if (!_route->is_master()) {
-                       rec_solo_table.attach (*show_sends_button, 0, 1, 0, 2);
+                       rec_mon_table.attach (*show_sends_button, 0, 1, 0, 2);
                        show_sends_button->show();
                }
        }
@@ -515,7 +579,10 @@ MixerStrip::set_route (boost::shared_ptr<Route> rt)
        _route->output()->changed.connect (*this, invalidator (*this), boost::bind (&MixerStrip::update_output_display, this), gui_context());
        _route->route_group_changed.connect (route_connections, invalidator (*this), boost::bind (&MixerStrip::route_group_changed, this), gui_context());
 
+       _route->io_changed.connect (route_connections, invalidator (*this), boost::bind (&MixerStrip::io_changed_proxy, this), gui_context ());
+
        if (_route->panner_shell()) {
+               update_panner_choices();
                _route->panner_shell()->Changed.connect (route_connections, invalidator (*this), boost::bind (&MixerStrip::connect_to_pan, this), gui_context());
        }
 
@@ -523,7 +590,7 @@ MixerStrip::set_route (boost::shared_ptr<Route> rt)
                audio_track()->DiskstreamChanged.connect (route_connections, invalidator (*this), boost::bind (&MixerStrip::diskstream_changed, this), gui_context());
        }
 
-       _route->comment_changed.connect (route_connections, invalidator (*this), boost::bind (&MixerStrip::comment_changed, this, _1), gui_context());
+       _route->comment_changed.connect (route_connections, invalidator (*this), boost::bind (&MixerStrip::setup_comment_button, this), gui_context());
        _route->PropertyChanged.connect (route_connections, invalidator (*this), boost::bind (&MixerStrip::property_changed, this, _1), gui_context());
 
        set_stuff_from_route ();
@@ -556,6 +623,7 @@ MixerStrip::set_route (boost::shared_ptr<Route> rt)
        if (!route()->is_master() && !route()->is_monitor()) {
                /* we don't allow master or control routes to be hidden */
                hide_button.show();
+               number_label.show();
        }
 
        gpm.reset_peak_display ();
@@ -564,11 +632,9 @@ MixerStrip::set_route (boost::shared_ptr<Route> rt)
 
        width_button.show();
        width_hide_box.show();
-       whvbox.show ();
        global_frame.show();
        global_vpacker.show();
-       button_table.show();
-       middle_button_table.show();
+       mute_solo_table.show();
        bottom_button_table.show();
        gpm.show_all ();
        meter_point_button.show();
@@ -579,7 +645,7 @@ MixerStrip::set_route (boost::shared_ptr<Route> rt)
        group_button.show();
        gpm.gain_automation_state_button.show();
 
-       parameter_changed ("mixer-strip-visibility");
+       parameter_changed ("mixer-element-visibility");
 
        show ();
 }
@@ -619,8 +685,7 @@ MixerStrip::set_width_enum (Width w, void* owner)
        case Wide:
 
                if (show_sends_button)  {
-                       show_sends_button->set_text (_("Aux\nSends"));
-                       show_sends_button->layout()->set_alignment (Pango::ALIGN_CENTER);
+                       show_sends_button->set_text (_("Aux"));
                }
 
                gpm.gain_automation_style_button.set_text (
@@ -636,8 +701,7 @@ MixerStrip::set_width_enum (Width w, void* owner)
                }
 
 
-               Gtkmm2ext::set_size_request_to_display_given_text (name_button, longest_label.c_str(), 2, 2);
-               set_size_request (-1, -1);
+               set_size_request (max (110, gpm.get_gm_width()+5), -1);
                break;
 
        case Narrow:
@@ -659,8 +723,7 @@ MixerStrip::set_width_enum (Width w, void* owner)
                        panners.short_astate_string(_route->panner()->automation_state()));
                }
 
-               Gtkmm2ext::set_size_request_to_display_given_text (name_button, "long", 2, 2);
-               set_size_request (max (50, gpm.get_gm_width()), -1);
+               set_size_request (max (60, gpm.get_gm_width() + 10), -1);
                break;
        }
 
@@ -693,6 +756,18 @@ struct RouteCompareByName {
        }
 };
 
+gint
+MixerStrip::output_release (GdkEventButton *ev)
+{
+       switch (ev->button) {
+       case 1:
+               edit_output_configuration ();
+               break;
+       }
+       
+       return false;
+}
+
 gint
 MixerStrip::output_press (GdkEventButton *ev)
 {
@@ -707,8 +782,7 @@ MixerStrip::output_press (GdkEventButton *ev)
        switch (ev->button) {
 
        case 1:
-               edit_output_configuration ();
-               break;
+               return false;  //wait for the mouse-up to pop the dialog
 
        case 3:
        {
@@ -770,51 +844,22 @@ MixerStrip::output_press (GdkEventButton *ev)
        return TRUE;
 }
 
-void
-MixerStrip::edit_output_configuration ()
+gint
+MixerStrip::input_release (GdkEventButton *ev)
 {
-       if (output_selector == 0) {
-
-               boost::shared_ptr<Send> send;
-               boost::shared_ptr<IO> output;
-
-               if ((send = boost::dynamic_pointer_cast<Send>(_current_delivery)) != 0) {
-                       if (!boost::dynamic_pointer_cast<InternalSend>(send)) {
-                               output = send->output();
-                       } else {
-                               output = _route->output ();
-                       }
-               } else {
-                       output = _route->output ();
-               }
+       switch (ev->button) {
 
-               output_selector = new IOSelectorWindow (_session, output);
-       }
+       case 1:
+               edit_input_configuration ();
+               break;
+       default:
+               break;
 
-       if (output_selector->is_visible()) {
-               output_selector->get_toplevel()->get_window()->raise();
-       } else {
-               output_selector->present ();
        }
 
-       output_selector->set_keep_above (true);
+       return false;
 }
 
-void
-MixerStrip::edit_input_configuration ()
-{
-       if (input_selector == 0) {
-               input_selector = new IOSelectorWindow (_session, _route->input());
-       }
-
-       if (input_selector->is_visible()) {
-               input_selector->get_toplevel()->get_window()->raise();
-       } else {
-               input_selector->present ();
-       }
-
-       input_selector->set_keep_above (true);
-}
 
 gint
 MixerStrip::input_press (GdkEventButton *ev)
@@ -837,8 +882,7 @@ MixerStrip::input_press (GdkEventButton *ev)
        switch (ev->button) {
 
        case 1:
-               edit_input_configuration ();
-               break;
+               return false;  //don't handle the mouse-down here.  wait for mouse-up to pop the menu
 
        case 3:
        {
@@ -988,18 +1032,11 @@ MixerStrip::maybe_add_bundle_to_output_menu (boost::shared_ptr<Bundle> b, ARDOUR
 void
 MixerStrip::update_diskstream_display ()
 {
-       if (is_track()) {
+        if (is_track() && input_selector) {
+                        input_selector->hide_all ();
+        }
 
-               if (input_selector) {
-                       input_selector->hide_all ();
-               }
-
-               route_color_changed ();
-
-       } else {
-
-               show_passthru_color ();
-       }
+        route_color_changed ();
 }
 
 void
@@ -1019,9 +1056,33 @@ MixerStrip::connect_to_pan ()
        p->automation_state_changed.connect (panstate_connection, invalidator (*this), boost::bind (&PannerUI::pan_automation_state_changed, &panners), gui_context());
        p->automation_style_changed.connect (panstyle_connection, invalidator (*this), boost::bind (&PannerUI::pan_automation_style_changed, &panners), gui_context());
 
-       panners.panshell_changed ();
+       /* This call reduncant, PannerUI::set_panner() connects to _panshell->Changed itself
+        * However, that only works a panner was previously set.
+        *
+        * PannerUI must remain subscribed to _panshell->Changed() in case
+        * we switch the panner eg. AUX-Send and back
+        * _route->panner_shell()->Changed() vs _panshell->Changed
+        */
+       if (panners._panner == 0) {
+               panners.panshell_changed ();
+       }
+       update_panner_choices();
 }
 
+void
+MixerStrip::update_panner_choices ()
+{
+       ENSURE_GUI_THREAD (*this, &MixerStrip::update_panner_choices)
+       if (!_route->panner_shell()) { return; }
+
+       uint32_t in = _route->output()->n_ports().n_audio();
+       uint32_t out = in;
+       if (_route->panner()) {
+               in = _route->panner()->in().n_audio();
+       }
+
+       panners.set_available_panners(PannerManager::instance().PannerManager::get_available_panners(in, out));
+}
 
 /*
  * Output port labelling
@@ -1272,6 +1333,12 @@ MixerStrip::diskstream_changed ()
        Gtkmm2ext::UI::instance()->call_slot (invalidator (*this), boost::bind (&MixerStrip::update_diskstream_display, this));
 }
 
+void
+MixerStrip::io_changed_proxy ()
+{
+       Glib::signal_idle().connect_once (sigc::mem_fun (*this, &MixerStrip::update_panner_choices));
+}
+
 void
 MixerStrip::port_connected_or_disconnected (boost::weak_ptr<Port> wa, boost::weak_ptr<Port> wb)
 {
@@ -1318,79 +1385,7 @@ MixerStrip::setup_comment_button ()
        ARDOUR_UI::instance()->set_tip (
                _comment_button, _route->comment().empty() ? _("Click to Add/Edit Comments") : _route->comment()
                );
-}
-
-void
-MixerStrip::comment_editor_done_editing ()
-{
-       string const str = comment_area->get_buffer()->get_text();
-       if (str == _route->comment ()) {
-               return;
-       }
-
-       _route->set_comment (str, this);
-       setup_comment_button ();
-}
-
-void
-MixerStrip::toggle_comment_editor ()
-{
-       if (ignore_toggle) {
-               return;
-       }
-
-       if (comment_window && comment_window->is_visible ()) {
-               comment_window->hide ();
-       } else {
-               open_comment_editor ();
-       }
-}
-
-void
-MixerStrip::open_comment_editor ()
-{
-       if (comment_window == 0) {
-               setup_comment_editor ();
-       }
-
-       string title;
-       title = _route->name();
-       title += _(": comment editor");
 
-       comment_window->set_title (title);
-       comment_window->present();
-}
-
-void
-MixerStrip::setup_comment_editor ()
-{
-       comment_window = new ArdourWindow (""); // title will be reset to show route
-       comment_window->set_skip_taskbar_hint (true);
-       comment_window->signal_hide().connect (sigc::mem_fun(*this, &MixerStrip::comment_editor_done_editing));
-       comment_window->set_default_size (400, 200);
-
-       comment_area = manage (new TextView());
-       comment_area->set_name ("MixerTrackCommentArea");
-       comment_area->set_wrap_mode (WRAP_WORD);
-       comment_area->set_editable (true);
-       comment_area->get_buffer()->set_text (_route->comment());
-       comment_area->show ();
-
-       comment_window->add (*comment_area);
-}
-
-void
-MixerStrip::comment_changed (void *src)
-{
-       ENSURE_GUI_THREAD (*this, &MixerStrip::comment_changed, src)
-
-       if (src != this) {
-               ignore_comment_edit = true;
-               if (comment_area) {
-                       comment_area->get_buffer()->set_text (_route->comment());
-               }
-               ignore_comment_edit = false;
-       }
 }
 
 bool
@@ -1445,7 +1440,7 @@ void
 MixerStrip::route_color_changed ()
 {
        name_button.modify_bg (STATE_NORMAL, color());
-       top_event_box.modify_bg (STATE_NORMAL, color());
+       number_label.set_fixed_colors (gdk_color_to_rgba (color()), gdk_color_to_rgba (color()));
        reset_strip_style ();
 }
 
@@ -1464,7 +1459,16 @@ MixerStrip::build_route_ops_menu ()
 
        MenuList& items = route_ops_menu->items();
 
-       items.push_back (MenuElem (_("Comments..."), sigc::mem_fun (*this, &MixerStrip::open_comment_editor)));
+       items.push_back (MenuElem (_("Color..."), sigc::mem_fun (*this, &RouteUI::choose_color)));
+
+       items.push_back (MenuElem (_("Comments..."), sigc::mem_fun (*this, &RouteUI::open_comment_editor)));
+
+       items.push_back (MenuElem (_("Inputs..."), sigc::mem_fun (*this, &RouteUI::edit_input_configuration)));
+
+       items.push_back (MenuElem (_("Outputs..."), sigc::mem_fun (*this, &RouteUI::edit_output_configuration)));
+
+       items.push_back (SeparatorElem());
+
        if (!_route->is_master()) {
                items.push_back (MenuElem (_("Save As Template..."), sigc::mem_fun(*this, &RouteUI::save_as_template)));
        }
@@ -1475,6 +1479,7 @@ MixerStrip::build_route_ops_menu ()
        items.push_back (CheckMenuElem (_("Active")));
        Gtk::CheckMenuItem* i = dynamic_cast<Gtk::CheckMenuItem *> (&items.back());
        i->set_active (_route->active());
+       i->set_sensitive(! _session->transport_rolling());
        i->signal_activate().connect (sigc::bind (sigc::mem_fun (*this, &RouteUI::set_route_active), !_route->active(), false));
 
        items.push_back (SeparatorElem());
@@ -1498,16 +1503,44 @@ MixerStrip::build_route_ops_menu ()
 gboolean
 MixerStrip::name_button_button_press (GdkEventButton* ev)
 {
-       /* show menu for either button 1 or 3, so as not to confuse people
-          and also not hide stuff from them.
-       */
+       if (ev->button == 3) {
+               list_route_operations ();
+
+               /* do not allow rename if the track is record-enabled */
+               rename_menu_item->set_sensitive (!_route->record_enabled());
+               route_ops_menu->popup (1, ev->time);
+
+               return true;
+       }
+
+       return false;
+}
+
+gboolean
+MixerStrip::name_button_button_release (GdkEventButton* ev)
+{
+       if (ev->button == 1) {
+               list_route_operations ();
+
+               /* do not allow rename if the track is record-enabled */
+               rename_menu_item->set_sensitive (!_route->record_enabled());
+               route_ops_menu->popup (1, ev->time);
+       }
+
+       return false;
+}
 
-       if (ev->button == 3 || ev->button == 1) {
+gboolean
+MixerStrip::number_button_button_press (GdkEventButton* ev)
+{
+       if (  ev->button == 3 ) {
                list_route_operations ();
 
                /* do not allow rename if the track is record-enabled */
                rename_menu_item->set_sensitive (!_route->record_enabled());
                route_ops_menu->popup (1, ev->time);
+               
+               return true;
        }
 
        return false;
@@ -1532,6 +1565,9 @@ MixerStrip::set_selected (bool yn)
                global_frame.set_name ("MixerStripFrame");
        }
        global_frame.queue_draw ();
+       
+//     if (!yn)
+//             processor_box.deselect_all_processors();
 }
 
 void
@@ -1548,21 +1584,32 @@ void
 MixerStrip::name_changed ()
 {
        switch (_width) {
-       case Wide:
-               name_button.set_text (_route->name());
-               break;
-       case Narrow:
-               name_button.set_text (PBD::short_version (_route->name(), 5));
-               break;
+               case Wide:
+                       name_button.set_text (_route->name());
+                       break;
+               case Narrow:
+                       name_button.set_text (PBD::short_version (_route->name(), 5));
+                       break;
        }
 
        ARDOUR_UI::instance()->set_tip (name_button, _route->name());
+
+       if (_session->config.get_track_name_number()) {
+               const int64_t track_number = _route->track_number ();
+               if (track_number == 0) {
+                       number_label.set_text ("-");
+               } else {
+                       number_label.set_text (PBD::to_string (abs(_route->track_number ()), std::dec));
+               }
+       } else {
+               number_label.set_text ("");
+       }
 }
 
 void
 MixerStrip::name_button_resized (Gtk::Allocation& alloc)
 {
-       name_button.layout()->set_width (alloc.get_width() * PANGO_SCALE);
+       name_button.set_layout_ellisize_width (alloc.get_width() * PANGO_SCALE);
 }
 
 bool
@@ -1714,48 +1761,48 @@ MixerStrip::meter_point_string (MeterPoint mp)
        case Wide:
                switch (mp) {
                case MeterInput:
-                       return _("in");
+                       return _("In");
                        break;
                        
                case MeterPreFader:
-                       return _("pre");
+                       return _("Pre");
                        break;
                        
                case MeterPostFader:
-                       return _("post");
+                       return _("Post");
                        break;
                        
                case MeterOutput:
-                       return _("out");
+                       return _("Out");
                        break;
                        
                case MeterCustom:
                default:
-                       return _("custom");
+                       return _("Custom");
                        break;
                }
                break;
        case Narrow:
                switch (mp) {
                case MeterInput:
-                       return _("in");
+                       return _("In");
                        break;
                        
                case MeterPreFader:
-                       return _("pr");
+                       return _("Pr");
                        break;
                        
                case MeterPostFader:
-                       return _("po");
+                       return _("Po");
                        break;
                        
                case MeterOutput:
-                       return _("o");
+                       return _("O");
                        break;
                        
                case MeterCustom:
                default:
-                       return _("c");
+                       return _("C");
                        break;
                }
                break;
@@ -1836,22 +1883,21 @@ MixerStrip::show_send (boost::shared_ptr<Send> send)
 
        set_current_delivery (send);
 
+       send->meter()->set_type(_route->shared_peak_meter()->get_type());
        send->set_metering (true);
        _current_delivery->DropReferences.connect (send_gone_connection, invalidator (*this), boost::bind (&MixerStrip::revert_to_default_display, this), gui_context());
 
        gain_meter().set_controls (_route, send->meter(), send->amp());
        gain_meter().setup_meters ();
 
+       uint32_t const in = _current_delivery->pans_required();
+       uint32_t const out = _current_delivery->pan_outs();
+
        panner_ui().set_panner (_current_delivery->panner_shell(), _current_delivery->panner());
+       panner_ui().set_available_panners(PannerManager::instance().PannerManager::get_available_panners(in, out));
        panner_ui().setup_pan ();
-
-       /* make sure the send has audio output */
-
-       if (_current_delivery->output() && _current_delivery->output()->n_ports().n_audio() > 0) {
-               panners.show_all ();
-       } else {
-               panners.hide_all ();
-       }
+       panner_ui().set_send_drawing_mode (true);
+       panner_ui().show_all ();
 
        input_button.set_sensitive (false);
        group_button.set_sensitive (false);
@@ -1884,7 +1930,9 @@ MixerStrip::revert_to_default_display ()
        gain_meter().setup_meters ();
 
        panner_ui().set_panner (_route->main_outs()->panner_shell(), _route->main_outs()->panner());
+       update_panner_choices();
        panner_ui().setup_pan ();
+       panner_ui().set_send_drawing_mode (false);
 
        if (has_audio_outputs ()) {
                panners.show_all ();
@@ -1900,7 +1948,6 @@ MixerStrip::set_button_names ()
 {
        switch (_width) {
        case Wide:
-               rec_enable_button->set_text (_("Rec"));
                mute_button->set_text (_("Mute"));
                monitor_input_button->set_text (_("In"));
                monitor_disk_button->set_text (_("Disk"));
@@ -1922,12 +1969,11 @@ MixerStrip::set_button_names ()
                                break;
                        }
                }
-               solo_isolated_led->set_text (_("iso"));
-               solo_safe_led->set_text (_("lock"));
+               solo_isolated_led->set_text (_("Iso"));
+               solo_safe_led->set_text (_("Lock"));
                break;
 
        default:
-               rec_enable_button->set_text (_("R"));
                mute_button->set_text (_("M"));
                monitor_input_button->set_text (_("I"));
                monitor_disk_button->set_text (_("D"));
@@ -1950,7 +1996,7 @@ MixerStrip::set_button_names ()
                        }
                }
 
-               solo_isolated_led->set_text (_("i"));
+               solo_isolated_led->set_text (_("I"));
                solo_safe_led->set_text (_("L"));
                break;
        }
@@ -2025,6 +2071,9 @@ MixerStrip::parameter_changed (string p)
                */
                _visibility.set_state (Config->get_mixer_strip_visibility ());
        }
+       else if (p == "track-name-number") {
+               name_changed ();
+       }
 }
 
 /** Called to decide whether the solo isolate / solo lock button visibility should
@@ -2085,9 +2134,15 @@ MixerStrip::select_all_processors ()
 }
 
 void
+MixerStrip::deselect_all_processors ()
+{
+       processor_box.processor_operation (ProcessorBox::ProcessorsSelectNone);
+}
+
+bool
 MixerStrip::delete_processors ()
 {
-       processor_box.processor_operation (ProcessorBox::ProcessorsDelete);
+       return processor_box.processor_operation (ProcessorBox::ProcessorsDelete);
 }
 
 void
@@ -2105,6 +2160,9 @@ MixerStrip::ab_plugins ()
 bool
 MixerStrip::level_meter_button_press (GdkEventButton* ev)
 {
+       if (_current_delivery && boost::dynamic_pointer_cast<Send>(_current_delivery)) {
+               return false;
+       }
        if (ev->button == 3) {
                popup_level_meter_menu (ev);
                return true;
@@ -2125,11 +2183,17 @@ MixerStrip::popup_level_meter_menu (GdkEventButton* ev)
 
        _suspend_menu_callbacks = true;
        add_level_meter_item_point (items, group, _("Input"), MeterInput);
-       add_level_meter_item_point (items, group, _("Pre-fader"), MeterPreFader);
-       add_level_meter_item_point (items, group, _("Post-fader"), MeterPostFader);
+       add_level_meter_item_point (items, group, _("Pre Fader"), MeterPreFader);
+       add_level_meter_item_point (items, group, _("Post Fader"), MeterPostFader);
        add_level_meter_item_point (items, group, _("Output"), MeterOutput);
        add_level_meter_item_point (items, group, _("Custom"), MeterCustom);
 
+       if (gpm.meter_channels().n_audio() == 0) {
+               m->popup (ev->button, ev->time);
+               _suspend_menu_callbacks = false;
+               return;
+       }
+
        RadioMenuItem::Group tgroup;
        items.push_back (SeparatorElem());
 
@@ -2141,6 +2205,7 @@ MixerStrip::popup_level_meter_menu (GdkEventButton* ev)
        add_level_meter_item_type (items, tgroup, ArdourMeter::meter_type_string(MeterIEC2EBU), MeterIEC2EBU);
        add_level_meter_item_type (items, tgroup, ArdourMeter::meter_type_string(MeterK20), MeterK20);
        add_level_meter_item_type (items, tgroup, ArdourMeter::meter_type_string(MeterK14), MeterK14);
+       add_level_meter_item_type (items, tgroup, ArdourMeter::meter_type_string(MeterK12), MeterK12);
        add_level_meter_item_type (items, tgroup, ArdourMeter::meter_type_string(MeterVU),  MeterVU);
 
        int _strip_type;