use VCA full name in strips and track headers
[ardour.git] / gtk2_ardour / editor.cc
index ccb839ccc18a7cc8ef3555d52562454b73d7d4c0..62ad00a1d99921bdb7ecc2c9ae995931e64e504f 100644 (file)
@@ -87,6 +87,7 @@
 
 #include "actions.h"
 #include "analysis_window.h"
+#include "ardour_spacer.h"
 #include "audio_clock.h"
 #include "audio_region_view.h"
 #include "audio_streamview.h"
 #include "sfdb_ui.h"
 #include "tempo_lines.h"
 #include "time_axis_view.h"
+#include "time_info_box.h"
 #include "timers.h"
 #include "tooltips.h"
 #include "ui_config.h"
@@ -242,6 +244,7 @@ Editor::Editor ()
        , editor_mixer_strip_width (Wide)
        , constructed (false)
        , _playlist_selector (0)
+       , _time_info_box (0)
        , no_save_visual (false)
        , leftmost_frame (0)
        , samples_per_pixel (2048)
@@ -384,14 +387,12 @@ Editor::Editor ()
        , _visible_track_count (-1)
        ,  toolbar_selection_clock_table (2,3)
        ,  automation_mode_button (_("mode"))
-       ,  _toolbar_viewport (*manage (new Gtk::Adjustment (0, 0, 1e10)), *manage (new Gtk::Adjustment (0, 0, 1e10)))
        , selection (new Selection (this))
        , cut_buffer (new Selection (this))
        , _selection_memento (new SelectionMemento())
        , _all_region_actions_sensitized (false)
        , _ignore_region_action (false)
        , _last_region_menu_was_main (false)
-       , _ignore_follow_edits (false)
        , cd_marker_bar_drag_rect (0)
        , range_bar_drag_rect (0)
        , transport_bar_drag_rect (0)
@@ -577,7 +578,6 @@ Editor::Editor ()
        _summary = new EditorSummary (this);
 
        selection->TimeChanged.connect (sigc::mem_fun(*this, &Editor::time_selection_changed));
-       selection->TracksChanged.connect (sigc::mem_fun(*this, &Editor::track_selection_changed));
 
        editor_regions_selection_changed_connection = selection->RegionsChanged.connect (sigc::mem_fun(*this, &Editor::region_selection_changed));
 
@@ -643,6 +643,7 @@ Editor::Editor ()
        _regions = new EditorRegions (this);
        _snapshots = new EditorSnapshots (this);
        _locations = new EditorLocations (this);
+       _time_info_box = new TimeInfoBox (true);
 
        /* these are static location signals */
 
@@ -722,11 +723,13 @@ Editor::Editor ()
        edit_pane.set_check_divider_position (true);
        edit_pane.add (editor_summary_pane);
        if (!ARDOUR::Profile->get_trx()) {
-               edit_pane.add (_the_notebook);
+               _editor_list_vbox.pack_start (*_time_info_box, false, false, 0);
+               _editor_list_vbox.pack_start (_the_notebook);
+               edit_pane.add (_editor_list_vbox);
+               edit_pane.set_child_minsize (_editor_list_vbox, 30); /* rough guess at width of notebook tabs */
        }
 
        edit_pane.set_drag_cursor (*_cursors->expand_left_right);
-       edit_pane.set_child_minsize (_the_notebook, 30); /* rough guess at width of notebook tabs */
        editor_summary_pane.set_drag_cursor (*_cursors->expand_up_down);
 
        float fract;
@@ -750,14 +753,26 @@ Editor::Editor ()
                }
        }
 
-       top_hbox.pack_start (toolbar_frame);
+       global_vpacker.set_spacing (2);
+       global_vpacker.set_border_width (0);
 
-       HBox *hbox = manage (new HBox);
-       hbox->pack_start (edit_pane, true, true);
+       //the next three EventBoxes provide the ability for their child widgets to have a background color.  That is all.
 
-       global_vpacker.pack_start (top_hbox, false, false);
-       global_vpacker.pack_start (*hbox, true, true);
-       global_hpacker.pack_start (global_vpacker, true, true);
+       Gtk::EventBox* ebox = manage (new Gtk::EventBox);  //a themeable box
+       ebox->set_name("EditorWindow");
+       ebox->add (toolbar_hbox);
+
+       Gtk::EventBox* epane_box = manage (new Gtk::EventBox);  //a themeable box
+       epane_box->set_name("EditorWindow");
+       epane_box->add (edit_pane);
+
+       Gtk::EventBox* epane_box2 = manage (new Gtk::EventBox);  //a themeable box
+       epane_box2->set_name("EditorWindow");
+       epane_box2->add (global_vpacker);
+
+       global_vpacker.pack_start (*ebox, false, false);
+       global_vpacker.pack_start (*epane_box, true, true);
+       global_hpacker.pack_start (*epane_box2, true, true);
 
        /* need to show the "contents" widget so that notebook will show if tab is switched to
         */
@@ -813,6 +828,8 @@ Editor::Editor ()
 
        BasicUI::AccessAction.connect (*this, invalidator (*this), boost::bind (&Editor::access_action, this, _1, _2), gui_context());
 
+       PresentationInfo::Change.connect (*this, invalidator (*this), boost::bind (&Editor::presentation_info_changed, this, _1), gui_context());
+
        /* handle escape */
 
        ARDOUR_UI::instance()->Escape.connect (*this, invalidator (*this), boost::bind (&Editor::escape, this), gui_context());
@@ -830,8 +847,6 @@ Editor::Editor ()
        _last_region_menu_was_main = false;
        _popup_region_menu_item = 0;
 
-       _ignore_follow_edits = false;
-
        _show_marker_lines = false;
 
         /* Button bindings */
@@ -875,10 +890,30 @@ Editor::~Editor()
        delete _snapshots;
        delete _locations;
        delete _playlist_selector;
+       delete _time_info_box;
+       delete selection;
+       delete cut_buffer;
+       delete _cursors;
+
+       LuaInstance::destroy_instance ();
 
        for (list<XMLNode *>::iterator i = selection_op_history.begin(); i != selection_op_history.end(); ++i) {
                delete *i;
        }
+       for (std::map<ARDOUR::FadeShape, Gtk::Image*>::const_iterator i = _xfade_in_images.begin(); i != _xfade_in_images.end (); ++i) {
+               delete i->second;
+       }
+       for (std::map<ARDOUR::FadeShape, Gtk::Image*>::const_iterator i = _xfade_out_images.begin(); i != _xfade_out_images.end (); ++i) {
+               delete i->second;
+       }
+}
+
+void
+Editor::presentation_info_changed (PropertyChange const & what_changed)
+{
+       if (what_changed.contains (Properties::selected)) {
+               track_selection_changed ();
+       }
 }
 
 XMLNode*
@@ -1312,6 +1347,7 @@ Editor::set_session (Session *t)
        _snapshots->set_session (_session);
        _routes->set_session (_session);
        _locations->set_session (_session);
+       _time_info_box->set_session (_session);
 
        if (rhythm_ferret) {
                rhythm_ferret->set_session (_session);
@@ -1368,7 +1404,7 @@ Editor::set_session (Session *t)
        _session->RouteAdded.connect (_session_connections, invalidator (*this), boost::bind (&Editor::add_routes, this, _1), gui_context());
        _session->DirtyChanged.connect (_session_connections, invalidator (*this), boost::bind (&Editor::update_title, this), gui_context());
        _session->tempo_map().PropertyChanged.connect (_session_connections, invalidator (*this), boost::bind (&Editor::tempo_map_changed, this, _1), gui_context());
-       _session->tempo_map().MetricPositionChanged.connect (_session_connections, invalidator (*this), boost::bind (&Editor::marker_position_changed, this), gui_context());
+       _session->tempo_map().MetricPositionChanged.connect (_session_connections, invalidator (*this), boost::bind (&Editor::tempometric_position_changed, this, _1), gui_context());
        _session->Located.connect (_session_connections, invalidator (*this), boost::bind (&Editor::located, this), gui_context());
        _session->config.ParameterChanged.connect (_session_connections, invalidator (*this), boost::bind (&Editor::parameter_changed, this, _1), gui_context());
        _session->StateSaved.connect (_session_connections, invalidator (*this), boost::bind (&Editor::session_state_saved, this, _1), gui_context());
@@ -1407,6 +1443,25 @@ Editor::set_session (Session *t)
                break;
        }
 
+       /* catch up on selection of stripables (other selection state is lost
+        * when a session is closed
+        */
+
+       StripableList sl;
+       TrackViewList tl;
+       _session->get_stripables (sl);
+       for (StripableList::const_iterator s = sl.begin(); s != sl.end(); ++s) {
+               if ((*s)->presentation_info().selected()) {
+                       RouteTimeAxisView* rtav = get_route_view_by_route_id ((*s)->id());
+                       if (rtav) {
+                               tl.push_back (rtav);
+                       }
+               }
+       }
+       if (!tl.empty()) {
+               selection->set (tl);
+       }
+
        /* register for undo history */
        _session->register_with_memento_command_factory(id(), this);
        _session->register_with_memento_command_factory(_selection_memento->id(), _selection_memento);
@@ -1928,7 +1983,7 @@ Editor::add_selection_context_items (Menu_Helpers::MenuList& edit_items)
        edit_items.push_back (MenuElem (_("Loop Range"), sigc::bind (sigc::mem_fun(*this, &Editor::set_loop_from_selection), true)));
 
        edit_items.push_back (SeparatorElem());
-       edit_items.push_back (MenuElem (_("Zoom to Range"), sigc::bind (sigc::mem_fun(*this, &Editor::temporal_zoom_selection), false)));
+       edit_items.push_back (MenuElem (_("Zoom to Range"), sigc::bind (sigc::mem_fun(*this, &Editor::temporal_zoom_selection), Horizontal)));
 
        edit_items.push_back (SeparatorElem());
        edit_items.push_back (MenuElem (_("Loudness Analysis"), sigc::mem_fun(*this, &Editor::loudness_analyze_range_selection)));
@@ -2220,10 +2275,8 @@ Editor::set_snap_to (SnapType st)
        case SnapToBeatDiv4:
        case SnapToBeatDiv3:
        case SnapToBeatDiv2: {
-               std::vector<TempoMap::BBTPoint> grid;
-               compute_current_bbt_points (grid, leftmost_frame, leftmost_frame + current_page_samples());
-               compute_bbt_ruler_scale (grid, leftmost_frame, leftmost_frame + current_page_samples());
-               update_tempo_based_rulers (grid);
+               compute_bbt_ruler_scale (leftmost_frame, leftmost_frame + current_page_samples());
+               update_tempo_based_rulers ();
                break;
        }
 
@@ -2714,18 +2767,18 @@ Editor::snap_to (framepos_t& start, RoundMode direction, bool for_mark, bool ens
 void
 Editor::timecode_snap_to_internal (framepos_t& start, RoundMode direction, bool /*for_mark*/)
 {
-       const framepos_t one_timecode_second = (framepos_t)(rint(_session->timecode_frames_per_second()) * _session->frames_per_timecode_frame());
-       framepos_t one_timecode_minute = (framepos_t)(rint(_session->timecode_frames_per_second()) * _session->frames_per_timecode_frame() * 60);
+       const framepos_t one_timecode_second = (framepos_t)(rint(_session->timecode_frames_per_second()) * _session->samples_per_timecode_frame());
+       framepos_t one_timecode_minute = (framepos_t)(rint(_session->timecode_frames_per_second()) * _session->samples_per_timecode_frame() * 60);
 
        switch (_snap_type) {
        case SnapToTimecodeFrame:
                if ((direction == RoundUpMaybe || direction == RoundDownMaybe) &&
-                   fmod((double)start, (double)_session->frames_per_timecode_frame()) == 0) {
+                   fmod((double)start, (double)_session->samples_per_timecode_frame()) == 0) {
                        /* start is already on a whole timecode frame, do nothing */
-               } else if (((direction == 0) && (fmod((double)start, (double)_session->frames_per_timecode_frame()) > (_session->frames_per_timecode_frame() / 2))) || (direction > 0)) {
-                       start = (framepos_t) (ceil ((double) start / _session->frames_per_timecode_frame()) * _session->frames_per_timecode_frame());
+               } else if (((direction == 0) && (fmod((double)start, (double)_session->samples_per_timecode_frame()) > (_session->samples_per_timecode_frame() / 2))) || (direction > 0)) {
+                       start = (framepos_t) (ceil ((double) start / _session->samples_per_timecode_frame()) * _session->samples_per_timecode_frame());
                } else {
-                       start = (framepos_t) (floor ((double) start / _session->frames_per_timecode_frame()) *  _session->frames_per_timecode_frame());
+                       start = (framepos_t) (floor ((double) start / _session->samples_per_timecode_frame()) *  _session->samples_per_timecode_frame());
                }
                break;
 
@@ -2900,11 +2953,16 @@ Editor::snap_to_internal (framepos_t& start, RoundMode direction, bool for_mark,
                } else if (after == max_framepos) {
                        start = before;
                } else if (before != max_framepos && after != max_framepos) {
-                       /* have before and after */
-                       if ((start - before) < (after - start)) {
-                               start = before;
-                       } else {
+                       if ((direction == RoundUpMaybe || direction == RoundUpAlways))
                                start = after;
+                       else if ((direction == RoundDownMaybe || direction == RoundDownAlways))
+                               start = before;
+                       else if (direction ==  0 ) {
+                               if ((start - before) < (after - start)) {
+                                       start = before;
+                               } else {
+                                       start = after;
+                               }
                        }
                }
 
@@ -2993,15 +3051,17 @@ Editor::setup_toolbar ()
        mouse_mode_size_group->add_widget (mouse_draw_button);
        mouse_mode_size_group->add_widget (mouse_content_button);
 
-       mouse_mode_size_group->add_widget (zoom_in_button);
-       mouse_mode_size_group->add_widget (zoom_out_button);
-       mouse_mode_size_group->add_widget (zoom_preset_selector);
-       mouse_mode_size_group->add_widget (zoom_out_full_button);
-       mouse_mode_size_group->add_widget (zoom_focus_selector);
-
-       mouse_mode_size_group->add_widget (tav_shrink_button);
-       mouse_mode_size_group->add_widget (tav_expand_button);
-       mouse_mode_size_group->add_widget (visible_tracks_selector);
+       if (!Profile->get_mixbus()) {
+               mouse_mode_size_group->add_widget (zoom_in_button);
+               mouse_mode_size_group->add_widget (zoom_out_button);
+               mouse_mode_size_group->add_widget (zoom_out_full_button);
+               mouse_mode_size_group->add_widget (zoom_focus_selector);
+               mouse_mode_size_group->add_widget (tav_shrink_button);
+               mouse_mode_size_group->add_widget (tav_expand_button);
+       } else {
+               mouse_mode_size_group->add_widget (zoom_preset_selector);
+               mouse_mode_size_group->add_widget (visible_tracks_selector);
+       }
 
        mouse_mode_size_group->add_widget (snap_type_selector);
        mouse_mode_size_group->add_widget (snap_mode_selector);
@@ -3056,8 +3116,7 @@ Editor::setup_toolbar ()
        RefPtr<Action> act;
 
        zoom_preset_selector.set_name ("zoom button");
-       zoom_preset_selector.set_image(::get_icon ("time_exp"));
-       zoom_preset_selector.set_size_request (42, -1);
+       zoom_preset_selector.set_icon (ArdourIcon::ZoomExpand);
 
        zoom_in_button.set_name ("zoom button");
        zoom_in_button.set_icon (ArdourIcon::ZoomIn);
@@ -3089,10 +3148,12 @@ Editor::setup_toolbar ()
        }
 
        /* Track zoom buttons */
+       _track_box.set_spacing (2);
+       _track_box.set_border_width (2);
+
        visible_tracks_selector.set_name ("zoom button");
        if (Profile->get_mixbus()) {
-               visible_tracks_selector.set_image(::get_icon ("tav_exp"));
-               visible_tracks_selector.set_size_request (42, -1);
+               visible_tracks_selector.set_icon (ArdourIcon::TimeAxisExpand);
        } else {
                set_size_request_to_display_given_text (visible_tracks_selector, _("All"), 30, 2);
        }
@@ -3108,14 +3169,14 @@ Editor::setup_toolbar ()
        tav_shrink_button.set_related_action (act);
 
        if (ARDOUR::Profile->get_mixbus()) {
-               _zoom_box.pack_start (visible_tracks_selector);
+               _track_box.pack_start (visible_tracks_selector);
        } else if (ARDOUR::Profile->get_trx()) {
-               _zoom_box.pack_start (tav_shrink_button);
-               _zoom_box.pack_start (tav_expand_button);
+               _track_box.pack_start (tav_shrink_button);
+               _track_box.pack_start (tav_expand_button);
        } else {
-               _zoom_box.pack_start (visible_tracks_selector);
-               _zoom_box.pack_start (tav_shrink_button);
-               _zoom_box.pack_start (tav_expand_button);
+               _track_box.pack_start (visible_tracks_selector);
+               _track_box.pack_start (tav_shrink_button);
+               _track_box.pack_start (tav_expand_button);
        }
 
        snap_box.set_spacing (2);
@@ -3129,7 +3190,13 @@ Editor::setup_toolbar ()
 
        snap_box.pack_start (snap_mode_selector, false, false);
        snap_box.pack_start (snap_type_selector, false, false);
-       snap_box.pack_start (edit_point_selector, false, false);
+
+       /* Edit Point*/
+       HBox *ep_box = manage (new HBox);
+       ep_box->set_spacing (2);
+       ep_box->set_border_width (2);
+
+       ep_box->pack_start (edit_point_selector, false, false);
 
        /* Nudge */
 
@@ -3147,35 +3214,35 @@ Editor::setup_toolbar ()
 
        /* Pack everything in... */
 
-       HBox* hbox = manage (new HBox);
-       hbox->set_spacing(2);
-
        toolbar_hbox.set_spacing (2);
-       toolbar_hbox.set_border_width (1);
+       toolbar_hbox.set_border_width (2);
 
        toolbar_hbox.pack_start (*mode_box, false, false);
+
        if (!ARDOUR::Profile->get_trx()) {
+
+               toolbar_hbox.pack_start (*(manage (new ArdourVSpacer ())), false, false, 3);
+
                toolbar_hbox.pack_start (_zoom_box, false, false);
-               toolbar_hbox.pack_start (*hbox, false, false);
-       }
 
-       if (!ARDOUR::Profile->get_trx()) {
-               hbox->pack_start (snap_box, false, false);
-               hbox->pack_start (*nudge_box, false, false);
-       }
+               toolbar_hbox.pack_start (*(manage (new ArdourVSpacer ())), false, false, 3);
+
+               toolbar_hbox.pack_start (_track_box, false, false);
+
+               toolbar_hbox.pack_start (*(manage (new ArdourVSpacer ())), false, false, 3);
+
+               toolbar_hbox.pack_start (snap_box, false, false);
 
-       hbox->show_all ();
+               toolbar_hbox.pack_start (*(manage (new ArdourVSpacer ())), false, false, 3);
 
-       toolbar_base.set_name ("ToolBarBase");
-       toolbar_base.add (toolbar_hbox);
+               toolbar_hbox.pack_start (*ep_box, false, false);
 
-       _toolbar_viewport.add (toolbar_base);
-       /* stick to the required height but allow width to vary if there's not enough room */
-       _toolbar_viewport.set_size_request (1, -1);
+               toolbar_hbox.pack_start (*(manage (new ArdourVSpacer ())), false, false, 3);
 
-       toolbar_frame.set_shadow_type (SHADOW_OUT);
-       toolbar_frame.set_name ("BaseFrame");
-       toolbar_frame.add (_toolbar_viewport);
+               toolbar_hbox.pack_start (*nudge_box, false, false);
+       }
+
+       toolbar_hbox.show_all ();
 }
 
 void
@@ -3777,7 +3844,7 @@ Editor::build_track_count_menu ()
                zoom_preset_selector.AddMenuElem (MenuElem (_("Zoom to 8 hours"), sigc::bind (sigc::mem_fun(*this, &Editor::set_zoom_preset), 8 * 60 * 60 * 1000)));
                zoom_preset_selector.AddMenuElem (MenuElem (_("Zoom to 24 hours"), sigc::bind (sigc::mem_fun(*this, &Editor::set_zoom_preset), 24 * 60 * 60 * 1000)));
                zoom_preset_selector.AddMenuElem (MenuElem (_("Zoom to Session"), sigc::mem_fun(*this, &Editor::temporal_zoom_session)));
-               zoom_preset_selector.AddMenuElem (MenuElem (_("Zoom to Range/Region Selection"), sigc::bind (sigc::mem_fun(*this, &Editor::temporal_zoom_selection), false)));
+               zoom_preset_selector.AddMenuElem (MenuElem (_("Zoom to Range/Region Selection"), sigc::bind (sigc::mem_fun(*this, &Editor::temporal_zoom_selection), Horizontal)));
        }
 }
 
@@ -4038,7 +4105,7 @@ Editor::get_grid_beat_divisions(framepos_t position)
     if the grid is snapped to bars, returns -1.
     @param event_state the current keyboard modifier mask.
 */
-unsigned
+int32_t
 Editor::get_grid_music_divisions (uint32_t event_state)
 {
        if (snap_mode() == Editing::SnapOff && !ArdourKeyboard::indicates_snap (event_state)) {
@@ -4313,13 +4380,13 @@ Editor::clear_playlists (TimeAxisView* v)
 void
 Editor::mapped_use_new_playlist (RouteTimeAxisView& atv, uint32_t sz, vector<boost::shared_ptr<ARDOUR::Playlist> > const & playlists)
 {
-       atv.use_new_playlist (sz > 1 ? false : true, playlists);
+       atv.use_new_playlist (sz > 1 ? false : true, playlists, false);
 }
 
 void
 Editor::mapped_use_copy_playlist (RouteTimeAxisView& atv, uint32_t sz, vector<boost::shared_ptr<ARDOUR::Playlist> > const & playlists)
 {
-       atv.use_copy_playlist (sz > 1 ? false : true, playlists);
+       atv.use_new_playlist (sz > 1 ? false : true, playlists, true);
 }
 
 void
@@ -4529,6 +4596,12 @@ Editor::set_samples_per_pixel (framecnt_t spp)
        instant_save ();
 }
 
+framepos_t
+Editor::playhead_cursor_sample () const
+{
+       return playhead_cursor->current_frame();
+}
+
 void
 Editor::queue_visual_videotimeline_update ()
 {
@@ -4595,10 +4668,8 @@ Editor::visual_changer (const VisualChange& vc)
 
                compute_fixed_ruler_scale ();
 
-               std::vector<TempoMap::BBTPoint> grid;
-               compute_current_bbt_points (grid, vc.time_origin, pending_visual_change.time_origin + current_page_samples());
-               compute_bbt_ruler_scale (grid, vc.time_origin, pending_visual_change.time_origin + current_page_samples());
-               update_tempo_based_rulers (grid);
+               compute_bbt_ruler_scale (vc.time_origin, pending_visual_change.time_origin + current_page_samples());
+               update_tempo_based_rulers ();
 
                update_video_timeline();
        }
@@ -4644,9 +4715,11 @@ Editor::get_preferred_edit_position (EditIgnoreOption ignore, bool from_context_
        framepos_t where = 0;
        EditPoint ep = _edit_point;
 
-       if (Profile->get_mixbus())
-               if (ep == EditAtSelectedMarker)
+       if (Profile->get_mixbus()) {
+               if (ep == EditAtSelectedMarker) {
                        ep = EditAtPlayhead;
+               }
+       }
 
        if (from_outside_canvas && (ep == EditAtMouse)) {
                ep = EditAtPlayhead;
@@ -4717,7 +4790,7 @@ Editor::set_loop_range (framepos_t start, framepos_t end, string cmd)
        Location* tll;
 
        if ((tll = transport_loop_location()) == 0) {
-               Location* loc = new Location (*_session, start, end, _("Loop"),  Location::IsAutoLoop);
+               Location* loc = new Location (*_session, start, end, _("Loop"),  Location::IsAutoLoop, get_grid_music_divisions(0));
                XMLNode &before = _session->locations()->get_state();
                _session->locations()->add (loc, true);
                _session->set_auto_loop_location (loc);
@@ -4744,7 +4817,7 @@ Editor::set_punch_range (framepos_t start, framepos_t end, string cmd)
        Location* tpl;
 
        if ((tpl = transport_punch_location()) == 0) {
-               Location* loc = new Location (*_session, start, end, _("Punch"),  Location::IsAutoPunch);
+               Location* loc = new Location (*_session, start, end, _("Punch"),  Location::IsAutoPunch, get_grid_music_divisions(0));
                XMLNode &before = _session->locations()->get_state();
                _session->locations()->add (loc, true);
                _session->set_auto_punch_location (loc);
@@ -5149,7 +5222,7 @@ Editor::region_view_added (RegionView * rv)
 
        MidiRegionView* mrv = dynamic_cast<MidiRegionView*> (rv);
        if (mrv) {
-               list<pair<PBD::ID const, list<boost::shared_ptr<Evoral::Note<Evoral::Beats> > > > >::iterator rnote;
+               list<pair<PBD::ID const, list<Evoral::event_id_t> > >::iterator rnote;
                for (rnote = selection->pending_midi_note_selection.begin(); rnote != selection->pending_midi_note_selection.end(); ++rnote) {
                        if (rv->region()->id () == (*rnote).first) {
                                mrv->select_notes ((*rnote).second);
@@ -5361,6 +5434,16 @@ Editor::timeaxisview_deleted (TimeAxisView *tv)
                        next_tv = (*i);
                }
 
+               // skip VCAs (cannot be selected, n/a in editor-mixer)
+               if (dynamic_cast<VCATimeAxisView*> (next_tv)) {
+                       /* VCAs are sorted last in line -- route_sorter.h, jump to top */
+                       next_tv = track_views.front();
+               }
+               if (dynamic_cast<VCATimeAxisView*> (next_tv)) {
+                       /* just in case: no master, only a VCA remains */
+                       next_tv = 0;
+               }
+
 
                if (next_tv) {
                        set_selected_mixer_strip (*next_tv);
@@ -5806,9 +5889,9 @@ void
 Editor::show_editor_list (bool yn)
 {
        if (yn) {
-               _the_notebook.show ();
+               _editor_list_vbox.show ();
        } else {
-               _the_notebook.hide ();
+               _editor_list_vbox.hide ();
        }
 }
 
@@ -5855,18 +5938,6 @@ Editor::update_region_layering_order_editor ()
 void
 Editor::setup_fade_images ()
 {
-       _fade_in_images[FadeLinear] = new Gtk::Image (get_icon_path (X_("fadein-linear")));
-       _fade_in_images[FadeSymmetric] = new Gtk::Image (get_icon_path (X_("fadein-symmetric")));
-       _fade_in_images[FadeFast] = new Gtk::Image (get_icon_path (X_("fadein-fast-cut")));
-       _fade_in_images[FadeSlow] = new Gtk::Image (get_icon_path (X_("fadein-slow-cut")));
-       _fade_in_images[FadeConstantPower] = new Gtk::Image (get_icon_path (X_("fadein-constant-power")));
-
-       _fade_out_images[FadeLinear] = new Gtk::Image (get_icon_path (X_("fadeout-linear")));
-       _fade_out_images[FadeSymmetric] = new Gtk::Image (get_icon_path (X_("fadeout-symmetric")));
-       _fade_out_images[FadeFast] = new Gtk::Image (get_icon_path (X_("fadeout-fast-cut")));
-       _fade_out_images[FadeSlow] = new Gtk::Image (get_icon_path (X_("fadeout-slow-cut")));
-       _fade_out_images[FadeConstantPower] = new Gtk::Image (get_icon_path (X_("fadeout-constant-power")));
-
        _xfade_in_images[FadeLinear] = new Gtk::Image (get_icon_path (X_("fadein-linear")));
        _xfade_in_images[FadeSymmetric] = new Gtk::Image (get_icon_path (X_("fadein-symmetric")));
        _xfade_in_images[FadeFast] = new Gtk::Image (get_icon_path (X_("fadein-fast-cut")));