leftmost_position => leftmost_sample, current_page_frames => current_page_samples
[ardour.git] / gtk2_ardour / route_ui.cc
index 2257acd83e66b7c7afff56ad72df8442a52b01c9..b10906192b04082c9b4d6118bf58e0213d4186a2 100644 (file)
 #include "route_time_axis.h"
 #include "group_tabs.h"
 
-#include "ardour/route.h"
-#include "ardour/event_type_map.h"
-#include "ardour/session.h"
-#include "ardour/audioengine.h"
 #include "ardour/audio_track.h"
+#include "ardour/audioengine.h"
+#include "ardour/filename_extensions.h"
 #include "ardour/midi_track.h"
+#include "ardour/route.h"
+#include "ardour/session.h"
 #include "ardour/template_utils.h"
-#include "ardour/filename_extensions.h"
-#include "ardour/directory_names.h"
-#include "ardour/profile.h"
 
 #include "i18n.h"
 using namespace Gtk;
@@ -123,20 +120,21 @@ RouteUI::init ()
 
        rec_enable_button = manage (new ArdourButton);
        rec_enable_button->set_name ("record enable button");
+       rec_enable_button->set_tweaks (ArdourButton::ImplicitUsesSolidColor);
        UI::instance()->set_tip (rec_enable_button, _("Enable recording on this track"), "");
 
        show_sends_button = manage (new ArdourButton);
        show_sends_button->set_name ("send alert button");
        UI::instance()->set_tip (show_sends_button, _("make mixer strips show sends to this bus"), "");
 
-       monitor_input_button = manage (new ArdourButton (ArdourButton::led_default_elements));
-       monitor_input_button->set_name ("monitor");
+       monitor_input_button = manage (new ArdourButton (ArdourButton::default_elements));
+       monitor_input_button->set_name ("monitor button");
        monitor_input_button->set_text (_("In"));
        UI::instance()->set_tip (monitor_input_button, _("Monitor input"), "");
        monitor_input_button->set_no_show_all (true);
-
-       monitor_disk_button = manage (new ArdourButton (ArdourButton::led_default_elements));
-       monitor_disk_button->set_name ("monitor");
+       
+       monitor_disk_button = manage (new ArdourButton (ArdourButton::default_elements));
+       monitor_disk_button->set_name ("monitor button");
        monitor_disk_button->set_text (_("Disk"));
        UI::instance()->set_tip (monitor_disk_button, _("Monitor playback"), "");
        monitor_disk_button->set_no_show_all (true);
@@ -145,8 +143,8 @@ RouteUI::init ()
        _session->TransportStateChange.connect (_session_connections, invalidator (*this), boost::bind (&RouteUI::check_rec_enable_sensitivity, this), gui_context());
        _session->RecordStateChanged.connect (_session_connections, invalidator (*this), boost::bind (&RouteUI::session_rec_enable_changed, this), gui_context());
 
-       _session->config.ParameterChanged.connect (*this, invalidator (*this), ui_bind (&RouteUI::parameter_changed, this, _1), gui_context());
-       Config->ParameterChanged.connect (*this, invalidator (*this), ui_bind (&RouteUI::parameter_changed, this, _1), gui_context());
+       _session->config.ParameterChanged.connect (*this, invalidator (*this), boost::bind (&RouteUI::parameter_changed, this, _1), gui_context());
+       Config->ParameterChanged.connect (*this, invalidator (*this), boost::bind (&RouteUI::parameter_changed, this, _1), gui_context());
 
        rec_enable_button->signal_button_press_event().connect (sigc::mem_fun(*this, &RouteUI::rec_enable_press), false);
        rec_enable_button->signal_button_release_event().connect (sigc::mem_fun(*this, &RouteUI::rec_enable_release), false);
@@ -210,7 +208,7 @@ RouteUI::set_route (boost::shared_ptr<Route> rp)
        solo_button->set_controllable (_route->solo_control());
 
        _route->active_changed.connect (route_connections, invalidator (*this), boost::bind (&RouteUI::route_active_changed, this), gui_context());
-       _route->mute_changed.connect (route_connections, invalidator (*this), ui_bind (&RouteUI::mute_changed, this, _1), gui_context());
+       _route->mute_changed.connect (route_connections, invalidator (*this), boost::bind (&RouteUI::mute_changed, this, _1), gui_context());
 
        _route->solo_changed.connect (route_connections, invalidator (*this), boost::bind (&RouteUI::update_solo_display, this), gui_context());
        _route->solo_safe_changed.connect (route_connections, invalidator (*this), boost::bind (&RouteUI::update_solo_display, this), gui_context());
@@ -218,10 +216,10 @@ RouteUI::set_route (boost::shared_ptr<Route> rp)
        _route->solo_isolated_changed.connect (route_connections, invalidator (*this), boost::bind (&RouteUI::update_solo_display, this), gui_context());
 
         _route->phase_invert_changed.connect (route_connections, invalidator (*this), boost::bind (&RouteUI::polarity_changed, this), gui_context());
-       _route->PropertyChanged.connect (route_connections, invalidator (*this), ui_bind (&RouteUI::property_changed, this, _1), gui_context());
+       _route->PropertyChanged.connect (route_connections, invalidator (*this), boost::bind (&RouteUI::property_changed, this, _1), gui_context());
 
-       _route->io_changed.connect (route_connections, invalidator (*this), ui_bind (&RouteUI::setup_invert_buttons, this), gui_context ());
-       _route->gui_changed.connect (route_connections, invalidator (*this), ui_bind (&RouteUI::route_gui_changed, this, _1), gui_context ());
+       _route->io_changed.connect (route_connections, invalidator (*this), boost::bind (&RouteUI::setup_invert_buttons, this), gui_context ());
+       _route->gui_changed.connect (route_connections, invalidator (*this), boost::bind (&RouteUI::route_gui_changed, this, _1), gui_context ());
 
        if (_session->writable() && is_track()) {
                boost::shared_ptr<Track> t = boost::dynamic_pointer_cast<Track>(_route);
@@ -231,14 +229,18 @@ RouteUI::set_route (boost::shared_ptr<Route> rp)
                rec_enable_button->show();
                rec_enable_button->set_controllable (t->rec_enable_control());
 
-               update_rec_display ();
-
                 if (is_midi_track()) {
                         midi_track()->StepEditStatusChange.connect (route_connections, invalidator (*this),
-                                                                    ui_bind (&RouteUI::step_edit_changed, this, _1), gui_context());
+                                                                    boost::bind (&RouteUI::step_edit_changed, this, _1), gui_context());
                 }
 
-       }
+       } 
+
+       /* this will work for busses and tracks, and needs to be called to
+          set up the name entry/name label display.
+       */
+
+       update_rec_display ();
 
        if (is_track()) {
                boost::shared_ptr<Track> t = boost::dynamic_pointer_cast<Track>(_route);
@@ -317,11 +319,31 @@ RouteUI::mute_press (GdkEventButton* ev)
 
                                if (Keyboard::modifier_state_equals (ev->state, Keyboard::ModifierMask (Keyboard::PrimaryModifier|Keyboard::TertiaryModifier))) {
 
+                                       /* toggle mute on everything (but
+                                        * exclude the master and monitor)
+                                        *
+                                        * because we are going to erase
+                                        * elements of the list we need to work
+                                        * on a copy.
+                                        */
+                                       
+                                       boost::shared_ptr<RouteList> copy (new RouteList);
+
+                                       *copy = *_session->get_routes ();
+
+                                       for (RouteList::iterator i = copy->begin(); i != copy->end(); ) {
+                                               if ((*i)->is_master() || (*i)->is_monitor()) {
+                                                       i = copy->erase (i);
+                                               } else {
+                                                       ++i;
+                                               }
+                                       }
+
                                        if (_mute_release) {
-                                               _mute_release->routes = _session->get_routes ();
+                                               _mute_release->routes = copy;
                                        }
 
-                                       _session->set_mute (_session->get_routes(), !_route->muted());
+                                       _session->set_mute (copy, !_route->muted());
 
                                } else if (Keyboard::modifier_state_equals (ev->state, Keyboard::PrimaryModifier)) {
 
@@ -329,12 +351,23 @@ RouteUI::mute_press (GdkEventButton* ev)
                                           NOTE: Primary-button2 is MIDI learn.
                                        */
 
-                                       if (ev->button == 1 && _route->route_group()) {
-                                               if (_mute_release) {
-                                                       _mute_release->routes = _session->get_routes ();
+                                       boost::shared_ptr<RouteList> rl;
+
+                                       if (ev->button == 1) { 
+
+                                               if (_route->route_group()) {
+                                                       
+                                                       rl = _route->route_group()->route_list();
+                                                       
+                                                       if (_mute_release) {
+                                                               _mute_release->routes = rl;
+                                                       }
+                                               } else {
+                                                       rl.reset (new RouteList);
+                                                       rl->push_back (_route);
                                                }
 
-                                               _session->set_mute (_session->get_routes(), !_route->muted(), Session::rt_cleanup, true);
+                                               _session->set_mute (rl, !_route->muted(), Session::rt_cleanup, true);
                                        }
 
                                } else {
@@ -343,7 +376,7 @@ RouteUI::mute_press (GdkEventButton* ev)
 
                                        boost::shared_ptr<RouteList> rl (new RouteList);
                                        rl->push_back (_route);
-
+                                       
                                        if (_mute_release) {
                                                _mute_release->routes = rl;
                                        }
@@ -466,16 +499,30 @@ RouteUI::solo_press(GdkEventButton* ev)
                                           NOTE: Primary-button2 is MIDI learn.
                                        */
 
-                                       if (ev->button == 1 && _route->route_group()) {
+                                       /* Primary-button1 applies change to the mix group even if it is not active
+                                          NOTE: Primary-button2 is MIDI learn.
+                                       */
+
+                                       boost::shared_ptr<RouteList> rl;
+
+                                       if (ev->button == 1) { 
 
-                                               if (_solo_release) {
-                                                       _solo_release->routes = _route->route_group()->route_list();
+                                               if (_route->route_group()) {
+                                                       
+                                                       rl = _route->route_group()->route_list();
+                                                       
+                                                       if (_solo_release) {
+                                                               _solo_release->routes = rl;
+                                                       }
+                                               } else {
+                                                       rl.reset (new RouteList);
+                                                       rl->push_back (_route);
                                                }
 
                                                if (Config->get_solo_control_is_listen_control()) {
-                                                       _session->set_listen (_route->route_group()->route_list(), !_route->listening_via_monitor(),  Session::rt_cleanup, true);
+                                                       _session->set_listen (rl, !_route->listening_via_monitor(),  Session::rt_cleanup, true);
                                                } else {
-                                                       _session->set_solo (_route->route_group()->route_list(), !_route->self_soloed(),  Session::rt_cleanup, true);
+                                                       _session->set_solo (rl, !_route->self_soloed(),  Session::rt_cleanup, true);
                                                }
                                        }
 
@@ -543,10 +590,11 @@ RouteUI::rec_enable_press(GdkEventButton* ev)
 
         if (is_midi_track()) {
 
-                /* cannot rec-enable while step-editing */
+                /* rec-enable button exits from step editing */
 
                 if (midi_track()->step_editing()) {
-                        return true;
+                       midi_track()->set_step_editing (false);
+                       return true;
                 }
         }
 
@@ -566,8 +614,21 @@ RouteUI::rec_enable_press(GdkEventButton* ev)
                        /* Primary-button1 applies change to the route group (even if it is not active)
                           NOTE: Primary-button2 is MIDI learn.
                        */
-                       if (ev->button == 1 && _route->route_group()) {
-                               _session->set_record_enabled (_route->route_group()->route_list(), !rec_enable_button->active_state(), Session::rt_cleanup, true);
+
+                       if (ev->button == 1) {
+
+                               boost::shared_ptr<RouteList> rl;
+                               
+                               if (_route->route_group()) {
+                                       
+                                       rl = _route->route_group()->route_list();
+                                       
+                               } else {
+                                       rl.reset (new RouteList);
+                                       rl->push_back (_route);
+                               }
+                               
+                               _session->set_record_enabled (rl, !rec_enable_button->active_state(), Session::rt_cleanup, true);
                        }
 
                } else if (Keyboard::is_context_menu_event (ev)) {
@@ -607,20 +668,20 @@ RouteUI::update_monitoring_display ()
        MonitorState ms = t->monitoring_state();
 
        if (t->monitoring_choice() & MonitorInput) {
-               monitor_input_button->set_active_state (Gtkmm2ext::Active);
+               monitor_input_button->set_active_state (Gtkmm2ext::ExplicitActive);
        } else {
                if (ms & MonitoringInput) {
-                       monitor_input_button->set_active_state (Gtkmm2ext::Mid);
+                       monitor_input_button->set_active_state (Gtkmm2ext::ImplicitActive);
                } else {
                        monitor_input_button->unset_active_state ();
                }
        }
 
        if (t->monitoring_choice() & MonitorDisk) {
-               monitor_disk_button->set_active_state (Gtkmm2ext::Active);
+               monitor_disk_button->set_active_state (Gtkmm2ext::ExplicitActive);
        } else {
                if (ms & MonitoringDisk) {
-                       monitor_disk_button->set_active_state (Gtkmm2ext::Mid);
+                       monitor_disk_button->set_active_state (Gtkmm2ext::ImplicitActive);
                } else {
                        monitor_disk_button->unset_active_state ();
                }
@@ -628,7 +689,7 @@ RouteUI::update_monitoring_display ()
 }
 
 bool
-RouteUI::monitor_input_press(GdkEventButton* ev)
+RouteUI::monitor_input_press(GdkEventButton*)
 {
        return true;
 }
@@ -640,7 +701,7 @@ RouteUI::monitor_input_release(GdkEventButton* ev)
 }
 
 bool
-RouteUI::monitor_disk_press (GdkEventButton* ev)
+RouteUI::monitor_disk_press (GdkEventButton*)
 {
        return true;
 }
@@ -743,7 +804,7 @@ RouteUI::step_edit_changed (bool yn)
 {
         if (yn) {
                 if (rec_enable_button) {
-                        rec_enable_button->set_active_state (Active);
+                        rec_enable_button->set_active_state (Gtkmm2ext::ExplicitActive);
                 }
 
                 start_step_editing ();
@@ -922,7 +983,7 @@ RouteUI::send_blink (bool onoff)
        }
 
        if (onoff) {
-               show_sends_button->set_active_state (Gtkmm2ext::Active);
+               show_sends_button->set_active_state (Gtkmm2ext::ExplicitActive);
        } else {
                show_sends_button->unset_active_state ();
        }
@@ -932,27 +993,27 @@ Gtkmm2ext::ActiveState
 RouteUI::solo_active_state (boost::shared_ptr<Route> r)
 {
        if (r->is_master() || r->is_monitor()) {
-               return ActiveState (0);
+               return Gtkmm2ext::Off;
        }
 
        if (Config->get_solo_control_is_listen_control()) {
 
                if (r->listening_via_monitor()) {
-                       return Active;
+                       return Gtkmm2ext::ExplicitActive;
                } else {
-                       return ActiveState (0);
+                       return Gtkmm2ext::Off;
                }
 
        }
 
        if (r->soloed()) {
                 if (!r->self_soloed()) {
-                        return Mid;
+                        return Gtkmm2ext::ImplicitActive;
                 } else {
-                        return Active;
+                        return Gtkmm2ext::ExplicitActive;
                 }
        } else {
-               return ActiveState(0);
+               return Gtkmm2ext::Off;
        }
 }
 
@@ -960,13 +1021,13 @@ Gtkmm2ext::ActiveState
 RouteUI::solo_isolate_active_state (boost::shared_ptr<Route> r)
 {
        if (r->is_master() || r->is_monitor()) {
-               return ActiveState (0);
+               return Gtkmm2ext::Off;
        }
 
        if (r->solo_isolated()) {
-               return Active;
+               return Gtkmm2ext::ExplicitActive;
        } else {
-               return ActiveState(0);
+               return Gtkmm2ext::Off;
        }
 }
 
@@ -974,43 +1035,19 @@ Gtkmm2ext::ActiveState
 RouteUI::solo_safe_active_state (boost::shared_ptr<Route> r)
 {
        if (r->is_master() || r->is_monitor()) {
-               return ActiveState (0);
+               return Gtkmm2ext::Off;
        }
 
        if (r->solo_safe()) {
-               return Active;
+               return Gtkmm2ext::ExplicitActive;
        } else {
-               return ActiveState (0);
+               return Gtkmm2ext::Off;
        }
 }
 
 void
 RouteUI::update_solo_display ()
 {
-       bool x;
-
-       if (Config->get_solo_control_is_listen_control()) {
-
-               if ((bool) solo_button->active_state() != (x = _route->listening_via_monitor())) {
-                       ++_i_am_the_modifier;
-                       solo_button->set_active_state (Active);
-                       --_i_am_the_modifier;
-               }
-
-       } else {
-
-               if ((bool) solo_button->active_state() != (x = _route->soloed())) {
-                       ++_i_am_the_modifier;
-                       if (x) {
-                               solo_button->set_active_state (Active);
-                       } else {
-                               solo_button->unset_active_state();
-                       }
-                       --_i_am_the_modifier;
-               }
-
-       }
-
        bool yn = _route->solo_safe ();
 
        if (solo_safe_check && solo_safe_check->get_active() != yn) {
@@ -1027,7 +1064,7 @@ RouteUI::update_solo_display ()
 
         if (solo_isolated_led) {
                if (_route->solo_isolated()) {
-                       solo_isolated_led->set_active_state (Gtkmm2ext::Active);
+                       solo_isolated_led->set_active_state (Gtkmm2ext::ExplicitActive);
                } else {
                        solo_isolated_led->unset_active_state ();
                }
@@ -1035,7 +1072,7 @@ RouteUI::update_solo_display ()
 
         if (solo_safe_led) {
                if (_route->solo_safe()) {
-                       solo_safe_led->set_active_state (Gtkmm2ext::Active);
+                       solo_safe_led->set_active_state (Gtkmm2ext::ExplicitActive);
                } else {
                        solo_safe_led->unset_active_state ();
                }
@@ -1073,23 +1110,23 @@ RouteUI::mute_active_state (Session* s, boost::shared_ptr<Route> r)
 
                if (r->muted ()) {
                        /* full mute */
-                       return Active;
+                       return Gtkmm2ext::ExplicitActive;
                } else if (!r->is_master() && s->soloing() && !r->soloed() && !r->solo_isolated()) {
                        /* master is NEVER muted by others */
-                       return Mid;
+                       return Gtkmm2ext::ImplicitActive;
                } else {
                        /* no mute at all */
-                       return ActiveState(0);
+                       return Gtkmm2ext::Off;
                }
 
        } else {
 
                if (r->muted()) {
                        /* full mute */
-                       return Active;
+                       return Gtkmm2ext::ExplicitActive;
                } else {
                        /* no mute at all */
-                       return ActiveState(0);
+                       return Gtkmm2ext::Off;
                }
        }
 
@@ -1130,12 +1167,12 @@ RouteUI::update_rec_display ()
        if (_route->record_enabled()) {
                 switch (_session->record_status ()) {
                 case Session::Recording:
-                        rec_enable_button->set_active_state (Active);
+                        rec_enable_button->set_active_state (Gtkmm2ext::ExplicitActive);
                         break;
 
                 case Session::Disabled:
                 case Session::Enabled:
-                        rec_enable_button->set_active_state (Mid);
+                        rec_enable_button->set_active_state (Gtkmm2ext::ImplicitActive);
                         break;
 
                 }
@@ -1310,10 +1347,13 @@ RouteUI::solo_isolate_button_release (GdkEventButton* ev)
 }
 
 bool
-RouteUI::solo_safe_button_release (GdkEventButton*)
+RouteUI::solo_safe_button_release (GdkEventButton* ev)
 {
-        _route->set_solo_safe (!solo_safe_led->active_state(), this);
-        return true;
+       if (ev->button == 1) {
+               _route->set_solo_safe (!solo_safe_led->active_state(), this);
+               return true;
+       }
+       return false;
 }
 
 void
@@ -1357,17 +1397,19 @@ RouteUI::choose_color ()
 void
 RouteUI::set_color (const Gdk::Color & c)
 {
-       char buf[64];
+       /* leave _color alone in the group case so that tracks can retain their
+        * own pre-group colors.
+        */
 
+       char buf[64];
        _color = c;
-
        snprintf (buf, sizeof (buf), "%d:%d:%d", c.get_red(), c.get_green(), c.get_blue());
-
+       
        /* note: we use the route state ID here so that color is the same for both
           the time axis view and the mixer strip
        */
        
-       gui_object_state().set<string> (route_state_id(), X_("color"), buf);
+       gui_object_state().set_property<string> (route_state_id(), X_("color"), buf);
        _route->gui_changed ("color", (void *) 0); /* EMIT_SIGNAL */
 }
 
@@ -1651,14 +1693,14 @@ RouteUI::adjust_latency ()
 void
 RouteUI::save_as_template ()
 {
-       sys::path path;
+       std::string path;
        std::string safe_name;
        string name;
 
        path = ARDOUR::user_route_template_directory ();
 
-       if (g_mkdir_with_parents (path.to_string().c_str(), 0755)) {
-               error << string_compose (_("Cannot create route template directory %1"), path.to_string()) << endmsg;
+       if (g_mkdir_with_parents (path.c_str(), 0755)) {
+               error << string_compose (_("Cannot create route template directory %1"), path) << endmsg;
                return;
        }
 
@@ -1680,9 +1722,9 @@ RouteUI::save_as_template ()
        safe_name = legalize_for_path (name);
        safe_name += template_suffix;
 
-       path /= safe_name;
+       path = Glib::build_filename (path, safe_name);
 
-       _route->save_as_template (path.to_string(), name);
+       _route->save_as_template (path, name);
 }
 
 void
@@ -1739,27 +1781,53 @@ void
 RouteUI::open_remote_control_id_dialog ()
 {
        ArdourDialog dialog (_("Remote Control ID"));
+       SpinButton* spin = 0;
 
-       uint32_t const limit = _session->ntracks() + _session->nbusses () + 4;
-
-       HBox* hbox = manage (new HBox);
-       hbox->set_spacing (6);
-       hbox->pack_start (*manage (new Label (_("Remote control ID:"))));
-       SpinButton* spin = manage (new SpinButton);
-       spin->set_digits (0);
-       spin->set_increments (1, 10);
-       spin->set_range (0, limit);
-       spin->set_value (_route->remote_control_id());
-       hbox->pack_start (*spin);
-       dialog.get_vbox()->pack_start (*hbox);
+       dialog.get_vbox()->set_border_width (18);
 
-       dialog.add_button (Stock::CANCEL, RESPONSE_CANCEL);
-       dialog.add_button (Stock::APPLY, RESPONSE_ACCEPT);
+       if (Config->get_remote_model() == UserOrdered) {
+               uint32_t const limit = _session->ntracks() + _session->nbusses () + 4;
+               
+               HBox* hbox = manage (new HBox);
+               hbox->set_spacing (6);
+               hbox->pack_start (*manage (new Label (_("Remote control ID:"))));
+               spin = manage (new SpinButton);
+               spin->set_digits (0);
+               spin->set_increments (1, 10);
+               spin->set_range (0, limit);
+               spin->set_value (_route->remote_control_id());
+               hbox->pack_start (*spin);
+               dialog.get_vbox()->pack_start (*hbox);
+               
+               dialog.add_button (Stock::CANCEL, RESPONSE_CANCEL);
+               dialog.add_button (Stock::APPLY, RESPONSE_ACCEPT);
+       } else {
+               Label* l = manage (new Label());
+               if (_route->is_master() || _route->is_monitor()) {
+                       l->set_markup (string_compose (_("The remote control ID of %1 is: %2\n\n\n"
+                                                        "The remote control ID of %3 cannot be changed."),
+                                                      Glib::Markup::escape_text (_route->name()),
+                                                      _route->remote_control_id(),
+                                                      (_route->is_master() ? _("the master bus") : _("the monitor bus"))));
+               } else {
+                       l->set_markup (string_compose (_("The remote control ID of %6 is: %3\n\n\n"
+                                                        "Remote Control IDs are currently determined by track/bus ordering in %1\n\n"
+                                                        "%4Use the User Interaction tab of the Preferences window if you want to change this%5"),
+                                                      (Config->get_remote_model() == MixerOrdered ? _("the mixer") : _("the editor")),
+                                                      (is_track() ? _("track") : _("bus")),
+                                                      _route->remote_control_id(),
+                                                      "<span size=\"small\" style=\"italic\">",
+                                                      "</span>",
+                                                      Glib::Markup::escape_text (_route->name())));
+               }
+               dialog.get_vbox()->pack_start (*l);
+               dialog.add_button (Stock::OK, RESPONSE_CANCEL);
+       }
 
        dialog.show_all ();
        int const r = dialog.run ();
 
-       if (r == RESPONSE_ACCEPT) {
+       if (r == RESPONSE_ACCEPT && spin) {
                _route->set_remote_control_id (spin->get_value_as_int ());
        }
 }
@@ -1784,20 +1852,25 @@ RouteUI::setup_invert_buttons ()
 
        for (uint32_t i = 0; i < to_add; ++i) {
                ArdourButton* b = manage (new ArdourButton);
+               b->set_size_request(20,20);
                b->signal_button_press_event().connect (sigc::mem_fun (*this, &RouteUI::invert_press));
                b->signal_button_release_event().connect (sigc::bind (sigc::mem_fun (*this, &RouteUI::invert_release), i));
 
-               b->set_name (X_("mixer strip button"));
+               b->set_name (X_("invert button"));
                if (to_add == 1) {
-                       b->set_text (X_("Ø"));
+                       if (N > 1) {
+                               b->set_text (string_compose (X_("Ø (%1)"), N));
+                       } else {
+                               b->set_text (X_("Ø"));
+                       }
                } else {
                        b->set_text (string_compose (X_("Ø%1"), i + 1));
                }
 
-               if (N <= 4) {
+               if (N <= _max_invert_buttons) {
                        UI::instance()->set_tip (*b, string_compose (_("Left-click to invert (phase reverse) channel %1 of this track.  Right-click to show menu."), i + 1));
                } else {
-                       UI::instance()->set_tip (*b, string_compose (_("Left-click to invert (phase reverse) all channels of this track.  Right-click to show menu."), i + 1));
+                       UI::instance()->set_tip (*b, _("Click to show a menu of channels for inversion (phase reverse)"));
                }
 
                _invert_buttons.push_back (b);
@@ -1815,14 +1888,30 @@ RouteUI::set_invert_button_state ()
 
        uint32_t const N = _route->input()->n_ports().n_audio();
        if (N > _max_invert_buttons) {
-               _invert_buttons.front()->set_active (_route->phase_invert().any());
-               --_i_am_the_modifier;
-               return;
-       }
 
-       int j = 0;
-       for (vector<ArdourButton*>::iterator i = _invert_buttons.begin(); i != _invert_buttons.end(); ++i, ++j) {
-               (*i)->set_active (_route->phase_invert (j));
+               /* One button for many channels; explicit active if all channels are inverted,
+                  implicit active if some are, off if none are.
+               */
+
+               ArdourButton* b = _invert_buttons.front ();
+               
+               if (_route->phase_invert().count() == _route->phase_invert().size()) {
+                       b->set_active_state (Gtkmm2ext::ExplicitActive);
+               } else if (_route->phase_invert().any()) {
+                       b->set_active_state (Gtkmm2ext::ImplicitActive);
+               } else {
+                       b->set_active_state (Gtkmm2ext::Off);
+               }
+
+       } else {
+
+               /* One button per channel; just set active */
+
+               int j = 0;
+               for (vector<ArdourButton*>::iterator i = _invert_buttons.begin(); i != _invert_buttons.end(); ++i, ++j) {
+                       (*i)->set_active (_route->phase_invert (j));
+               }
+               
        }
 
        --_i_am_the_modifier;
@@ -1832,8 +1921,12 @@ bool
 RouteUI::invert_release (GdkEventButton* ev, uint32_t i)
 {
        if (ev->button == 1 && i < _invert_buttons.size()) {
-               _route->set_phase_invert (i, !_invert_buttons[i]->get_active());
-               return true;
+               uint32_t const N = _route->input()->n_ports().n_audio ();
+               if (N <= _max_invert_buttons) {
+                       /* left-click inverts phase so long as we have a button per channel */
+                       _route->set_phase_invert (i, !_invert_buttons[i]->get_active());
+                       return true;
+               }
        }
        return false;
 }
@@ -1843,17 +1936,21 @@ bool
 RouteUI::invert_press (GdkEventButton* ev)
 {
        using namespace Menu_Helpers;
-       
-       if (ev->button != 3) {
+
+       uint32_t const N = _route->input()->n_ports().n_audio();
+       if (N <= _max_invert_buttons && ev->button != 3) {
+               /* If we have an invert button per channel, we only pop
+                  up a menu on right-click; left click is handled
+                  on release.
+               */
                return true;
        }
-
+       
        delete _invert_menu;
        _invert_menu = new Menu;
        _invert_menu->set_name ("ArdourContextMenu");
        MenuList& items = _invert_menu->items ();
 
-       uint32_t const N = _route->input()->n_ports().n_audio();
        for (uint32_t i = 0; i < N; ++i) {
                items.push_back (CheckMenuElem (string_compose (X_("Ø%1"), i + 1), sigc::bind (sigc::mem_fun (*this, &RouteUI::invert_menu_toggled), i)));
                CheckMenuItem* e = dynamic_cast<CheckMenuItem*> (&items.back ());
@@ -1930,10 +2027,10 @@ void
 RouteUI::bus_send_display_changed (boost::shared_ptr<Route> send_to)
 {
        if (_route == send_to) {
-               show_sends_button->set_active_state (Gtkmm2ext::Active);
+               show_sends_button->set_active (true);
                send_blink_connection = ARDOUR_UI::instance()->Blink.connect (sigc::mem_fun (*this, &RouteUI::send_blink));
        } else {
-               show_sends_button->unset_active_state ();
+               show_sends_button->set_active (false);
                send_blink_connection.disconnect ();
        }
 }