return to the old way of editing track names, but hopefully better than it was. likel...
[ardour.git] / gtk2_ardour / mixer_strip.cc
index bf82130b95e441585109e6f8150da37036231513..0f1d78d5bff567cd64b9fd8a2bef39c1d8d9ccae 100644 (file)
 #include <gtkmm2ext/slider_controller.h>
 #include <gtkmm2ext/bindable_button.h>
 
-#include "ardour/ardour.h"
-#include "ardour/amp.h"
-#include "ardour/session.h"
+#include "ardour/audio_track.h"
 #include "ardour/audioengine.h"
 #include "ardour/internal_send.h"
-#include "ardour/route.h"
-#include "ardour/route_group.h"
-#include "ardour/audio_track.h"
 #include "ardour/midi_track.h"
 #include "ardour/pannable.h"
 #include "ardour/panner.h"
 #include "ardour/panner_shell.h"
-#include "ardour/send.h"
-#include "ardour/processor.h"
+#include "ardour/port.h"
 #include "ardour/profile.h"
-#include "ardour/ladspa_plugin.h"
+#include "ardour/route.h"
+#include "ardour/route_group.h"
+#include "ardour/send.h"
+#include "ardour/session.h"
+#include "ardour/types.h"
 #include "ardour/user_bundle.h"
-#include "ardour/port.h"
 
 #include "ardour_ui.h"
-#include "ardour_dialog.h"
+#include "ardour_window.h"
 #include "mixer_strip.h"
 #include "mixer_ui.h"
 #include "keyboard.h"
@@ -91,7 +88,7 @@ MixerStrip::MixerStrip (Mixer_UI& mx, Session* sess, bool in_mixer)
        , top_button_table (1, 2)
        , middle_button_table (1, 2)
        , bottom_button_table (1, 2)
-       , meter_point_label (_("pre"))
+       , meter_point_button (_("pre"))
        , midi_input_enable_button (0)
        , _comment_button (_("Comments"))
        , _visibility (X_("mixer-strip-visibility"))
@@ -112,14 +109,14 @@ MixerStrip::MixerStrip (Mixer_UI& mx, Session* sess, boost::shared_ptr<Route> rt
        , RouteUI (sess)
        , _mixer(mx)
        , _mixer_owned (in_mixer)
-       , processor_box (sess, sigc::mem_fun(*this, &MixerStrip::plugin_selector), mx.selection(), this, in_mixer)
+       , processor_box (sess, boost::bind (&MixerStrip::plugin_selector, this), mx.selection(), this, in_mixer)
        , 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)
-       , meter_point_label (_("pre"))
+       , meter_point_button (_("pre"))
        , midi_input_enable_button (0)
        , _comment_button (_("Comments"))
        , _visibility (X_("mixer-strip-visibility"))
@@ -145,36 +142,28 @@ MixerStrip::init ()
        /* the length of this string determines the width of the mixer strip when it is set to `wide' */
        longest_label = "longest label";
 
-       Gtk::Image* img;
-
-       img = manage (new Gtk::Image (::get_icon("strip_width")));
-       img->show ();
-
-       width_button.add (*img);
-
-       img = manage (new Gtk::Image (::get_icon("hide")));
-       img->show ();
+       string t = _("Click to toggle the width of this mixer strip.");
+       if (_mixer_owned) {
+               t += string_compose (_("\n%1-%2-click to toggle the width of all strips."), Keyboard::primary_modifier_name(), Keyboard::tertiary_modifier_name ());
+       }
+       
+       width_button.set_image (::get_icon("strip_width"));
+       ARDOUR_UI::instance()->set_tip (width_button, t);
 
-       hide_button.add (*img);
+       hide_button.set_image(::get_icon("hide"));
+       ARDOUR_UI::instance()->set_tip (&hide_button, _("Hide this mixer strip"));
 
-       input_label.set_text (_("Input"));
-       ARDOUR_UI::instance()->set_tip (&input_button, _("Button 1 to choose inputs from a port matrix, button 3 to select inputs from a menu"), "");
-       input_button.add (input_label);
-       input_button.set_name ("MixerIOButton");
-       input_label.set_name ("MixerIOButtonLabel");
+       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_label.set_text (_("Output"));
-       ARDOUR_UI::instance()->set_tip (&output_button, _("Button 1 to choose outputs from a port matrix, button 3 to select inputs from a menu"), "");
-       output_button.add (output_label);
-       output_button.set_name ("MixerIOButton");
-       output_label.set_name ("MixerIOButtonLabel");
+       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, _("Select metering point"), "");
-       meter_point_button.add (meter_point_label);
-       meter_point_button.set_name ("MixerStripMeterPreButton");
-       meter_point_label.set_name ("MixerStripMeterPreButton");
+       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".
@@ -218,7 +207,7 @@ MixerStrip::init ()
         top_button_table.attach (*monitor_disk_button, 1, 2, 0, 1);
        top_button_table.show ();
 
-       rec_solo_table.set_homogeneous (false);
+       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);
@@ -226,7 +215,7 @@ MixerStrip::init ()
         rec_solo_table.show ();
 
        button_table.set_homogeneous (false);
-       button_table.set_spacings (0);
+       button_table.set_spacings (2);
 
        if (solo_isolated_led) {
                button_size_group->add_widget (*solo_isolated_led);
@@ -251,33 +240,35 @@ MixerStrip::init ()
        middle_button_table.set_homogeneous (true);
        middle_button_table.set_spacings (2);
 
-       bottom_button_table.set_col_spacings (0);
+       bottom_button_table.set_spacings (2);
        bottom_button_table.set_homogeneous (true);
-       bottom_button_table.attach (group_button, 0, 1, 0, 1);
-
-       name_button.add (name_label);
-       name_button.set_name ("MixerNameButton");
+//     bottom_button_table.attach (group_button, 0, 1, 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.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);
 
-       name_label.set_name ("MixerNameButtonLabel");
        ARDOUR_UI::instance()->set_tip (&group_button, _("Mix group"), "");
-       group_button.add (group_label);
-       group_button.set_name ("MixerGroupButton");
+       group_button.set_name ("mixer strip button");
        Gtkmm2ext::set_size_request_to_display_given_text (group_button, "Group", 2, 2);
-       group_label.set_name ("MixerGroupButtonLabel");
 
-       _comment_button.set_name (X_("MixerCommentButton"));
-       _comment_button.signal_clicked().connect (sigc::mem_fun (*this, &MixerStrip::toggle_comment));
+       _comment_button.set_name (X_("mixer strip button"));
+       _comment_button.signal_clicked.connect (sigc::mem_fun (*this, &MixerStrip::toggle_comment_editor));
 
        global_vpacker.set_border_width (0);
        global_vpacker.set_spacing (0);
 
-       width_button.set_name ("MixerWidthButton");
-       hide_button.set_name ("MixerHideButton");
-       top_event_box.set_name ("MixerTopEventBox");
+       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_clicked().connect (sigc::mem_fun(*this, &MixerStrip::width_clicked));
-       hide_button.signal_clicked().connect (sigc::mem_fun(*this, &MixerStrip::hide_clicked));
+       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.pack_start (width_button, false, true);
        width_hide_box.pack_start (top_event_box, true, true);
@@ -285,13 +276,14 @@ MixerStrip::init ()
 
        whvbox.pack_start (width_hide_box, true, 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);
        global_vpacker.pack_start (panners, Gtk::PACK_SHRINK);
-       global_vpacker.pack_start (top_button_table, Gtk::PACK_SHRINK, 2);
-       global_vpacker.pack_start (rec_solo_table, Gtk::PACK_SHRINK, 2);
-       global_vpacker.pack_start (middle_button_table, Gtk::PACK_SHRINK, 2);
+       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 (gpm, Gtk::PACK_SHRINK);
        global_vpacker.pack_start (bottom_button_table, Gtk::PACK_SHRINK);
        global_vpacker.pack_start (output_button, Gtk::PACK_SHRINK);
@@ -363,7 +355,7 @@ MixerStrip::init ()
 
        parameter_changed (X_("mixer-strip-visibility"));
 
-       Config->ParameterChanged.connect (_config_connection, MISSING_INVALIDATOR, ui_bind (&MixerStrip::parameter_changed, this, _1), gui_context());
+       Config->ParameterChanged.connect (_config_connection, MISSING_INVALIDATOR, boost::bind (&MixerStrip::parameter_changed, this, _1), gui_context());
 
        gpm.LevelMeterButtonPress.connect_same_thread (_level_meter_connection, boost::bind (&MixerStrip::level_meter_button_press, this, _1));
 }
@@ -388,10 +380,13 @@ MixerStrip::set_route (boost::shared_ptr<Route> rt)
                rec_solo_table.remove (*show_sends_button);
        }
 
-       processor_box.set_route (rt);
-
        RouteUI::set_route (rt);
 
+       /* ProcessorBox needs access to _route so that it can read
+          GUI object state.
+       */
+       processor_box.set_route (rt);
+
        /* map the current state */
 
        mute_changed (0);
@@ -405,6 +400,14 @@ MixerStrip::set_route (boost::shared_ptr<Route> rt)
 
        revert_to_default_display ();
 
+       if (gpm.gain_display.get_parent()) {
+               middle_button_table.remove (gpm.gain_display);
+       }
+
+       if (gpm.peak_display.get_parent()) {
+               middle_button_table.remove (gpm.peak_display);
+       }
+
        if (solo_button->get_parent()) {
                middle_button_table.remove (*solo_button);
        }
@@ -449,10 +452,9 @@ MixerStrip::set_route (boost::shared_ptr<Route> rt)
 
        if (is_midi_track()) {
                if (midi_input_enable_button == 0) {
-                       Image* img = manage (new Image (get_icon (X_("midi_socket_small"))));
-                       midi_input_enable_button = manage (new StatefulToggleButton);
-                       midi_input_enable_button->set_name ("MixerMidiInputEnableButton");
-                       midi_input_enable_button->set_image (*img);
+                       midi_input_enable_button = manage (new ArdourButton);
+                       midi_input_enable_button->set_name ("midi input button");
+                       midi_input_enable_button->set_image (::get_icon (X_("midi_socket_small")));
                        midi_input_enable_button->signal_button_press_event().connect (sigc::mem_fun (*this, &MixerStrip::input_active_button_press), false);
                        midi_input_enable_button->signal_button_release_event().connect (sigc::mem_fun (*this, &MixerStrip::input_active_button_release), false);
                        ARDOUR_UI::instance()->set_tip (midi_input_enable_button, _("Enable/Disable MIDI input"));
@@ -499,7 +501,7 @@ MixerStrip::set_route (boost::shared_ptr<Route> rt)
                }
        }
 
-       meter_point_label.set_text (meter_point_string (_route->meter_point()));
+       meter_point_button.set_text (meter_point_string (_route->meter_point()));
 
        delete route_ops_menu;
        route_ops_menu = 0;
@@ -515,7 +517,8 @@ 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), ui_bind (&MixerStrip::comment_changed, this, _1), gui_context());
+       _route->comment_changed.connect (route_connections, invalidator (*this), boost::bind (&MixerStrip::comment_changed, this, _1), gui_context());
+       _route->PropertyChanged.connect (route_connections, invalidator (*this), boost::bind (&MixerStrip::property_changed, this, _1), gui_context());
 
        set_stuff_from_route ();
 
@@ -543,6 +546,10 @@ MixerStrip::set_route (boost::shared_ptr<Route> rt)
                hide_button.show();
        }
 
+       gpm.reset_peak_display ();
+       gpm.gain_display.show ();
+       gpm.peak_display.show ();
+
        width_button.show();
        width_hide_box.show();
        whvbox.show ();
@@ -552,20 +559,13 @@ MixerStrip::set_route (boost::shared_ptr<Route> rt)
        middle_button_table.show();
        bottom_button_table.show();
        gpm.show_all ();
-       gain_unit_button.show();
-       gain_unit_label.show();
        meter_point_button.show();
-       meter_point_label.show();
-       diskstream_button.show();
-       diskstream_label.show();
        input_button_box.show_all();
        output_button.show();
-       output_label.show();
-       name_label.show();
        name_button.show();
        _comment_button.show();
        group_button.show();
-       group_label.show();
+       gpm.gain_automation_state_button.show();
 
        parameter_changed ("mixer-strip-visibility");
 
@@ -605,13 +605,24 @@ MixerStrip::set_width_enum (Width w, void* owner)
 
        switch (w) {
        case Wide:
+               if (!gpm.peak_display.get_parent()) {
+                       middle_button_table.attach (gpm.peak_display,1,2,1,2);
+               }
+               if (gpm.gain_display.get_parent()) {
+                       middle_button_table.remove (gpm.gain_display);
+               }
+               if (!gpm.gain_display.get_parent()) {
+                       middle_button_table.attach (gpm.gain_display,0,1,1,2);
+               }
+
                if (show_sends_button)  {
-                       show_sends_button->set_text (_("Sends"));
+                       show_sends_button->set_text (_("Aux\nSends"));
+                       show_sends_button->layout()->set_alignment (Pango::ALIGN_CENTER);
                }
 
-               ((Gtk::Label*)gpm.gain_automation_style_button.get_child())->set_text (
+               gpm.gain_automation_style_button.set_text (
                                gpm.astyle_string(gain_automation->automation_style()));
-               ((Gtk::Label*)gpm.gain_automation_state_button.get_child())->set_text (
+               gpm.gain_automation_state_button.set_text (
                                gpm.astate_string(gain_automation->automation_state()));
 
                if (_route->panner()) {
@@ -624,18 +635,29 @@ MixerStrip::set_width_enum (Width w, void* owner)
                solo_isolated_led->set_text (_("iso"));
                solo_safe_led->set_text (_("lock"));
 
-               Gtkmm2ext::set_size_request_to_display_given_text (name_button, "long", 2, 2);
+               Gtkmm2ext::set_size_request_to_display_given_text (name_button, longest_label.c_str(), 2, 2);
                set_size_request (-1, -1);
                break;
 
        case Narrow:
+               if (gpm.peak_display.get_parent()) {
+                       middle_button_table.remove (gpm.peak_display);
+               }
+
+               if (gpm.gain_display.get_parent()) {
+                       middle_button_table.remove (gpm.gain_display);
+               }
+               if (!gpm.gain_display.get_parent()) {
+                       middle_button_table.attach (gpm.gain_display,0,2,1,2);
+               }
+
                if (show_sends_button) {
                        show_sends_button->set_text (_("Snd"));
                }
 
-               ((Gtk::Label*)gpm.gain_automation_style_button.get_child())->set_text (
+               gpm.gain_automation_style_button.set_text (
                                gpm.short_astyle_string(gain_automation->automation_style()));
-               ((Gtk::Label*)gpm.gain_automation_state_button.get_child())->set_text (
+               gpm.gain_automation_state_button.set_text (
                                gpm.short_astate_string(gain_automation->automation_state()));
 
                if (_route->panner()) {
@@ -645,10 +667,10 @@ MixerStrip::set_width_enum (Width w, void* owner)
                        panners.short_astate_string(_route->panner()->automation_state()));
                }
                
-               solo_isolated_led->set_text ("");
-               solo_safe_led->set_text ("");
+               solo_isolated_led->set_text (_("iso"));
+               solo_safe_led->set_text (_("Lck"));
 
-               Gtkmm2ext::set_size_request_to_display_given_text (name_button, longest_label.c_str(), 2, 2);
+               Gtkmm2ext::set_size_request_to_display_given_text (name_button, "long", 2, 2);
                set_size_request (max (50, gpm.get_gm_width()), -1);
                break;
        }
@@ -717,6 +739,7 @@ MixerStrip::output_press (GdkEventButton *ev)
                }
                
                citems.push_back (SeparatorElem());
+               uint32_t const n_with_separator = citems.size ();
 
                ARDOUR::BundleList current = _route->output()->bundles_connected ();
 
@@ -743,7 +766,7 @@ MixerStrip::output_press (GdkEventButton *ev)
                        maybe_add_bundle_to_output_menu ((*i)->input()->bundle(), current);
                }
 
-               if (citems.size() == 2) {
+               if (citems.size() == n_with_separator) {
                        /* no routes added; remove the separator */
                        citems.pop_back ();
                }
@@ -784,6 +807,8 @@ MixerStrip::edit_output_configuration ()
        } else {
                output_selector->present ();
        }
+
+       output_selector->set_keep_above (true);
 }
 
 void
@@ -798,6 +823,8 @@ MixerStrip::edit_input_configuration ()
        } else {
                input_selector->present ();
        }
+
+       input_selector->set_keep_above (true);
 }
 
 gint
@@ -838,6 +865,8 @@ MixerStrip::input_press (GdkEventButton *ev)
                }
 
                citems.push_back (SeparatorElem());
+               uint32_t const n_with_separator = citems.size ();
+               
                input_menu_bundles.clear ();
 
                ARDOUR::BundleList current = _route->input()->bundles_connected ();
@@ -865,7 +894,7 @@ MixerStrip::input_press (GdkEventButton *ev)
                        maybe_add_bundle_to_input_menu ((*i)->output()->bundle(), current);
                }
 
-               if (citems.size() == 2) {
+               if (citems.size() == n_with_separator) {
                        /* no routes added; remove the separator */
                        citems.pop_back ();
                }
@@ -880,7 +909,7 @@ MixerStrip::input_press (GdkEventButton *ev)
 }
 
 void
-MixerStrip::bundle_input_toggled (boost::shared_ptr<ARDOUR::Bundle> c)
+MixerStrip::bundle_input_chosen (boost::shared_ptr<ARDOUR::Bundle> c)
 {
        if (ignore_toggle) {
                return;
@@ -889,14 +918,14 @@ MixerStrip::bundle_input_toggled (boost::shared_ptr<ARDOUR::Bundle> c)
        ARDOUR::BundleList current = _route->input()->bundles_connected ();
 
        if (std::find (current.begin(), current.end(), c) == current.end()) {
-               _route->input()->connect_ports_to_bundle (c, this);
+               _route->input()->connect_ports_to_bundle (c, true, this);
        } else {
                _route->input()->disconnect_ports_from_bundle (c, this);
        }
 }
 
 void
-MixerStrip::bundle_output_toggled (boost::shared_ptr<ARDOUR::Bundle> c)
+MixerStrip::bundle_output_chosen (boost::shared_ptr<ARDOUR::Bundle> c)
 {
        if (ignore_toggle) {
                return;
@@ -905,14 +934,14 @@ MixerStrip::bundle_output_toggled (boost::shared_ptr<ARDOUR::Bundle> c)
        ARDOUR::BundleList current = _route->output()->bundles_connected ();
 
        if (std::find (current.begin(), current.end(), c) == current.end()) {
-               _route->output()->connect_ports_to_bundle (c, this);
+               _route->output()->connect_ports_to_bundle (c, true, this);
        } else {
                _route->output()->disconnect_ports_from_bundle (c, this);
        }
 }
 
 void
-MixerStrip::maybe_add_bundle_to_input_menu (boost::shared_ptr<Bundle> b, ARDOUR::BundleList const & current)
+MixerStrip::maybe_add_bundle_to_input_menu (boost::shared_ptr<Bundle> b, ARDOUR::BundleList const& /*current*/)
 {
        using namespace Menu_Helpers;
 
@@ -936,17 +965,11 @@ MixerStrip::maybe_add_bundle_to_input_menu (boost::shared_ptr<Bundle> b, ARDOUR:
        std::string n = b->name ();
        replace_all (n, "_", " ");
 
-       citems.push_back (CheckMenuElem (n, sigc::bind (sigc::mem_fun(*this, &MixerStrip::bundle_input_toggled), b)));
-
-       if (std::find (current.begin(), current.end(), b) != current.end()) {
-               ignore_toggle = true;
-               dynamic_cast<CheckMenuItem *> (&citems.back())->set_active (true);
-               ignore_toggle = false;
-       }
+       citems.push_back (MenuElem (n, sigc::bind (sigc::mem_fun(*this, &MixerStrip::bundle_input_chosen), b)));
 }
 
 void
-MixerStrip::maybe_add_bundle_to_output_menu (boost::shared_ptr<Bundle> b, ARDOUR::BundleList const & current)
+MixerStrip::maybe_add_bundle_to_output_menu (boost::shared_ptr<Bundle> b, ARDOUR::BundleList const& /*current*/)
 {
        using namespace Menu_Helpers;
 
@@ -970,13 +993,7 @@ MixerStrip::maybe_add_bundle_to_output_menu (boost::shared_ptr<Bundle> b, ARDOUR
        std::string n = b->name ();
        replace_all (n, "_", " ");
 
-       citems.push_back (CheckMenuElem (n, sigc::bind (sigc::mem_fun(*this, &MixerStrip::bundle_output_toggled), b)));
-
-       if (std::find (current.begin(), current.end(), b) != current.end()) {
-               ignore_toggle = true;
-               dynamic_cast<CheckMenuItem *> (&citems.back())->set_active (true);
-               ignore_toggle = false;
-       }
+       citems.push_back (MenuElem (n, sigc::bind (sigc::mem_fun(*this, &MixerStrip::bundle_output_chosen), b)));
 }
 
 void
@@ -1085,14 +1102,15 @@ MixerStrip::update_io_button (boost::shared_ptr<ARDOUR::Route> route, Width widt
        ostringstream tooltip;
        char * tooltip_cstr;
 
-       tooltip << route->name();
-
        if (for_input) {
                io_count = route->n_inputs().n_total();
+               tooltip << string_compose (_("<b>INPUT</b> to %1"), Glib::Markup::escape_text(route->name()));
        } else {
                io_count = route->n_outputs().n_total();
+               tooltip << string_compose (_("<b>OUTPUT</b> from %1"), Glib::Markup::escape_text(route->name()));
        }
 
+
        for (io_index = 0; io_index < io_count; ++io_index) {
                if (for_input) {
                        port = route->input()->nth (io_index);
@@ -1109,9 +1127,9 @@ MixerStrip::update_io_button (boost::shared_ptr<ARDOUR::Route> route, Width widt
                                string& connection_name (*i);
 
                                if (io_connection_count == 0) {
-                                       tooltip << endl << port->name().substr(port->name().find("/") + 1) << " -> " << connection_name;
+                                       tooltip << endl << Glib::Markup::escape_text(port->name().substr(port->name().find("/") + 1)) << " -> " << Glib::Markup::escape_text(connection_name);
                                } else {
-                                       tooltip << ", " << connection_name;
+                                       tooltip << ", " << Glib::Markup::escape_text(connection_name);
                                }
 
                                if (connection_name.find("ardour:") == 0) {
@@ -1177,7 +1195,7 @@ MixerStrip::update_io_button (boost::shared_ptr<ARDOUR::Route> route, Width widt
        }
 
        if (each_io_has_one_connection) {
-               if ((total_connection_count == ardour_connection_count)) {
+               if (total_connection_count == ardour_connection_count) {
                        // all connections are to the same track in ardour
                        // "ardour:Master/" -> "Master"
                        string::size_type slash = ardour_track_name.find("/");
@@ -1211,7 +1229,7 @@ MixerStrip::update_io_button (boost::shared_ptr<ARDOUR::Route> route, Width widt
 
        switch (width) {
        case Wide:
-               label_string = label.str().substr(0, 6);
+               label_string = label.str().substr(0, 7);
                break;
        case Narrow:
                label_string = label.str().substr(0, 3);
@@ -1219,9 +1237,9 @@ MixerStrip::update_io_button (boost::shared_ptr<ARDOUR::Route> route, Width widt
        }
 
        if (for_input) {
-               input_label.set_text (label_string);
+               input_button.set_text (label_string);
        } else {
-               output_label.set_text (label_string);
+               output_button.set_text (label_string);
        }
 }
 
@@ -1277,20 +1295,20 @@ MixerStrip::setup_comment_button ()
        case Wide:
                if (_route->comment().empty ()) {
                        _comment_button.unset_bg (STATE_NORMAL);
-                       ((Gtk::Label *) _comment_button.get_child ())->set_text (_("Comments"));
+                       _comment_button.set_text (_("Comments"));
                } else {
                        _comment_button.modify_bg (STATE_NORMAL, color ());
-                       ((Gtk::Label *) _comment_button.get_child ())->set_text (_("*Comments*"));
+                       _comment_button.set_text (_("*Comments*"));
                }
                break;
 
        case Narrow:
                if (_route->comment().empty ()) {
                        _comment_button.unset_bg (STATE_NORMAL);
-                       ((Gtk::Label *) _comment_button.get_child ())->set_text (_("Cmt"));
+                       _comment_button.set_text (_("Cmt"));
                } else {
                        _comment_button.modify_bg (STATE_NORMAL, color ());
-                       ((Gtk::Label *) _comment_button.get_child ())->set_text (_("*Cmt*"));
+                       _comment_button.set_text (_("*Cmt*"));
                }
                break;
        }
@@ -1313,19 +1331,24 @@ MixerStrip::comment_editor_done_editing ()
 }
 
 void
-MixerStrip::toggle_comment ()
+MixerStrip::toggle_comment_editor ()
 {
        if (ignore_toggle) {
                return;
        }
 
-       if (comment_window == 0) {
-               setup_comment_editor ();
+       if (comment_window && comment_window->is_visible ()) {
+               comment_window->hide ();
+       } else {
+               open_comment_editor ();
        }
+}
 
-       if (comment_window->is_visible ()) {
-               comment_window->hide ();
-               return;
+void
+MixerStrip::open_comment_editor ()
+{
+       if (comment_window == 0) {
+               setup_comment_editor ();
        }
 
        string title;
@@ -1339,7 +1362,7 @@ MixerStrip::toggle_comment ()
 void
 MixerStrip::setup_comment_editor ()
 {
-       comment_window = new ArdourDialog ("", false); // title will be reset to show route
+       comment_window = new ArdourWindow (""); // title will be reset to show route
        comment_window->set_position (Gtk::WIN_POS_MOUSE);
        comment_window->set_skip_taskbar_hint (true);
        comment_window->signal_hide().connect (sigc::mem_fun(*this, &MixerStrip::comment_editor_done_editing));
@@ -1352,8 +1375,7 @@ MixerStrip::setup_comment_editor ()
        comment_area->get_buffer()->set_text (_route->comment());
        comment_area->show ();
 
-       comment_window->get_vbox()->pack_start (*comment_area);
-       comment_window->get_action_area()->hide();
+       comment_window->add (*comment_area);
 }
 
 void
@@ -1405,14 +1427,14 @@ MixerStrip::route_group_changed ()
        RouteGroup *rg = _route->route_group();
 
        if (rg) {
-               group_label.set_text (PBD::short_version (rg->name(), 5));
+               group_button.set_text (PBD::short_version (rg->name(), 5));
        } else {
                switch (_width) {
                case Wide:
-                       group_label.set_text (_("Grp"));
+                       group_button.set_text (_("Grp"));
                        break;
                case Narrow:
-                       group_label.set_text (_("~G"));
+                       group_button.set_text (_("~G"));
                        break;
                }
        }
@@ -1441,16 +1463,16 @@ MixerStrip::build_route_ops_menu ()
 
        MenuList& items = route_ops_menu->items();
 
-       items.push_back (CheckMenuElem (_("Comments..."), sigc::mem_fun (*this, &MixerStrip::toggle_comment)));
-       CheckMenuItem* i = dynamic_cast<CheckMenuItem*> (&items.back ());
-       i->set_active (comment_window && comment_window->is_visible ());
-       items.push_back (MenuElem (_("Save As Template..."), sigc::mem_fun(*this, &RouteUI::save_as_template)));
+       items.push_back (MenuElem (_("Comments..."), sigc::mem_fun (*this, &MixerStrip::open_comment_editor)));
+       if (!_route->is_master()) {
+               items.push_back (MenuElem (_("Save As Template..."), sigc::mem_fun(*this, &RouteUI::save_as_template)));
+       }
        items.push_back (MenuElem (_("Rename..."), sigc::mem_fun(*this, &RouteUI::route_rename)));
        rename_menu_item = &items.back();
 
        items.push_back (SeparatorElem());
        items.push_back (CheckMenuElem (_("Active")));
-       i = dynamic_cast<CheckMenuItem *> (&items.back());
+       CheckMenuItem* i = dynamic_cast<CheckMenuItem *> (&items.back());
        i->set_active (_route->active());
        i->signal_activate().connect (sigc::bind (sigc::mem_fun (*this, &RouteUI::set_route_active), !_route->active(), false));
 
@@ -1475,15 +1497,16 @@ MixerStrip::build_route_ops_menu ()
 gboolean
 MixerStrip::name_button_button_press (GdkEventButton* ev)
 {
-       if (ev->button == 3) {
+       /* 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 || 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);
-
-       } else if (ev->button == 1) {
-               revert_to_default_display ();
        }
 
        return false;
@@ -1510,30 +1533,66 @@ MixerStrip::set_selected (bool yn)
        global_frame.queue_draw ();
 }
 
+void
+MixerStrip::property_changed (const PropertyChange& what_changed)
+{
+       RouteUI::property_changed (what_changed);
+
+       if (what_changed.contains (ARDOUR::Properties::name)) {
+               name_changed ();
+       }
+}
+
 void
 MixerStrip::name_changed ()
 {
        switch (_width) {
        case Wide:
-               RouteUI::property_changed (PropertyChange (ARDOUR::Properties::name));
+               name_button.set_text (_route->name());
                break;
        case Narrow:
-               name_label.set_text (PBD::short_version (_route->name(), 5));
+               name_button.set_text (PBD::short_version (_route->name(), 5));
                break;
        }
+
+       ARDOUR_UI::instance()->set_tip (name_button, _route->name());
 }
 
 void
-MixerStrip::width_clicked ()
+MixerStrip::name_button_resized (Gtk::Allocation& alloc)
 {
-       switch (_width) {
-       case Wide:
-               set_width_enum (Narrow, this);
-               break;
-       case Narrow:
-               set_width_enum (Wide, this);
-               break;
+       name_button.layout()->set_width (alloc.get_width() * PANGO_SCALE);
+}
+
+bool
+MixerStrip::width_button_pressed (GdkEventButton* ev)
+{
+       if (ev->button != 1) {
+               return false;
        }
+       
+       if (Keyboard::modifier_state_contains (ev->state, Keyboard::ModifierMask (Keyboard::PrimaryModifier | Keyboard::TertiaryModifier)) && _mixer_owned) {
+               switch (_width) {
+               case Wide:
+                       _mixer.set_strip_width (Narrow);
+                       break;
+
+               case Narrow:
+                       _mixer.set_strip_width (Wide);
+                       break;
+               }
+       } else {
+               switch (_width) {
+               case Wide:
+                       set_width_enum (Narrow, this);
+                       break;
+               case Narrow:
+                       set_width_enum (Wide, this);
+                       break;
+               }
+       }
+
+       return true;
 }
 
 void
@@ -1656,35 +1715,65 @@ MixerStrip::engine_running ()
 string
 MixerStrip::meter_point_string (MeterPoint mp)
 {
-       switch (mp) {
-       case MeterInput:
-               return _("in");
-               break;
-
-       case MeterPreFader:
-               return _("pre");
-               break;
-
-       case MeterPostFader:
-               return _("post");
-               break;
-
-       case MeterOutput:
-               return _("out");
+       switch (_width) {
+       case Wide:
+               switch (mp) {
+               case MeterInput:
+                       return _("in");
+                       break;
+                       
+               case MeterPreFader:
+                       return _("pre");
+                       break;
+                       
+               case MeterPostFader:
+                       return _("post");
+                       break;
+                       
+               case MeterOutput:
+                       return _("out");
+                       break;
+                       
+               case MeterCustom:
+               default:
+                       return _("custom");
+                       break;
+               }
                break;
-
-       case MeterCustom:
-       default:
-               return _("custom");
+       case Narrow:
+               switch (mp) {
+               case MeterInput:
+                       return _("in");
+                       break;
+                       
+               case MeterPreFader:
+                       return _("pr");
+                       break;
+                       
+               case MeterPostFader:
+                       return _("po");
+                       break;
+                       
+               case MeterOutput:
+                       return _("o");
+                       break;
+                       
+               case MeterCustom:
+               default:
+                       return _("c");
+                       break;
+               }
                break;
        }
+
+       return string();
 }
 
 /** Called when the metering point has changed */
 void
 MixerStrip::meter_changed ()
 {
-       meter_point_label.set_text (meter_point_string (_route->meter_point()));
+       meter_point_button.set_text (meter_point_string (_route->meter_point()));
        gpm.setup_meters ();
        // reset peak when meter point changes
        gpm.reset_peak_display();
@@ -1700,6 +1789,7 @@ MixerStrip::bus_send_display_changed (boost::shared_ptr<Route> send_to)
 
        if (send_to) {
                boost::shared_ptr<Send> send = _route->internal_send_for (send_to);
+
                if (send) {
                        show_send (send);
                } else {
@@ -1715,7 +1805,7 @@ MixerStrip::drop_send ()
 {
        boost::shared_ptr<Send> current_send;
 
-       if (_current_delivery && (current_send = boost::dynamic_pointer_cast<Send>(_current_delivery))) {
+       if (_current_delivery && ((current_send = boost::dynamic_pointer_cast<Send>(_current_delivery)) != 0)) {
                current_send->set_metering (false);
        }
 
@@ -1859,8 +1949,13 @@ MixerStrip::set_button_names ()
                break;
 
        }
-}
 
+       if (_route) {
+               meter_point_button.set_text (meter_point_string (_route->meter_point()));
+       } else {
+               meter_point_button.set_text ("");
+       }
+}
 
 PluginSelector*
 MixerStrip::plugin_selector()
@@ -1890,22 +1985,12 @@ MixerStrip::input_active_button_release (GdkEventButton* ev)
                return true;
        }
 
-       if (mt->input_active()) {
-               if (Keyboard::modifier_state_contains (ev->state, Keyboard::ModifierMask (Keyboard::PrimaryModifier|Keyboard::SecondaryModifier))) {
-                       /* turn all other tracks using this input off */
-                       _session->set_exclusive_input_active (mt, false);
-               } else {
-                       mt->set_input_active (false);
-               }
+       boost::shared_ptr<RouteList> rl (new RouteList);
 
-       } else {
-               if (Keyboard::modifier_state_contains (ev->state, Keyboard::ModifierMask (Keyboard::PrimaryModifier|Keyboard::SecondaryModifier))) {
-                       /* turn all other tracks using this input on */
-                       _session->set_exclusive_input_active (mt, true);
-               } else {
-                       mt->set_input_active (true);
-               }
-       }
+       rl->push_back (route());
+
+       _session->set_exclusive_input_active (rl, !mt->input_active(),
+                                             Keyboard::modifier_state_contains (ev->state, Keyboard::ModifierMask (Keyboard::PrimaryModifier|Keyboard::SecondaryModifier)));
 
        return true;
 }