the endless quest to plug memory leaks -- episode 379
[ardour.git] / gtk2_ardour / editor.cc
index edc1aa44789a832259f10e3c982555e9a652a21f..757b767b7ed338897c0b33b0e85578b5abc1d21d 100644 (file)
 #include "ardour/lmath.h"
 #include "ardour/location.h"
 #include "ardour/profile.h"
+#include "ardour/route.h"
 #include "ardour/route_group.h"
 #include "ardour/session_playlists.h"
 #include "ardour/tempo.h"
 #include "ardour/utils.h"
+#include "ardour/vca_manager.h"
+#include "ardour/vca.h"
 
 #include "canvas/debug.h"
 #include "canvas/text.h"
 #include "region_layering_order_editor.h"
 #include "rgb_macros.h"
 #include "rhythm_ferret.h"
+#include "route_sorter.h"
 #include "selection.h"
 #include "simple_progress_dialog.h"
 #include "sfdb_ui.h"
 #include "tooltips.h"
 #include "ui_config.h"
 #include "utils.h"
+#include "vca_time_axis.h"
 #include "verbose_cursor.h"
 
-#include "i18n.h"
+#include "pbd/i18n.h"
 
 using namespace std;
 using namespace ARDOUR;
@@ -212,8 +217,8 @@ static const gchar *_zoom_focus_strings[] = {
        N_("Right"),
        N_("Center"),
        N_("Playhead"),
-       N_("Mouse"),
-       N_("Edit point"),
+       N_("Mouse"),
+       N_("Edit point"),
        0
 };
 
@@ -232,30 +237,6 @@ static const gchar *_rb_opt_strings[] = {
 
 #define COMBO_TRIANGLE_WIDTH 25 // ArdourButton _diameter (11) + 2 * arrow-padding (2*2) + 2 * text-padding (2*5)
 
-static void
-pane_size_watcher (Paned* pane)
-{
-       /* if the handle of a pane vanishes into (at least) the tabs of a notebook,
-          it is:
-
-             X: hard to access
-             Quartz: impossible to access
-
-          so stop that by preventing it from ever getting too narrow. 35
-          pixels is basically a rough guess at the tab width.
-
-          ugh.
-       */
-
-       int max_width_of_lhs = GTK_WIDGET(pane->gobj())->allocation.width - 35;
-
-       gint pos = pane->get_position ();
-
-       if (pos > max_width_of_lhs) {
-               pane->set_position (max_width_of_lhs);
-       }
-}
-
 Editor::Editor ()
        : PublicEditor (global_hpacker)
        , editor_mixer_strip_width (Wide)
@@ -391,7 +372,8 @@ Editor::Editor ()
        , tempo_lines (0)
        , global_rect_group (0)
        , time_line_group (0)
-       , tempo_or_meter_marker_menu (0)
+       , tempo_marker_menu (0)
+       , meter_marker_menu (0)
        , marker_menu (0)
        , range_marker_menu (0)
        , transport_marker_menu (0)
@@ -508,6 +490,7 @@ Editor::Editor ()
 
        TimeAxisView::setup_sizes ();
        ArdourMarker::setup_sizes (timebar_height);
+       TempoCurve::setup_sizes (timebar_height);
 
        bbt_label.set_name ("EditorRulerLabel");
        bbt_label.set_size_request (-1, (int)timebar_height);
@@ -681,7 +664,18 @@ Editor::Editor ()
 
        _notebook_shrunk = false;
 
-       editor_summary_pane.pack1(edit_packer);
+
+       /* Pick up some settings we need to cache, early */
+
+       XMLNode* settings = ARDOUR_UI::instance()->editor_settings();
+       XMLProperty* prop;
+
+       if (settings && (prop = settings->property ("notebook-shrunk"))) {
+               _notebook_shrunk = string_is_affirmative (prop->value ());
+       }
+
+       editor_summary_pane.set_check_divider_position (true);
+       editor_summary_pane.add (edit_packer);
 
        Button* summary_arrows_left_left = manage (new Button);
        summary_arrows_left_left->add (*manage (new Arrow (ARROW_LEFT, SHADOW_NONE)));
@@ -722,22 +716,39 @@ Editor::Editor ()
        _summary_hbox.pack_start (*summary_arrows_right, false, false);
 
        if (!ARDOUR::Profile->get_trx()) {
-               editor_summary_pane.pack2 (_summary_hbox);
+               editor_summary_pane.add (_summary_hbox);
        }
 
-       edit_pane.pack1 (editor_summary_pane, true, true);
+       edit_pane.set_check_divider_position (true);
+       edit_pane.add (editor_summary_pane);
        if (!ARDOUR::Profile->get_trx()) {
-               edit_pane.pack2 (_the_notebook, false, true);
+               edit_pane.add (_the_notebook);
        }
 
-       editor_summary_pane.signal_size_allocate().connect (sigc::bind (sigc::mem_fun (*this, &Editor::pane_allocation_handler), static_cast<Paned*> (&editor_summary_pane)));
+       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);
 
-       /* XXX: editor_summary_pane might need similar to the edit_pane */
+       float fract;
+
+       {
+               LocaleGuard lg;
+
+               if (!settings || ((prop = settings->property ("edit-horizontal-pane-pos")) == 0) || ((fract = atof (prop->value())) > 1.0)) {
+                       /* initial allocation is 90% to canvas, 10% to notebook */
+                       edit_pane.set_divider (0, 0.90);
+               } else {
+                       edit_pane.set_divider (0, fract);
+               }
 
-       edit_pane.signal_size_allocate().connect (sigc::bind (sigc::mem_fun(*this, &Editor::pane_allocation_handler), static_cast<Paned*> (&edit_pane)));
+               if (!settings || ((prop = settings->property ("edit-vertical-pane-pos")) == 0) || ((fract = atof (prop->value())) > 1.0)) {
+                       /* initial allocation is 90% to canvas, 10% to summary */
+                       editor_summary_pane.set_divider (0, 0.90);
+               } else {
 
-       Glib::PropertyProxy<int> proxy = edit_pane.property_position();
-       proxy.signal_changed().connect (bind (sigc::ptr_fun (pane_size_watcher), static_cast<Paned*> (&edit_pane)));
+                       editor_summary_pane.set_divider (0, fract);
+               }
+       }
 
        top_hbox.pack_start (toolbar_frame);
 
@@ -794,14 +805,18 @@ Editor::Editor ()
        ControlProtocol::VerticalZoomInSelected.connect (*this, invalidator (*this), boost::bind (&Editor::control_vertical_zoom_in_selected, this), gui_context());
        ControlProtocol::VerticalZoomOutSelected.connect (*this, invalidator (*this), boost::bind (&Editor::control_vertical_zoom_out_selected, this), gui_context());
 
-       ControlProtocol::AddRouteToSelection.connect (*this, invalidator (*this), boost::bind (&Editor::control_select, this, _1, Selection::Add), gui_context());
-       ControlProtocol::RemoveRouteFromSelection.connect (*this, invalidator (*this), boost::bind (&Editor::control_select, this, _1, Selection::Toggle), gui_context());
-       ControlProtocol::SetRouteSelection.connect (*this, invalidator (*this), boost::bind (&Editor::control_select, this, _1, Selection::Set), gui_context());
-       ControlProtocol::ToggleRouteSelection.connect (*this, invalidator (*this), boost::bind (&Editor::control_select, this, _1, Selection::Toggle), gui_context());
-       ControlProtocol::ClearRouteSelection.connect (*this, invalidator (*this), boost::bind (&Editor::control_unselect, this), gui_context());
+       ControlProtocol::AddStripableToSelection.connect (*this, invalidator (*this), boost::bind (&Editor::control_select, this, _1, Selection::Add), gui_context());
+       ControlProtocol::RemoveStripableFromSelection.connect (*this, invalidator (*this), boost::bind (&Editor::control_select, this, _1, Selection::Toggle), gui_context());
+       ControlProtocol::SetStripableSelection.connect (*this, invalidator (*this), boost::bind (&Editor::control_select, this, _1, Selection::Set), gui_context());
+       ControlProtocol::ToggleStripableSelection.connect (*this, invalidator (*this), boost::bind (&Editor::control_select, this, _1, Selection::Toggle), gui_context());
+       ControlProtocol::ClearStripableSelection.connect (*this, invalidator (*this), boost::bind (&Editor::control_unselect, this), gui_context());
 
        BasicUI::AccessAction.connect (*this, invalidator (*this), boost::bind (&Editor::access_action, this, _1, _2), gui_context());
 
+       /* handle escape */
+
+       ARDOUR_UI::instance()->Escape.connect (*this, invalidator (*this), boost::bind (&Editor::escape, this), gui_context());
+
        /* problematic: has to return a value and thus cannot be x-thread */
 
        Session::AskAboutPlaylistDeletion.connect_same_thread (*this, boost::bind (&Editor::playlist_deletion_dialog, this, _1));
@@ -846,12 +861,13 @@ Editor::Editor ()
 
 Editor::~Editor()
 {
-        delete button_bindings;
+       delete button_bindings;
        delete _routes;
        delete _route_groups;
        delete _track_canvas_viewport;
        delete _drags;
        delete nudge_clock;
+       delete _verbose_cursor;
        delete quantize_dialog;
        delete _summary;
        delete _group_tabs;
@@ -998,23 +1014,9 @@ Editor::control_unselect ()
 }
 
 void
-Editor::control_select (uint32_t rid, Selection::Operation op)
+Editor::control_select (boost::shared_ptr<Stripable> s, Selection::Operation op)
 {
-       /* handles the (static) signal from the ControlProtocol class that
-        * requests setting the selected track to a given RID
-        */
-
-       if (!_session) {
-               return;
-       }
-
-       boost::shared_ptr<Route> r = _session->route_by_remote_id (rid);
-
-       if (!r) {
-               return;
-       }
-
-       TimeAxisView* tav = axis_view_from_route (r);
+       TimeAxisView* tav = axis_view_from_stripable (s);
 
        if (tav) {
                switch (op) {
@@ -1362,9 +1364,11 @@ Editor::set_session (Session *t)
        _session->StepEditStatusChange.connect (_session_connections, invalidator (*this), boost::bind (&Editor::step_edit_status_change, this, _1), gui_context());
        _session->TransportStateChange.connect (_session_connections, invalidator (*this), boost::bind (&Editor::map_transport_state, this), gui_context());
        _session->PositionChanged.connect (_session_connections, invalidator (*this), boost::bind (&Editor::map_position_change, this, _1), gui_context());
+       _session->vca_manager().VCAAdded.connect (_session_connections, invalidator (*this), boost::bind (&Editor::add_vcas, this, _1), gui_context());
        _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->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());
@@ -1403,6 +1407,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);
@@ -1670,7 +1693,7 @@ Editor::build_track_context_menu ()
 {
        using namespace Menu_Helpers;
 
-       MenuList& edit_items = track_context_menu.items();
+       MenuList& edit_items = track_context_menu.items();
        edit_items.clear();
 
        add_dstream_context_items (edit_items);
@@ -1682,7 +1705,7 @@ Editor::build_track_bus_context_menu ()
 {
        using namespace Menu_Helpers;
 
-       MenuList& edit_items = track_context_menu.items();
+       MenuList& edit_items = track_context_menu.items();
        edit_items.clear();
 
        add_bus_context_items (edit_items);
@@ -2135,6 +2158,37 @@ Editor::snap_type() const
        return _snap_type;
 }
 
+bool
+Editor::snap_musical() const
+{
+       switch (_snap_type) {
+       case SnapToBeatDiv128:
+       case SnapToBeatDiv64:
+       case SnapToBeatDiv32:
+       case SnapToBeatDiv28:
+       case SnapToBeatDiv24:
+       case SnapToBeatDiv20:
+       case SnapToBeatDiv16:
+       case SnapToBeatDiv14:
+       case SnapToBeatDiv12:
+       case SnapToBeatDiv10:
+       case SnapToBeatDiv8:
+       case SnapToBeatDiv7:
+       case SnapToBeatDiv6:
+       case SnapToBeatDiv5:
+       case SnapToBeatDiv4:
+       case SnapToBeatDiv3:
+       case SnapToBeatDiv2:
+       case SnapToBeat:
+       case SnapToBar:
+               return true;
+       default:
+               break;
+       }
+
+       return false;
+}
+
 SnapMode
 Editor::snap_mode() const
 {
@@ -2185,14 +2239,10 @@ Editor::set_snap_to (SnapType st)
        case SnapToBeatDiv4:
        case SnapToBeatDiv3:
        case SnapToBeatDiv2: {
-               ARDOUR::TempoMap::BBTPointList::const_iterator current_bbt_points_begin;
-               ARDOUR::TempoMap::BBTPointList::const_iterator current_bbt_points_end;
-
-               compute_current_bbt_points (leftmost_frame, leftmost_frame + current_page_samples(),
-                                           current_bbt_points_begin, current_bbt_points_end);
-               compute_bbt_ruler_scale (leftmost_frame, leftmost_frame + current_page_samples(),
-                                        current_bbt_points_begin, current_bbt_points_end);
-               update_tempo_based_rulers (current_bbt_points_begin, current_bbt_points_end);
+               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);
                break;
        }
 
@@ -2291,6 +2341,7 @@ Editor::set_state (const XMLNode& node, int version)
        XMLProperty const * prop;
        set_id (node);
        PBD::Unwinder<bool> nsi (no_save_instant, true);
+       LocaleGuard lg;
 
        Tabbable::set_state (node, version);
 
@@ -2313,6 +2364,8 @@ Editor::set_state (const XMLNode& node, int version)
 
        if ((prop = node.property ("zoom-focus"))) {
                zoom_focus_selection_done ((ZoomFocus) string_2_enum (prop->value(), zoom_focus));
+       } else {
+               zoom_focus_selection_done (zoom_focus);
        }
 
        if ((prop = node.property ("zoom"))) {
@@ -2329,10 +2382,20 @@ Editor::set_state (const XMLNode& node, int version)
 
        if ((prop = node.property ("snap-to"))) {
                snap_type_selection_done ((SnapType) string_2_enum (prop->value(), _snap_type));
+               set_snap_to ((SnapType) string_2_enum (prop->value(), _snap_type));
+       } else {
+               set_snap_to (_snap_type);
        }
 
        if ((prop = node.property ("snap-mode"))) {
                snap_mode_selection_done((SnapMode) string_2_enum (prop->value(), _snap_mode));
+               /* set text of Dropdown. in case _snap_mode == SnapOff (default)
+                * snap_mode_selection_done() will only mark an already active item as active
+                * which does not trigger set_text().
+                */
+               set_snap_mode ((SnapMode) string_2_enum (prop->value(), _snap_mode));
+       } else {
+               set_snap_mode (_snap_mode);
        }
 
        if ((prop = node.property ("internal-snap-to"))) {
@@ -2385,6 +2448,8 @@ Editor::set_state (const XMLNode& node, int version)
 
        if ((prop = node.property ("edit-point"))) {
                set_edit_point_preference ((EditPoint) string_2_enum (prop->value(), _edit_point), true);
+       } else {
+               set_edit_point_preference (_edit_point);
        }
 
        if ((prop = node.property ("show-measures"))) {
@@ -2519,16 +2584,17 @@ Editor::get_state ()
 {
        XMLNode* node = new XMLNode (X_("Editor"));
        char buf[32];
+       LocaleGuard lg;
 
        id().print (buf, sizeof (buf));
        node->add_property ("id", buf);
 
        node->add_child_nocopy (Tabbable::get_state());
 
-       snprintf(buf,sizeof(buf), "%d",gtk_paned_get_position (static_cast<Paned*>(&edit_pane)->gobj()));
+       snprintf(buf,sizeof(buf), "%f", edit_pane.get_divider ());
        node->add_property("edit-horizontal-pane-pos", string(buf));
        node->add_property("notebook-shrunk", _notebook_shrunk ? "1" : "0");
-       snprintf(buf,sizeof(buf), "%d",gtk_paned_get_position (static_cast<Paned*>(&editor_summary_pane)->gobj()));
+       snprintf(buf,sizeof(buf), "%f", editor_summary_pane.get_divider());
        node->add_property("edit-vertical-pane-pos", string(buf));
 
        maybe_add_mixer_strip_width (*node);
@@ -3867,74 +3933,6 @@ Editor::cycle_zoom_focus ()
        }
 }
 
-void
-Editor::pane_allocation_handler (Allocation &alloc, Paned* which)
-{
-       /* recover or initialize pane positions. do this here rather than earlier because
-          we don't want the positions to change the child allocations, which they seem to do.
-        */
-
-       int pos;
-       XMLProperty const * prop;
-       char buf[32];
-       XMLNode* node = ARDOUR_UI::instance()->editor_settings();
-
-       enum Pane {
-               Horizontal = 0x1,
-               Vertical = 0x2
-       };
-
-       static Pane done;
-
-       XMLNode* geometry = find_named_node (*node, "geometry");
-
-       if (which == static_cast<Paned*> (&edit_pane)) {
-
-               if (done & Horizontal) {
-                       return;
-               }
-
-               if (geometry && (prop = geometry->property ("notebook-shrunk"))) {
-                       _notebook_shrunk = string_is_affirmative (prop->value ());
-               }
-
-               if (!geometry || (prop = geometry->property ("edit-horizontal-pane-pos")) == 0) {
-                       /* initial allocation is 90% to canvas, 10% to notebook */
-                       pos = (int) floor (alloc.get_width() * 0.90f);
-                       snprintf (buf, sizeof(buf), "%d", pos);
-               } else {
-                       pos = atoi (prop->value());
-               }
-
-               if (GTK_WIDGET(edit_pane.gobj())->allocation.width > pos) {
-                       edit_pane.set_position (pos);
-               }
-
-               done = (Pane) (done | Horizontal);
-
-       } else if (which == static_cast<Paned*> (&editor_summary_pane)) {
-
-               if (done & Vertical) {
-                       return;
-               }
-
-               if (!geometry || (prop = geometry->property ("edit-vertical-pane-pos")) == 0) {
-                       /* initial allocation is 90% to canvas, 10% to summary */
-                       pos = (int) floor (alloc.get_height() * 0.90f);
-                       snprintf (buf, sizeof(buf), "%d", pos);
-               } else {
-
-                       pos = atoi (prop->value());
-               }
-
-               if (GTK_WIDGET(editor_summary_pane.gobj())->allocation.height > pos) {
-                       editor_summary_pane.set_position (pos);
-               }
-
-               done = (Pane) (done | Vertical);
-       }
-}
-
 void
 Editor::set_show_measures (bool yn)
 {
@@ -3946,11 +3944,9 @@ Editor::set_show_measures (bool yn)
                                tempo_lines->show();
                        }
 
-                       ARDOUR::TempoMap::BBTPointList::const_iterator begin;
-                       ARDOUR::TempoMap::BBTPointList::const_iterator end;
-
-                       compute_current_bbt_points (leftmost_frame, leftmost_frame + current_page_samples(), begin, end);
-                       draw_measures (begin, end);
+                       std::vector<TempoMap::BBTPoint> grid;
+                       compute_current_bbt_points (grid, leftmost_frame, leftmost_frame + current_page_samples());
+                       draw_measures (grid);
                }
 
                instant_save ();
@@ -4056,6 +4052,47 @@ Editor::get_grid_beat_divisions(framepos_t position)
        return 0;
 }
 
+/** returns the current musical grid divisiions using the supplied modifier mask from a GtkEvent.
+    if the grid is non-musical, returns 0.
+    if the grid is snapped to bars, returns -1.
+    @param event_state the current keyboard modifier mask.
+*/
+unsigned
+Editor::get_grid_music_divisions (uint32_t event_state)
+{
+       if (snap_mode() == Editing::SnapOff && !ArdourKeyboard::indicates_snap (event_state)) {
+               return 0;
+       }
+
+       if (snap_mode() != Editing::SnapOff && ArdourKeyboard::indicates_snap (event_state)) {
+               return 0;
+       }
+
+       switch (_snap_type) {
+       case SnapToBeatDiv128: return 128;
+       case SnapToBeatDiv64:  return 64;
+       case SnapToBeatDiv32:  return 32;
+       case SnapToBeatDiv28:  return 28;
+       case SnapToBeatDiv24:  return 24;
+       case SnapToBeatDiv20:  return 20;
+       case SnapToBeatDiv16:  return 16;
+       case SnapToBeatDiv14:  return 14;
+       case SnapToBeatDiv12:  return 12;
+       case SnapToBeatDiv10:  return 10;
+       case SnapToBeatDiv8:   return 8;
+       case SnapToBeatDiv7:   return 7;
+       case SnapToBeatDiv6:   return 6;
+       case SnapToBeatDiv5:   return 5;
+       case SnapToBeatDiv4:   return 4;
+       case SnapToBeatDiv3:   return 3;
+       case SnapToBeatDiv2:   return 2;
+       case SnapToBeat:       return 1;
+       case SnapToBar :       return -1;
+       default:               return 0;
+       }
+       return 0;
+}
+
 Evoral::Beats
 Editor::get_grid_type_as_beats (bool& success, framepos_t position)
 {
@@ -4071,7 +4108,7 @@ Editor::get_grid_type_as_beats (bool& success, framepos_t position)
                return Evoral::Beats(1.0);
        case SnapToBar:
                if (_session) {
-                       return Evoral::Beats(_session->tempo_map().meter_at (position).divisions_per_bar());
+                       return Evoral::Beats(_session->tempo_map().meter_at_frame (position).divisions_per_bar());
                }
                break;
        default:
@@ -4256,7 +4293,7 @@ Editor::new_playlists (TimeAxisView* v)
        begin_reversible_command (_("new playlists"));
        vector<boost::shared_ptr<ARDOUR::Playlist> > playlists;
        _session->playlists->get (playlists);
-       mapover_tracks (sigc::bind (sigc::mem_fun (*this, &Editor::mapped_use_new_playlist), playlists), v, ARDOUR::Properties::select.property_id);
+       mapover_tracks (sigc::bind (sigc::mem_fun (*this, &Editor::mapped_use_new_playlist), playlists), v, ARDOUR::Properties::group_select.property_id);
        commit_reversible_command ();
 }
 
@@ -4272,7 +4309,7 @@ Editor::copy_playlists (TimeAxisView* v)
        begin_reversible_command (_("copy playlists"));
        vector<boost::shared_ptr<ARDOUR::Playlist> > playlists;
        _session->playlists->get (playlists);
-       mapover_tracks (sigc::bind (sigc::mem_fun (*this, &Editor::mapped_use_copy_playlist), playlists), v, ARDOUR::Properties::select.property_id);
+       mapover_tracks (sigc::bind (sigc::mem_fun (*this, &Editor::mapped_use_copy_playlist), playlists), v, ARDOUR::Properties::group_select.property_id);
        commit_reversible_command ();
 }
 
@@ -4287,7 +4324,7 @@ Editor::clear_playlists (TimeAxisView* v)
        begin_reversible_command (_("clear playlists"));
        vector<boost::shared_ptr<ARDOUR::Playlist> > playlists;
        _session->playlists->get (playlists);
-       mapover_tracks (sigc::mem_fun (*this, &Editor::mapped_clear_playlist), v, ARDOUR::Properties::select.property_id);
+       mapover_tracks (sigc::mem_fun (*this, &Editor::mapped_clear_playlist), v, ARDOUR::Properties::group_select.property_id);
        commit_reversible_command ();
 }
 
@@ -4377,7 +4414,7 @@ Editor::current_visual_state (bool with_tracks)
        vs->zoom_focus = zoom_focus;
 
        if (with_tracks) {
-               *vs->gui_state = *ARDOUR_UI::instance()->gui_object_state;
+               vs->gui_state->set_state (ARDOUR_UI::instance()->gui_object_state->get_state());
        }
 
        return vs;
@@ -4442,7 +4479,7 @@ Editor::use_visual_state (VisualState& vs)
        reposition_and_zoom (vs.leftmost_frame, vs.samples_per_pixel);
 
        if (vs.gui_state) {
-               *ARDOUR_UI::instance()->gui_object_state = *vs.gui_state;
+               ARDOUR_UI::instance()->gui_object_state->set_state (vs.gui_state->get_state());
 
                for (TrackViewList::iterator i = track_views.begin(); i != track_views.end(); ++i) {
                        (*i)->clear_property_cache();
@@ -4576,14 +4613,10 @@ Editor::visual_changer (const VisualChange& vc)
 
                compute_fixed_ruler_scale ();
 
-               ARDOUR::TempoMap::BBTPointList::const_iterator current_bbt_points_begin;
-               ARDOUR::TempoMap::BBTPointList::const_iterator current_bbt_points_end;
-
-               compute_current_bbt_points (vc.time_origin, pending_visual_change.time_origin + current_page_samples(),
-                                           current_bbt_points_begin, current_bbt_points_end);
-               compute_bbt_ruler_scale (vc.time_origin, pending_visual_change.time_origin + current_page_samples(),
-                                        current_bbt_points_begin, current_bbt_points_end);
-               update_tempo_based_rulers (current_bbt_points_begin, current_bbt_points_end);
+               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);
 
                update_video_timeline();
        }
@@ -5017,7 +5050,7 @@ Editor::first_idle ()
                        true
                        );
                dialog->present ();
-               ARDOUR_UI::instance()->flush_pending ();
+               ARDOUR_UI::instance()->flush_pending (60);
        }
 
        for (TrackViewList::iterator t = track_views.begin(); t != track_views.end(); ++t) {
@@ -5153,16 +5186,13 @@ Editor::region_view_removed ()
        _summary->set_background_dirty ();
 }
 
-RouteTimeAxisView*
-Editor::axis_view_from_route (boost::shared_ptr<Route> r) const
+TimeAxisView*
+Editor::axis_view_from_stripable (boost::shared_ptr<Stripable> s) const
 {
-       TrackViewList::const_iterator j = track_views.begin ();
-       while (j != track_views.end()) {
-               RouteTimeAxisView* rtv = dynamic_cast<RouteTimeAxisView*> (*j);
-               if (rtv && rtv->route() == r) {
-                       return rtv;
+       for (TrackViewList::const_iterator j = track_views.begin (); j != track_views.end(); ++j) {
+               if ((*j)->stripable() == s) {
+                       return *j;
                }
-               ++j;
        }
 
        return 0;
@@ -5175,7 +5205,7 @@ Editor::axis_views_from_routes (boost::shared_ptr<RouteList> r) const
        TrackViewList t;
 
        for (RouteList::const_iterator i = r->begin(); i != r->end(); ++i) {
-               TimeAxisView* tv = axis_view_from_route (*i);
+               TimeAxisView* tv = axis_view_from_stripable (*i);
                if (tv) {
                        t.push_back (tv);
                }
@@ -5202,56 +5232,94 @@ Editor::resume_route_redisplay ()
 }
 
 void
-Editor::add_routes (RouteList& routes)
+Editor::add_vcas (VCAList& vlist)
+{
+       StripableList sl;
+
+       for (VCAList::iterator v = vlist.begin(); v != vlist.end(); ++v) {
+               sl.push_back (boost::dynamic_pointer_cast<Stripable> (*v));
+       }
+
+       add_stripables (sl);
+}
+
+void
+Editor::add_routes (RouteList& rlist)
 {
-       ENSURE_GUI_THREAD (*this, &Editor::handle_new_route, routes)
+       StripableList sl;
+
+       for (RouteList::iterator r = rlist.begin(); r != rlist.end(); ++r) {
+               sl.push_back (*r);
+       }
 
-       RouteTimeAxisView *rtv;
-       list<RouteTimeAxisView*> new_views;
+       add_stripables (sl);
+}
+
+void
+Editor::add_stripables (StripableList& sl)
+{
+       list<TimeAxisView*> new_views;
+       boost::shared_ptr<VCA> v;
+       boost::shared_ptr<Route> r;
        TrackViewList new_selection;
        bool from_scratch = (track_views.size() == 0);
 
-       for (RouteList::iterator x = routes.begin(); x != routes.end(); ++x) {
-               boost::shared_ptr<Route> route = (*x);
+       sl.sort (StripablePresentationInfoSorter());
 
-               if (route->is_auditioner() || route->is_monitor()) {
-                       continue;
-               }
+       for (StripableList::iterator s = sl.begin(); s != sl.end(); ++s) {
 
-               DataType dt = route->input()->default_type();
+               if ((v = boost::dynamic_pointer_cast<VCA> (*s)) != 0) {
 
-               if (dt == ARDOUR::DataType::AUDIO) {
-                       rtv = new AudioTimeAxisView (*this, _session, *_track_canvas);
-                       rtv->set_route (route);
-               } else if (dt == ARDOUR::DataType::MIDI) {
-                       rtv = new MidiTimeAxisView (*this, _session, *_track_canvas);
-                       rtv->set_route (route);
-               } else {
-                       throw unknown_type();
-               }
+                       VCATimeAxisView* vtv = new VCATimeAxisView (*this, _session, *_track_canvas);
+                       vtv->set_vca (v);
+                       new_views.push_back (vtv);
 
-               new_views.push_back (rtv);
-               track_views.push_back (rtv);
-               new_selection.push_back (rtv);
+               } else if ((r = boost::dynamic_pointer_cast<Route> (*s)) != 0) {
 
-               rtv->effective_gain_display ();
+                       if (r->is_auditioner() || r->is_monitor()) {
+                               continue;
+                       }
+
+                       RouteTimeAxisView* rtv;
+                       DataType dt = r->input()->default_type();
+
+                       if (dt == ARDOUR::DataType::AUDIO) {
+                               rtv = new AudioTimeAxisView (*this, _session, *_track_canvas);
+                               rtv->set_route (r);
+                       } else if (dt == ARDOUR::DataType::MIDI) {
+                               rtv = new MidiTimeAxisView (*this, _session, *_track_canvas);
+                               rtv->set_route (r);
+                       } else {
+                               throw unknown_type();
+                       }
+
+                       new_views.push_back (rtv);
+                       track_views.push_back (rtv);
+                       new_selection.push_back (rtv);
 
-               rtv->view()->RegionViewAdded.connect (sigc::mem_fun (*this, &Editor::region_view_added));
-               rtv->view()->RegionViewRemoved.connect (sigc::mem_fun (*this, &Editor::region_view_removed));
+                       rtv->effective_gain_display ();
+
+                       rtv->view()->RegionViewAdded.connect (sigc::mem_fun (*this, &Editor::region_view_added));
+                       rtv->view()->RegionViewRemoved.connect (sigc::mem_fun (*this, &Editor::region_view_removed));
+               }
        }
 
        if (new_views.size() > 0) {
-               _routes->routes_added (new_views);
-               _summary->routes_added (new_views);
+               _routes->time_axis_views_added (new_views);
+               //_summary->routes_added (new_selection); /* XXX requires RouteTimeAxisViewList */
        }
 
-       if (!from_scratch) {
+       /* note: !new_selection.empty() means that we got some routes rather
+        * than just VCAs
+        */
+
+       if (!from_scratch && !new_selection.empty()) {
                selection->tracks.clear();
                selection->add (new_selection);
                begin_selection_op_history();
        }
 
-       if (show_editor_mixer_when_tracks_arrive) {
+       if (show_editor_mixer_when_tracks_arrive && !new_selection.empty()) {
                show_editor_mixer (true);
        }
 
@@ -5866,16 +5934,16 @@ Editor::notebook_tab_clicked (GdkEventButton* ev, Gtk::Widget* page)
 
                if (_notebook_shrunk) {
                        if (pre_notebook_shrink_pane_width) {
-                               edit_pane.set_position (*pre_notebook_shrink_pane_width);
+                               edit_pane.set_divider (0, *pre_notebook_shrink_pane_width);
                        }
                        _notebook_shrunk = false;
                } else {
-                       pre_notebook_shrink_pane_width = edit_pane.get_position();
+                       pre_notebook_shrink_pane_width = edit_pane.get_divider();
 
                        /* this expands the LHS of the edit pane to cover the notebook
                           PAGE but leaves the tabs visible.
                         */
-                       edit_pane.set_position (edit_pane.get_position() + page->get_width());
+                       edit_pane.set_divider (0, edit_pane.get_divider() + page->get_width());
                        _notebook_shrunk = true;
                }
        }
@@ -5972,6 +6040,9 @@ Editor::ui_parameter_changed (string parameter)
                }
                _cursors->set_cursor_set (UIConfiguration::instance().get_icon_set());
                _cursor_stack.push_back(_cursors->grabber);
+               edit_pane.set_drag_cursor (*_cursors->expand_left_right);
+               editor_summary_pane.set_drag_cursor (*_cursors->expand_up_down);
+
        } else if (parameter == "draggable-playhead") {
                if (_verbose_cursor) {
                        playhead_cursor->set_sensitive (UIConfiguration::instance().get_draggable_playhead());