check dependents in Playlist after moving a region to a specific layer; make region layering editor appear reliably; click on xfade sets clicked_routeview
git-svn-id: svn://localhost/ardour2/branches/3.0@7692 d708f5d6-7413-0410-9779-e7cbd77b26cf
This commit is contained in:
parent
b92ebde801
commit
884ee2a3cc
|
@ -2017,7 +2017,7 @@ Editor::add_region_context_items (StreamView* sv, list<boost::shared_ptr<Region>
|
|||
|
||||
edit_items.push_back (MenuElem (menu_item_name, *region_menu));
|
||||
if (multiple_regions_at_position && (layering_order_editor == 0 || !layering_order_editor->is_visible ())) {
|
||||
edit_items.push_back (MenuElem (_("Choose top region"), (bind (mem_fun(*this, &Editor::change_region_layering_order), position))));
|
||||
edit_items.push_back (MenuElem (_("Choose Top Region"), (bind (mem_fun(*this, &Editor::change_region_layering_order), position))));
|
||||
}
|
||||
edit_items.push_back (SeparatorElem());
|
||||
}
|
||||
|
@ -5572,17 +5572,13 @@ Editor::show_editor_list (bool yn)
|
|||
void
|
||||
Editor::change_region_layering_order (framepos_t position)
|
||||
{
|
||||
if (!clicked_regionview) {
|
||||
if (!clicked_routeview) {
|
||||
if (layering_order_editor) {
|
||||
layering_order_editor->hide ();
|
||||
}
|
||||
return;
|
||||
}
|
||||
|
||||
if (!clicked_routeview) {
|
||||
return;
|
||||
}
|
||||
|
||||
boost::shared_ptr<Track> track = boost::dynamic_pointer_cast<Track> (clicked_routeview->route());
|
||||
|
||||
if (!track) {
|
||||
|
|
|
@ -518,6 +518,7 @@ Editor::canvas_crossfade_view_event (GdkEvent* event, ArdourCanvas::Item* item,
|
|||
case GDK_BUTTON_PRESS:
|
||||
clicked_crossfadeview = xfv;
|
||||
clicked_axisview = &clicked_crossfadeview->get_time_axis_view();
|
||||
clicked_routeview = dynamic_cast<RouteTimeAxisView*>(clicked_axisview);
|
||||
if (event->button.button == 3) {
|
||||
return button_press_handler (item, event, CrossfadeViewItem);
|
||||
}
|
||||
|
|
|
@ -219,6 +219,7 @@ RegionLayeringOrderEditor::maybe_present ()
|
|||
hide ();
|
||||
return;
|
||||
}
|
||||
|
||||
present ();
|
||||
}
|
||||
|
||||
|
|
|
@ -399,7 +399,7 @@ AudioPlaylist::check_dependents (boost::shared_ptr<Region> r, bool norefresh)
|
|||
touched_regions = 0;
|
||||
|
||||
try {
|
||||
nframes_t xfade_length;
|
||||
framecnt_t xfade_length;
|
||||
switch (c) {
|
||||
case OverlapNone:
|
||||
break;
|
||||
|
|
|
@ -2603,6 +2603,8 @@ Playlist::move_region_to_layer (layer_t target_layer, boost::shared_ptr<Region>
|
|||
}
|
||||
}
|
||||
|
||||
freeze ();
|
||||
|
||||
/* now reset the layers without holding the region lock */
|
||||
|
||||
for (list<LayerInfo>::iterator x = layerinfo.begin(); x != layerinfo.end(); ++x) {
|
||||
|
@ -2611,15 +2613,16 @@ Playlist::move_region_to_layer (layer_t target_layer, boost::shared_ptr<Region>
|
|||
|
||||
region->set_layer (target_layer);
|
||||
|
||||
#if 0
|
||||
/* now check all dependents */
|
||||
/* now check all dependents, since we changed the layering */
|
||||
|
||||
for (list<LayerInfo>::iterator x = layerinfo.begin(); x != layerinfo.end(); ++x) {
|
||||
check_dependents (x->first, false);
|
||||
}
|
||||
|
||||
check_dependents (region, false);
|
||||
#endif
|
||||
notify_layering_changed ();
|
||||
|
||||
thaw ();
|
||||
|
||||
return 0;
|
||||
}
|
||||
|
|
Loading…
Reference in New Issue
Block a user