Make send automation work (#4734).
[ardour.git] / gtk2_ardour / group_tabs.cc
index ec80bc4291261de3bdbe7ca1553e110c13dd5af4..9ce3b150460029822cabdf1ebc0ec7269f76126a 100644 (file)
@@ -56,7 +56,16 @@ GroupTabs::set_session (Session* s)
        SessionHandlePtr::set_session (s);
 
        if (_session) {
-               _session->RouteGroupChanged.connect (_session_connections, invalidator (*this), boost::bind (&GroupTabs::set_dirty, this), gui_context());
+               _session->RouteGroupPropertyChanged.connect (
+                       _session_connections, invalidator (*this), boost::bind (&GroupTabs::route_group_property_changed, this, _1), gui_context()
+                       );
+               _session->RouteAddedToRouteGroup.connect (
+                       _session_connections, invalidator (*this), boost::bind (&GroupTabs::route_added_to_route_group, this, _1, _2), gui_context()
+                       );
+               _session->RouteRemovedFromRouteGroup.connect (
+                       _session_connections, invalidator (*this), boost::bind (&GroupTabs::route_removed_from_route_group, this, _1, _2), gui_context()
+                       );
+               
                _session->route_group_removed.connect (_session_connections, invalidator (*this), boost::bind (&GroupTabs::set_dirty, this), gui_context());
        }
 }
@@ -104,6 +113,7 @@ GroupTabs::on_button_press_event (GdkEventButton* ev)
 
                } else {
                        _dragging_new_tab = false;
+                       _initial_dragging_routes = routes_for_tab (t);
                }
 
                _dragging = t;
@@ -124,11 +134,17 @@ GroupTabs::on_button_press_event (GdkEventButton* ev)
        } else if (ev->button == 3) {
 
                RouteGroup* g = t ? t->group : 0;
-               Menu* m = get_menu (g);
-               if (m) {
-                       m->popup (ev->button, ev->time);
+               
+               if (Keyboard::modifier_state_equals (ev->state, Keyboard::PrimaryModifier) && g) {
+                       /* edit */
+                       RouteGroupDialog d (g, false);
+                       d.do_run ();
+               } else {
+                       Menu* m = get_menu (g);
+                       if (m) {
+                               m->popup (ev->button, ev->time);
+                       }
                }
-
        }
 
        return true;
@@ -164,34 +180,23 @@ GroupTabs::on_motion_notify_event (GdkEventMotion* ev)
 
 
 bool
-GroupTabs::on_button_release_event (GdkEventButton* ev)
+GroupTabs::on_button_release_event (GdkEventButton*)
 {
        if (_dragging == 0) {
                return false;
        }
-
+       
        if (!_drag_moved) {
-
+               
                if (_dragging->group) {
-
-                       if (Keyboard::modifier_state_equals (ev->state, Keyboard::PrimaryModifier)) {
-
-                               /* edit */
-                               RouteGroupDialog d (_dragging->group, false);
-                               d.do_run ();
-
-                       } else {
-
-                               /* toggle active state */
-                               _dragging->group->set_active (!_dragging->group->is_active (), this);
-
-                       }
+                       /* toggle active state */
+                       _dragging->group->set_active (!_dragging->group->is_active (), this);
                }
-
+               
        } else {
                /* finish drag */
                RouteList routes = routes_for_tab (_dragging);
-
+               
                if (!routes.empty()) {
                        if (_dragging_new_tab) {
                                RouteGroup* g = create_and_add_group ();
@@ -203,24 +208,28 @@ GroupTabs::on_button_release_event (GdkEventButton* ev)
                        } else {
                                boost::shared_ptr<RouteList> r = _session->get_routes ();
                                for (RouteList::iterator i = r->begin(); i != r->end(); ++i) {
-
-                                       if (find (routes.begin(), routes.end(), *i) == routes.end()) {
-                                               /* this route is not on the list of those that should be in _dragging's group */
-                                               if ((*i)->route_group() == _dragging->group) {
-                                                       _dragging->group->remove (*i);
-                                               }
-                                       } else {
+                                       
+                                       bool const was_in_tab = find (
+                                               _initial_dragging_routes.begin(), _initial_dragging_routes.end(), *i
+                                               ) != _initial_dragging_routes.end ();
+                                       
+                                       bool const now_in_tab = find (routes.begin(), routes.end(), *i) != routes.end();
+                                       
+                                       if (was_in_tab && !now_in_tab) {
+                                               _dragging->group->remove (*i);
+                                       } else if (!was_in_tab && now_in_tab) {
                                                _dragging->group->add (*i);
                                        }
                                }
                        }
                }
-
+               
                set_dirty ();
                queue_draw ();
        }
-
+       
        _dragging = 0;
+       _initial_dragging_routes.clear ();
 
        return true;
 }
@@ -234,10 +243,12 @@ GroupTabs::render (cairo_t* cr)
 
        /* background */
 
-       cairo_set_source_rgb (cr, 0, 0, 0);
-       cairo_rectangle (cr, 0, 0, _width, _height);
-       cairo_fill (cr);
+       Gdk::Color c = get_style()->get_base (Gtk::STATE_NORMAL);
 
+       cairo_set_source_rgb (cr, c.get_red_p(), c.get_green_p(), c.get_blue_p());
+       cairo_rectangle (cr, 0, 0, get_width(), get_height());
+       cairo_fill (cr);
+       
        /* tabs */
 
        for (list<Tab>::const_iterator i = _tabs.begin(); i != _tabs.end(); ++i) {
@@ -300,23 +311,24 @@ GroupTabs::get_menu (RouteGroup* g)
        _menu->set_name ("ArdourContextMenu");
        MenuList& items = _menu->items();
 
-       items.push_back (MenuElem (_("New..."), hide_return (sigc::mem_fun(*this, &GroupTabs::create_and_add_group))));
-       items.push_back (MenuElem (_("New From"), *new_from));
+       items.push_back (MenuElem (_("Create New Group ..."), hide_return (sigc::mem_fun(*this, &GroupTabs::create_and_add_group))));
+       items.push_back (MenuElem (_("Create New Group From"), *new_from));
 
        if (g) {
-               items.push_back (MenuElem (_("Edit..."), sigc::bind (sigc::mem_fun (*this, &GroupTabs::edit_group), g)));
+               items.push_back (MenuElem (_("Edit Group..."), sigc::bind (sigc::mem_fun (*this, &GroupTabs::edit_group), g)));
+               items.push_back (MenuElem (_("Collect Group"), sigc::bind (sigc::mem_fun (*this, &GroupTabs::collect), g)));
+               items.push_back (MenuElem (_("Remove Group"), sigc::bind (sigc::mem_fun (*this, &GroupTabs::remove_group), g)));
+               items.push_back (SeparatorElem());
                items.push_back (MenuElem (_("Add New Subgroup Bus"), sigc::bind (sigc::mem_fun (*this, &GroupTabs::subgroup), g, false, PreFader)));
                items.push_back (MenuElem (_("Add New Aux Bus (pre-fader)"), sigc::bind (sigc::mem_fun (*this, &GroupTabs::subgroup), g, true, PreFader)));
                items.push_back (MenuElem (_("Add New Aux Bus (post-fader)"), sigc::bind (sigc::mem_fun (*this, &GroupTabs::subgroup), g, true, PostFader)));
-               items.push_back (MenuElem (_("Collect"), sigc::bind (sigc::mem_fun (*this, &GroupTabs::collect), g)));
-               items.push_back (MenuElem (_("Remove"), sigc::bind (sigc::mem_fun (*this, &GroupTabs::remove_group), g)));
        }
 
        add_menu_items (_menu, g);
 
        items.push_back (SeparatorElem());
-       items.push_back (MenuElem (_("Activate All"), sigc::mem_fun(*this, &GroupTabs::activate_all)));
-       items.push_back (MenuElem (_("Disable All"), sigc::mem_fun(*this, &GroupTabs::disable_all)));
+       items.push_back (MenuElem (_("Enable All Groups"), sigc::mem_fun(*this, &GroupTabs::activate_all)));
+       items.push_back (MenuElem (_("Disable All Groups"), sigc::mem_fun(*this, &GroupTabs::disable_all)));
 
        return _menu;
 
@@ -423,13 +435,24 @@ GroupTabs::subgroup (RouteGroup* g, bool aux, Placement placement)
 }
 
 struct CollectSorter {
-       CollectSorter (std::string const & key) : _key (key) {}
+       CollectSorter (string const & key) : _key (key) {}
 
        bool operator () (boost::shared_ptr<Route> a, boost::shared_ptr<Route> b) {
                return a->order_key (_key) < b->order_key (_key);
        }
 
-       std::string _key;
+       string _key;
+};
+
+struct OrderSorter {
+       OrderSorter (string const & key) : _key (key) {}
+       
+       bool operator() (boost::shared_ptr<Route> a, boost::shared_ptr<Route> b) {
+               /* use of ">" forces the correct sort order */
+               return a->order_key (_key) < b->order_key (_key);
+       }
+
+       string _key;
 };
 
 /** Collect all members of a RouteGroup so that they are together in the Editor or Mixer.
@@ -444,6 +467,7 @@ GroupTabs::collect (RouteGroup* g)
 
        RouteList::iterator i = group_routes->begin ();
        boost::shared_ptr<RouteList> routes = _session->get_routes ();
+       routes->sort (OrderSorter (order_key ()));
        RouteList::const_iterator j = routes->begin ();
 
        int diff = 0;
@@ -511,12 +535,30 @@ void
 GroupTabs::set_group_color (RouteGroup* group, Gdk::Color color)
 {
        assert (group);
+
+       /* Hack to disallow black route groups; force a dark grey instead */
+       if (color.get_red() == 0 && color.get_green() == 0 && color.get_blue() == 0) {
+               color.set_grey_p (0.1);
+       }
        
        GUIObjectState& gui_state = *ARDOUR_UI::instance()->gui_object_state;
 
        char buf[64];
        snprintf (buf, sizeof (buf), "%d:%d:%d", color.get_red(), color.get_green(), color.get_blue());
-       gui_state.set (group_gui_id (group), "color", buf);
+       gui_state.set_property (group_gui_id (group), "color", buf);
+       
+       /* the group color change notification */
+       
+       PBD::PropertyChange change;
+       change.add (Properties::color);
+       group->PropertyChanged (change);
+
+       /* This is a bit of a hack, but this might change
+          our route's effective color, so emit gui_changed
+          for our routes.
+       */
+
+       emit_gui_changed_for_members (group);
 }
 
 /** @return the ID string to use for the GUI state of a route group */
@@ -563,3 +605,53 @@ GroupTabs::group_color (RouteGroup* group)
        return c;
 }
 
+void
+GroupTabs::route_group_property_changed (RouteGroup* rg)
+{
+       /* This is a bit of a hack, but this might change
+          our route's effective color, so emit gui_changed
+          for our routes.
+       */
+
+       emit_gui_changed_for_members (rg);
+       
+       set_dirty ();
+}
+
+void
+GroupTabs::route_added_to_route_group (RouteGroup*, boost::weak_ptr<Route> w)
+{
+       /* Similarly-spirited hack as in route_group_property_changed */
+
+       boost::shared_ptr<Route> r = w.lock ();
+       if (!r) {
+               return;
+       }
+
+       r->gui_changed (X_("color"), 0);
+
+       set_dirty ();
+}
+
+void
+GroupTabs::route_removed_from_route_group (RouteGroup*, boost::weak_ptr<Route> w)
+{
+       /* Similarly-spirited hack as in route_group_property_changed */
+
+       boost::shared_ptr<Route> r = w.lock ();
+       if (!r) {
+               return;
+       }
+
+       r->gui_changed (X_("color"), 0);
+
+       set_dirty ();
+}
+
+void
+GroupTabs::emit_gui_changed_for_members (RouteGroup* rg)
+{
+       for (RouteList::iterator i = rg->route_list()->begin(); i != rg->route_list()->end(); ++i) {
+               (*i)->gui_changed (X_("color"), 0);
+       }
+}