X-Git-Url: https://main.carlh.net/gitweb/?a=blobdiff_plain;f=gtk2_ardour%2Feditor_route_groups.cc;h=3cfa1eeafaf7c45ea4bd43c065f6253c4089f87e;hb=5583929a3491075df4e1862579e8df7318dc7370;hp=d74ba50d64bca7f8cb9045c9e6fae80fc5eb2bf6;hpb=a3c378cf62939c85bcfa3b4c55b68c9163c5e21e;p=ardour.git diff --git a/gtk2_ardour/editor_route_groups.cc b/gtk2_ardour/editor_route_groups.cc index d74ba50d64..3cfa1eeafa 100644 --- a/gtk2_ardour/editor_route_groups.cc +++ b/gtk2_ardour/editor_route_groups.cc @@ -45,9 +45,10 @@ #include "route_group_dialog.h" #include "route_time_axis.h" #include "time_axis_view.h" +#include "tooltips.h" #include "utils.h" -#include "i18n.h" +#include "pbd/i18n.h" using namespace std; using namespace ARDOUR; @@ -64,7 +65,6 @@ struct ColumnInfo { EditorRouteGroups::EditorRouteGroups (Editor* e) : EditorComponent (e) - , _all_group_active_button (_("No Selection = All Tracks?")) , _in_row_change (false) , _in_rebuild (false) { @@ -75,7 +75,7 @@ EditorRouteGroups::EditorRouteGroups (Editor* e) TreeViewColumn* color_column = manage (new TreeViewColumn ("", *color_renderer)); color_column->add_attribute (color_renderer->property_color(), _columns.gdkcolor); - + _display.append_column (*color_column); _display.append_column ("", _columns.text); @@ -94,18 +94,18 @@ EditorRouteGroups::EditorRouteGroups (Editor* e) Gtk::Label* l; ColumnInfo ci[] = { - { 0, _("Col"), _("Group Tab Color") }, - { 1, _("Name"), _("Name of Group") }, - { 2, _("V"), _("Group is visible?") }, - { 3, _("On"), _("Group is enabled?") }, - { 4, S_("group|G"), _("Sharing Gain?") }, - { 5, S_("relative|Rel"), _("Relative Gain Changes?") }, - { 6, S_("mute|M"), _("Sharing Mute?") }, - { 7, S_("solo|S"), _("Sharing Solo?") }, - { 8, _("Rec"), _("Sharing Record-enable Status?") }, - { 9, S_("monitoring|Mon"), _("Sharing Monitoring Choice?") }, - { 10, S_("selection|Sel"), _("Sharing Selected/Editing Status?") }, - { 11, S_("active|A"), _("Sharing Active Status?") }, + { 0, _("Col"), _("Group Tab Color") }, + { 1, _("Name"), _("Name of Group") }, + { 2, S_("Visible|V"), _("Group is visible?") }, + { 3, _("On"), _("Group is enabled?") }, + { 4, S_("Group|G"), _("Sharing Gain?") }, + { 5, S_("Relative|Rel"), _("Relative Gain Changes?") }, + { 6, S_("Mute|M"), _("Sharing Mute?") }, + { 7, S_("Solo|S"), _("Sharing Solo?") }, + { 8, _("Rec"), _("Sharing Record-enable Status?") }, + { 9, S_("Monitoring|Mon"), _("Sharing Monitoring Choice?") }, + { 10, S_("Selection|Sel"), _("Sharing Selected/Editing Status?") }, + { 11, S_("Active|A"), _("Sharing Active Status?") }, { -1, 0, 0 } }; @@ -113,7 +113,7 @@ EditorRouteGroups::EditorRouteGroups (Editor* e) for (int i = 0; ci[i].index >= 0; ++i) { col = _display.get_column (ci[i].index); l = manage (new Label (ci[i].label)); - ARDOUR_UI::instance()->set_tip (*l, ci[i].tooltip); + set_tooltip (*l, ci[i].tooltip); col->set_widget (*l); l->show (); @@ -138,7 +138,7 @@ EditorRouteGroups::EditorRouteGroups (Editor* e) CellRendererText* name_cell = dynamic_cast(_display.get_column_cell_renderer (1)); name_cell->property_editable() = true; name_cell->signal_edited().connect (sigc::mem_fun (*this, &EditorRouteGroups::name_edit)); - + for (int i = 1; ci[i].index >= 0; ++i) { CellRendererToggle* active_cell = dynamic_cast (_display.get_column_cell_renderer (i)); @@ -188,15 +188,8 @@ EditorRouteGroups::EditorRouteGroups (Editor* e) button_box->pack_start (*add_button); button_box->pack_start (*remove_button); - _all_group_active_button.show (); - _display_packer.pack_start (_scroller, true, true); - _display_packer.pack_start (_all_group_active_button, false, false); _display_packer.pack_start (*button_box, false, false); - - _all_group_active_button.signal_toggled().connect (sigc::mem_fun (*this, &EditorRouteGroups::all_group_toggled)); - _all_group_active_button.set_name (X_("EditorRouteGroupsAllGroupButton")); - ARDOUR_UI::instance()->set_tip (_all_group_active_button, _("Activate this button to operate on all tracks when none are selected.")); } void @@ -205,7 +198,7 @@ EditorRouteGroups::remove_selected () Glib::RefPtr selection = _display.get_selection(); TreeView::Selection::ListHandle_Path rows = selection->get_selected_rows (); - if (rows.empty()) { + if (rows.empty() || _session->deletion_in_progress()) { return; } @@ -251,12 +244,12 @@ EditorRouteGroups::button_press_event (GdkEventButton* ev) if (iter) { group = (*iter)[_columns.routegroup]; - } + } if (Keyboard::is_context_menu_event (ev)) { _editor->_group_tabs->get_menu(group)->popup (1, ev->time); return true; - } + } if (!p) { /* cancel selection */ @@ -269,7 +262,7 @@ EditorRouteGroups::button_press_event (GdkEventButton* ev) group = (*iter)[_columns.routegroup]; switch (GPOINTER_TO_UINT (column->get_data (X_("colnum")))) { - case 0: + case 0: c = (*iter)[_columns.gdkcolor]; color_dialog.get_colorsel()->set_previous_color (c); color_dialog.get_colorsel()->set_current_color (c); @@ -280,12 +273,11 @@ EditorRouteGroups::button_press_event (GdkEventButton* ev) case RESPONSE_ACCEPT: c = color_dialog.get_colorsel()->get_current_color(); GroupTabs::set_group_color (group, gdk_color_to_rgba (c)); - ARDOUR_UI::config()->set_dirty (); break; - + default: break; - + } color_dialog.hide (); @@ -309,7 +301,7 @@ EditorRouteGroups::button_press_event (GdkEventButton* ev) ret = true; break; - + case 3: val = (*iter)[_columns.active_state]; group->set_active (!val, this); @@ -388,21 +380,22 @@ EditorRouteGroups::row_change (const Gtk::TreeModel::Path&, const Gtk::TreeModel plist.add (Properties::name, string ((*iter)[_columns.text])); bool val = (*iter)[_columns.gain]; - plist.add (Properties::gain, val); + plist.add (Properties::group_gain, val); val = (*iter)[_columns.gain_relative]; - plist.add (Properties::relative, val); + plist.add (Properties::group_relative, val); val = (*iter)[_columns.mute]; - plist.add (Properties::mute, val); + plist.add (Properties::group_mute, val); val = (*iter)[_columns.solo]; - plist.add (Properties::solo, val); + plist.add (Properties::group_solo, val); val = (*iter)[_columns.record]; - plist.add (Properties::recenable, val); + plist.add (Properties::group_recenable, val); val = (*iter)[_columns.monitoring]; - plist.add (Properties::monitoring, val); + plist.add (Properties::group_monitoring, val); val = (*iter)[_columns.select]; - plist.add (Properties::select, val); + plist.add (Properties::group_select, val); val = (*iter)[_columns.active_shared]; - plist.add (Properties::route_active, val); + plist.add (Properties::group_route_active, val); + val = (*iter)[_columns.active_state]; plist.add (Properties::active, val); val = (*iter)[_columns.is_visible]; @@ -431,11 +424,11 @@ EditorRouteGroups::add (RouteGroup* group) row[_columns.active_shared] = group->is_route_active (); row[_columns.active_state] = group->is_active (); row[_columns.is_visible] = !group->is_hidden(); - + Gdk::Color c; set_color_from_rgba (c, GroupTabs::group_color (group)); row[_columns.gdkcolor] = c; - + _in_row_change = true; row[_columns.routegroup] = group; @@ -481,6 +474,7 @@ EditorRouteGroups::groups_changed () void EditorRouteGroups::property_changed (RouteGroup* group, const PropertyChange&) { + assert(group); _in_row_change = true; Gtk::TreeModel::Children children = _model->children(); @@ -512,7 +506,7 @@ EditorRouteGroups::property_changed (RouteGroup* group, const PropertyChange&) break; } } - + _in_row_change = false; for (TrackViewList::const_iterator i = _editor->get_track_views().begin(); i != _editor->get_track_views().end(); ++i) { @@ -559,10 +553,6 @@ EditorRouteGroups::set_session (Session* s) if (_session) { - RouteGroup& arg (_session->all_route_group()); - - arg.PropertyChanged.connect (all_route_groups_changed_connection, MISSING_INVALIDATOR, boost::bind (&EditorRouteGroups::all_group_changed, this, _1), gui_context()); - _session->route_group_added.connect (_session_connections, MISSING_INVALIDATOR, boost::bind (&EditorRouteGroups::add, this, _1), gui_context()); _session->route_group_removed.connect ( _session_connections, MISSING_INVALIDATOR, boost::bind (&EditorRouteGroups::groups_changed, this), gui_context() @@ -573,9 +563,8 @@ EditorRouteGroups::set_session (Session* s) } PBD::PropertyChange pc; - pc.add (Properties::select); + pc.add (Properties::group_select); pc.add (Properties::active); - all_group_changed (pc); groups_changed (); } @@ -583,28 +572,7 @@ EditorRouteGroups::set_session (Session* s) void EditorRouteGroups::run_new_group_dialog () { - RouteList rl; - - return _editor->_group_tabs->run_new_group_dialog (rl); -} - -void -EditorRouteGroups::all_group_toggled () -{ - if (_session) { - _session->all_route_group().set_select (_all_group_active_button.get_active()); - } -} - -void -EditorRouteGroups::all_group_changed (const PropertyChange&) -{ - if (_session) { - RouteGroup& arg (_session->all_route_group()); - _all_group_active_button.set_active (arg.is_active() && arg.is_select()); - } else { - _all_group_active_button.set_active (false); - } + return _editor->_group_tabs->run_new_group_dialog (0, false); } /** Called when a model row is deleted, but also when the model is @@ -614,7 +582,7 @@ EditorRouteGroups::all_group_changed (const PropertyChange&) void EditorRouteGroups::row_deleted (Gtk::TreeModel::Path const &) { - if (_in_rebuild) { + if (_in_rebuild || !_session || _session->deletion_in_progress()) { /* We need to ignore this in cases where we're not doing a drag-and-drop re-order. */