more fiddling about with placing bits of the scroll/zoom operation in the idle callba...
[ardour.git] / gtk2_ardour / editor.cc
index 06412abcc88ff1bc14e864ecbd3b18dd1431b2b3..6b8fc782dc79be8e526eb4a0445e64baee5bd8f1 100644 (file)
@@ -28,6 +28,7 @@
 
 #include <pbd/convert.h>
 #include <pbd/error.h>
+#include <pbd/memento_command.h>
 
 #include <gtkmm/image.h>
 #include <gdkmm/color.h>
@@ -38,7 +39,7 @@
 #include <gtkmm2ext/utils.h>
 
 #include <ardour/audio_track.h>
-#include <ardour/diskstream.h>
+#include <ardour/audio_diskstream.h>
 #include <ardour/plugin_manager.h>
 #include <ardour/location.h>
 #include <ardour/audioplaylist.h>
@@ -47,7 +48,8 @@
 #include <ardour/session_route.h>
 #include <ardour/tempo.h>
 #include <ardour/utils.h>
-#include <ardour/control_protocol.h>
+
+#include <control_protocol/control_protocol.h>
 
 #include "ardour_ui.h"
 #include "editor.h"
 #include "keyboard.h"
 #include "marker.h"
 #include "playlist_selector.h"
-#include "regionview.h"
+#include "audio_region_view.h"
 #include "rgb_macros.h"
 #include "selection.h"
-#include "streamview.h"
+#include "audio_streamview.h"
 #include "time_axis_view.h"
 #include "utils.h"
 #include "crossfade_view.h"
 using namespace std;
 using namespace sigc;
 using namespace ARDOUR;
+using namespace PBD;
 using namespace Gtk;
 using namespace Glib;
 using namespace Gtkmm2ext;
 using namespace Editing;
 
 using PBD::internationalize;
+using PBD::atoi;
 
 const double Editor::timebar_height = 15.0;
 
@@ -98,8 +102,8 @@ static const int32_t slide_index = 0;
 static const int32_t splice_index = 1;
 
 static const gchar *edit_mode_strings[] = {
-       N_("Slide"),
-       N_("Splice"),
+       N_("Slide Edit"),
+       N_("Splice Edit"),
        0
 };
 
@@ -128,17 +132,17 @@ static const gchar *snap_type_strings[] = {
 };
 
 static const gchar *snap_mode_strings[] = {
-       N_("Normal"),
-       N_("Magnetic"),
+       N_("Normal Snap"),
+       N_("Magnetic Snap"),
        0
 };
 
 static const gchar *zoom_focus_strings[] = {
-       N_("Left"),
-       N_("Right"),
-       N_("Center"),
-       N_("Playhead"),
-       N_("Edit Cursor"),
+       N_("Focus Left"),
+       N_("Focus Right"),
+       N_("Focus Center"),
+       N_("Focus Play"),
+       N_("Focus Edit"),
        0
 };
 
@@ -201,37 +205,20 @@ Editor::Editor (AudioEngine& eng)
 
          /* tool bar related */
 
-         selection_start_clock (X_("SelectionStartClock"), true),
-         selection_end_clock (X_("SelectionEndClock"), true),
          edit_cursor_clock (X_("EditCursorClock"), true),
          zoom_range_clock (X_("ZoomRangeClock"), true, true),
          
          toolbar_selection_clock_table (2,3),
          
-         mouse_mode_button_table (2, 3),
-
-         mouse_select_button (_("range")),
-         mouse_move_button (_("object")),
-         mouse_gain_button (_("gain")),
-         mouse_zoom_button (_("zoom")),
-         mouse_timefx_button (_("timefx")),
-         mouse_audition_button (_("listen")),
-
          automation_mode_button (_("mode")),
          global_automation_button (_("automation")),
 
-         edit_mode_label (_("Edit Mode")),
-         snap_type_label (_("Snap To")),
-         snap_mode_label(_("Snap Mode")),
-         zoom_focus_label (_("Zoom Focus")),
-
          /* <CMT Additions> */
          image_socket_listener(0),
          /* </CMT Additions> */
 
          /* nudge */
 
-         nudge_label (_("Nudge")),
          nudge_clock (X_("NudgeClock"), true, true)
 
 {
@@ -259,7 +246,6 @@ Editor::Editor (AudioEngine& eng)
        latest_regionview = 0;
        last_update_frame = 0;
        drag_info.item = 0;
-       last_audition_region = 0;
        current_mixer_strip = 0;
        current_bbt_points = 0;
 
@@ -285,7 +271,6 @@ Editor::Editor (AudioEngine& eng)
        first_action_message = 0;
        export_dialog = 0;
        show_gain_after_trim = false;
-       no_zoom_repos_update = false;
        ignore_route_list_reorder = false;
        no_route_list_redisplay = false;
        verbose_cursor_on = true;
@@ -298,7 +283,6 @@ Editor::Editor (AudioEngine& eng)
        _xfade_visibility = true;
        editor_ruler_menu = 0;
        no_ruler_shown_update = false;
-       edit_hscroll_dragging = false;
        edit_group_list_menu = 0;
        route_list_menu = 0;
        region_list_menu = 0;
@@ -325,6 +309,7 @@ Editor::Editor (AudioEngine& eng)
        last_canvas_frame = 0;
        edit_cursor = 0;
        playhead_cursor = 0;
+       button_release_can_deselect = true;
 
        location_marker_color = color_map[cLocationMarker];
        location_range_color = color_map[cLocationRange];
@@ -341,6 +326,7 @@ Editor::Editor (AudioEngine& eng)
        reset_hscrollbar_stepping ();
        
        zoom_focus = ZoomFocusLeft;
+       set_zoom_focus (ZoomFocusLeft);
        zoom_range_clock.ValueChanged.connect (mem_fun(*this, &Editor::zoom_adjustment_changed));
 
        initialize_rulers ();
@@ -461,36 +447,7 @@ Editor::Editor (AudioEngine& eng)
 
        edit_packer.attach (edit_hscrollbar,         2, 3, 2, 3,    FILL|EXPAND,  FILL, 0, 0);
 
-       zoom_in_button.set_name ("EditorTimeButton");
-       zoom_out_button.set_name ("EditorTimeButton");
-       ARDOUR_UI::instance()->tooltips().set_tip (zoom_in_button, _("Zoom in"));
-       ARDOUR_UI::instance()->tooltips().set_tip (zoom_out_button, _("Zoom out"));
-
-       zoom_out_full_button.set_name ("EditorTimeButton");
-       ARDOUR_UI::instance()->tooltips().set_tip (zoom_out_full_button, _("Zoom to session"));
-
-       zoom_in_button.add (*(manage (new Image (Stock::ZOOM_IN, ICON_SIZE_BUTTON))));
-       zoom_out_button.add (*(manage (new Image (Stock::ZOOM_OUT, ICON_SIZE_BUTTON))));
-       zoom_out_full_button.add (*(manage (new Image (Stock::ZOOM_FIT, ICON_SIZE_BUTTON))));
-       
-       zoom_in_button.signal_clicked().connect (bind (mem_fun(*this, &Editor::temporal_zoom_step), false));
-       zoom_out_button.signal_clicked().connect (bind (mem_fun(*this, &Editor::temporal_zoom_step), true));
-       zoom_out_full_button.signal_clicked().connect (mem_fun(*this, &Editor::temporal_zoom_session));
-       
-       zoom_indicator_box.pack_start (zoom_out_button, false, false);
-       zoom_indicator_box.pack_start (zoom_in_button, false, false);
-       zoom_indicator_box.pack_start (zoom_range_clock, false, false); 
-       zoom_indicator_box.pack_start (zoom_out_full_button, false, false);
-       
-       zoom_indicator_label.set_text (_("Zoom Span"));
-       zoom_indicator_label.set_name ("ToolBarLabel");
-
-       zoom_indicator_vbox.set_spacing (3);
-       zoom_indicator_vbox.set_border_width (3);
-       zoom_indicator_vbox.pack_start (zoom_indicator_label, false, false);
-       zoom_indicator_vbox.pack_start (zoom_indicator_box, false, false);
-
-       bottom_hbox.set_border_width (3);
+       bottom_hbox.set_border_width (2);
        bottom_hbox.set_spacing (3);
 
        route_display_model = ListStore::create(route_display_columns);
@@ -525,6 +482,9 @@ Editor::Editor (AudioEngine& eng)
        edit_group_display.get_column (0)->set_data (X_("colnum"), GUINT_TO_POINTER(0));
        edit_group_display.get_column (1)->set_data (X_("colnum"), GUINT_TO_POINTER(1));
        edit_group_display.get_column (2)->set_data (X_("colnum"), GUINT_TO_POINTER(2));
+       edit_group_display.get_column (0)->set_expand (true);
+       edit_group_display.get_column (1)->set_expand (false);
+       edit_group_display.get_column (2)->set_expand (false);
        edit_group_display.set_headers_visible (true);
 
        /* name is directly editable */
@@ -594,6 +554,7 @@ Editor::Editor (AudioEngine& eng)
        region_list_display.set_model (region_list_model);
        region_list_display.append_column (_("Regions"), region_list_columns.name);
        region_list_display.set_headers_visible (false);
+       region_list_display.set_hover_expand (true);
 
        region_list_display.get_selection()->set_select_function (mem_fun (*this, &Editor::region_list_selection_filter));
        
@@ -682,7 +643,7 @@ Editor::Editor (AudioEngine& eng)
        edit_pane.pack1 (edit_packer, true, true);
        edit_pane.pack2 (the_notebook, false, true);
        
-       edit_pane.signal_size_allocate().connect_notify (bind (mem_fun(*this, &Editor::pane_allocation_handler), static_cast<Paned*> (&edit_pane)));
+       edit_pane.signal_size_allocate().connect (bind (mem_fun(*this, &Editor::pane_allocation_handler), static_cast<Paned*> (&edit_pane)));
 
        top_hbox.pack_start (toolbar_frame, true, true);
 
@@ -709,15 +670,15 @@ Editor::Editor (AudioEngine& eng)
        _playlist_selector = new PlaylistSelector();
        _playlist_selector->signal_delete_event().connect (bind (sigc::ptr_fun (just_hide_it), static_cast<Window *> (_playlist_selector)));
 
-       AudioRegionView::AudioRegionViewGoingAway.connect (mem_fun(*this, &Editor::catch_vanishing_audio_regionview));
+       RegionView::RegionViewGoingAway.connect (mem_fun(*this, &Editor::catch_vanishing_regionview));
 
        /* nudge stuff */
 
        nudge_forward_button.add (*(manage (new Image (get_xpm("right_arrow.xpm")))));
        nudge_backward_button.add (*(manage (new Image (get_xpm("left_arrow.xpm")))));
 
-       ARDOUR_UI::instance()->tooltips().set_tip (nudge_forward_button, _("Nudge region/selection forwards"));
-       ARDOUR_UI::instance()->tooltips().set_tip (nudge_backward_button, _("Nudge region/selection backwards"));
+       ARDOUR_UI::instance()->tooltips().set_tip (nudge_forward_button, _("Nudge Region/Selection Forwards"));
+       ARDOUR_UI::instance()->tooltips().set_tip (nudge_backward_button, _("Nudge Region/Selection Backwards"));
 
        nudge_forward_button.set_name ("TransportButton");
        nudge_backward_button.set_name ("TransportButton");
@@ -741,6 +702,7 @@ Editor::Editor (AudioEngine& eng)
        ControlProtocol::ScrollTimeline.connect (mem_fun (*this, &Editor::control_scroll));
        constructed = true;
        instant_save ();
+
 }
 
 Editor::~Editor()
@@ -767,7 +729,7 @@ Editor::add_toplevel_controls (Container& cont)
 }
 
 void
-Editor::catch_vanishing_audio_regionview (AudioRegionView *rv)
+Editor::catch_vanishing_regionview (RegionView *rv)
 {
        /* note: the selection will take care of the vanishing
           audioregionview by itself.
@@ -783,7 +745,7 @@ Editor::catch_vanishing_audio_regionview (AudioRegionView *rv)
 }
 
 void
-Editor::set_entered_regionview (AudioRegionView* rv)
+Editor::set_entered_regionview (RegionView* rv)
 {
        if (rv == entered_regionview) {
                return;
@@ -810,15 +772,6 @@ Editor::set_entered_track (TimeAxisView* tav)
        }
 }
 
-gint
-Editor::left_track_canvas (GdkEventCrossing *ev)
-{
-       set_entered_track (0);
-       set_entered_regionview (0);
-       return FALSE;
-}
-
-
 void
 Editor::show_window ()
 {
@@ -877,16 +830,6 @@ Editor::set_frames_per_unit (double fpu)
                zoom_range_clock.set (frames);
        }
 
-       /* only update these if we not about to call reposition_x_origin,
-          which will do the same updates.
-       */
-       
-       if (!no_zoom_repos_update) {
-               horizontal_adjustment.set_value (leftmost_frame/frames_per_unit);
-               update_fixed_rulers ();
-               tempo_map_changed (Change (0));
-       }
-
        if (mouse_mode == MouseRange && selection->time.start () != selection->time.end_frame ()) {
                if (!selection->tracks.empty()) {
                        for (TrackSelection::iterator i = selection->tracks.begin(); i != selection->tracks.end(); ++i) {
@@ -914,7 +857,7 @@ Editor::set_frames_per_unit (double fpu)
 void
 Editor::instant_save ()
 {
-        if (!constructed || !ARDOUR_UI::instance()->session_loaded) {
+       if (!constructed || !ARDOUR_UI::instance()->session_loaded) {
                return;
        }
 
@@ -939,6 +882,9 @@ Editor::reposition_x_origin (jack_nframes_t frame)
                }
 
                horizontal_adjustment.set_value (frame/frames_per_unit);
+       } else {
+               update_fixed_rulers();
+               tempo_map_changed (Change (0));
        }
 }
 
@@ -954,7 +900,7 @@ Editor::edit_cursor_clock_changed()
 void
 Editor::zoom_adjustment_changed ()
 {
-       if (session == 0 || no_zoom_repos_update) {
+       if (session == 0) {
                return;
        }
 
@@ -1021,40 +967,28 @@ Editor::deferred_control_scroll (jack_nframes_t target)
 void 
 Editor::canvas_horizontally_scrolled ()
 {
-       leftmost_frame = (jack_nframes_t) floor (horizontal_adjustment.get_value() * frames_per_unit);
 
+       leftmost_frame = (jack_nframes_t) floor (horizontal_adjustment.get_value() * frames_per_unit);
        update_fixed_rulers ();
-       
-       if (!edit_hscroll_dragging) {
-               tempo_map_changed (Change (0));
-       } else {
-               update_tempo_based_rulers();
-       }
+       tempo_map_changed (Change (0));
+
 }
 
 void
 Editor::reposition_and_zoom (jack_nframes_t frame, double nfpu)
 {
        if (!repos_zoom_queued) {
-               Glib::signal_idle().connect (bind (mem_fun(*this, &Editor::deferred_reposition_and_zoom), frame, nfpu));
                repos_zoom_queued = true;
+               Glib::signal_idle().connect (bind (mem_fun(*this, &Editor::deferred_reposition_and_zoom), frame, nfpu));
        }
 }
 
 gint
 Editor::deferred_reposition_and_zoom (jack_nframes_t frame, double nfpu)
 {
-       /* if we need to force an update to the hscroller stuff,
-          don't set no_zoom_repos_update.
-       */
 
-       no_zoom_repos_update = (frame != leftmost_frame);
-       
        set_frames_per_unit (nfpu);
-       if (no_zoom_repos_update) {
-               reposition_x_origin  (frame);
-       }
-       no_zoom_repos_update = false;
+       reposition_x_origin  (frame);
        repos_zoom_queued = false;
        
        return FALSE;
@@ -1107,7 +1041,6 @@ void
 Editor::stop_scrolling ()
 {
        scroll_connection.disconnect ();
-       slower_update_connection.disconnect ();
 }
 
 void
@@ -1213,7 +1146,7 @@ Editor::connect_to_session (Session *t)
 
        update_title ();
 
-       session->going_away.connect (mem_fun(*this, &Editor::session_going_away));
+       session->GoingAway.connect (mem_fun(*this, &Editor::session_going_away));
 
        /* These signals can all be emitted by a non-GUI thread. Therefore the
           handlers for them must not attempt to directly interact with the GUI,
@@ -1222,7 +1155,7 @@ Editor::connect_to_session (Session *t)
 
        session_connections.push_back (session->TransportStateChange.connect (mem_fun(*this, &Editor::map_transport_state)));
        session_connections.push_back (session->PositionChanged.connect (mem_fun(*this, &Editor::map_position_change)));
-       session_connections.push_back (session->RouteAdded.connect (mem_fun(*this, &Editor::handle_new_route_p)));
+       session_connections.push_back (session->RouteAdded.connect (mem_fun(*this, &Editor::handle_new_route)));
        session_connections.push_back (session->AudioRegionAdded.connect (mem_fun(*this, &Editor::handle_new_audio_region)));
        session_connections.push_back (session->AudioRegionRemoved.connect (mem_fun(*this, &Editor::handle_audio_region_removed)));
        session_connections.push_back (session->DurationChanged.connect (mem_fun(*this, &Editor::handle_new_duration)));
@@ -1243,8 +1176,6 @@ Editor::connect_to_session (Session *t)
        edit_groups_changed ();
 
        edit_cursor_clock.set_session (session);
-       selection_start_clock.set_session (session);
-       selection_end_clock.set_session (session);
        zoom_range_clock.set_session (session);
        _playlist_selector->set_session (session);
        nudge_clock.set_session (session);
@@ -1353,7 +1284,8 @@ Editor::connect_to_session (Session *t)
        horizontal_adjustment.set_value (0);
 
        restore_ruler_visibility ();
-       tempo_map_changed (Change (0));
+       //tempo_map_changed (Change (0));
+       session->tempo_map().apply_with_metrics (*this, &Editor::draw_metric_marks);
 
        edit_cursor->set_position (0);
        playhead_cursor->set_position (0);
@@ -1377,7 +1309,7 @@ Editor::connect_to_session (Session *t)
                        AudioTimeAxisView *atv;
                        
                        if ((atv = dynamic_cast<AudioTimeAxisView*>(tv)) != 0) {
-                               if (atv->route().master()) {
+                               if (atv->route()->master()) {
                                        route_list_display.get_selection()->unselect (i);
                                }
                        }
@@ -1386,6 +1318,10 @@ Editor::connect_to_session (Session *t)
                no_route_list_redisplay = false;
                redisplay_route_list ();
        }
+
+        /* register for undo history */
+
+        session->register_with_memento_command_factory(_id, this);
 }
 
 void
@@ -1442,13 +1378,14 @@ Editor::popup_fade_context_menu (int button, int32_t time, ArdourCanvas::Item* i
        }
 
        MenuList& items (fade_context_menu.items());
+       AudioRegion& ar (*arv->audio_region().get()); // FIXME
 
        items.clear ();
 
        switch (item_type) {
        case FadeInItem:
        case FadeInHandleItem:
-               if (arv->region.fade_in_active()) {
+               if (arv->audio_region()->fade_in_active()) {
                        items.push_back (MenuElem (_("Deactivate"), bind (mem_fun (*arv, &AudioRegionView::set_fade_in_active), false)));
                } else {
                        items.push_back (MenuElem (_("Activate"), bind (mem_fun (*arv, &AudioRegionView::set_fade_in_active), true)));
@@ -1456,16 +1393,16 @@ Editor::popup_fade_context_menu (int button, int32_t time, ArdourCanvas::Item* i
                
                items.push_back (SeparatorElem());
                
-               items.push_back (MenuElem (_("Linear"), bind (mem_fun (arv->region, &AudioRegion::set_fade_in_shape), AudioRegion::Linear)));
-               items.push_back (MenuElem (_("Slowest"), bind (mem_fun (arv->region, &AudioRegion::set_fade_in_shape), AudioRegion::LogB)));
-               items.push_back (MenuElem (_("Slow"), bind (mem_fun (arv->region, &AudioRegion::set_fade_in_shape), AudioRegion::Fast)));
-               items.push_back (MenuElem (_("Fast"), bind (mem_fun (arv->region, &AudioRegion::set_fade_in_shape), AudioRegion::LogA)));
-               items.push_back (MenuElem (_("Fastest"), bind (mem_fun (arv->region, &AudioRegion::set_fade_in_shape), AudioRegion::Slow)));
+               items.push_back (MenuElem (_("Linear"), bind (mem_fun (ar, &AudioRegion::set_fade_in_shape), AudioRegion::Linear)));
+               items.push_back (MenuElem (_("Slowest"), bind (mem_fun (ar, &AudioRegion::set_fade_in_shape), AudioRegion::LogB)));
+               items.push_back (MenuElem (_("Slow"), bind (mem_fun (ar, &AudioRegion::set_fade_in_shape), AudioRegion::Fast)));
+               items.push_back (MenuElem (_("Fast"), bind (mem_fun (ar, &AudioRegion::set_fade_in_shape), AudioRegion::LogA)));
+               items.push_back (MenuElem (_("Fastest"), bind (mem_fun (ar, &AudioRegion::set_fade_in_shape), AudioRegion::Slow)));
                break;
 
        case FadeOutItem:
        case FadeOutHandleItem:
-               if (arv->region.fade_out_active()) {
+               if (arv->audio_region()->fade_out_active()) {
                        items.push_back (MenuElem (_("Deactivate"), bind (mem_fun (*arv, &AudioRegionView::set_fade_out_active), false)));
                } else {
                        items.push_back (MenuElem (_("Activate"), bind (mem_fun (*arv, &AudioRegionView::set_fade_out_active), true)));
@@ -1473,11 +1410,11 @@ Editor::popup_fade_context_menu (int button, int32_t time, ArdourCanvas::Item* i
                
                items.push_back (SeparatorElem());
                
-               items.push_back (MenuElem (_("Linear"), bind (mem_fun (arv->region, &AudioRegion::set_fade_out_shape), AudioRegion::Linear)));
-               items.push_back (MenuElem (_("Slowest"), bind (mem_fun (arv->region, &AudioRegion::set_fade_out_shape), AudioRegion::Fast)));
-               items.push_back (MenuElem (_("Slow"), bind (mem_fun (arv->region, &AudioRegion::set_fade_out_shape), AudioRegion::LogB)));
-               items.push_back (MenuElem (_("Fast"), bind (mem_fun (arv->region, &AudioRegion::set_fade_out_shape), AudioRegion::LogA)));
-               items.push_back (MenuElem (_("Fastest"), bind (mem_fun (arv->region, &AudioRegion::set_fade_out_shape), AudioRegion::Slow)));
+               items.push_back (MenuElem (_("Linear"), bind (mem_fun (ar, &AudioRegion::set_fade_out_shape), AudioRegion::Linear)));
+               items.push_back (MenuElem (_("Slowest"), bind (mem_fun (ar, &AudioRegion::set_fade_out_shape), AudioRegion::Fast)));
+               items.push_back (MenuElem (_("Slow"), bind (mem_fun (ar, &AudioRegion::set_fade_out_shape), AudioRegion::LogB)));
+               items.push_back (MenuElem (_("Fast"), bind (mem_fun (ar, &AudioRegion::set_fade_out_shape), AudioRegion::LogA)));
+               items.push_back (MenuElem (_("Fastest"), bind (mem_fun (ar, &AudioRegion::set_fade_out_shape), AudioRegion::Slow)));
 
                break;
        default:
@@ -1499,8 +1436,8 @@ Editor::popup_track_context_menu (int button, int32_t time, ItemType item_type,
 
        switch (item_type) {
        case RegionItem:
-       case AudioRegionViewName:
-       case AudioRegionViewNameHighlight:
+       case RegionViewName:
+       case RegionViewNameHighlight:
                if (with_selection) {
                        build_menu_function = &Editor::build_track_selection_context_menu;
                } else {
@@ -1540,25 +1477,26 @@ Editor::popup_track_context_menu (int button, int32_t time, ItemType item_type,
 
        switch (item_type) {
        case RegionItem:
-       case AudioRegionViewName:
-       case AudioRegionViewNameHighlight:
+       case RegionViewName:
+       case RegionViewNameHighlight:
                if (!with_selection) {
                        if (region_edit_menu_split_item) {
-                               if (clicked_regionview && clicked_regionview->region.covers (edit_cursor->current_frame)) {
+                               if (clicked_regionview && clicked_regionview->region()->covers (edit_cursor->current_frame)) {
                                        ActionManager::set_sensitive (ActionManager::edit_cursor_in_region_sensitive_actions, true);
                                } else {
                                        ActionManager::set_sensitive (ActionManager::edit_cursor_in_region_sensitive_actions, false);
                                }
                        }
+                       /*
                        if (region_edit_menu_split_multichannel_item) {
-                               if (clicked_regionview && clicked_regionview->region.n_channels() > 1) {
+                               if (clicked_regionview && clicked_regionview->region().n_channels() > 1) {
                                        // GTK2FIX find the action, change its sensitivity
                                        // region_edit_menu_split_multichannel_item->set_sensitive (true);
                                } else {
                                        // GTK2FIX see above
                                        // region_edit_menu_split_multichannel_item->set_sensitive (false);
                                }
-                       }
+                       }*/
                }
                break;
 
@@ -1638,13 +1576,13 @@ Editor::build_track_region_context_menu (jack_nframes_t frame)
        AudioTimeAxisView* atv = dynamic_cast<AudioTimeAxisView*> (clicked_trackview);
 
        if (atv) {
-               DiskStream* ds;
+               boost::shared_ptr<Diskstream> ds;
                Playlist* pl;
                
                if ((ds = atv->get_diskstream()) && ((pl = ds->playlist()))) {
                        Playlist::RegionList* regions = pl->regions_at ((jack_nframes_t) floor ( (double)frame * ds->speed()));
                        for (Playlist::RegionList::iterator i = regions->begin(); i != regions->end(); ++i) {
-                               add_region_context_items (atv->view, (*i), edit_items);
+                               add_region_context_items (atv->audio_view(), (*i), edit_items);
                        }
                        delete regions;
                }
@@ -1665,7 +1603,7 @@ Editor::build_track_crossfade_context_menu (jack_nframes_t frame)
        AudioTimeAxisView* atv = dynamic_cast<AudioTimeAxisView*> (clicked_trackview);
 
        if (atv) {
-               DiskStream* ds;
+               boost::shared_ptr<Diskstream> ds;
                Playlist* pl;
                AudioPlaylist* apl;
 
@@ -1679,11 +1617,11 @@ Editor::build_track_crossfade_context_menu (jack_nframes_t frame)
                        bool many = xfades.size() > 1;
 
                        for (AudioPlaylist::Crossfades::iterator i = xfades.begin(); i != xfades.end(); ++i) {
-                               add_crossfade_context_items (atv->view, (*i), edit_items, many);
+                               add_crossfade_context_items (atv->audio_view(), (*i), edit_items, many);
                        }
 
                        for (Playlist::RegionList::iterator i = regions->begin(); i != regions->end(); ++i) {
-                               add_region_context_items (atv->view, (*i), edit_items);
+                               add_region_context_items (atv->audio_view(), (*i), edit_items);
                        }
 
                        delete regions;
@@ -1749,7 +1687,7 @@ Editor::build_track_selection_context_menu (jack_nframes_t ignored)
 }
 
 void
-Editor::add_crossfade_context_items (StreamView* view, Crossfade* xfade, Menu_Helpers::MenuList& edit_items, bool many)
+Editor::add_crossfade_context_items (AudioStreamView* view, Crossfade* xfade, Menu_Helpers::MenuList& edit_items, bool many)
 {
        using namespace Menu_Helpers;
        Menu     *xfade_menu = manage (new Menu);
@@ -1778,9 +1716,9 @@ Editor::add_crossfade_context_items (StreamView* view, Crossfade* xfade, Menu_He
        }
 
        if (many) {
-               str = xfade->out().name();
+               str = xfade->out()->name();
                str += "->";
-               str += xfade->in().name();
+               str += xfade->in()->name();
        } else {
                str = _("Crossfade");
        }
@@ -1806,17 +1744,17 @@ Editor::xfade_edit_right_region ()
 }
 
 void
-Editor::add_region_context_items (StreamView* sv, Region* region, Menu_Helpers::MenuList& edit_items)
+Editor::add_region_context_items (AudioStreamView* sv, boost::shared_ptr<Region> region, Menu_Helpers::MenuList& edit_items)
 {
        using namespace Menu_Helpers;
        Menu     *region_menu = manage (new Menu);
        MenuList& items       = region_menu->items();
        region_menu->set_name ("ArdourContextMenu");
        
-       AudioRegion* ar = 0;
+       boost::shared_ptr<AudioRegion> ar;
 
        if (region) {
-               ar = dynamic_cast<AudioRegion*> (region);
+               ar = boost::dynamic_pointer_cast<AudioRegion> (region);
        }
 
        /* when this particular menu pops up, make the relevant region 
@@ -2165,6 +2103,9 @@ Editor::set_state (const XMLNode& node)
        int x, y, xoff, yoff;
        Gdk::Geometry g;
 
+       if ((prop = node.property ("id")) != 0) {
+               _id = prop->value ();
+       }
 
        if ((geometry = find_named_node (node, "geometry")) == 0) {
 
@@ -2185,7 +2126,6 @@ Editor::set_state (const XMLNode& node)
                yoff = atoi(geometry->property("y_off")->value());
        }
 
-       set_geometry_hints (vpacker, g, Gdk::HINT_BASE_SIZE);
        set_default_size (g.base_width, g.base_height);
        move (x, y);
 
@@ -2194,7 +2134,7 @@ Editor::set_state (const XMLNode& node)
        }
 
        if ((prop = node.property ("zoom"))) {
-               set_frames_per_unit (atof (prop->value()));
+               set_frames_per_unit (PBD::atof (prop->value()));
        }
 
        if ((prop = node.property ("snap-to"))) {
@@ -2287,6 +2227,7 @@ Editor::set_state (const XMLNode& node)
                }
        }
 
+
        return 0;
 }
 
@@ -2296,8 +2237,11 @@ Editor::get_state ()
        XMLNode* node = new XMLNode ("Editor");
        char buf[32];
 
+       _id.print (buf);
+       node->add_property ("id", buf);
+       
        if (is_realized()) {
-               Glib::RefPtr<Gdk::Window> win = get_window();
+               Glib::RefPtr<Gdk::Window> win = get_window();
                
                int x, y, xoff, yoff, width, height;
                win->get_root_origin(x, y);
@@ -2305,7 +2249,7 @@ Editor::get_state ()
                win->get_size(width, height);
                
                XMLNode* geometry = new XMLNode ("geometry");
-               char buf[32];
+
                snprintf(buf, sizeof(buf), "%d", width);
                geometry->add_property("x_size", string(buf));
                snprintf(buf, sizeof(buf), "%d", height);
@@ -2586,30 +2530,56 @@ void
 Editor::setup_toolbar ()
 {
        string pixmap_path;
+
+       const guint32 FUDGE = 18; // Combo's are stupid - they steal space from the entry for the button
+
+
+       /* Mode Buttons (tool selection) */
+
        vector<ToggleButton *> mouse_mode_buttons;
 
+       mouse_move_button.add (*(manage (new Image (get_xpm("tool_object.xpm")))));
+       mouse_move_button.set_relief(Gtk::RELIEF_NONE);
        mouse_mode_buttons.push_back (&mouse_move_button);
+       mouse_select_button.add (*(manage (new Image (get_xpm("tool_range.xpm")))));
+       mouse_select_button.set_relief(Gtk::RELIEF_NONE);
        mouse_mode_buttons.push_back (&mouse_select_button);
+       mouse_gain_button.add (*(manage (new Image (get_xpm("tool_gain.xpm")))));
+       mouse_gain_button.set_relief(Gtk::RELIEF_NONE);
        mouse_mode_buttons.push_back (&mouse_gain_button);
+       mouse_zoom_button.add (*(manage (new Image (get_xpm("tool_zoom.xpm")))));
+       mouse_zoom_button.set_relief(Gtk::RELIEF_NONE);
        mouse_mode_buttons.push_back (&mouse_zoom_button);
+       mouse_timefx_button.add (*(manage (new Image (get_xpm("tool_stretch.xpm")))));
+       mouse_timefx_button.set_relief(Gtk::RELIEF_NONE);
        mouse_mode_buttons.push_back (&mouse_timefx_button);
+       mouse_audition_button.add (*(manage (new Image (get_xpm("tool_audition.xpm")))));
+       mouse_audition_button.set_relief(Gtk::RELIEF_NONE);
        mouse_mode_buttons.push_back (&mouse_audition_button);
+       
        mouse_mode_button_set = new GroupedButtons (mouse_mode_buttons);
 
-       mouse_mode_button_table.set_homogeneous (true);
-       mouse_mode_button_table.set_col_spacings (2);
-       mouse_mode_button_table.set_row_spacings (2);
-       mouse_mode_button_table.set_border_width (5);
+       HBox* mode_box = manage(new HBox);
+       mode_box->set_border_width (2);
+       mode_box->set_spacing(4);
+       mouse_mode_button_box.set_spacing(1);
+       mouse_mode_button_box.pack_start(mouse_move_button, true, true);
+       mouse_mode_button_box.pack_start(mouse_select_button, true, true);
+       mouse_mode_button_box.pack_start(mouse_zoom_button, true, true);
+       mouse_mode_button_box.pack_start(mouse_gain_button, true, true);
+       mouse_mode_button_box.pack_start(mouse_timefx_button, true, true);
+       mouse_mode_button_box.pack_start(mouse_audition_button, true, true);
+       mouse_mode_button_box.set_homogeneous(true);
 
-       mouse_mode_button_table.attach (mouse_move_button, 0, 1, 0, 1);
-       mouse_mode_button_table.attach (mouse_select_button, 1, 2, 0, 1);
-       mouse_mode_button_table.attach (mouse_zoom_button, 2, 3, 0, 1);
-       mouse_mode_button_table.attach (mouse_gain_button, 0, 1, 1, 2);
-       mouse_mode_button_table.attach (mouse_timefx_button, 1, 2, 1, 2);
-       mouse_mode_button_table.attach (mouse_audition_button, 2, 3, 1, 2);
+       edit_mode_selector.set_name ("EditModeSelector");
+       Gtkmm2ext::set_size_request_to_display_given_text (edit_mode_selector, "Splice Edit", 2+FUDGE, 10);
+       set_popdown_strings (edit_mode_selector, internationalize (edit_mode_strings));
+       edit_mode_selector.signal_changed().connect (mem_fun(*this, &Editor::edit_mode_selection_done));
 
-       mouse_mode_tearoff = manage (new TearOff (mouse_mode_button_table));
+       mode_box->pack_start(edit_mode_selector);
+       mode_box->pack_start(mouse_mode_button_box);
+       
+       mouse_mode_tearoff = manage (new TearOff (*mode_box));
        mouse_mode_tearoff->set_name ("MouseModeBase");
 
        mouse_mode_tearoff->Detach.connect (bind (mem_fun(*this, &Editor::detach_tearoff), static_cast<Box*>(&toolbar_hbox), 
@@ -2628,12 +2598,12 @@ Editor::setup_toolbar ()
        mouse_timefx_button.set_name ("MouseModeButton");
        mouse_audition_button.set_name ("MouseModeButton");
 
-       ARDOUR_UI::instance()->tooltips().set_tip (mouse_move_button, _("select/move objects"));
-       ARDOUR_UI::instance()->tooltips().set_tip (mouse_select_button, _("select/move ranges"));
-       ARDOUR_UI::instance()->tooltips().set_tip (mouse_gain_button, _("draw gain automation"));
-       ARDOUR_UI::instance()->tooltips().set_tip (mouse_zoom_button, _("select zoom range"));
-       ARDOUR_UI::instance()->tooltips().set_tip (mouse_timefx_button, _("stretch/shrink regions"));
-       ARDOUR_UI::instance()->tooltips().set_tip (mouse_audition_button, _("listen to specific regions"));
+       ARDOUR_UI::instance()->tooltips().set_tip (mouse_move_button, _("Select/Move Objects"));
+       ARDOUR_UI::instance()->tooltips().set_tip (mouse_select_button, _("Select/Move Ranges"));
+       ARDOUR_UI::instance()->tooltips().set_tip (mouse_gain_button, _("Draw Gain Automation"));
+       ARDOUR_UI::instance()->tooltips().set_tip (mouse_zoom_button, _("Select Zoom Range"));
+       ARDOUR_UI::instance()->tooltips().set_tip (mouse_timefx_button, _("Stretch/Shrink Regions"));
+       ARDOUR_UI::instance()->tooltips().set_tip (mouse_audition_button, _("Listen to Specific Regions"));
 
        mouse_move_button.unset_flags (CAN_FOCUS);
        mouse_select_button.unset_flags (CAN_FOCUS);
@@ -2652,160 +2622,82 @@ Editor::setup_toolbar ()
        mouse_audition_button.signal_toggled().connect (bind (mem_fun(*this, &Editor::mouse_mode_toggled), Editing::MouseAudition));
 
        // mouse_move_button.set_active (true);
+       
 
-       /* automation control */
-
-       global_automation_button.set_name ("MouseModeButton");
-       automation_mode_button.set_name ("MouseModeButton");
-
-       automation_box.set_spacing (2);
-       automation_box.set_border_width (2);
-       automation_box.pack_start (global_automation_button, false, false);
-       automation_box.pack_start (automation_mode_button, false, false);
-
-       /* Edit mode */
-
-       edit_mode_label.set_name ("ToolBarLabel");
-
-       edit_mode_selector.set_name ("EditModeSelector");
+       /* Zoom */
+       
+       zoom_box.set_spacing (1);
+       zoom_box.set_border_width (2);
 
-       edit_mode_box.set_spacing (3);
-       edit_mode_box.set_border_width (3);
+       zoom_in_button.set_name ("EditorTimeButton");
+       zoom_in_button.add (*(manage (new Image (get_xpm("zoom_in.xpm")))));
+       zoom_in_button.signal_clicked().connect (bind (mem_fun(*this, &Editor::temporal_zoom_step), false));
+       ARDOUR_UI::instance()->tooltips().set_tip (zoom_in_button, _("Zoom In"));
+       
+       zoom_out_button.set_name ("EditorTimeButton");
+       zoom_out_button.add (*(manage (new Image (get_xpm("zoom_out.xpm")))));
+       zoom_out_button.signal_clicked().connect (bind (mem_fun(*this, &Editor::temporal_zoom_step), true));
+       ARDOUR_UI::instance()->tooltips().set_tip (zoom_out_button, _("Zoom Out"));
 
-       /* XXX another disgusting hack because of the way combo boxes size themselves */
+       zoom_out_full_button.set_name ("EditorTimeButton");
+       zoom_out_full_button.add (*(manage (new Image (get_xpm("zoom_full.xpm")))));
+       zoom_out_full_button.signal_clicked().connect (mem_fun(*this, &Editor::temporal_zoom_session));
+       ARDOUR_UI::instance()->tooltips().set_tip (zoom_out_full_button, _("Zoom to Session"));
+       
+       zoom_box.pack_start (zoom_out_button, false, false);
+       zoom_box.pack_start (zoom_in_button, false, false);
+       zoom_box.pack_start (zoom_range_clock, false, false);   
+       zoom_box.pack_start (zoom_out_full_button, false, false);
+       
+       ARDOUR_UI::instance()->tooltips().set_tip (zoom_range_clock, _("Current Zoom Range\n(Width of visible area)"));
+       
+       zoom_focus_selector.set_name ("ZoomFocusSelector");
+       Gtkmm2ext::set_size_request_to_display_given_text (zoom_focus_selector, "Focus Center", 2+FUDGE, 0);
+       set_popdown_strings (zoom_focus_selector, internationalize (zoom_focus_strings));
+       zoom_focus_selector.signal_changed().connect (mem_fun(*this, &Editor::zoom_focus_selection_done));
 
-       const guint32 FUDGE = 20; // Combo's are stupid - they steal space from the entry for the button
-       Gtkmm2ext::set_size_request_to_display_given_text (edit_mode_selector, "EdgtMode", 2+FUDGE, 10);
-       set_popdown_strings (edit_mode_selector, internationalize (edit_mode_strings));
-       edit_mode_box.pack_start (edit_mode_label, false, false);
-       edit_mode_box.pack_start (edit_mode_selector, false, false);
+       zoom_box.pack_start (zoom_focus_selector, false, false);
 
-       edit_mode_selector.signal_changed().connect (mem_fun(*this, &Editor::edit_mode_selection_done));
 
-       /* Snap Type */
+       /* Edit Cursor / Snap */
 
-       snap_type_label.set_name ("ToolBarLabel");
+       snap_box.set_spacing (1);
+       snap_box.set_border_width (2);
 
        snap_type_selector.set_name ("SnapTypeSelector");
-
-       snap_type_box.set_spacing (3);
-       snap_type_box.set_border_width (3);
-
-       /* XXX another disgusting hack because of the way combo boxes size themselves */
-
        Gtkmm2ext::set_size_request_to_display_given_text (snap_type_selector, "SMPTE Seconds", 2+FUDGE, 10);
        set_popdown_strings (snap_type_selector, internationalize (snap_type_strings));
-
-       snap_type_box.pack_start (snap_type_label, false, false);
-       snap_type_box.pack_start (snap_type_selector, false, false);
-
        snap_type_selector.signal_changed().connect (mem_fun(*this, &Editor::snap_type_selection_done));
-
-       /* Snap mode, not snap type */
-
-       snap_mode_label.set_name ("ToolBarLabel");
+       ARDOUR_UI::instance()->tooltips().set_tip (snap_type_selector, _("Unit to snap cursors and ranges to"));
 
        snap_mode_selector.set_name ("SnapModeSelector");
-       
-       snap_mode_box.set_spacing (3);
-       snap_mode_box.set_border_width (3);
-
-       Gtkmm2ext::set_size_request_to_display_given_text (snap_mode_selector, "SngpMode", 2+FUDGE, 10);
+       Gtkmm2ext::set_size_request_to_display_given_text (snap_mode_selector, "Magnetic Snap", 2+FUDGE, 10);
        set_popdown_strings (snap_mode_selector, internationalize (snap_mode_strings));
-
-       snap_mode_box.pack_start (snap_mode_label, false, false);
-       snap_mode_box.pack_start (snap_mode_selector, false, false);
-
        snap_mode_selector.signal_changed().connect (mem_fun(*this, &Editor::snap_mode_selection_done));
 
-       /* Zoom focus mode */
-
-       zoom_focus_label.set_name ("ToolBarLabel");
-
-       zoom_focus_selector.set_name ("ZoomFocusSelector");
-
-       zoom_focus_box.set_spacing (3);
-       zoom_focus_box.set_border_width (3);
+       snap_box.pack_start (edit_cursor_clock, false, false);
+       snap_box.pack_start (snap_mode_selector, false, false);
+       snap_box.pack_start (snap_type_selector, false, false);
 
-       /* XXX another disgusting hack because of the way combo boxes size themselves */
 
-       Gtkmm2ext::set_size_request_to_display_given_text (zoom_focus_selector, "Edgt Cursor", 2+FUDGE, 10);
-       set_popdown_strings (zoom_focus_selector, internationalize (zoom_focus_strings));
+       /* Nudge */
 
-       zoom_focus_box.pack_start (zoom_focus_label, false, false);
-       zoom_focus_box.pack_start (zoom_focus_selector, false, false);
-
-       zoom_focus_selector.signal_changed().connect (mem_fun(*this, &Editor::zoom_focus_selection_done));
+       HBox *nudge_box = manage (new HBox);
+       nudge_box->set_spacing(1);
+       nudge_box->set_border_width (2);
 
-       /* selection/cursor clocks */
-
-       toolbar_selection_cursor_label.set_name ("ToolBarLabel");
-       selection_start_clock_label.set_name ("ToolBarLabel");
-       selection_end_clock_label.set_name ("ToolBarLabel");
-       edit_cursor_clock_label.set_name ("ToolBarLabel");
-
-       selection_start_clock_label.set_text (_("Start:"));
-       selection_end_clock_label.set_text (_("End:"));
-       edit_cursor_clock_label.set_text (_("Edit"));
-
-       /* the zoom in/out buttons are generally taller than the clocks, so
-          put all the toolbar clocks into a size group with one of the 
-          buttons to make them all equal height.
-
-          this also applies to the various toolbar combos
-       */
-
-       RefPtr<SizeGroup> toolbar_clock_size_group = SizeGroup::create (SIZE_GROUP_VERTICAL);
-       toolbar_clock_size_group->add_widget (zoom_out_button);
-       toolbar_clock_size_group->add_widget (edit_cursor_clock);
-       toolbar_clock_size_group->add_widget (zoom_range_clock);
-       toolbar_clock_size_group->add_widget (nudge_clock);
-       toolbar_clock_size_group->add_widget (edit_mode_selector);
-       toolbar_clock_size_group->add_widget (snap_type_selector);
-       toolbar_clock_size_group->add_widget (snap_mode_selector);
-       toolbar_clock_size_group->add_widget (zoom_focus_selector);
-
-       HBox* edit_clock_hbox = manage (new HBox());
-       VBox* edit_clock_vbox = manage (new VBox());
-
-       edit_clock_hbox->pack_start (edit_cursor_clock, false, false);
-
-       edit_clock_vbox->set_spacing (3);
-       edit_clock_vbox->set_border_width (3);
-       edit_clock_vbox->pack_start (edit_cursor_clock_label, false, false);
-       edit_clock_vbox->pack_start (*edit_clock_hbox, false, false);
-
-       HBox* hbox = new HBox;
-
-       hbox->pack_start (*edit_clock_vbox, false, false);
-       hbox->pack_start (zoom_indicator_vbox, false, false); 
-       hbox->pack_start (zoom_focus_box, false, false);
-       hbox->pack_start (snap_type_box, false, false);
-       hbox->pack_start (snap_mode_box, false, false);
-       hbox->pack_start (edit_mode_box, false, false);
-
-       VBox *vbox = manage (new VBox);
-
-       vbox->set_spacing (3);
-       vbox->set_border_width (3);
-
-       HBox *nbox = manage (new HBox);
-       
        nudge_forward_button.signal_clicked().connect (bind (mem_fun(*this, &Editor::nudge_forward), false));
        nudge_backward_button.signal_clicked().connect (bind (mem_fun(*this, &Editor::nudge_backward), false));
 
-       nbox->pack_start (nudge_backward_button, false, false);
-       nbox->pack_start (nudge_forward_button, false, false);
-       nbox->pack_start (nudge_clock, false, false, 5);
-
-       nudge_label.set_name ("ToolBarLabel");
+       nudge_box->pack_start (nudge_backward_button, false, false);
+       nudge_box->pack_start (nudge_forward_button, false, false);
+       nudge_box->pack_start (nudge_clock, false, false);
 
-       vbox->pack_start (nudge_label, false, false);
-       vbox->pack_start (*nbox, false, false);
 
-       hbox->pack_start (*vbox, false, false);
+       /* Pack everything in... */
 
-       hbox->show_all ();
+       HBox* hbox = new HBox;
+       hbox->set_spacing(10);
 
        tools_tearoff = new TearOff (*hbox);
        tools_tearoff->set_name ("MouseModeBase");
@@ -2819,11 +2711,18 @@ Editor::setup_toolbar ()
        tools_tearoff->Visible.connect (bind (mem_fun(*this, &Editor::reattach_tearoff), static_cast<Box*> (&toolbar_hbox), 
                                              &tools_tearoff->tearoff_window(), 0));
 
-       toolbar_hbox.set_spacing (8);
-       toolbar_hbox.set_border_width (2);
+       toolbar_hbox.set_spacing (10);
+       toolbar_hbox.set_border_width (1);
 
-       toolbar_hbox.pack_start (*tools_tearoff, false, false);
        toolbar_hbox.pack_start (*mouse_mode_tearoff, false, false);
+       toolbar_hbox.pack_start (*tools_tearoff, false, false);
+
+       
+       hbox->pack_start (snap_box, false, false);
+       hbox->pack_start (zoom_box, false, false); 
+       hbox->pack_start (*nudge_box, false, false);
+
+       hbox->show_all ();
        
        toolbar_base.set_name ("ToolBarBase");
        toolbar_base.add (toolbar_hbox);
@@ -2833,117 +2732,6 @@ Editor::setup_toolbar ()
        toolbar_frame.add (toolbar_base);
 }
 
-gint
-Editor::_autoscroll_canvas (void *arg)
-{
-       return ((Editor *) arg)->autoscroll_canvas ();
-}
-
-gint
-Editor::autoscroll_canvas ()
-{
-       jack_nframes_t new_frame;
-       bool keep_calling = true;
-       jack_nframes_t limit = max_frames - current_page_frames();
-       GdkEventMotion ev;
-
-       autoscroll_distance = current_page_frames() * 3 / 4;
-
-       if (autoscroll_direction < 0) {
-               if (leftmost_frame < autoscroll_distance) {
-                       new_frame = 0;
-               } else {
-                       new_frame = leftmost_frame - autoscroll_distance;
-               }
-               ev.x = drag_info.current_pointer_x - autoscroll_distance;
-       } else {
-               if (leftmost_frame > limit - autoscroll_distance) {
-                       new_frame = limit;
-               } else {
-                       new_frame = leftmost_frame + autoscroll_distance;
-               }
-               ev.x = drag_info.current_pointer_x + autoscroll_distance;
-       }
-
-       if (new_frame != leftmost_frame) {
-               cerr << "move to " << new_frame << " which is " << autoscroll_distance << " away" << endl;
-               reposition_x_origin (new_frame);
-       }
-
-       /* now fake a motion event to get the object that is being dragged to move too */
-
-       ev.type = GDK_MOTION_NOTIFY;
-       ev.x = frame_to_unit (ev.x);
-       ev.y = frame_to_unit (drag_info.current_pointer_y);
-       motion_handler (drag_info.item, (GdkEvent*) &ev, drag_info.item_type, true);
-
-       if (new_frame == 0 || new_frame == limit) {
-               /* we are done */
-               return false;
-       }
-
-       return false;
-
-       autoscroll_cnt++;
-
-       if (autoscroll_cnt == 1) {
-
-               /* connect the timeout so that we get called repeatedly */
-
-               autoscroll_timeout_tag = gtk_timeout_add (20, _autoscroll_canvas, this);
-               keep_calling = false;
-
-       } else if (autoscroll_cnt == 50) { /* 0.5 seconds */
-               
-               /* after about a while, speed up a bit by changing the timeout interval */
-
-               autoscroll_distance = (jack_nframes_t) floor (current_page_frames()/50.0f);
-               cerr << "change distance to " << autoscroll_distance << endl;
-               
-       } else if (autoscroll_cnt == 75) { /* 1.0 seconds */
-
-               autoscroll_distance = (jack_nframes_t) floor (current_page_frames()/20.0f);
-               cerr << "change distance to " << autoscroll_distance << endl;
-
-       } else if (autoscroll_cnt == 100) { /* 1.5 seconds */
-
-               /* after about another while, speed up by increasing the shift per callback */
-
-               autoscroll_distance =  (jack_nframes_t) floor (current_page_frames()/10.0f);
-               cerr << "change distance to " << autoscroll_distance << endl;
-
-       } 
-
-       return keep_calling;
-}
-
-void
-Editor::start_canvas_autoscroll (int dir)
-{
-       if (!session) {
-               return;
-       }
-
-       stop_canvas_autoscroll ();
-
-       autoscroll_direction = dir;
-       autoscroll_distance = (jack_nframes_t) floor (current_page_frames()/100.0);
-       autoscroll_cnt = 0;
-       
-       /* do it right now, which will start the repeated callbacks */
-       
-       autoscroll_canvas ();
-}
-
-void
-Editor::stop_canvas_autoscroll ()
-{
-       if (autoscroll_timeout_tag >= 0) {
-               gtk_timeout_remove (autoscroll_timeout_tag);
-               autoscroll_timeout_tag = -1;
-       }
-}
-
 int
 Editor::convert_drop_to_paths (vector<ustring>& paths, 
                               const RefPtr<Gdk::DragContext>& context,
@@ -3013,7 +2801,7 @@ Editor::convert_drop_to_paths (vector<ustring>& paths,
        for (vector<ustring>::iterator i = uris.begin(); i != uris.end(); ++i) {
                if ((*i).substr (0,7) == "file://") {
                        string p = *i;
-                       url_decode (p);
+                        PBD::url_decode (p);
                        paths.push_back (p.substr (7));
                }
        }
@@ -3082,8 +2870,8 @@ void
 Editor::begin_reversible_command (string name)
 {
        if (session) {
-               UndoAction ua = get_memento();
-               session->begin_reversible_command (name, &ua);
+                before = &get_state();
+               session->begin_reversible_command (name);
        }
 }
 
@@ -3091,36 +2879,35 @@ void
 Editor::commit_reversible_command ()
 {
        if (session) {
-               UndoAction ua = get_memento();
-               session->commit_reversible_command (&ua);
+               session->commit_reversible_command (new MementoCommand<Editor>(*this, before, &get_state()));
        }
 }
 
-void
-Editor::set_selected_track_from_click (Selection::Operation op, bool with_undo, bool no_remove)
+bool
+Editor::set_selected_track_from_click (bool press, Selection::Operation op, bool with_undo, bool no_remove)
 {
+       bool commit = false;
+
        if (!clicked_trackview) {
-               return;
+               return false;
        }
 
-       if (with_undo) {
-               begin_reversible_command (_("set selected trackview"));
-       }
-       
        switch (op) {
        case Selection::Toggle:
                if (selection->selected (clicked_trackview)) {
                        if (!no_remove) {
                                selection->remove (clicked_trackview);
+                               commit = true;
                        }
                } else {
-                       selection->toggle (clicked_trackview);
+                       selection->add (clicked_trackview);
+                       commit = false;
                }
                break;
+
        case Selection::Set:
                if (selection->selected (clicked_trackview) && selection->tracks.size() == 1) {
                        /* no commit necessary */
-                       return;
                } 
 
                selection->set (clicked_trackview);
@@ -3130,27 +2917,19 @@ Editor::set_selected_track_from_click (Selection::Operation op, bool with_undo,
                /* not defined yet */
                break;
        }
-       
-       if (with_undo) {
-               commit_reversible_command ();
-       }
+
+       return commit;
 }
 
-void
-Editor::set_selected_control_point_from_click (Selection::Operation op, bool with_undo, bool no_remove)
+bool
+Editor::set_selected_control_point_from_click (bool press, Selection::Operation op, bool with_undo, bool no_remove)
 {
        if (!clicked_control_point) {
-               return;
+               return false;
        }
 
        /* select this point and any others that it represents */
 
-       bool commit;
-       
-       if (with_undo) {
-               begin_reversible_command (_("select control points"));
-       }
-       
        double y1, y2;
        jack_nframes_t x1, x2;
 
@@ -3159,18 +2938,12 @@ Editor::set_selected_control_point_from_click (Selection::Operation op, bool wit
        y1 = clicked_control_point->get_x() - 10;
        y2 = clicked_control_point->get_y() + 10;
 
-       commit = select_all_within (x1, x2, y1, y2, op);
-       
-       if (with_undo && commit) {
-               commit_reversible_command ();
-       }
+       return select_all_within (x1, x2, y1, y2, op);
 }
 
 void
-Editor::mapover_audio_tracks (slot<void,AudioTimeAxisView&,uint32_t> sl)
+Editor::get_relevant_audio_tracks (AudioTimeAxisView& base, set<AudioTimeAxisView*>& relevant_tracks)
 {
-       set<AudioTimeAxisView*> relevant_tracks;
-
        /* step one: get all selected tracks and all tracks in the relevant edit groups */
 
        for (TrackSelection::iterator ti = selection->tracks.begin(); ti != selection->tracks.end(); ++ti) {
@@ -3181,7 +2954,7 @@ Editor::mapover_audio_tracks (slot<void,AudioTimeAxisView&,uint32_t> sl)
                        continue;
                }
 
-               RouteGroup* group = atv->route().edit_group();
+               RouteGroup* group = atv->route()->edit_group();
 
                if (group && group->is_active()) {
                        
@@ -3193,7 +2966,7 @@ Editor::mapover_audio_tracks (slot<void,AudioTimeAxisView&,uint32_t> sl)
                                
                                if ((tatv = dynamic_cast<AudioTimeAxisView*> (*i)) != 0) {
                                        
-                                       if (tatv->route().edit_group() == group) {
+                                       if (tatv->route()->edit_group() == group) {
                                                relevant_tracks.insert (tatv);
                                        }
                                }
@@ -3203,12 +2976,22 @@ Editor::mapover_audio_tracks (slot<void,AudioTimeAxisView&,uint32_t> sl)
 
                        /* no active group, or no group */
 
-                       relevant_tracks.insert (atv);
+                       relevant_tracks.insert (&base);
                }
 
        }
+}
+
+void
+Editor::mapover_audio_tracks (slot<void,AudioTimeAxisView&,uint32_t> sl)
+{
+       set<AudioTimeAxisView*> relevant_tracks;
+
+       if (!clicked_audio_trackview) {
+               return;
+       }
 
-       /* step two: apply operation to each track */
+       get_relevant_audio_tracks (*clicked_audio_trackview, relevant_tracks);
 
        uint32_t sz = relevant_tracks.size();
        
@@ -3218,128 +3001,251 @@ Editor::mapover_audio_tracks (slot<void,AudioTimeAxisView&,uint32_t> sl)
 }
 
 void
-Editor::mapped_set_selected_regionview_from_click (AudioTimeAxisView& atv, uint32_t ignored, 
-                                                 AudioRegionView* basis, vector<AudioRegionView*>* all_equivs)
+Editor::mapped_set_selected_regionview_from_click (RouteTimeAxisView& tv, uint32_t ignored, 
+                                                  RegionView* basis, vector<RegionView*>* all_equivs)
 {
-       AudioPlaylist* pl;
-       vector<AudioRegion*> results;
-       AudioRegionView* marv;
-       DiskStream* ds;
-       
-       if ((ds = atv.get_diskstream()) == 0) {
+       Playlist* pl;
+       vector<boost::shared_ptr<Region> > results;
+       RegionView* marv;
+       boost::shared_ptr<Diskstream> ds;
+
+       if ((ds = tv.get_diskstream()) == 0) {
                /* bus */
                return;
        }
+
+       if (&tv == &basis->get_time_axis_view()) {
+               /* looking in same track as the original */
+               return;
+       }
+
        
-       if ((pl = ds->playlist()) != 0) {
-               pl->get_equivalent_regions (basis->region, results);
+       if ((pl = dynamic_cast<Playlist*>(ds->playlist())) != 0) {
+               pl->get_equivalent_regions (basis->region(), results);
        }
        
-       for (vector<AudioRegion*>::iterator ir = results.begin(); ir != results.end(); ++ir) {
-               if ((marv = atv.view->find_view (**ir)) != 0) {
+       for (vector<boost::shared_ptr<Region> >::iterator ir = results.begin(); ir != results.end(); ++ir) {
+               if ((marv = tv.view()->find_view (*ir)) != 0) {
                        all_equivs->push_back (marv);
                }
        }
 }
 
-void
-Editor::set_selected_regionview_from_click (Selection::Operation op, bool no_track_remove)
+bool
+Editor::set_selected_regionview_from_click (bool press, Selection::Operation op, bool no_track_remove)
 {
-       cerr << "In SSRfC\n";
+       vector<RegionView*> all_equivalent_regions;
+       bool commit = false;
 
-       vector<AudioRegionView*> all_equivalent_regions;
-
-       if (!clicked_regionview) {
-               return;
+       if (!clicked_regionview || !clicked_audio_trackview) {
+               return false;
        }
 
-       mapover_audio_tracks (bind (mem_fun (*this, &Editor::mapped_set_selected_regionview_from_click), 
-                                   clicked_regionview, &all_equivalent_regions));
-       
+       if (op == Selection::Toggle || op == Selection::Set) {
+               
+               mapover_audio_tracks (bind (mem_fun (*this, &Editor::mapped_set_selected_regionview_from_click), 
+                                           clicked_regionview, &all_equivalent_regions));
+               
+               
+               /* add clicked regionview since we skipped all other regions in the same track as the one it was in */
+               
+               all_equivalent_regions.push_back (clicked_regionview);
+               
+               switch (op) {
+               case Selection::Toggle:
+                       
+                       if (clicked_regionview->get_selected()) {
+                               if (press) {
 
-       cerr << "mapover done\n";
+                                       /* whatever was clicked was selected already; do nothing here but allow
+                                          the button release to deselect it
+                                       */
 
-       begin_reversible_command (_("set selected regionview"));
+                                       button_release_can_deselect = true;
+
+                               } else {
+
+                                       if (button_release_can_deselect) {
+
+                                               /* just remove this one region, but only on a permitted button release */
+
+                                               selection->remove (clicked_regionview);
+                                               commit = true;
+
+                                               /* no more deselect action on button release till a new press
+                                                  finds an already selected object.
+                                               */
+
+                                               button_release_can_deselect = false;
+                                       }
+                               } 
 
-       switch (op) {
-       case Selection::Toggle:
-               selection->toggle (clicked_regionview);
-#if 0
-               if (clicked_regionview->get_selected()) {
-                       if (/* group && group->is_active() && */ selection->audio_regions.size() > 1) {
-                               /* reduce selection down to just the one clicked */
-                               selection->set (clicked_regionview);
                        } else {
-                               selection->remove (clicked_regionview);
+
+                               if (press) {
+                                       /* add all the equivalent regions, but only on button press */
+                                       
+                                       if (!all_equivalent_regions.empty()) {
+                                               commit = true;
+                                       }
+                                       
+                                       for (vector<RegionView*>::iterator i = all_equivalent_regions.begin(); i != all_equivalent_regions.end(); ++i) {
+                                               selection->add (*i);
+                                       }
+                               } 
                        }
-               } else {
-                       selection->add (all_equivalent_regions);
+                       break;
+                       
+               case Selection::Set:
+                       if (!clicked_regionview->get_selected()) {
+                               selection->set (all_equivalent_regions);
+                               commit = true;
+                       } else {
+                               /* no commit necessary: clicked on an already selected region */
+                               goto out;
+                       }
+                       break;
+
+               default:
+                       /* silly compiler */
+                       break;
                }
-#endif
-               set_selected_track_from_click (op, false, no_track_remove);
-               break;
 
-       case Selection::Set:
-               // karsten wiese suggested these two lines to make
-               // a selected region rise to the top. but this
-               // leads to a mismatch between actual layering
-               // and visual layering. resolution required ....
-               //
-               // gnome_canvas_item_raise_to_top (clicked_regionview->get_canvas_group());
-               // gnome_canvas_item_raise_to_top (clicked_regionview->get_time_axis_view().canvas_display);
-
-               if (clicked_regionview->get_selected()) {
-                       /* no commit necessary: we are the one selected. */
-                       return;
+       } else if (op == Selection::Extend) {
 
-               } else {
-                       
-                       selection->set (all_equivalent_regions);
-                       set_selected_track_from_click (op, false, false);
+               list<Selectable*> results;
+               jack_nframes_t last_frame;
+               jack_nframes_t first_frame;
+
+               /* 1. find the last selected regionview in the track that was clicked in */
+
+               last_frame = 0;
+               first_frame = max_frames;
+
+               for (RegionSelection::iterator x = selection->regions.begin(); x != selection->regions.end(); ++x) {
+                       if (&(*x)->get_time_axis_view() == &clicked_regionview->get_time_axis_view()) {
+
+                               if ((*x)->region()->last_frame() > last_frame) {
+                                       last_frame = (*x)->region()->last_frame();
+                               }
+
+                               if ((*x)->region()->first_frame() < first_frame) {
+                                       first_frame = (*x)->region()->first_frame();
+                               }
+                       }
                }
-               break;
 
-       case Selection::Extend:
-               /* not defined yet */
-               break;
+               /* 2. figure out the boundaries for our search for new objects */
+
+               switch (clicked_regionview->region()->coverage (first_frame, last_frame)) {
+               case OverlapNone:
+                       cerr << "no overlap, first = " << first_frame << " last = " << last_frame << " region = " 
+                            << clicked_regionview->region()->first_frame() << " .. " << clicked_regionview->region()->last_frame() << endl;
+
+                       if (last_frame < clicked_regionview->region()->first_frame()) {
+                               first_frame = last_frame;
+                               last_frame = clicked_regionview->region()->last_frame();
+                       } else {
+                               last_frame = first_frame;
+                               first_frame = clicked_regionview->region()->first_frame();
+                       }
+                       break;
+
+               case OverlapExternal:
+                       cerr << "external overlap, first = " << first_frame << " last = " << last_frame << " region = " 
+                            << clicked_regionview->region()->first_frame() << " .. " << clicked_regionview->region()->last_frame() << endl;
+
+                       if (last_frame < clicked_regionview->region()->first_frame()) {
+                               first_frame = last_frame;
+                               last_frame = clicked_regionview->region()->last_frame();
+                       } else {
+                               last_frame = first_frame;
+                               first_frame = clicked_regionview->region()->first_frame();
+                       }
+                       break;
+
+               case OverlapInternal:
+                       cerr << "internal overlap, first = " << first_frame << " last = " << last_frame << " region = " 
+                            << clicked_regionview->region()->first_frame() << " .. " << clicked_regionview->region()->last_frame() << endl;
+
+                       if (last_frame < clicked_regionview->region()->first_frame()) {
+                               first_frame = last_frame;
+                               last_frame = clicked_regionview->region()->last_frame();
+                       } else {
+                               last_frame = first_frame;
+                               first_frame = clicked_regionview->region()->first_frame();
+                       }
+                       break;
+
+               case OverlapStart:
+               case OverlapEnd:
+                       /* nothing to do except add clicked region to selection, since it
+                          overlaps with the existing selection in this track.
+                       */
+                       break;
+               }
+
+               /* 2. find all selectable objects (regionviews in this case) between that one and the end of the
+                     one that was clicked.
+               */
+
+               set<AudioTimeAxisView*> relevant_tracks;
+               
+               get_relevant_audio_tracks (*clicked_audio_trackview, relevant_tracks);
+               
+               for (set<AudioTimeAxisView*>::iterator t = relevant_tracks.begin(); t != relevant_tracks.end(); ++t) {
+                       (*t)->get_selectables (first_frame, last_frame, -1.0, -1.0, results);
+               }
+               
+               /* 3. convert to a vector of audio regions */
+
+               vector<RegionView*> regions;
+               
+               for (list<Selectable*>::iterator x = results.begin(); x != results.end(); ++x) {
+                       RegionView* arv;
+
+                       if ((arv = dynamic_cast<RegionView*>(*x)) != 0) {
+                               regions.push_back (arv);
+                       }
+               }
+
+               if (!regions.empty()) {
+                       selection->add (regions);
+                       commit = true;
+               }
        }
-       cerr << "case done\n";
 
-       commit_reversible_command () ;
+  out:
+       return commit;
 }
 
 void
-Editor::set_selected_regionview_from_region_list (Region& r, Selection::Operation op)
+Editor::set_selected_regionview_from_region_list (boost::shared_ptr<Region> region, Selection::Operation op)
 {
-       vector<AudioRegionView*> all_equivalent_regions;
-       AudioRegion* region;
-
-       if ((region = dynamic_cast<AudioRegion*>(&r)) == 0) {
-               return;
-       }
+       vector<RegionView*> all_equivalent_regions;
 
        for (TrackViewList::iterator i = track_views.begin(); i != track_views.end(); ++i) {
                
-               AudioTimeAxisView* tatv;
+               RouteTimeAxisView* tatv;
                
-               if ((tatv = dynamic_cast<AudioTimeAxisView*> (*i)) != 0) {
+               if ((tatv = dynamic_cast<RouteTimeAxisView*> (*i)) != 0) {
                        
-                       AudioPlaylist* pl;
-                       vector<AudioRegion*> results;
-                       AudioRegionView* marv;
-                       DiskStream* ds;
+                       Playlist* pl;
+                       vector<boost::shared_ptr<Region> > results;
+                       RegionView* marv;
+                       boost::shared_ptr<Diskstream> ds;
                        
                        if ((ds = tatv->get_diskstream()) == 0) {
                                /* bus */
                                continue;
                        }
 
-                       if ((pl = ds->playlist()) != 0) {
-                               pl->get_region_list_equivalent_regions (*region, results);
+                       if ((pl = dynamic_cast<Playlist*>(ds->playlist())) != 0) {
+                               pl->get_region_list_equivalent_regions (region, results);
                        }
                        
-                       for (vector<AudioRegion*>::iterator ir = results.begin(); ir != results.end(); ++ir) {
-                               if ((marv = tatv->view->find_view (**ir)) != 0) {
+                       for (vector<boost::shared_ptr<Region> >::iterator ir = results.begin(); ir != results.end(); ++ir) {
+                               if ((marv = tatv->view()->find_view (*ir)) != 0) {
                                        all_equivalent_regions.push_back (marv);
                                }
                        }
@@ -3352,13 +3258,13 @@ Editor::set_selected_regionview_from_region_list (Region& r, Selection::Operatio
        switch (op) {
        case Selection::Toggle:
                /* XXX this is not correct */
-               selection->add (all_equivalent_regions);
+               selection->toggle (all_equivalent_regions);
                break;
        case Selection::Set:
                selection->set (all_equivalent_regions);
                break;
        case Selection::Extend:
-               /* not defined yet */
+               selection->add (all_equivalent_regions);
                break;
        }
 
@@ -3366,25 +3272,25 @@ Editor::set_selected_regionview_from_region_list (Region& r, Selection::Operatio
 }
 
 bool
-Editor::set_selected_regionview_from_map_event (GdkEventAny* ev, StreamView* sv, Region* r)
+Editor::set_selected_regionview_from_map_event (GdkEventAny* ev, StreamView* sv, boost::shared_ptr<Region> r)
 {
-       AudioRegionView* rv;
-       AudioRegion* ar;
+       RegionView* rv;
+       boost::shared_ptr<AudioRegion> ar;
 
-       if ((ar = dynamic_cast<AudioRegion*> (r)) == 0) {
-               return TRUE;
+       if ((ar = boost::dynamic_pointer_cast<AudioRegion> (r)) == 0) {
+               return true;
        }
 
-       if ((rv = sv->find_view (*ar)) == 0) {
-               return TRUE;
+       if ((rv = sv->find_view (ar)) == 0) {
+               return true;
        }
 
        /* don't reset the selection if its something other than 
           a single other region.
        */
 
-       if (selection->audio_regions.size() > 1) {
-               return TRUE;
+       if (selection->regions.size() > 1) {
+               return true;
        }
        
        begin_reversible_command (_("set selected regions"));
@@ -3393,7 +3299,7 @@ Editor::set_selected_regionview_from_map_event (GdkEventAny* ev, StreamView* sv,
 
        commit_reversible_command () ;
 
-       return TRUE;
+       return true;
 }
 
 void
@@ -3548,7 +3454,7 @@ Editor::duplicate_dialog (bool dup_region)
 
        if (sscanf (text.c_str(), "%f", &times) == 1) {
                if (dup_region) {
-                       AudioRegionSelection regions;
+                       RegionSelection regions;
                        regions.add (clicked_regionview);
                        duplicate_some_regions (regions, times);
                } else {
@@ -3599,9 +3505,9 @@ Editor::edit_mode_selection_done ()
        string choice = edit_mode_selector.get_active_text();
        EditMode mode = Slide;
 
-       if (choice == _("Splice")) {
+       if (choice == _("Splice Edit")) {
                mode = Splice;
-       } else if (choice == _("Slide")) {
+       } else if (choice == _("Slide Edit")) {
                mode = Slide;
        }
 
@@ -3617,18 +3523,18 @@ Editor::snap_type_selection_done ()
 
        string choice = snap_type_selector.get_active_text();
        SnapType snaptype = SnapToFrame;
-       
+
        if (choice == _("Beats/3")) {
-                snaptype = SnapToAThirdBeat;
-        } else if (choice == _("Beats/4")) {
-                snaptype = SnapToAQuarterBeat;
-        } else if (choice == _("Beats/8")) {
-                snaptype = SnapToAEighthBeat;
-        } else if (choice == _("Beats/16")) {
-                snaptype = SnapToASixteenthBeat;
-        } else if (choice == _("Beats/32")) {
-                snaptype = SnapToAThirtysecondBeat;
-        } else if (choice == _("Beats")) {
+               snaptype = SnapToAThirdBeat;
+       } else if (choice == _("Beats/4")) {
+               snaptype = SnapToAQuarterBeat;
+       } else if (choice == _("Beats/8")) {
+               snaptype = SnapToAEighthBeat;
+       } else if (choice == _("Beats/16")) {
+               snaptype = SnapToASixteenthBeat;
+       } else if (choice == _("Beats/32")) {
+               snaptype = SnapToAThirtysecondBeat;
+       } else if (choice == _("Beats")) {
                snaptype = SnapToBeat;
        } else if (choice == _("Bars")) {
                snaptype = SnapToBar;
@@ -3656,10 +3562,10 @@ Editor::snap_type_selection_done ()
                snaptype = SnapToSeconds;
        } else if (choice == _("Minutes")) {
                snaptype = SnapToMinutes;
-        } else if (choice == _("None")) {
+       } else if (choice == _("None")) {
                snaptype = SnapToFrame;
        }
-       
+
        set_snap_to (snaptype);
 }      
 
@@ -3673,9 +3579,9 @@ Editor::snap_mode_selection_done ()
        string choice = snap_mode_selector.get_active_text();
        SnapMode mode = SnapNormal;
 
-       if (choice == _("Normal")) {
+       if (choice == _("Normal Snap")) {
                mode = SnapNormal;
-       } else if (choice == _("Magnetic")) {
+       } else if (choice == _("Magnetic Snap")) {
                mode = SnapMagnetic;
        }
 
@@ -3692,15 +3598,15 @@ Editor::zoom_focus_selection_done ()
        string choice = zoom_focus_selector.get_active_text();
        ZoomFocus focus_type = ZoomFocusLeft;
 
-       if (choice == _("Left")) {
+       if (choice == _("Focus Left")) {
                focus_type = ZoomFocusLeft;
-       } else if (choice == _("Right")) {
+       } else if (choice == _("Focus Right")) {
                focus_type = ZoomFocusRight;
-       } else if (choice == _("Center")) {
+       } else if (choice == _("Focus Center")) {
                focus_type = ZoomFocusCenter;
-        } else if (choice == _("Playhead")) {
+       } else if (choice == _("Focus Playhead")) {
                focus_type = ZoomFocusPlayhead;
-        } else if (choice == _("Edit Cursor")) {
+       } else if (choice == _("Focus Edit Cursor")) {
                focus_type = ZoomFocusEdit;
        } 
 
@@ -3770,7 +3676,7 @@ void
 Editor::region_selection_changed ()
 {
        for (TrackViewList::iterator i = track_views.begin(); i != track_views.end(); ++i) {
-               (*i)->set_selected_regionviews (selection->audio_regions);
+               (*i)->set_selected_regionviews (selection->regions);
        }
 }
 
@@ -3835,10 +3741,12 @@ Editor::get_valid_views (TimeAxisView* track, RouteGroup* group)
 void
 Editor::set_zoom_focus (ZoomFocus f)
 {
+       vector<string> txt = internationalize (zoom_focus_strings);
+       zoom_focus_selector.set_active_text (txt[(int)f]);
+       
        if (zoom_focus != f) {
                zoom_focus = f;
-               vector<string> txt = internationalize (zoom_focus_strings);
-               zoom_focus_selector.set_active_text (txt[(int)f]);
+
                ZoomFocusChanged (); /* EMIT_SIGNAL */
 
                instant_save ();
@@ -4040,8 +3948,8 @@ Editor::playlist_deletion_dialog (Playlist* pl)
 bool
 Editor::audio_region_selection_covers (jack_nframes_t where)
 {
-       for (AudioRegionSelection::iterator a = selection->audio_regions.begin(); a != selection->audio_regions.end(); ++a) {
-               if ((*a)->region.covers (where)) {
+       for (RegionSelection::iterator a = selection->regions.begin(); a != selection->regions.end(); ++a) {
+               if ((*a)->region()->covers (where)) {
                        return true;
                }
        }
@@ -4052,10 +3960,10 @@ Editor::audio_region_selection_covers (jack_nframes_t where)
 void
 Editor::prepare_for_cleanup ()
 {
-       cut_buffer->clear_audio_regions ();
+       cut_buffer->clear_regions ();
        cut_buffer->clear_playlists ();
 
-       selection->clear_audio_regions ();
+       selection->clear_regions ();
        selection->clear_playlists ();
 }