X-Git-Url: https://main.carlh.net/gitweb/?a=blobdiff_plain;f=gtk2_ardour%2Feditor_selection.cc;h=0a7ad1958c642671c61b41614fb14fd4729f0e7c;hb=25a702798e7fc2f3875208ceb67e0b6a7dfb1715;hp=893adf0262b58c29b8cbb775c83bcc23c442f531;hpb=5da31b63a9d047acb80a61ac6ac7eed67edba799;p=ardour.git diff --git a/gtk2_ardour/editor_selection.cc b/gtk2_ardour/editor_selection.cc index 893adf0262..0a7ad1958c 100644 --- a/gtk2_ardour/editor_selection.cc +++ b/gtk2_ardour/editor_selection.cc @@ -26,6 +26,7 @@ #include "ardour/playlist.h" #include "ardour/route_group.h" #include "ardour/profile.h" +#include "ardour/midi_region.h" #include "editor.h" #include "actions.h" @@ -35,6 +36,7 @@ #include "automation_line.h" #include "control_point.h" #include "editor_regions.h" +#include "editor_cursors.h" #include "i18n.h" @@ -174,7 +176,7 @@ Editor::select_all_tracks () * tracks, in which case nothing will happen unless `force' is true. */ void -Editor::set_selected_track_as_side_effect (Selection::Operation op, bool force) +Editor::set_selected_track_as_side_effect (Selection::Operation op, bool /*force*/) { if (!clicked_axisview) { return; @@ -430,7 +432,7 @@ Editor::mapped_get_equivalent_regions (RouteTimeAxisView& tv, uint32_t, RegionVi void Editor::get_equivalent_regions (RegionView* basis, vector& equivalent_regions, PBD::PropertyID property) const { - mapover_tracks (sigc::bind (sigc::mem_fun (*this, &Editor::mapped_get_equivalent_regions), basis, &equivalent_regions), &basis->get_trackview(), property); + mapover_tracks (sigc::bind (sigc::mem_fun (*this, &Editor::mapped_get_equivalent_regions), basis, &equivalent_regions), &basis->get_time_axis_view(), property); /* add clicked regionview since we skipped all other regions in the same track as the one it was in */ @@ -448,7 +450,7 @@ Editor::get_equivalent_regions (RegionSelection & basis, PBD::PropertyID prop) c mapover_tracks ( sigc::bind (sigc::mem_fun (*this, &Editor::mapped_get_equivalent_regions), *i, &eq), - &(*i)->get_trackview(), prop + &(*i)->get_time_axis_view(), prop ); for (vector::iterator j = eq.begin(); j != eq.end(); ++j) { @@ -586,14 +588,14 @@ Editor::set_selected_regionview_from_click (bool press, Selection::Operation op, } else if (op == Selection::Extend) { list results; - nframes64_t last_frame; - nframes64_t first_frame; + framepos_t last_frame; + framepos_t first_frame; bool same_track = false; /* 1. find the last selected regionview in the track that was clicked in */ last_frame = 0; - first_frame = max_frames; + first_frame = max_framepos; for (RegionSelection::iterator x = selection->regions.begin(); x != selection->regions.end(); ++x) { if (&(*x)->get_time_axis_view() == &clicked_regionview->get_time_axis_view()) { @@ -661,8 +663,8 @@ Editor::set_selected_regionview_from_click (bool press, Selection::Operation op, */ - first_frame = entered_regionview->region()->position(); - last_frame = entered_regionview->region()->last_frame(); + first_frame = clicked_regionview->region()->position(); + last_frame = clicked_regionview->region()->last_frame(); for (RegionSelection::iterator i = selection->regions.begin(); i != selection->regions.end(); ++i) { if ((*i)->region()->position() < first_frame) { @@ -696,9 +698,9 @@ Editor::set_selected_regionview_from_click (bool press, Selection::Operation op, any selected ones. */ - if (!selection->selected (entered_regionview)) { + if (!selection->selected (clicked_regionview)) { - RouteTimeAxisView* rtv = dynamic_cast (&entered_regionview->get_time_axis_view()); + RouteTimeAxisView* rtv = dynamic_cast (&clicked_regionview->get_time_axis_view()); if (rtv) { @@ -927,27 +929,222 @@ Editor::time_selection_changed () } } +/** Set all region actions to have a given sensitivity */ void -Editor::sensitize_the_right_region_actions (bool have_selected_regions) +Editor::sensitize_all_region_actions (bool s) { - for (vector >::iterator x = ActionManager::region_selection_sensitive_actions.begin(); - x != ActionManager::region_selection_sensitive_actions.end(); ++x) { + Glib::ListHandle > all = _region_actions->get_actions (); - string accel_path = (*x)->get_accel_path (); - AccelKey key; + for (Glib::ListHandle >::iterator i = all.begin(); i != all.end(); ++i) { + (*i)->set_sensitive (s); + } - /* if there is an accelerator, it should always be sensitive - to allow for keyboard ops on entered regions. - */ + _all_region_actions_sensitized = s; +} + +/** Sensitize region-based actions based on the selection ONLY, ignoring the entered_regionview. + * This method should be called just before displaying a Region menu. When a Region menu is not + * currently being shown, all region actions are sensitized so that hotkey-triggered actions + * on entered_regionviews work without having to check sensitivity every time the selection or + * entered_regionview changes. + * + * This method also sets up toggle action state as appropriate. + */ +void +Editor::sensitize_the_right_region_actions () +{ + if ((mouse_mode == MouseRange) || (mouse_mode != MouseObject && _join_object_range_state == JOIN_OBJECT_RANGE_RANGE)) { + sensitize_all_region_actions (false); + if (!selection->time.empty()) { + _region_actions->get_action("split-region")->set_sensitive (true); + } + + return; + + } else if (mouse_mode != MouseObject) { + sensitize_all_region_actions (false); + return; + } + + /* We get here if we are in Object mode */ + + RegionSelection rs = get_regions_from_selection_and_entered (); + sensitize_all_region_actions (!rs.empty ()); + + _ignore_region_action = true; + + /* Look through the regions that are selected and make notes about what we have got */ + + bool have_audio = false; + bool have_midi = false; + bool have_locked = false; + bool have_unlocked = false; + bool have_position_lock_style_audio = false; + bool have_position_lock_style_music = false; + bool have_muted = false; + bool have_unmuted = false; + bool have_opaque = false; + bool have_non_opaque = false; + bool have_not_at_natural_position = false; + bool have_envelope_visible = false; + bool have_envelope_invisible = false; + bool have_envelope_active = false; + bool have_envelope_inactive = false; + bool have_non_unity_scale_amplitude = false; + + for (list::const_iterator i = rs.begin(); i != rs.end(); ++i) { + + boost::shared_ptr r = (*i)->region (); + boost::shared_ptr ar = boost::dynamic_pointer_cast (r); + + if (ar) { + have_audio = true; + } + + if (boost::dynamic_pointer_cast (r)) { + have_midi = true; + } - bool known = ActionManager::lookup_entry (accel_path, key); + if (r->locked()) { + have_locked = true; + } else { + have_unlocked = true; + } - if (known && ((key.get_key() != GDK_VoidSymbol) && (key.get_key() != 0))) { - (*x)->set_sensitive (true); + if (r->position_lock_style() == MusicTime) { + have_position_lock_style_music = true; } else { - (*x)->set_sensitive (have_selected_regions); + have_position_lock_style_audio = true; } + + if (r->muted()) { + have_muted = true; + } else { + have_unmuted = true; + } + + if (r->opaque()) { + have_opaque = true; + } else { + have_non_opaque = true; + } + + if (!r->at_natural_position()) { + have_not_at_natural_position = true; + } + + if (ar) { + /* its a bit unfortunate that "envelope visible" is a view-only + property. we have to find the regionview to able to check + its current setting. + */ + + have_envelope_invisible = true; + + if (*i) { + AudioRegionView* arv = dynamic_cast (*i); + if (arv && arv->envelope_visible()) { + have_envelope_visible = true; + } + } + + if (ar->envelope_active()) { + have_envelope_active = true; + } else { + have_envelope_inactive = true; + } + + if (ar->scale_amplitude() != 1) { + have_non_unity_scale_amplitude = true; + } + } + } + + if (rs.size() > 1) { + _region_actions->get_action("show-region-list-editor")->set_sensitive (false); + _region_actions->get_action("show-region-properties")->set_sensitive (false); + _region_actions->get_action("rename-region")->set_sensitive (false); + } else if (rs.size() == 1) { + _region_actions->get_action("add-range-markers-from-region")->set_sensitive (false); + _region_actions->get_action("close-region-gaps")->set_sensitive (false); + } + + if (!have_midi) { + _region_actions->get_action("show-region-list-editor")->set_sensitive (false); + _region_actions->get_action("quantize-region")->set_sensitive (false); + _region_actions->get_action("fork-region")->set_sensitive (false); + } + + if (_edit_point == EditAtMouse) { + _region_actions->get_action("set-region-sync-position")->set_sensitive (false); + _region_actions->get_action("trim-front")->set_sensitive (false); + _region_actions->get_action("trim-back")->set_sensitive (false); + _region_actions->get_action("split-region")->set_sensitive (false); + _region_actions->get_action("place-transient")->set_sensitive (false); + } + + if (have_audio) { + + if (have_envelope_visible && !have_envelope_invisible) { + Glib::RefPtr::cast_dynamic (_region_actions->get_action("toggle-region-gain-envelope-visible"))->set_active (); + } else if (have_envelope_visible && have_envelope_invisible) { +// _region_actions->get_action("toggle-region-gain-envelope-visible")->set_inconsistent (); + } + + if (have_envelope_active && !have_envelope_inactive) { + Glib::RefPtr::cast_dynamic (_region_actions->get_action("toggle-region-gain-envelope-active"))->set_active (); + } else if (have_envelope_active && have_envelope_inactive) { +// _region_actions->get_action("toggle-region-gain-envelope-active")->set_inconsistent (); + } + + } else { + + _region_actions->get_action("analyze-region")->set_sensitive (false); + _region_actions->get_action("reset-region-gain-envelopes")->set_sensitive (false); + _region_actions->get_action("toggle-region-gain-envelope-visible")->set_sensitive (false); + _region_actions->get_action("toggle-region-gain-envelope-active")->set_sensitive (false); + + } + + if (!have_non_unity_scale_amplitude || !have_audio) { + _region_actions->get_action("reset-region-scale-amplitude")->set_sensitive (false); + } + + Glib::RefPtr::cast_dynamic (_region_actions->get_action("toggle-region-lock"))->set_active (have_locked && !have_unlocked); + if (have_locked && have_unlocked) { +// _region_actions->get_action("toggle-region-lock")->set_inconsistent (); } + + Glib::RefPtr::cast_dynamic (_region_actions->get_action("toggle-region-lock-style"))->set_active (have_position_lock_style_music && !have_position_lock_style_audio); + + if (have_position_lock_style_music && have_position_lock_style_audio) { +// _region_actions->get_action("toggle-region-lock-style")->set_inconsistent (); + } + + Glib::RefPtr::cast_dynamic (_region_actions->get_action("toggle-region-mute"))->set_active (have_muted && !have_unmuted); + if (have_muted && have_unmuted) { +// _region_actions->get_action("toggle-region-mute")->set_inconsistent (); + } + + Glib::RefPtr::cast_dynamic (_region_actions->get_action("toggle-opaque-region"))->set_active (have_opaque && !have_non_opaque); + if (have_opaque && have_non_opaque) { +// _region_actions->get_action("toggle-opaque-region")->set_inconsistent (); + } + + if (!have_not_at_natural_position) { + _region_actions->get_action("naturalize-region")->set_sensitive (false); + } + + /* XXX: should also check that there is a track of the appropriate type for the selected region */ + if (_edit_point == EditAtMouse || _regions->get_single_selection() == 0 || selection->tracks.empty()) { + _region_actions->get_action("insert-region-from-region-list")->set_sensitive (false); + } else { + _region_actions->get_action("insert-region-from-region-list")->set_sensitive (true); + } + + _ignore_region_action = false; + + _all_region_actions_sensitized = false; } @@ -957,18 +1154,27 @@ Editor::region_selection_changed () _regions->block_change_connection (true); editor_regions_selection_changed_connection.block(true); - _regions->unselect_all (); + if (_region_selection_change_updates_region_list) { + _regions->unselect_all (); + } for (TrackViewList::iterator i = track_views.begin(); i != track_views.end(); ++i) { (*i)->set_selected_regionviews (selection->regions); } - _regions->set_selected (selection->regions); - - sensitize_the_right_region_actions (!selection->regions.empty()); + if (_region_selection_change_updates_region_list) { + _regions->set_selected (selection->regions); + } _regions->block_change_connection (false); editor_regions_selection_changed_connection.block(false); + + if (!_all_region_actions_sensitized) { + /* This selection change might have changed what region actions + are allowed, so sensitize them all in case a key is pressed. + */ + sensitize_all_region_actions (true); + } } void @@ -988,7 +1194,7 @@ Editor::select_all_in_track (Selection::Operation op) return; } - clicked_routeview->get_selectables (0, max_frames, 0, DBL_MAX, touched); + clicked_routeview->get_selectables (0, max_framepos, 0, DBL_MAX, touched); switch (op) { case Selection::Toggle: @@ -1015,7 +1221,7 @@ Editor::select_all (Selection::Operation op) if ((*iter)->hidden()) { continue; } - (*iter)->get_selectables (0, max_frames, 0, DBL_MAX, touched); + (*iter)->get_selectables (0, max_framepos, 0, DBL_MAX, touched); } begin_reversible_command (_("select all")); switch (op) { @@ -1178,8 +1384,8 @@ Editor::select_all_selectables_using_time_selection () return; } - nframes64_t start = selection->time[clicked_selection].start; - nframes64_t end = selection->time[clicked_selection].end; + framepos_t start = selection->time[clicked_selection].start; + framepos_t end = selection->time[clicked_selection].end; if (end - start < 1) { return; @@ -1271,13 +1477,13 @@ Editor::select_all_selectables_using_loop() void Editor::select_all_selectables_using_cursor (EditorCursor *cursor, bool after) { - nframes64_t start; - nframes64_t end; + framepos_t start; + framepos_t end; list touched; if (after) { begin_reversible_command (_("select all after cursor")); - start = cursor->current_frame ; + start = cursor->current_frame; end = _session->current_end_frame(); } else { if (cursor->current_frame > 0) { @@ -1311,8 +1517,8 @@ Editor::select_all_selectables_using_cursor (EditorCursor *cursor, bool after) void Editor::select_all_selectables_using_edit (bool after) { - nframes64_t start; - nframes64_t end; + framepos_t start; + framepos_t end; list touched; if (after) { @@ -1351,8 +1557,8 @@ Editor::select_all_selectables_using_edit (bool after) void Editor::select_all_selectables_between (bool /*within*/) { - nframes64_t start; - nframes64_t end; + framepos_t start; + framepos_t end; list touched; if (!get_edit_op_range (start, end)) { @@ -1380,8 +1586,8 @@ Editor::select_all_selectables_between (bool /*within*/) void Editor::select_range_between () { - nframes64_t start; - nframes64_t end; + framepos_t start; + framepos_t end; if (mouse_mode == MouseRange && !selection->time.empty()) { selection->clear_time (); @@ -1396,9 +1602,9 @@ Editor::select_range_between () } bool -Editor::get_edit_op_range (nframes64_t& start, nframes64_t& end) const +Editor::get_edit_op_range (framepos_t& start, framepos_t& end) const { - nframes64_t m; + framepos_t m; bool ignored; /* in range mode, use any existing selection */