Merge with 2.0-ongoing R2885.
[ardour.git] / gtk2_ardour / editor.cc
index bdb8e841cb6f503528465063ac52beb6fd5c0779..41261b7079b868813b56a7aad44f64f3b627e67a 100644 (file)
@@ -1,5 +1,5 @@
 /*
-    Copyright (C) 2000-2006 Paul Davis 
+    Copyright (C) 2000-2007 Paul Davis 
 
     This program is free software; you can redistribute it and/or modify
     it under the terms of the GNU General Public License as published by
     along with this program; if not, write to the Free Software
     Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
 
-    $Id$
 */
 
+/* Note: public Editor methods are documented in public_editor.h */
+
 #include <unistd.h>
 #include <cstdlib>
 #include <cmath>
 #include <string>
 #include <algorithm>
 
+#include <boost/none.hpp>
+
 #include <sigc++/bind.h>
 
+#include <pbd/convert.h>
 #include <pbd/error.h>
+#include <pbd/enumwriter.h>
+#include <pbd/memento_command.h>
 
+#include <glibmm/miscutils.h>
 #include <gtkmm/image.h>
 #include <gdkmm/color.h>
 #include <gdkmm/bitmap.h>
 
+#include <gtkmm2ext/grouped_buttons.h>
 #include <gtkmm2ext/gtk_ui.h>
 #include <gtkmm2ext/tearoff.h>
 #include <gtkmm2ext/utils.h>
+#include <gtkmm2ext/window_title.h>
+#include <gtkmm2ext/choice.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>
 #include <ardour/audioregion.h>
-#include <ardour/region.h>
+#include <ardour/midi_region.h>
 #include <ardour/session_route.h>
+#include <ardour/session_directory.h>
+#include <ardour/session_state_utils.h>
 #include <ardour/tempo.h>
 #include <ardour/utils.h>
+#include <ardour/profile.h>
+
+#include <control_protocol/control_protocol.h>
 
 #include "ardour_ui.h"
 #include "editor.h"
-#include "grouped_buttons.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 "audio_time_axis.h"
 #include "utils.h"
 #include "crossfade_view.h"
 #include "editing.h"
 #include "public_editor.h"
 #include "crossfade_edit.h"
-#include "audio_time_axis.h"
 #include "canvas_impl.h"
 #include "actions.h"
+#include "sfdb_ui.h"
 #include "gui_thread.h"
 
 #ifdef FFT_ANALYSIS
 
 #include "i18n.h"
 
-/* <CMT Additions> */
+#ifdef WITH_CMT
 #include "imageframe_socket_handler.h"
-/* </CMT Additions> */
+#endif
 
 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;
 
 #include "editor_xpms"
 
-static const int32_t slide_index = 0;
-static const int32_t splice_index = 1;
-
-static const gchar *edit_mode_strings[] = {
-       N_("Slide"),
-       N_("Splice"),
-       0
-};
-
-static const gchar *snap_type_strings[] = {
-       N_("None"),
+static const gchar *_snap_type_strings[] = {
        N_("CD Frames"),
        N_("SMPTE Frames"),
        N_("SMPTE Seconds"),
@@ -115,7 +124,6 @@ static const gchar *snap_type_strings[] = {
        N_("Beats"),
        N_("Bars"),
        N_("Marks"),
-       N_("Edit Cursor"),
        N_("Region starts"),
        N_("Region ends"),
        N_("Region syncs"),
@@ -123,20 +131,41 @@ static const gchar *snap_type_strings[] = {
        0
 };
 
-static const gchar *snap_mode_strings[] = {
-       N_("Normal"),
+static const gchar *_snap_mode_strings[] = {
+       N_("No Grid"),
+       N_("Grid"),
        N_("Magnetic"),
        0
 };
 
-static const gchar *zoom_focus_strings[] = {
+static const gchar *_edit_point_strings[] = {
+       N_("Playhead"),
+       N_("Marker"),
+       N_("Mouse"),
+       0
+};
+
+static const gchar *_zoom_focus_strings[] = {
        N_("Left"),
        N_("Right"),
        N_("Center"),
        N_("Playhead"),
-       N_("Edit Cursor"),
+       N_("Mouse"),
+       N_("Edit Point"),
+       0
+};
+
+#ifdef USE_RUBBERBAND
+static const gchar *_rb_opt_strings[] = {
+       N_("Mushy"),
+       N_("Smooth"),
+       N_("Balanced multitimbral mixture"),
+       N_("Unpitched percussion with stable notes"),
+       N_("Crisp monophonic instrumental"),
+       N_("Unpitched solo percussion"),
        0
 };
+#endif
 
 /* Soundfile  drag-n-drop */
 
@@ -148,8 +177,12 @@ Gdk::Cursor* Editor::zoom_cursor = 0;
 Gdk::Cursor* Editor::time_fx_cursor = 0;
 Gdk::Cursor* Editor::fader_cursor = 0;
 Gdk::Cursor* Editor::speaker_cursor = 0;
+Gdk::Cursor* Editor::midi_pencil_cursor = 0;
+Gdk::Cursor* Editor::midi_select_cursor = 0;
+Gdk::Cursor* Editor::midi_erase_cursor = 0;
 Gdk::Cursor* Editor::wait_cursor = 0;
 Gdk::Cursor* Editor::timebar_cursor = 0;
+Gdk::Cursor* Editor::transparent_cursor = 0;
 
 void
 show_me_the_size (Requisition* r, const char* what)
@@ -157,64 +190,51 @@ show_me_the_size (Requisition* r, const char* what)
        cerr << "size of " << what << " = " << r->width << " x " << r->height << endl;
 }
 
-Editor::Editor (AudioEngine& eng) 
-       : engine (eng),
-
+Editor::Editor ()
+       : 
          /* time display buttons */
 
          minsec_label (_("Mins:Secs")),
          bbt_label (_("Bars:Beats")),
-         smpte_label (_("SMPTE")),
-         frame_label (_("Frames")),
+         smpte_label (_("Timecode")),
+         frame_label (_("Samples")),
          tempo_label (_("Tempo")),
          meter_label (_("Meter")),
          mark_label (_("Location Markers")),
          range_mark_label (_("Range Markers")),
          transport_mark_label (_("Loop/Punch Ranges")),
+         cd_mark_label (_("CD Markers")),
 
-         edit_packer (3, 3, false),
+         edit_packer (3, 3, true),
 
          /* the values here don't matter: layout widgets
             reset them as needed.
          */
 
-         vertical_adjustment (0.0, 0.0, 400.0, 10),
-         horizontal_adjustment (0.0, 0.0, 1200.0, 20),
+         vertical_adjustment (0.0, 0.0, 10.0, 400.0),
+         horizontal_adjustment (0.0, 0.0, 20.0, 1200.0),
+
+         tempo_lines(0),
+         marker_tempo_lines(0),
 
          /* 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),
+         edit_point_clock (X_("editpoint"), false, X_("EditPointClock"), true),
+         zoom_range_clock (X_("zoomrange"), false, 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> */
+#ifdef WITH_CMT
          image_socket_listener(0),
-         /* </CMT Additions> */
+#endif
 
          /* nudge */
 
-         nudge_label (_("Nudge")),
-         nudge_clock (X_("NudgeClock"), true, true)
+         nudge_clock (X_("nudge"), false, X_("NudgeClock"), true, true),
+         meters_running(false)
 
 {
        constructed = false;
@@ -225,36 +245,41 @@ Editor::Editor (AudioEngine& eng)
 
        session = 0;
 
-       selection = new Selection;
-       cut_buffer = new Selection;
+       selection = new Selection (this);
+       cut_buffer = new Selection (this);
 
        selection->TimeChanged.connect (mem_fun(*this, &Editor::time_selection_changed));
        selection->TracksChanged.connect (mem_fun(*this, &Editor::track_selection_changed));
        selection->RegionsChanged.connect (mem_fun(*this, &Editor::region_selection_changed));
        selection->PointsChanged.connect (mem_fun(*this, &Editor::point_selection_changed));
+       selection->MarkersChanged.connect (mem_fun(*this, &Editor::marker_selection_changed));
 
        clicked_regionview = 0;
-       clicked_trackview = 0;
-       clicked_audio_trackview = 0;
+       clicked_axisview = 0;
+       clicked_routeview = 0;
        clicked_crossfadeview = 0;
        clicked_control_point = 0;
-       latest_regionview = 0;
        last_update_frame = 0;
        drag_info.item = 0;
-       last_audition_region = 0;
        current_mixer_strip = 0;
        current_bbt_points = 0;
-
-       snap_type = SnapToFrame;
-       set_snap_to (snap_type);
-       snap_mode = SnapNormal;
-       set_snap_mode (snap_mode);
+       
+       snap_type_strings =  I18N (_snap_type_strings);
+       snap_mode_strings =  I18N (_snap_mode_strings);
+       zoom_focus_strings = I18N (_zoom_focus_strings);
+       edit_point_strings = I18N (_edit_point_strings);
+#ifdef USE_RUBBERBAND
+       rb_opt_strings = I18N (_rb_opt_strings);
+#endif
+       
        snap_threshold = 5.0;
        bbt_beat_subdivision = 4;
        canvas_width = 0;
        canvas_height = 0;
+       autoscroll_active = false;
        autoscroll_timeout_tag = -1;
        interthread_progress_window = 0;
+       logo_item = 0;
 
 #ifdef FFT_ANALYSIS
        analysis_window = 0;
@@ -266,68 +291,89 @@ Editor::Editor (AudioEngine& eng)
        _show_waveforms_recording = true;
        first_action_message = 0;
        export_dialog = 0;
+       export_range_markers_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;
        route_removal = false;
-       track_spacing = 0;
        show_automatic_regions_in_region_list = true;
-       region_list_sort_type = (Editing::RegionListSortType) 0; 
+
+       region_list_sort_type = (Editing::RegionListSortType) 0;
        have_pending_keyboard_selection = false;
        _follow_playhead = true;
        _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;
        marker_menu = 0;
+       start_end_marker_menu = 0;
+       range_marker_menu = 0;
        marker_menu_item = 0;
        tm_marker_menu = 0;
        transport_marker_menu = 0;
        new_transport_marker_menu = 0;
        editor_mixer_strip_width = Wide;
-       repos_zoom_queued = false;
+       show_editor_mixer_when_tracks_arrive = false;
+       region_edit_menu_split_multichannel_item = 0;
        region_edit_menu_split_item = 0;
        temp_location = 0;
-       region_edit_menu_split_multichannel_item = 0;
        leftmost_frame = 0;
        ignore_mouse_mode_toggle = false;
+       ignore_midi_edit_mode_toggle = false;
        current_stepping_trackview = 0;
        entered_track = 0;
        entered_regionview = 0;
+       entered_marker = 0;
        clear_entered_track = false;
        _new_regionviews_show_envelope = false;
-       current_timestretch = 0;
+       current_timefx = 0;
        in_edit_group_row_change = false;
-
-       edit_cursor = 0;
+       last_canvas_frame = 0;
        playhead_cursor = 0;
+       button_release_can_deselect = true;
+       canvas_idle_queued = false;
+       _dragging_playhead = false;
+       _dragging_edit_point = false;
+       _dragging_hscrollbar = false;
+       select_new_marker = false;
+       zoomed_to_region = false;
+
+       scrubbing_direction = 0;
 
-       location_marker_color = color_map[cLocationMarker];
-       location_range_color = color_map[cLocationRange];
-       location_cd_marker_color = color_map[cLocationCDMarker];
-       location_loop_color = color_map[cLocationLoop];
-       location_punch_color = color_map[cLocationPunch];
+       sfbrowser = 0;
+       ignore_route_order_sync = false;
+
+       location_marker_color = ARDOUR_UI::config()->canvasvar_LocationMarker.get();
+       location_range_color = ARDOUR_UI::config()->canvasvar_LocationRange.get();
+       location_cd_marker_color = ARDOUR_UI::config()->canvasvar_LocationCDMarker.get();
+       location_loop_color = ARDOUR_UI::config()->canvasvar_LocationLoop.get();
+       location_punch_color = ARDOUR_UI::config()->canvasvar_LocationPunch.get();
 
        range_marker_drag_rect = 0;
        marker_drag_line = 0;
-       
+       tempo_map_change_idle_handler_id = -1;
+       canvas_hroizontally_scrolled_handler_id = -1;
+       set_midi_edit_mode (MidiEditPencil, true);
        set_mouse_mode (MouseObject, true);
 
-       frames_per_unit = 2048; /* too early to use set_frames_per_unit */
+       last_visual_state.frames_per_unit = 0;
+
+       frames_per_unit = 2048; /* too early to use reset_zoom () */
+       reset_hscrollbar_stepping ();
+       
        zoom_focus = ZoomFocusLeft;
+       set_zoom_focus (ZoomFocusLeft);
        zoom_range_clock.ValueChanged.connect (mem_fun(*this, &Editor::zoom_adjustment_changed));
 
        initialize_rulers ();
        initialize_canvas ();
 
        edit_controls_vbox.set_spacing (0);
-       horizontal_adjustment.signal_value_changed().connect (mem_fun(*this, &Editor::canvas_horizontally_scrolled));
-       vertical_adjustment.signal_value_changed().connect (mem_fun(*this, &Editor::tie_vertical_scrolling));
+       horizontal_adjustment.signal_value_changed().connect (mem_fun(*this, &Editor::canvas_horizontally_scrolled), false);
+       vertical_adjustment.signal_value_changed().connect (mem_fun(*this, &Editor::tie_vertical_scrolling), true);
        
        track_canvas.set_hadjustment (horizontal_adjustment);
        track_canvas.set_vadjustment (vertical_adjustment);
@@ -348,21 +394,32 @@ Editor::Editor (AudioEngine& eng)
        edit_vscrollbar.set_adjustment (vertical_adjustment);
        edit_hscrollbar.set_adjustment (horizontal_adjustment);
 
-       edit_hscrollbar.signal_button_press_event().connect (mem_fun(*this, &Editor::hscrollbar_button_press));
-       edit_hscrollbar.signal_button_release_event().connect (mem_fun(*this, &Editor::hscrollbar_button_release));
+       edit_hscrollbar.signal_button_press_event().connect (mem_fun(*this, &Editor::hscrollbar_button_press), false);
+       edit_hscrollbar.signal_button_release_event().connect (mem_fun(*this, &Editor::hscrollbar_button_release), false);
        edit_hscrollbar.signal_size_allocate().connect (mem_fun(*this, &Editor::hscrollbar_allocate));
 
+       edit_hscrollbar.set_name ("EditorHScrollbar");
+
        build_cursors ();
        setup_toolbar ();
+       setup_midi_toolbar ();
 
-       edit_cursor_clock.ValueChanged.connect (mem_fun(*this, &Editor::edit_cursor_clock_changed));
+       edit_point_clock.ValueChanged.connect (mem_fun(*this, &Editor::edit_point_clock_changed));
        
+       ArdourCanvas::Canvas* time_pad = manage(new ArdourCanvas::Canvas());
+       ArdourCanvas::SimpleLine* pad_line_1 = manage(new ArdourCanvas::SimpleLine(*time_pad->root(),
+                       0.0, 1.0, 100.0, 1.0));
+       pad_line_1->property_color_rgba() = 0xFF0000FF;
+       pad_line_1->show();
+       time_pad->show();
+
+       time_canvas_vbox.pack_start (*_ruler_separator, false, false);
        time_canvas_vbox.pack_start (*minsec_ruler, false, false);
        time_canvas_vbox.pack_start (*smpte_ruler, false, false);
        time_canvas_vbox.pack_start (*frames_ruler, false, false);
        time_canvas_vbox.pack_start (*bbt_ruler, false, false);
        time_canvas_vbox.pack_start (time_canvas, true, true);
-       time_canvas_vbox.set_size_request (-1, (int)(timebar_height * visible_timebars));
+       time_canvas_vbox.set_size_request (-1, (int)(timebar_height * visible_timebars) + 5);
 
        bbt_label.set_name ("EditorTimeButton");
        bbt_label.set_size_request (-1, (int)timebar_height);
@@ -392,6 +449,10 @@ Editor::Editor (AudioEngine& eng)
        mark_label.set_size_request (-1, (int)timebar_height);
        mark_label.set_alignment (1.0, 0.5);
        mark_label.set_padding (5,0);
+       cd_mark_label.set_name ("EditorTimeButton");
+       cd_mark_label.set_size_request (-1, (int)timebar_height);
+       cd_mark_label.set_alignment (1.0, 0.5);
+       cd_mark_label.set_padding (5,0);
        range_mark_label.set_name ("EditorTimeButton");
        range_mark_label.set_size_request (-1, (int)timebar_height);
        range_mark_label.set_alignment (1.0, 0.5);
@@ -410,11 +471,16 @@ Editor::Editor (AudioEngine& eng)
        time_button_vbox.pack_start (mark_label, false, false);
 
        time_button_event_box.add (time_button_vbox);
+       time_button_event_box.set_name ("TimebarLabelBase");
+       time_button_frame.set_shadow_type (Gtk::SHADOW_NONE);
        
        time_button_event_box.set_events (Gdk::BUTTON_PRESS_MASK|Gdk::BUTTON_RELEASE_MASK);
-       time_button_event_box.set_name ("TimebarLabelBase");
        time_button_event_box.signal_button_release_event().connect (mem_fun(*this, &Editor::ruler_label_button_release));
 
+       time_button_frame.add (time_button_event_box);
+       time_button_frame.set_name ("TimebarLabelBase");
+       time_button_frame.set_shadow_type (Gtk::SHADOW_ETCHED_OUT);
+
        /* these enable us to have a dedicated window (for cursor setting, etc.) 
           for the canvas areas.
        */
@@ -430,51 +496,23 @@ Editor::Editor (AudioEngine& eng)
        edit_packer.set_border_width (0);
        edit_packer.set_name ("EditorWindow");
        
-       edit_packer.attach (edit_vscrollbar,         0, 1, 1, 3,    FILL,        FILL|EXPAND, 0, 0);
+       edit_packer.attach (edit_vscrollbar,         3, 4, 1, 2,    FILL,        FILL|EXPAND, 0, 0);
 
-       edit_packer.attach (time_button_event_box,   1, 2, 0, 1,    FILL,        FILL, 0, 0);
-       edit_packer.attach (time_canvas_event_box,   2, 3, 0, 1,    FILL|EXPAND, FILL, 0, 0);
+       edit_packer.attach (time_button_frame,       0, 2, 0, 1,    FILL,        SHRINK, 0, 0);
+       edit_packer.attach (time_canvas_event_box,   2, 4, 0, 1,    FILL|EXPAND, FILL, 0, 0);
 
        edit_packer.attach (controls_layout,         1, 2, 1, 2,    FILL,        FILL|EXPAND, 0, 0);
        edit_packer.attach (track_canvas_event_box,  2, 3, 1, 2,    FILL|EXPAND, FILL|EXPAND, 0, 0);
 
+       edit_packer.attach (zoom_box,                1, 2, 2, 3,    FILL,         FILL, 0, 0);
        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);
        route_list_display.set_model (route_display_model);
-       route_list_display.append_column (_("Visible"), route_display_columns.visible);
+       route_list_display.append_column (_("Show"), route_display_columns.visible);
        route_list_display.append_column (_("Name"), route_display_columns.text);
        route_list_display.get_column (0)->set_data (X_("colnum"), GUINT_TO_POINTER(0));
        route_list_display.get_column (1)->set_data (X_("colnum"), GUINT_TO_POINTER(1));
@@ -487,7 +525,7 @@ Editor::Editor (AudioEngine& eng)
        CellRendererToggle* route_list_visible_cell = dynamic_cast<CellRendererToggle*>(route_list_display.get_column_cell_renderer (0));
        route_list_visible_cell->property_activatable() = true;
        route_list_visible_cell->property_radio() = false;
-       
+
        route_display_model->signal_row_deleted().connect (mem_fun (*this, &Editor::route_list_delete));
        route_display_model->signal_row_changed().connect (mem_fun (*this, &Editor::route_list_change));
 
@@ -500,10 +538,13 @@ Editor::Editor (AudioEngine& eng)
        edit_group_display.set_model (group_model);
        edit_group_display.append_column (_("Name"), group_columns.text);
        edit_group_display.append_column (_("Active"), group_columns.is_active);
-       edit_group_display.append_column (_("Visible"), group_columns.is_visible);
+       edit_group_display.append_column (_("Show"), group_columns.is_visible);
        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 */
@@ -522,12 +563,11 @@ Editor::Editor (AudioEngine& eng)
        active_cell->property_activatable() = true;
        active_cell->property_radio() = false;
 
-       edit_group_display.set_name ("EditGroupList");
-
        group_model->signal_row_changed().connect (mem_fun (*this, &Editor::edit_group_row_change));
 
        edit_group_display.set_name ("EditGroupList");
        edit_group_display.get_selection()->set_mode (SELECTION_SINGLE);
+       edit_group_display.set_headers_visible (true);
        edit_group_display.set_reorderable (false);
        edit_group_display.set_rules_hint (true);
        edit_group_display.set_size_request (75, -1);
@@ -538,7 +578,7 @@ Editor::Editor (AudioEngine& eng)
        edit_group_display.signal_button_press_event().connect (mem_fun(*this, &Editor::edit_group_list_button_press_event), false);
 
        VBox* edit_group_display_packer = manage (new VBox());
-       HButtonBox* edit_group_display_button_box = manage (new HButtonBox());
+       HBox* edit_group_display_button_box = manage (new HBox());
        edit_group_display_button_box->set_homogeneous (true);
 
        Button* edit_group_add_button = manage (new Button ());
@@ -574,6 +614,10 @@ Editor::Editor (AudioEngine& eng)
        region_list_display.append_column (_("Regions"), region_list_columns.name);
        region_list_display.set_headers_visible (false);
 
+       CellRendererText* region_name_cell = dynamic_cast<CellRendererText*>(region_list_display.get_column_cell_renderer (0));
+       region_name_cell->property_editable() = true;
+       region_name_cell->signal_edited().connect (mem_fun (*this, &Editor::region_name_edit));
+
        region_list_display.get_selection()->set_select_function (mem_fun (*this, &Editor::region_list_selection_filter));
        
        TreeViewColumn* tv_col = region_list_display.get_column(0);
@@ -613,11 +657,12 @@ Editor::Editor (AudioEngine& eng)
        named_selection_display.append_column (_("Chunks"), named_selection_columns.text);
        named_selection_display.set_headers_visible (false);
        named_selection_display.set_size_request (100, -1);
-       named_selection_display.set_name ("RegionListDisplay");
+       named_selection_display.set_name ("NamedSelectionDisplay");
        
        named_selection_display.get_selection()->set_mode (SELECTION_SINGLE);
        named_selection_display.set_size_request (100, -1);
-       named_selection_display.signal_button_press_event().connect (mem_fun(*this, &Editor::named_selection_display_button_press), false);
+       named_selection_display.signal_button_release_event().connect (mem_fun(*this, &Editor::named_selection_display_button_release), false);
+       named_selection_display.signal_key_release_event().connect (mem_fun(*this, &Editor::named_selection_display_key_release), false);
        named_selection_display.get_selection()->signal_changed().connect (mem_fun (*this, &Editor::named_selection_display_selection_changed));
 
        /* SNAPSHOTS */
@@ -625,7 +670,7 @@ Editor::Editor (AudioEngine& eng)
        snapshot_display_model = ListStore::create (snapshot_display_columns);
        snapshot_display.set_model (snapshot_display_model);
        snapshot_display.append_column (X_("snapshot"), snapshot_display_columns.visible_name);
-       snapshot_display.set_name ("SnapshotDisplayList");
+       snapshot_display.set_name ("SnapshotDisplay");
        snapshot_display.set_size_request (75, -1);
        snapshot_display.set_headers_visible (false);
        snapshot_display.set_reorderable (false);
@@ -639,31 +684,37 @@ Editor::Editor (AudioEngine& eng)
 
        nlabel = manage (new Label (_("Regions")));
        nlabel->set_angle (-90);
-               the_notebook.append_page (region_list_scroller, *nlabel);
+       the_notebook.append_page (region_list_scroller, *nlabel);
        nlabel = manage (new Label (_("Tracks/Busses")));
        nlabel->set_angle (-90);
-               the_notebook.append_page (route_list_scroller, *nlabel);
+       the_notebook.append_page (route_list_scroller, *nlabel);
        nlabel = manage (new Label (_("Snapshots")));
        nlabel->set_angle (-90);
        the_notebook.append_page (snapshot_display_scroller, *nlabel);
        nlabel = manage (new Label (_("Edit Groups")));
        nlabel->set_angle (-90);
        the_notebook.append_page (*edit_group_display_packer, *nlabel);
-       nlabel = manage (new Label (_("Chunks")));
-       nlabel->set_angle (-90);
-       the_notebook.append_page (named_selection_scroller, *nlabel);
+       
+       if (!Profile->get_sae()) {
+               nlabel = manage (new Label (_("Chunks")));
+               nlabel->set_angle (-90);
+               the_notebook.append_page (named_selection_scroller, *nlabel);
+       }
 
        the_notebook.set_show_tabs (true);
        the_notebook.set_scrollable (true);
        the_notebook.popup_enable ();
        the_notebook.set_tab_pos (Gtk::POS_RIGHT);
 
+       post_maximal_editor_width = 0;
+       post_maximal_pane_position = 0;
        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);
+       top_hbox.pack_start (toolbar_frame, false, true);
+       top_hbox.pack_start (midi_toolbar_frame, false, true);
 
        HBox *hbox = manage (new HBox);
        hbox->pack_start (edit_pane, true, true);
@@ -676,35 +727,69 @@ Editor::Editor (AudioEngine& eng)
        set_name ("EditorWindow");
        add_accel_group (ActionManager::ui_manager->get_accel_group());
 
+       status_bar_hpacker.show ();
+
+       vpacker.pack_end (status_bar_hpacker, false, false);
        vpacker.pack_end (global_hpacker, true, true);
 
        /* register actions now so that set_state() can find them and set toggles/checks etc */
        
        register_actions ();
-       
+
+       snap_type = SnapToBeat;
+       set_snap_to (snap_type);
+       snap_mode = SnapOff;
+       set_snap_mode (snap_mode);
+       _edit_point = EditAtMouse;
+       set_edit_point_preference (_edit_point);
+
        XMLNode* node = ARDOUR_UI::instance()->editor_settings();
        set_state (*node);
 
        _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")))));
+       nudge_forward_button.add (*(manage (new Image (::get_icon("nudge_right")))));
+       nudge_backward_button.add (*(manage (new Image (::get_icon("nudge_left")))));
 
-       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");
 
        fade_context_menu.set_name ("ArdourContextMenu");
 
-       set_title (_("ardour: editor"));
-       set_wmclass (_("ardour_editor"), "Ardour");
+       /* icons, titles, WM stuff */
+
+       list<Glib::RefPtr<Gdk::Pixbuf> > window_icons;
+       Glib::RefPtr<Gdk::Pixbuf> icon;
+
+       if ((icon = ::get_icon ("ardour_icon_16px")) != 0) {
+               window_icons.push_back (icon);
+       }
+       if ((icon = ::get_icon ("ardour_icon_22px")) != 0) {
+               window_icons.push_back (icon);
+       }
+       if ((icon = ::get_icon ("ardour_icon_32px")) != 0) {
+               window_icons.push_back (icon);
+       }
+       if ((icon = ::get_icon ("ardour_icon_48px")) != 0) {
+               window_icons.push_back (icon);
+       }
+       if (!window_icons.empty()) {
+               set_icon_list (window_icons);
+               set_default_icon_list (window_icons);
+       }
+
+       WindowTitle title(Glib::get_application_name());
+       title += _("Editor");
+       set_title (title.get_string());
+       set_wmclass (X_("ardour_editor"), "Ardour");
 
        add (vpacker);
        add_events (Gdk::KEY_PRESS_MASK|Gdk::KEY_RELEASE_MASK);
@@ -712,13 +797,23 @@ Editor::Editor (AudioEngine& eng)
        signal_configure_event().connect (mem_fun (*ARDOUR_UI::instance(), &ARDOUR_UI::configure_handler));
        signal_delete_event().connect (mem_fun (*ARDOUR_UI::instance(), &ARDOUR_UI::exit_on_main_window_close));
 
+       /* allow external control surfaces/protocols to do various things */
+
+       ControlProtocol::ZoomToSession.connect (mem_fun (*this, &Editor::temporal_zoom_session));
+       ControlProtocol::ZoomIn.connect (bind (mem_fun (*this, &Editor::temporal_zoom_step), false));
+       ControlProtocol::ZoomOut.connect (bind (mem_fun (*this, &Editor::temporal_zoom_step), true));
+       ControlProtocol::ScrollTimeline.connect (mem_fun (*this, &Editor::control_scroll));
+
+       Config->ParameterChanged.connect (mem_fun (*this, &Editor::parameter_changed));
+       Route::SyncOrderKeys.connect (mem_fun (*this, &Editor::sync_order_keys));
+
        constructed = true;
        instant_save ();
 }
 
 Editor::~Editor()
 {
-       /* <CMT Additions> */
+#ifdef WITH_CMT
        if(image_socket_listener)
        {
                if(image_socket_listener->is_connected())
@@ -729,7 +824,7 @@ Editor::~Editor()
                delete image_socket_listener ;
                image_socket_listener = 0 ;
        }
-       /* </CMT Additions> */
+#endif
 }
 
 void
@@ -740,7 +835,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.
@@ -756,7 +851,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;
@@ -783,20 +878,13 @@ 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 ()
 {
-       show_all ();
-       present ();
+       show_all_children ();
+       
+       /* re-hide editor list if necessary */
+       editor_list_button_toggled ();
 
        /* now reset all audio_time_axis heights, because widgets might need
           to be re-hidden
@@ -808,124 +896,49 @@ Editor::show_window ()
                tv = (static_cast<TimeAxisView*>(*i));
                tv->reset_height ();
        }
-}
 
-void
-Editor::tie_vertical_scrolling ()
-{
-       double y1 = vertical_adjustment.get_value();
-       controls_layout.get_vadjustment()->set_value (y1);
-       playhead_cursor->set_y_axis(y1);
-       edit_cursor->set_y_axis(y1);
+       present ();
 }
 
 void
-Editor::set_frames_per_unit (double fpu)
+Editor::instant_save ()
 {
-       jack_nframes_t frames;
-
-       if (fpu == frames_per_unit) {
-               return;
-       }
-
-       if (fpu < 1.0) {
-               fpu = 1.0;
-       }
-
-       // convert fpu to frame count
-
-       frames = (jack_nframes_t) floor (fpu * canvas_width);
-       
-       /* don't allow zooms that fit more than the maximum number
-          of frames into an 800 pixel wide space.
-       */
-
-       if (max_frames / fpu < 800.0) {
+       if (!constructed || !ARDOUR_UI::instance()->session_loaded) {
                return;
        }
 
-       frames_per_unit = fpu;
-
-       if (frames != zoom_range_clock.current_duration()) {
-               zoom_range_clock.set (frames);
-       }
-
-       /* only update these if we not about to call reposition_x_origin,
-          which will do the same updates.
-       */
-       
-       if (session && !no_zoom_repos_update) {
-               horizontal_adjustment.set_upper (session->current_end_frame() / frames_per_unit);
-       }
-       
-       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) {
-                               (*i)->reshow_selection (selection->time);
-                       }
-               } else {
-                       for (TrackViewList::iterator i = track_views.begin(); i != track_views.end(); ++i) {
-                               (*i)->reshow_selection (selection->time);
-                       }
-               }
+       if (session) {
+               session->add_instant_xml(get_state());
+       } else {
+               Config->add_instant_xml(get_state());
        }
-
-       ZoomChanged (); /* EMIT_SIGNAL */
-
-       if (edit_cursor) edit_cursor->set_position (edit_cursor->current_frame);
-       if (playhead_cursor) playhead_cursor->set_position (playhead_cursor->current_frame);
-
-       instant_save ();
-
 }
 
 void
-Editor::instant_save ()
+Editor::edit_point_clock_changed()
 {
-        if (!constructed || !ARDOUR_UI::instance()->session_loaded) {
+       if (_dragging_edit_point) {
                return;
        }
 
-       if (session) {
-               session->add_instant_xml(get_state(), session->path());
-       } else {
-               Config->add_instant_xml(get_state(), get_user_ardour_path());
+       if (selection->markers.empty()) {
+               return;
        }
-}
 
-void
-Editor::reposition_x_origin (jack_nframes_t frame)
-{
-       if (frame != leftmost_frame) {
-               leftmost_frame = frame;
-               double pixel = frame_to_pixel (frame);
-               if (pixel >= horizontal_adjustment.get_upper()) {
-                       horizontal_adjustment.set_upper (frame_to_pixel (frame + (current_page_frames())));
-               }
-               horizontal_adjustment.set_value (frame/frames_per_unit);
-               XOriginChanged (); /* EMIT_SIGNAL */
-       }
-}
+       bool ignored;
+       Location* loc = find_location_from_marker (selection->markers.front(), ignored);
 
-void
-Editor::edit_cursor_clock_changed()
-{
-       if (edit_cursor->current_frame != edit_cursor_clock.current_time()) {
-               edit_cursor->set_position (edit_cursor_clock.current_time());
+       if (!loc) {
+               return;
        }
-}
 
+       loc->move_to (edit_point_clock.current_time());
+}
 
 void
 Editor::zoom_adjustment_changed ()
 {
-       if (session == 0 || no_zoom_repos_update) {
+       if (session == 0) {
                return;
        }
 
@@ -933,109 +946,92 @@ Editor::zoom_adjustment_changed ()
 
        if (fpu < 1.0) {
                fpu = 1.0;
-               zoom_range_clock.set ((jack_nframes_t) floor (fpu * canvas_width));
+               zoom_range_clock.set ((nframes_t) floor (fpu * canvas_width));
        } else if (fpu > session->current_end_frame() / canvas_width) {
                fpu = session->current_end_frame() / canvas_width;
-               zoom_range_clock.set ((jack_nframes_t) floor (fpu * canvas_width));
+               zoom_range_clock.set ((nframes_t) floor (fpu * canvas_width));
        }
        
        temporal_zoom (fpu);
 }
 
-void 
-Editor::canvas_horizontally_scrolled ()
-{
-       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();
-       }
-}
-
 void
-Editor::reposition_and_zoom (jack_nframes_t frame, double nfpu)
+Editor::control_scroll (float fraction)
 {
-       if (!repos_zoom_queued) {
-         Glib::signal_idle().connect (bind (mem_fun(*this, &Editor::deferred_reposition_and_zoom), frame, nfpu));
-               repos_zoom_queued = true;
+       ENSURE_GUI_THREAD(bind (mem_fun (*this, &Editor::control_scroll), fraction));
+
+       if (!session) {
+               return;
        }
-}
 
-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.
-       */
+       double step = fraction * current_page_frames();
 
-       no_zoom_repos_update = (frame != leftmost_frame);
+       /*
+               _control_scroll_target is an optional<T>
        
-       set_frames_per_unit (nfpu);
-       if (no_zoom_repos_update) {
-               reposition_x_origin  (frame);
+               it acts like a pointer to an nframes_t, with
+               a operator conversion to boolean to check
+               that it has a value could possibly use
+               playhead_cursor->current_frame to store the
+               value and a boolean in the class to know
+               when it's out of date
+       */
+
+       if (!_control_scroll_target) {
+               _control_scroll_target = session->transport_frame();
+               _dragging_playhead = true;
        }
-       no_zoom_repos_update = false;
-       repos_zoom_queued = false;
-       
-       return FALSE;
-}
 
-void
-Editor::on_realize ()
-{
-       Window::on_realize ();
-       Realized ();
-}
+       if ((fraction < 0.0f) && (*_control_scroll_target < (nframes_t) fabs(step))) {
+               *_control_scroll_target = 0;
+       } else if ((fraction > 0.0f) && (max_frames - *_control_scroll_target < step)) {
+               *_control_scroll_target = max_frames - (current_page_frames()*2); // allow room for slop in where the PH is on the screen
+       } else {
+               *_control_scroll_target += (nframes_t) floor (step);
+       }
 
-void
-Editor::queue_session_control_changed (Session::ControlType t)
-{
-       Gtkmm2ext::UI::instance()->call_slot (bind (mem_fun(*this, &Editor::session_control_changed), t));
-}
+       /* move visuals, we'll catch up with it later */
 
-void
-Editor::session_control_changed (Session::ControlType t)
-{
-       // right now we're only tracking some state here 
+       playhead_cursor->set_position (*_control_scroll_target);
+       UpdateAllTransportClocks (*_control_scroll_target);
+       
+       if (*_control_scroll_target > (current_page_frames() / 2)) {
+               /* try to center PH in window */
+               reset_x_origin (*_control_scroll_target - (current_page_frames()/2));
+       } else {
+               reset_x_origin (0);
+       }
 
-       switch (t) {
-       case Session::AutoLoop:
-               update_loop_range_view (true);
-               break;
-       case Session::PunchIn:
-       case Session::PunchOut:
-               update_punch_range_view (true);
-               break;
+       /*
+               Now we do a timeout to actually bring the session to the right place
+               according to the playhead. This is to avoid reading disk buffers on every
+               call to control_scroll, which is driven by ScrollTimeline and therefore
+               probably by a control surface wheel which can generate lots of events.
+       */
+       /* cancel the existing timeout */
 
-       case Session::LayeringModel:
-               update_layering_model ();
-               break;
+       control_scroll_connection.disconnect ();
 
-       default:
-               break;
-       }
-}
+       /* add the next timeout */
 
-void
-Editor::fake_handle_new_audio_region (AudioRegion *region)
-{
-       Gtkmm2ext::UI::instance()->call_slot (bind (mem_fun(*this, &Editor::handle_new_audio_region), region));
+       control_scroll_connection = Glib::signal_timeout().connect (bind (mem_fun (*this, &Editor::deferred_control_scroll), *_control_scroll_target), 250);
 }
 
-void
-Editor::fake_handle_audio_region_removed (AudioRegion *region)
+bool
+Editor::deferred_control_scroll (nframes_t target)
 {
-       Gtkmm2ext::UI::instance()->call_slot (bind (mem_fun(*this, &Editor::handle_audio_region_removed), region));
+       session->request_locate (*_control_scroll_target, session->transport_rolling());
+       // reset for next stream
+       _control_scroll_target = boost::none;
+       _dragging_playhead = false;
+       return false;
 }
 
 void
-Editor::fake_handle_new_duration ()
+Editor::on_realize ()
 {
-       Gtkmm2ext::UI::instance()->call_slot (mem_fun(*this, &Editor::handle_new_duration));
+       Window::on_realize ();
+       Realized ();
 }
 
 void
@@ -1043,20 +1039,16 @@ Editor::start_scrolling ()
 {
        scroll_connection = ARDOUR_UI::instance()->SuperRapidScreenUpdate.connect 
                (mem_fun(*this, &Editor::update_current_screen));
-
-       slower_update_connection = ARDOUR_UI::instance()->RapidScreenUpdate.connect 
-               (mem_fun(*this, &Editor::update_slower));
 }
 
 void
 Editor::stop_scrolling ()
 {
        scroll_connection.disconnect ();
-       slower_update_connection.disconnect ();
 }
 
 void
-Editor::map_position_change (jack_nframes_t frame)
+Editor::map_position_change (nframes_t frame)
 {
        ENSURE_GUI_THREAD (bind (mem_fun(*this, &Editor::map_position_change), frame));
 
@@ -1069,7 +1061,7 @@ Editor::map_position_change (jack_nframes_t frame)
 }      
 
 void
-Editor::center_screen (jack_nframes_t frame)
+Editor::center_screen (nframes_t frame)
 {
        double page = canvas_width * frames_per_unit;
 
@@ -1082,28 +1074,33 @@ Editor::center_screen (jack_nframes_t frame)
 }
 
 void
-Editor::center_screen_internal (jack_nframes_t frame, float page)
+Editor::center_screen_internal (nframes_t frame, float page)
 {
        page /= 2;
                
        if (frame > page) {
-               frame -= (jack_nframes_t) page;
+               frame -= (nframes_t) page;
        } else {
                frame = 0;
        }
 
-    reposition_x_origin (frame);
+       reset_x_origin (frame);
 }
 
 void
 Editor::handle_new_duration ()
 {
-       reset_scrolling_region ();
+       ENSURE_GUI_THREAD (mem_fun (*this, &Editor::handle_new_duration));
 
-       if (session) {
-               horizontal_adjustment.set_upper (session->current_end_frame() / frames_per_unit);
-               horizontal_adjustment.set_value (leftmost_frame/frames_per_unit);
+       nframes_t new_end = session->get_maximum_extent() + (nframes_t) floorf (current_page_frames() * 0.10f);
+                                 
+       if (new_end > last_canvas_frame) {
+               last_canvas_frame = new_end;
+               horizontal_adjustment.set_upper (last_canvas_frame / frames_per_unit);
+               reset_scrolling_region ();
        }
+
+       horizontal_adjustment.set_value (leftmost_frame/frames_per_unit);
 }
 
 void
@@ -1122,24 +1119,21 @@ Editor::update_title ()
        if (session) {
                bool dirty = session->dirty();
 
-               string wintitle = _("ardour: editor: ");
-
-               if (dirty) {
-                       wintitle += '[';
-               }
-
-               wintitle += session->name();
+               string session_name;
 
                if (session->snap_name() != session->name()) {
-                       wintitle += ':';
-                       wintitle += session->snap_name();
+                       session_name = session->snap_name();
+               } else {
+                       session_name = session->name();
                }
 
                if (dirty) {
-                       wintitle += ']';
+                       session_name = "*" + session_name;
                }
 
-               set_title (wintitle);
+               WindowTitle title(session_name);
+               title += Glib::get_application_name();
+               set_title (title.get_string());
        }
 }
 
@@ -1148,13 +1142,21 @@ Editor::connect_to_session (Session *t)
 {
        session = t;
 
+       XMLNode* node = ARDOUR_UI::instance()->editor_settings();
+       set_state (*node);
+
+       /* catch up with the playhead */
+
+       session->request_locate (playhead_cursor->current_frame);
+
        if (first_action_message) {
                first_action_message->hide();
        }
 
        update_title ();
 
-       session->going_away.connect (mem_fun(*this, &Editor::session_going_away));
+       session->GoingAway.connect (mem_fun(*this, &Editor::session_going_away));
+       session->history().Changed.connect (mem_fun (*this, &Editor::history_changed));
 
        /* 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,
@@ -1163,10 +1165,10 @@ 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->AudioRegionAdded.connect (mem_fun(*this, &Editor::fake_handle_new_audio_region)));
-       session_connections.push_back (session->AudioRegionRemoved.connect (mem_fun(*this, &Editor::fake_handle_audio_region_removed)));
-       session_connections.push_back (session->DurationChanged.connect (mem_fun(*this, &Editor::fake_handle_new_duration)));
+       session_connections.push_back (session->RouteAdded.connect (mem_fun(*this, &Editor::handle_new_route)));
+       session_connections.push_back (session->RegionAdded.connect (mem_fun(*this, &Editor::handle_new_region)));
+       session_connections.push_back (session->RegionRemoved.connect (mem_fun(*this, &Editor::handle_region_removed)));
+       session_connections.push_back (session->DurationChanged.connect (mem_fun(*this, &Editor::handle_new_duration)));
        session_connections.push_back (session->edit_group_added.connect (mem_fun(*this, &Editor::add_edit_group)));
        session_connections.push_back (session->edit_group_removed.connect (mem_fun(*this, &Editor::edit_groups_changed)));
        session_connections.push_back (session->NamedSelectionAdded.connect (mem_fun(*this, &Editor::handle_new_named_selection)));
@@ -1177,15 +1179,12 @@ Editor::connect_to_session (Session *t)
        session_connections.push_back (session->RegionHiddenChange.connect (mem_fun(*this, &Editor::region_hidden)));
 
        session_connections.push_back (session->SMPTEOffsetChanged.connect (mem_fun(*this, &Editor::update_just_smpte)));
-       session_connections.push_back (session->SMPTETypeChanged.connect (mem_fun(*this, &Editor::update_just_smpte)));
 
        session_connections.push_back (session->tempo_map().StateChanged.connect (mem_fun(*this, &Editor::tempo_map_changed)));
 
        edit_groups_changed ();
 
-       edit_cursor_clock.set_session (session);
-       selection_start_clock.set_session (session);
-       selection_end_clock.set_session (session);
+       edit_point_clock.set_session (session);
        zoom_range_clock.set_session (session);
        _playlist_selector->set_session (session);
        nudge_clock.set_session (session);
@@ -1195,16 +1194,6 @@ Editor::connect_to_session (Session *t)
                analysis_window->set_session (session);
 #endif
 
-       switch (session->get_edit_mode()) {
-       case Splice:
-               edit_mode_selector.set_active_text (edit_mode_strings[splice_index]);
-               break;
-
-       case Slide:
-               edit_mode_selector.set_active_text (edit_mode_strings[slide_index]);
-               break;
-       }
-
        Location* loc = session->locations()->auto_loop_location();
        if (loc == 0) {
                loc = new Location (0, session->current_end_frame(), _("Loop"),(Location::Flags) (Location::IsAutoLoop | Location::IsHidden));
@@ -1213,8 +1202,7 @@ Editor::connect_to_session (Session *t)
                }
                session->locations()->add (loc, false);
                session->set_auto_loop_location (loc);
-       }
-       else {
+       } else {
                // force name
                loc->set_name (_("Loop"));
        }
@@ -1227,16 +1215,13 @@ Editor::connect_to_session (Session *t)
                }
                session->locations()->add (loc, false);
                session->set_auto_punch_location (loc);
-       }
-       else {
+       } else {
                // force name
                loc->set_name (_("Punch"));
        }
 
-       update_loop_range_view (true);
-       update_punch_range_view (true);
+       Config->map_parameters (mem_fun (*this, &Editor::parameter_changed));
        
-       session->ControlChanged.connect (mem_fun(*this, &Editor::queue_session_control_changed));
        session->StateSaved.connect (mem_fun(*this, &Editor::session_state_saved));
        
        refresh_location_display ();
@@ -1246,33 +1231,11 @@ Editor::connect_to_session (Session *t)
        session->locations()->StateChanged.connect (mem_fun(*this, &Editor::refresh_location_display_s));
        session->locations()->end_location()->changed.connect (mem_fun(*this, &Editor::end_location_changed));
 
-       bool yn;
-       RefPtr<Action> act;
-
-       act = ActionManager::get_action (X_("Editor"), X_("toggle-xfades-active"));
-       if (act) {
-               RefPtr<ToggleAction> tact = RefPtr<ToggleAction>::cast_dynamic(act);
-               /* do it twice to force the change */
-               yn = session->get_crossfades_active();
-               tact->set_active (!yn);
-               tact->set_active (yn);
-       }
-
-       act = ActionManager::get_action (X_("Editor"), X_("toggle-auto-xfades"));
-       if (act) {
-               RefPtr<ToggleAction> tact = RefPtr<ToggleAction>::cast_dynamic(act);
-               /* do it twice to force the change */
-               yn = Config->get_auto_xfade ();
-               tact->set_active (!yn);
-               tact->set_active (yn);
+       if (sfbrowser) {
+               sfbrowser->set_session (session);
        }
 
-       /* xfade visibility state set from editor::set_state() */
-       
-       update_crossfade_model ();
-       update_layering_model ();
-
-       reset_scrolling_region ();
+       handle_new_duration ();
 
        redisplay_regions ();
        redisplay_named_selections ();
@@ -1284,42 +1247,27 @@ Editor::connect_to_session (Session *t)
                (static_cast<TimeAxisView*>(*i))->set_samples_per_unit (frames_per_unit);
        }
 
-       /* ::reposition_x_origin() doesn't work right here, since the old
-          position may be zero already, and it does nothing in such
-          circumstances.
-       */
-
-       leftmost_frame = 0;
-       
-       horizontal_adjustment.set_upper (session->current_end_frame() / frames_per_unit);
-       horizontal_adjustment.set_value (0);
-
        restore_ruler_visibility ();
-       tempo_map_changed (Change (0));
-
-       edit_cursor->set_position (0);
-       playhead_cursor->set_position (0);
+       //tempo_map_changed (Change (0));
+       session->tempo_map().apply_with_metrics (*this, &Editor::draw_metric_marks);
 
        start_scrolling ();
 
-       XMLNode* node = ARDOUR_UI::instance()->editor_settings();
-       set_state (*node);
-
        /* don't show master bus in a new session */
 
        if (ARDOUR_UI::instance()->session_is_new ()) {
 
-               TreeModel::Children rows = route_display_model->children();
+               TreeModel::Children rows = route_display_model->children();
                TreeModel::Children::iterator i;
        
                no_route_list_redisplay = true;
                
                for (i = rows.begin(); i != rows.end(); ++i) {
                        TimeAxisView *tv =  (*i)[route_display_columns.tv];
-                       AudioTimeAxisView *atv;
+                       RouteTimeAxisView *rtv;
                        
-                       if ((atv = dynamic_cast<AudioTimeAxisView*>(tv)) != 0) {
-                               if (atv->route().master()) {
+                       if ((rtv = dynamic_cast<RouteTimeAxisView*>(tv)) != 0) {
+                               if (rtv->route()->is_master()) {
                                        route_list_display.get_selection()->unselect (i);
                                }
                        }
@@ -1328,6 +1276,12 @@ 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);
+
+               start_updating ();
 }
 
 void
@@ -1362,6 +1316,22 @@ Editor::build_cursors ()
                mask = Bitmap::create (speaker_cursor_mask_bits, speaker_cursor_width, speaker_cursor_height);
                speaker_cursor = new Gdk::Cursor (source, mask, ffg, fbg, speaker_cursor_x_hot, speaker_cursor_y_hot);
        }
+       
+       { 
+               RefPtr<Bitmap> bits;
+               char pix[4] = { 0, 0, 0, 0 };
+               bits = Bitmap::create (pix, 2, 2);
+               Gdk::Color c;
+               transparent_cursor = new Gdk::Cursor (bits, bits, c, c, 0, 0);
+       }
+
+       { 
+               RefPtr<Bitmap> bits;
+               char pix[4] = { 0, 0, 0, 0 };
+               bits = Bitmap::create (pix, 2, 2);
+               Gdk::Color c;
+               transparent_cursor = new Gdk::Cursor (bits, bits, c, c, 0, 0);
+       }
 
        grabber_cursor = new Gdk::Cursor (HAND2);
        cross_hair_cursor = new Gdk::Cursor (CROSSHAIR);
@@ -1370,8 +1340,12 @@ Editor::build_cursors ()
        time_fx_cursor = new Gdk::Cursor (SIZING);
        wait_cursor = new Gdk::Cursor  (WATCH);
        timebar_cursor = new Gdk::Cursor(LEFT_PTR);
+       midi_pencil_cursor = new Gdk::Cursor (PENCIL);
+       midi_select_cursor = new Gdk::Cursor (CENTER_PTR);
+       midi_erase_cursor = new Gdk::Cursor (DRAPED_BOX);
 }
 
+/** Pop up a context menu for when the user clicks on a fade in or fade out */
 void
 Editor::popup_fade_context_menu (int button, int32_t time, ArdourCanvas::Item* item, ItemType item_type)
 {
@@ -1390,38 +1364,50 @@ Editor::popup_fade_context_menu (int button, int32_t time, ArdourCanvas::Item* i
        switch (item_type) {
        case FadeInItem:
        case FadeInHandleItem:
-               if (arv->region.fade_in_active()) {
-                       items.push_back (MenuElem (_("Deactivate"), bind (mem_fun (*arv, &AudioRegionView::set_fade_in_active), false)));
+               if (arv->audio_region()->fade_in_active()) {
+                       items.push_back (MenuElem (_("Deactivate"), bind (mem_fun (*this, &Editor::set_fade_in_active), false)));
                } else {
-                       items.push_back (MenuElem (_("Activate"), bind (mem_fun (*arv, &AudioRegionView::set_fade_in_active), true)));
+                       items.push_back (MenuElem (_("Activate"), bind (mem_fun (*this, &Editor::set_fade_in_active), true)));
                }
                
                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)));
+
+               if (Profile->get_sae()) {
+                       items.push_back (MenuElem (_("Linear"), bind (mem_fun (*this, &Editor::set_fade_in_shape), AudioRegion::Linear)));
+                       items.push_back (MenuElem (_("Slowest"), bind (mem_fun (*this, &Editor::set_fade_in_shape), AudioRegion::Fast)));
+               } else {
+                       items.push_back (MenuElem (_("Linear"), bind (mem_fun (*this, &Editor::set_fade_in_shape), AudioRegion::Linear)));
+                       items.push_back (MenuElem (_("Slowest"), bind (mem_fun (*this, &Editor::set_fade_in_shape), AudioRegion::Fast)));
+                       items.push_back (MenuElem (_("Slow"), bind (mem_fun (*this, &Editor::set_fade_in_shape), AudioRegion::LogB)));
+                       items.push_back (MenuElem (_("Fast"), bind (mem_fun (*this, &Editor::set_fade_in_shape), AudioRegion::LogA)));
+                       items.push_back (MenuElem (_("Fastest"), bind (mem_fun (*this, &Editor::set_fade_in_shape), AudioRegion::Slow)));
+               }
+
                break;
 
        case FadeOutItem:
        case FadeOutHandleItem:
-               if (arv->region.fade_out_active()) {
-                       items.push_back (MenuElem (_("Deactivate"), bind (mem_fun (*arv, &AudioRegionView::set_fade_out_active), false)));
+               if (arv->audio_region()->fade_out_active()) {
+                       items.push_back (MenuElem (_("Deactivate"), bind (mem_fun (*this, &Editor::set_fade_out_active), false)));
                } else {
-                       items.push_back (MenuElem (_("Activate"), bind (mem_fun (*arv, &AudioRegionView::set_fade_out_active), true)));
+                       items.push_back (MenuElem (_("Activate"), bind (mem_fun (*this, &Editor::set_fade_out_active), true)));
                }
                
                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)));
+               if (Profile->get_sae()) {
+                       items.push_back (MenuElem (_("Linear"), bind (mem_fun (*this, &Editor::set_fade_out_shape), AudioRegion::Linear)));
+                       items.push_back (MenuElem (_("Slowest"), bind (mem_fun (*this, &Editor::set_fade_out_shape), AudioRegion::Slow)));
+               } else {
+                       items.push_back (MenuElem (_("Linear"), bind (mem_fun (*this, &Editor::set_fade_out_shape), AudioRegion::Linear)));
+                       items.push_back (MenuElem (_("Slowest"), bind (mem_fun (*this, &Editor::set_fade_out_shape), AudioRegion::Slow)));
+                       items.push_back (MenuElem (_("Slow"), bind (mem_fun (*this, &Editor::set_fade_out_shape), AudioRegion::LogA)));
+                       items.push_back (MenuElem (_("Fast"), bind (mem_fun (*this, &Editor::set_fade_out_shape), AudioRegion::LogB)));
+                       items.push_back (MenuElem (_("Fastest"), bind (mem_fun (*this, &Editor::set_fade_out_shape), AudioRegion::Fast)));
+               }
 
                break;
+
        default:
                fatal << _("programming error: ")
                      << X_("non-fade canvas item passed to popup_fade_context_menu()")
@@ -1433,16 +1419,16 @@ Editor::popup_fade_context_menu (int button, int32_t time, ArdourCanvas::Item* i
 }
 
 void
-Editor::popup_track_context_menu (int button, int32_t time, ItemType item_type, bool with_selection, jack_nframes_t frame)
+Editor::popup_track_context_menu (int button, int32_t time, ItemType item_type, bool with_selection, nframes_t frame)
 {
        using namespace Menu_Helpers;
-       Menu* (Editor::*build_menu_function)(jack_nframes_t);
+       Menu* (Editor::*build_menu_function)(nframes_t);
        Menu *menu;
 
        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 {
@@ -1463,7 +1449,7 @@ Editor::popup_track_context_menu (int button, int32_t time, ItemType item_type,
                break;
 
        case StreamItem:
-               if (clicked_audio_trackview->get_diskstream()) {
+               if (clicked_routeview->get_diskstream()) {
                        build_menu_function = &Editor::build_track_context_menu;
                } else {
                        build_menu_function = &Editor::build_track_bus_context_menu;
@@ -1482,25 +1468,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)) {
-                                       ActionManager::set_sensitive (ActionManager::edit_cursor_in_region_sensitive_actions, true);
+                               if (clicked_regionview && clicked_regionview->region()->covers (get_preferred_edit_position())) {
+                                       ActionManager::set_sensitive (ActionManager::edit_point_in_region_sensitive_actions, true);
                                } else {
-                                       ActionManager::set_sensitive (ActionManager::edit_cursor_in_region_sensitive_actions, false);
+                                       ActionManager::set_sensitive (ActionManager::edit_point_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;
 
@@ -1518,7 +1505,7 @@ Editor::popup_track_context_menu (int button, int32_t time, ItemType item_type,
                return;
        }
 
-       if (clicked_audio_trackview && clicked_audio_trackview->audio_track()) {
+       if (item_type != SelectionItem && clicked_routeview && clicked_routeview->audio_track()) {
 
                /* Bounce to disk */
                
@@ -1527,7 +1514,7 @@ Editor::popup_track_context_menu (int button, int32_t time, ItemType item_type,
                
                edit_items.push_back (SeparatorElem());
 
-               switch (clicked_audio_trackview->audio_track()->freeze_state()) {
+               switch (clicked_routeview->audio_track()->freeze_state()) {
                case AudioTrack::NoFreeze:
                        edit_items.push_back (MenuElem (_("Freeze"), mem_fun(*this, &Editor::freeze_route)));
                        break;
@@ -1547,7 +1534,7 @@ Editor::popup_track_context_menu (int button, int32_t time, ItemType item_type,
 }
 
 Menu*
-Editor::build_track_context_menu (jack_nframes_t ignored)
+Editor::build_track_context_menu (nframes_t ignored)
 {
        using namespace Menu_Helpers;
 
@@ -1559,7 +1546,7 @@ Editor::build_track_context_menu (jack_nframes_t ignored)
 }
 
 Menu*
-Editor::build_track_bus_context_menu (jack_nframes_t ignored)
+Editor::build_track_bus_context_menu (nframes_t ignored)
 {
        using namespace Menu_Helpers;
 
@@ -1571,22 +1558,22 @@ Editor::build_track_bus_context_menu (jack_nframes_t ignored)
 }
 
 Menu*
-Editor::build_track_region_context_menu (jack_nframes_t frame)
+Editor::build_track_region_context_menu (nframes_t frame)
 {
        using namespace Menu_Helpers;
        MenuList& edit_items  = track_region_context_menu.items();
        edit_items.clear();
 
-       AudioTimeAxisView* atv = dynamic_cast<AudioTimeAxisView*> (clicked_trackview);
+       RouteTimeAxisView* rtv = dynamic_cast<RouteTimeAxisView*> (clicked_axisview);
 
-       if (atv) {
-               DiskStream* ds;
-               Playlist* pl;
+       if (rtv) {
+               boost::shared_ptr<Diskstream> ds;
+               boost::shared_ptr<Playlist> pl;
                
-               if ((ds = atv->get_diskstream()) && ((pl = ds->playlist()))) {
-                       Playlist::RegionList* regions = pl->regions_at ((jack_nframes_t) floor ( (double)frame * ds->speed()));
+               if ((ds = rtv->get_diskstream()) && ((pl = ds->playlist()))) {
+                       Playlist::RegionList* regions = pl->regions_at ((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 (rtv->view(), (*i), edit_items);
                        }
                        delete regions;
                }
@@ -1598,20 +1585,20 @@ Editor::build_track_region_context_menu (jack_nframes_t frame)
 }
 
 Menu*
-Editor::build_track_crossfade_context_menu (jack_nframes_t frame)
+Editor::build_track_crossfade_context_menu (nframes_t frame)
 {
        using namespace Menu_Helpers;
        MenuList& edit_items  = track_crossfade_context_menu.items();
        edit_items.clear ();
 
-       AudioTimeAxisView* atv = dynamic_cast<AudioTimeAxisView*> (clicked_trackview);
+       AudioTimeAxisView* atv = dynamic_cast<AudioTimeAxisView*> (clicked_axisview);
 
        if (atv) {
-               DiskStream* ds;
-               Playlist* pl;
-               AudioPlaylist* apl;
+               boost::shared_ptr<Diskstream> ds;
+               boost::shared_ptr<Playlist> pl;
+               boost::shared_ptr<AudioPlaylist> apl;
 
-               if ((ds = atv->get_diskstream()) && ((pl = ds->playlist()) != 0) && ((apl = dynamic_cast<AudioPlaylist*> (pl)) != 0)) {
+               if ((ds = atv->get_diskstream()) && ((pl = ds->playlist()) != 0) && ((apl = boost::dynamic_pointer_cast<AudioPlaylist> (pl)) != 0)) {
 
                        Playlist::RegionList* regions = pl->regions_at (frame);
                        AudioPlaylist::Crossfades xfades;
@@ -1621,11 +1608,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;
@@ -1675,23 +1662,25 @@ Editor::analyze_range_selection()
 }
 #endif /* FFT_ANALYSIS */
 
-
-
 Menu*
-Editor::build_track_selection_context_menu (jack_nframes_t ignored)
+Editor::build_track_selection_context_menu (nframes_t ignored)
 {
        using namespace Menu_Helpers;
        MenuList& edit_items  = track_selection_context_menu.items();
        edit_items.clear ();
 
        add_selection_context_items (edit_items);
-       add_dstream_context_items (edit_items);
+       // edit_items.push_back (SeparatorElem());
+       // add_dstream_context_items (edit_items);
 
        return &track_selection_context_menu;
 }
 
+/** Add context menu items relevant to crossfades.
+ * @param edit_items List to add the items to.
+ */
 void
-Editor::add_crossfade_context_items (StreamView* view, Crossfade* xfade, Menu_Helpers::MenuList& edit_items, bool many)
+Editor::add_crossfade_context_items (AudioStreamView* view, boost::shared_ptr<Crossfade> xfade, Menu_Helpers::MenuList& edit_items, bool many)
 {
        using namespace Menu_Helpers;
        Menu     *xfade_menu = manage (new Menu);
@@ -1705,8 +1694,8 @@ Editor::add_crossfade_context_items (StreamView* view, Crossfade* xfade, Menu_He
                str = _("Unmute");
        }
 
-       items.push_back (MenuElem (str, bind (mem_fun(*this, &Editor::toggle_xfade_active), xfade)));
-       items.push_back (MenuElem (_("Edit"), bind (mem_fun(*this, &Editor::edit_xfade), xfade)));
+       items.push_back (MenuElem (str, bind (mem_fun(*this, &Editor::toggle_xfade_active), boost::weak_ptr<Crossfade> (xfade))));
+       items.push_back (MenuElem (_("Edit"), bind (mem_fun(*this, &Editor::edit_xfade), boost::weak_ptr<Crossfade> (xfade))));
 
        if (xfade->can_follow_overlap()) {
 
@@ -1720,9 +1709,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");
        }
@@ -1747,81 +1736,149 @@ Editor::xfade_edit_right_region ()
        }
 }
 
+/** Add an element to a menu, settings its sensitivity.
+ * @param m Menu to add to.
+ * @param e Element to add.
+ * @param s true to make sensitive, false to make insensitive
+ */
 void
-Editor::add_region_context_items (StreamView* sv, Region* region, Menu_Helpers::MenuList& edit_items)
+Editor::add_item_with_sensitivity (Menu_Helpers::MenuList& m, Menu_Helpers::MenuElem e, bool s) const
+{
+       m.push_back (e);
+       if (!s) {
+               m.back().set_sensitive (false);
+       }
+}
+
+void
+Editor::add_region_context_items (StreamView* 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;
+       boost::shared_ptr<MidiRegion>  mr;
 
        if (region) {
-               ar = dynamic_cast<AudioRegion*> (region);
+               ar = boost::dynamic_pointer_cast<AudioRegion> (region);
+               mr = boost::dynamic_pointer_cast<MidiRegion> (region);
        }
 
        /* when this particular menu pops up, make the relevant region 
           become selected.
        */
 
-       region_menu->signal_map_event().connect (bind (mem_fun(*this, &Editor::set_selected_regionview_from_map_event), sv, region));
+       region_menu->signal_map_event().connect (bind (mem_fun(*this, &Editor::set_selected_regionview_from_map_event), sv, boost::weak_ptr<Region>(region)));
 
+       items.push_back (MenuElem (_("Rename"), mem_fun(*this, &Editor::rename_region)));
        items.push_back (MenuElem (_("Popup region editor"), mem_fun(*this, &Editor::edit_region)));
        items.push_back (MenuElem (_("Raise to top layer"), mem_fun(*this, &Editor::raise_region_to_top)));
        items.push_back (MenuElem (_("Lower to bottom layer"), mem_fun  (*this, &Editor::lower_region_to_bottom)));
        items.push_back (SeparatorElem());
-       items.push_back (MenuElem (_("Define sync point"), mem_fun(*this, &Editor::set_region_sync_from_edit_cursor)));
+       items.push_back (MenuElem (_("Define sync point"), mem_fun(*this, &Editor::set_region_sync_from_edit_point)));
        items.push_back (MenuElem (_("Remove sync point"), mem_fun(*this, &Editor::remove_region_sync)));
        items.push_back (SeparatorElem());
 
-       items.push_back (MenuElem (_("Audition"), mem_fun(*this, &Editor::audition_selected_region)));
+       items.push_back (MenuElem (_("Audition"), mem_fun(*this, &Editor::play_selected_region)));
        items.push_back (MenuElem (_("Export"), mem_fun(*this, &Editor::export_region)));
        items.push_back (MenuElem (_("Bounce"), mem_fun(*this, &Editor::bounce_region_selection)));
 
 #ifdef FFT_ANALYSIS
-       items.push_back (MenuElem (_("Analyze region"), mem_fun(*this, &Editor::analyze_region_selection)));
+       if (ar)
+               items.push_back (MenuElem (_("Analyze region"), mem_fun(*this, &Editor::analyze_region_selection)));
 #endif
 
        items.push_back (SeparatorElem());
 
-       /* XXX hopefully this nonsense will go away with SigC++ 2.X, where the compiler
-          might be able to figure out which overloaded member function to use in
-          a bind() call ...
-       */
-
-       void (Editor::*type_A_pmf)(void (Region::*pmf)(bool), bool) = &Editor::region_selection_op;
-
-       items.push_back (MenuElem (_("Lock"), bind (mem_fun(*this, type_A_pmf), &Region::set_locked, true)));
-       items.push_back (MenuElem (_("Unlock"), bind (mem_fun(*this, type_A_pmf), &Region::set_locked, false)));
-       items.push_back (SeparatorElem());
+       sigc::connection fooc;
 
+       items.push_back (CheckMenuElem (_("Lock")));
+       CheckMenuItem* region_lock_item = static_cast<CheckMenuItem*>(&items.back());
+       fooc = region_lock_item->signal_activate().connect (mem_fun(*this, &Editor::toggle_region_lock));
+       if (region->locked()) {
+               fooc.block (true);
+               region_lock_item->set_active();
+               fooc.block (false);
+       }
+       items.push_back (CheckMenuElem (_("Mute")));
+       CheckMenuItem* region_mute_item = static_cast<CheckMenuItem*>(&items.back());
+       fooc = region_mute_item->signal_activate().connect (mem_fun(*this, &Editor::toggle_region_mute));
        if (region->muted()) {
-               items.push_back (MenuElem (_("Unmute"), bind (mem_fun(*this, type_A_pmf), &Region::set_muted, false)));
-       } else {
-               items.push_back (MenuElem (_("Mute"), bind (mem_fun(*this, type_A_pmf), &Region::set_muted, true)));
+               fooc.block (true);
+               region_mute_item->set_active();
+               fooc.block (false);
+       }
+       
+       if (!Profile->get_sae()) {
+               items.push_back (CheckMenuElem (_("Opaque")));
+               CheckMenuItem* region_opaque_item = static_cast<CheckMenuItem*>(&items.back());
+               fooc = region_opaque_item->signal_activate().connect (mem_fun(*this, &Editor::toggle_region_opaque));
+               if (region->opaque()) {
+                       fooc.block (true);
+                       region_opaque_item->set_active();
+                       fooc.block (false);
+               }
        }
-       items.push_back (SeparatorElem());
 
-       items.push_back (MenuElem (_("Original position"), mem_fun(*this, &Editor::naturalize)));
+       items.push_back (CheckMenuElem (_("Original position"), mem_fun(*this, &Editor::naturalize)));
+       if (region->at_natural_position()) {
+               items.back().set_sensitive (false);
+       }
+       
        items.push_back (SeparatorElem());
-
-
+       
        if (ar) {
+               
+               RegionView* rv = sv->find_view (ar);
+               AudioRegionView* arv = dynamic_cast<AudioRegionView*>(rv);
+               
+               if (!Profile->get_sae()) {
+                       items.push_back (MenuElem (_("Reset Envelope"), mem_fun(*this, &Editor::reset_region_gain_envelopes)));
+
+                       items.push_back (CheckMenuElem (_("Envelope Visible")));
+                       CheckMenuItem* region_envelope_visible_item = static_cast<CheckMenuItem*> (&items.back());
+                       fooc = region_envelope_visible_item->signal_activate().connect (mem_fun(*this, &Editor::toggle_gain_envelope_visibility));
+                       if (arv->envelope_visible()) {
+                               fooc.block (true);
+                               region_envelope_visible_item->set_active (true);
+                               fooc.block (false);
+                       }
+               
+                       items.push_back (CheckMenuElem (_("Envelope Active")));
+                       CheckMenuItem* region_envelope_active_item = static_cast<CheckMenuItem*> (&items.back());
+                       fooc = region_envelope_active_item->signal_activate().connect (mem_fun(*this, &Editor::toggle_gain_envelope_active));
+                       
+                       if (ar->envelope_active()) {
+                               fooc.block (true);
+                               region_envelope_active_item->set_active (true);
+                               fooc.block (false);
+                       }
 
-               items.push_back (MenuElem (_("Toggle envelope visibility"), mem_fun(*this, &Editor::toggle_gain_envelope_visibility)));
-               items.push_back (MenuElem (_("Toggle envelope active"), mem_fun(*this, &Editor::toggle_gain_envelope_active)));
-               items.push_back (SeparatorElem());
+                       items.push_back (SeparatorElem());
+               }
 
                if (ar->scale_amplitude() != 1.0f) {
                        items.push_back (MenuElem (_("DeNormalize"), mem_fun(*this, &Editor::denormalize_region)));
                } else {
                        items.push_back (MenuElem (_("Normalize"), mem_fun(*this, &Editor::normalize_region)));
                }
+
+       } else if (mr) {
+               items.push_back (MenuElem (_("Quantize"), mem_fun(*this, &Editor::quantize_region)));
+               items.push_back (SeparatorElem());
        }
+
        items.push_back (MenuElem (_("Reverse"), mem_fun(*this, &Editor::reverse_region)));
        items.push_back (SeparatorElem());
 
+       /* range related stuff */
+
+       items.push_back (MenuElem (_("Add Range Markers"), mem_fun (*this, &Editor::add_location_from_audio_region)));
+       items.push_back (MenuElem (_("Set Range Selection"), mem_fun (*this, &Editor::set_selection_from_region)));
+       items.push_back (SeparatorElem());
+                        
        /* Nudge region */
 
        Menu *nudge_menu = manage (new Menu());
@@ -1840,8 +1897,10 @@ Editor::add_region_context_items (StreamView* sv, Region* region, Menu_Helpers::
        MenuList& trim_items = trim_menu->items();
        trim_menu->set_name ("ArdourContextMenu");
        
-       trim_items.push_back (MenuElem (_("Start to edit cursor"), mem_fun(*this, &Editor::trim_region_from_edit_cursor)));
-       trim_items.push_back (MenuElem (_("Edit cursor to end"), mem_fun(*this, &Editor::trim_region_to_edit_cursor)));
+       trim_items.push_back (MenuElem (_("Start to edit point"), mem_fun(*this, &Editor::trim_region_from_edit_point)));
+       trim_items.push_back (MenuElem (_("Edit point to end"), mem_fun(*this, &Editor::trim_region_to_edit_point)));
+       trim_items.push_back (MenuElem (_("Trim To Loop"), mem_fun(*this, &Editor::trim_region_to_loop)));
+       trim_items.push_back (MenuElem (_("Trim To Punch"), mem_fun(*this, &Editor::trim_region_to_punch)));
                             
        items.push_back (MenuElem (_("Trim"), *trim_menu));
        items.push_back (SeparatorElem());
@@ -1852,12 +1911,11 @@ Editor::add_region_context_items (StreamView* sv, Region* region, Menu_Helpers::
        items.push_back (MenuElem (_("Make mono regions"), (mem_fun(*this, &Editor::split_multichannel_region))));
        region_edit_menu_split_multichannel_item = &items.back();
 
-       items.push_back (MenuElem (_("Duplicate"), (bind (mem_fun(*this, &Editor::duplicate_dialog), true))));
+       items.push_back (MenuElem (_("Duplicate"), (bind (mem_fun(*this, &Editor::duplicate_dialog), false))));
+       items.push_back (MenuElem (_("Multi-Duplicate"), (bind (mem_fun(*this, &Editor::duplicate_dialog), true))));
        items.push_back (MenuElem (_("Fill Track"), (mem_fun(*this, &Editor::region_fill_track))));
        items.push_back (SeparatorElem());
        items.push_back (MenuElem (_("Remove"), mem_fun(*this, &Editor::remove_clicked_region)));
-       items.push_back (SeparatorElem());
-       items.push_back (MenuElem (_("Destroy"), mem_fun(*this, &Editor::destroy_clicked_region)));
 
        /* OK, stick the region submenu at the top of the list, and then add
           the standard items.
@@ -1879,6 +1937,9 @@ Editor::add_region_context_items (StreamView* sv, Region* region, Menu_Helpers::
        edit_items.push_back (SeparatorElem());
 }
 
+/** Add context menu items relevant to selection ranges.
+ * @param edit_items List to add the items to.
+ */
 void
 Editor::add_selection_context_items (Menu_Helpers::MenuList& edit_items)
 {
@@ -1888,7 +1949,7 @@ Editor::add_selection_context_items (Menu_Helpers::MenuList& edit_items)
        selection_menu->set_name ("ArdourContextMenu");
 
        items.push_back (MenuElem (_("Play range"), mem_fun(*this, &Editor::play_selection)));
-       items.push_back (MenuElem (_("Loop range"), mem_fun(*this, &Editor::set_route_loop_selection)));
+       items.push_back (MenuElem (_("Loop range"), bind (mem_fun(*this, &Editor::set_loop_from_selection), true)));
 
 #ifdef FFT_ANALYSIS
        items.push_back (SeparatorElem());
@@ -1896,27 +1957,33 @@ Editor::add_selection_context_items (Menu_Helpers::MenuList& edit_items)
 #endif
        
        items.push_back (SeparatorElem());
-       items.push_back (MenuElem (_("Separate range to track"), mem_fun(*this, &Editor::separate_region_from_selection)));
-       items.push_back (MenuElem (_("Separate range to region list"), mem_fun(*this, &Editor::new_region_from_selection)));
+       items.push_back (MenuElem (_("Extend Range to End of Region"), bind (mem_fun(*this, &Editor::extend_selection_to_end_of_region), false)));
+       items.push_back (MenuElem (_("Extend Range to Start of Region"), bind (mem_fun(*this, &Editor::extend_selection_to_start_of_region), false)));
+
+       items.push_back (SeparatorElem());
+       items.push_back (MenuElem (_("Convert to region in-place"), mem_fun(*this, &Editor::separate_region_from_selection)));
+       items.push_back (MenuElem (_("Convert to region in region list"), mem_fun(*this, &Editor::new_region_from_selection)));
        
        items.push_back (SeparatorElem());
        items.push_back (MenuElem (_("Select all in range"), mem_fun(*this, &Editor::select_all_selectables_using_time_selection)));
+
+       items.push_back (SeparatorElem());
+       items.push_back (MenuElem (_("Set loop from selection"), bind (mem_fun(*this, &Editor::set_loop_from_selection), false)));
+       items.push_back (MenuElem (_("Set punch from selection"), mem_fun(*this, &Editor::set_punch_from_selection)));
+       
        items.push_back (SeparatorElem());
-       items.push_back (MenuElem (_("Set range to loop range"), mem_fun(*this, &Editor::set_selection_from_loop)));
-       items.push_back (MenuElem (_("Set range to punch range"), mem_fun(*this, &Editor::set_selection_from_punch)));
+       items.push_back (MenuElem (_("Add Range Markers"), mem_fun (*this, &Editor::add_location_from_selection)));
        items.push_back (SeparatorElem());
        items.push_back (MenuElem (_("Crop region to range"), mem_fun(*this, &Editor::crop_region_to_selection)));
        items.push_back (MenuElem (_("Fill range with region"), mem_fun(*this, &Editor::region_fill_selection)));
        items.push_back (MenuElem (_("Duplicate range"), bind (mem_fun(*this, &Editor::duplicate_dialog), false)));
-       items.push_back (MenuElem (_("Create chunk from range"), mem_fun(*this, &Editor::name_selection)));
+       items.push_back (MenuElem (_("Create chunk from range"), mem_fun(*this, &Editor::create_named_selection)));
        items.push_back (SeparatorElem());
        items.push_back (MenuElem (_("Bounce range"), mem_fun(*this, &Editor::bounce_range_selection)));
        items.push_back (MenuElem (_("Export range"), mem_fun(*this, &Editor::export_selection)));
-
-       edit_items.push_back (MenuElem (_("Range"), *selection_menu));
-       edit_items.push_back (SeparatorElem());
 }
 
+       
 void
 Editor::add_dstream_context_items (Menu_Helpers::MenuList& edit_items)
 {
@@ -1928,7 +1995,7 @@ Editor::add_dstream_context_items (Menu_Helpers::MenuList& edit_items)
        MenuList& play_items = play_menu->items();
        play_menu->set_name ("ArdourContextMenu");
        
-       play_items.push_back (MenuElem (_("Play from edit cursor")));
+       play_items.push_back (MenuElem (_("Play from edit point"), mem_fun(*this, &Editor::play_from_edit_point)));
        play_items.push_back (MenuElem (_("Play from start"), mem_fun(*this, &Editor::play_from_start)));
        play_items.push_back (MenuElem (_("Play region"), mem_fun(*this, &Editor::play_selected_region)));
        play_items.push_back (SeparatorElem());
@@ -1950,10 +2017,14 @@ Editor::add_dstream_context_items (Menu_Helpers::MenuList& edit_items)
        select_items.push_back (MenuElem (_("Set range to loop range"), mem_fun(*this, &Editor::set_selection_from_loop)));
        select_items.push_back (MenuElem (_("Set range to punch range"), mem_fun(*this, &Editor::set_selection_from_punch)));
        select_items.push_back (SeparatorElem());
-       select_items.push_back (MenuElem (_("Select all after edit cursor"), bind (mem_fun(*this, &Editor::select_all_selectables_using_cursor), edit_cursor, true)));
-       select_items.push_back (MenuElem (_("Select all before edit cursor"), bind (mem_fun(*this, &Editor::select_all_selectables_using_cursor), edit_cursor, false)));
-       select_items.push_back (MenuElem (_("Select all after playhead"), bind (mem_fun(*this, &Editor::select_all_selectables_using_cursor), playhead_cursor, true)));
-       select_items.push_back (MenuElem (_("Select all before playhead"), bind (mem_fun(*this, &Editor::select_all_selectables_using_cursor), playhead_cursor, false)));
+       select_items.push_back (MenuElem (_("Select All After Edit Point"), bind (mem_fun(*this, &Editor::select_all_selectables_using_edit), true)));
+       select_items.push_back (MenuElem (_("Select All Before Edit Point"), bind (mem_fun(*this, &Editor::select_all_selectables_using_edit), false)));
+       select_items.push_back (MenuElem (_("Select All After Playhead"), bind (mem_fun(*this, &Editor::select_all_selectables_using_cursor), playhead_cursor, true)));
+       select_items.push_back (MenuElem (_("Select All Before Playhead"), bind (mem_fun(*this, &Editor::select_all_selectables_using_cursor), playhead_cursor, false)));
+       select_items.push_back (MenuElem (_("Select All Between Playhead & Edit Point"), bind (mem_fun(*this, &Editor::select_all_selectables_between), false)));
+       select_items.push_back (MenuElem (_("Select All Within Playhead & Edit Point"), bind (mem_fun(*this, &Editor::select_all_selectables_between), true)));
+       select_items.push_back (MenuElem (_("Select Range Between Playhead & Edit Point"), mem_fun(*this, &Editor::select_range_between)));
+
        select_items.push_back (SeparatorElem());
 
        edit_items.push_back (MenuElem (_("Select"), *select_menu));
@@ -1966,8 +2037,7 @@ Editor::add_dstream_context_items (Menu_Helpers::MenuList& edit_items)
        
        cutnpaste_items.push_back (MenuElem (_("Cut"), mem_fun(*this, &Editor::cut)));
        cutnpaste_items.push_back (MenuElem (_("Copy"), mem_fun(*this, &Editor::copy)));
-       cutnpaste_items.push_back (MenuElem (_("Paste at edit cursor"), bind (mem_fun(*this, &Editor::paste), 1.0f)));
-       cutnpaste_items.push_back (MenuElem (_("Paste at mouse"), mem_fun(*this, &Editor::mouse_paste)));
+       cutnpaste_items.push_back (MenuElem (_("Paste"), bind (mem_fun(*this, &Editor::paste), 1.0f)));
 
        cutnpaste_items.push_back (SeparatorElem());
 
@@ -1994,9 +2064,9 @@ Editor::add_dstream_context_items (Menu_Helpers::MenuList& edit_items)
        
        edit_items.push_back (SeparatorElem());
        nudge_items.push_back (MenuElem (_("Nudge entire track fwd"), (bind (mem_fun(*this, &Editor::nudge_track), false, true))));
-       nudge_items.push_back (MenuElem (_("Nudge track after edit cursor fwd"), (bind (mem_fun(*this, &Editor::nudge_track), true, true))));
+       nudge_items.push_back (MenuElem (_("Nudge track after edit point fwd"), (bind (mem_fun(*this, &Editor::nudge_track), true, true))));
        nudge_items.push_back (MenuElem (_("Nudge entire track bwd"), (bind (mem_fun(*this, &Editor::nudge_track), false, false))));
-       nudge_items.push_back (MenuElem (_("Nudge track after edit cursor bwd"), (bind (mem_fun(*this, &Editor::nudge_track), true, false))));
+       nudge_items.push_back (MenuElem (_("Nudge track after edit point bwd"), (bind (mem_fun(*this, &Editor::nudge_track), true, false))));
 
        edit_items.push_back (MenuElem (_("Nudge"), *nudge_menu));
 }
@@ -2012,7 +2082,7 @@ Editor::add_bus_context_items (Menu_Helpers::MenuList& edit_items)
        MenuList& play_items = play_menu->items();
        play_menu->set_name ("ArdourContextMenu");
        
-       play_items.push_back (MenuElem (_("Play from edit cursor")));
+       play_items.push_back (MenuElem (_("Play from edit point"), mem_fun(*this, &Editor::play_from_edit_point)));
        play_items.push_back (MenuElem (_("Play from start"), mem_fun(*this, &Editor::play_from_start)));
        edit_items.push_back (MenuElem (_("Play"), *play_menu));
 
@@ -2027,8 +2097,8 @@ Editor::add_bus_context_items (Menu_Helpers::MenuList& edit_items)
        select_items.push_back (MenuElem (_("Invert selection in track"), mem_fun(*this, &Editor::invert_selection_in_track)));
        select_items.push_back (MenuElem (_("Invert selection"), mem_fun(*this, &Editor::invert_selection)));
        select_items.push_back (SeparatorElem());
-       select_items.push_back (MenuElem (_("Select all after edit cursor"), bind (mem_fun(*this, &Editor::select_all_selectables_using_cursor), edit_cursor, true)));
-       select_items.push_back (MenuElem (_("Select all before edit cursor"), bind (mem_fun(*this, &Editor::select_all_selectables_using_cursor), edit_cursor, false)));
+       select_items.push_back (MenuElem (_("Select all after edit point"), bind (mem_fun(*this, &Editor::select_all_selectables_using_edit), true)));
+       select_items.push_back (MenuElem (_("Select all before edit point"), bind (mem_fun(*this, &Editor::select_all_selectables_using_edit), false)));
        select_items.push_back (MenuElem (_("Select all after playhead"), bind (mem_fun(*this, &Editor::select_all_selectables_using_cursor), playhead_cursor, true)));
        select_items.push_back (MenuElem (_("Select all before playhead"), bind (mem_fun(*this, &Editor::select_all_selectables_using_cursor), playhead_cursor, false)));
 
@@ -2050,9 +2120,9 @@ Editor::add_bus_context_items (Menu_Helpers::MenuList& edit_items)
        
        edit_items.push_back (SeparatorElem());
        nudge_items.push_back (MenuElem (_("Nudge entire track fwd"), (bind (mem_fun(*this, &Editor::nudge_track), false, true))));
-       nudge_items.push_back (MenuElem (_("Nudge track after edit cursor fwd"), (bind (mem_fun(*this, &Editor::nudge_track), true, true))));
+       nudge_items.push_back (MenuElem (_("Nudge track after edit point fwd"), (bind (mem_fun(*this, &Editor::nudge_track), true, true))));
        nudge_items.push_back (MenuElem (_("Nudge entire track bwd"), (bind (mem_fun(*this, &Editor::nudge_track), false, false))));
-       nudge_items.push_back (MenuElem (_("Nudge track after edit cursor bwd"), (bind (mem_fun(*this, &Editor::nudge_track), true, false))));
+       nudge_items.push_back (MenuElem (_("Nudge track after edit point bwd"), (bind (mem_fun(*this, &Editor::nudge_track), true, false))));
 
        edit_items.push_back (MenuElem (_("Nudge"), *nudge_menu));
 }
@@ -2061,10 +2131,20 @@ Editor::add_bus_context_items (Menu_Helpers::MenuList& edit_items)
 
 void
 Editor::set_snap_to (SnapType st)
-{
+{      
+   unsigned int snap_ind = (unsigned int)st;
        snap_type = st;
-       vector<string> txt = internationalize (snap_type_strings);
-       snap_type_selector.set_active_text (txt[(int)st]);
+
+   if ( snap_ind > snap_type_strings.size() - 1 ) {
+      snap_ind = 0;
+      snap_type = (SnapType)snap_ind;
+   }
+
+       string str = snap_type_strings[snap_ind];
+
+       if (str != snap_type_selector.get_active_text()) {
+               snap_type_selector.set_active_text (str);
+       }
 
        instant_save ();
 
@@ -2074,7 +2154,9 @@ Editor::set_snap_to (SnapType st)
        case SnapToAEighthBeat:
        case SnapToAQuarterBeat:
        case SnapToAThirdBeat:
+                compute_bbt_ruler_scale (leftmost_frame, leftmost_frame + (nframes_t)(canvas_width * frames_per_unit));
                 update_tempo_based_rulers ();
+               break;
        default:
                /* relax */
                break;
@@ -2085,49 +2167,56 @@ void
 Editor::set_snap_mode (SnapMode mode)
 {
        snap_mode = mode;
-       vector<string> txt = internationalize (snap_mode_strings);
-       snap_mode_selector.set_active_text (txt[(int)mode]);
+       string str = snap_mode_strings[(int)mode];
+
+       if (str != snap_mode_selector.get_active_text ()) {
+               snap_mode_selector.set_active_text (str);
+       }
 
        instant_save ();
 }
-
 void
-Editor::add_location_from_selection ()
+Editor::set_edit_point_preference (EditPoint ep)
 {
-       if (selection->time.empty()) {
-               return;
+       bool changed = _edit_point != ep;
+
+       _edit_point = ep;
+       string str = edit_point_strings[(int)ep];
+
+       if (str != edit_point_selector.get_active_text ()) {
+               edit_point_selector.set_active_text (str);
        }
 
-       if (session == 0 || clicked_trackview == 0) {
+       if (!changed) {
                return;
        }
 
-       jack_nframes_t start = selection->time[clicked_selection].start;
-       jack_nframes_t end = selection->time[clicked_selection].end;
-
-       Location *location = new Location (start, end, "selection");
-
-       session->begin_reversible_command (_("add marker"));
-       session->add_undo (session->locations()->get_memento());
-       session->locations()->add (location, true);
-       session->add_redo_no_execute (session->locations()->get_memento());
-       session->commit_reversible_command ();
-}
-
-void
-Editor::add_location_from_playhead_cursor ()
-{
-       jack_nframes_t where = session->audible_frame();
-       
-       Location *location = new Location (where, where, "mark", Location::IsMark);
-       session->begin_reversible_command (_("add marker"));
-       session->add_undo (session->locations()->get_memento());
-       session->locations()->add (location, true);
-       session->add_redo_no_execute (session->locations()->get_memento());
-       session->commit_reversible_command ();
+       if (Profile->get_sae()) {
+
+               switch (zoom_focus) {
+               case ZoomFocusMouse:
+               case ZoomFocusPlayhead:
+               case ZoomFocusEdit:
+                       switch (_edit_point) {
+                       case EditAtMouse:
+                               set_zoom_focus (ZoomFocusMouse);
+                               break;
+                       case EditAtPlayhead:
+                               set_zoom_focus (ZoomFocusPlayhead);
+                               break;
+                       case EditAtSelectedMarker:
+                               set_zoom_focus (ZoomFocusEdit);
+                               break;
+                       }
+                       break;
+               default:
+                       break;
+               }
+       } 
+                                               
+       instant_save ();
 }
 
-
 int
 Editor::set_state (const XMLNode& node)
 {
@@ -2136,6 +2225,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) {
 
@@ -2156,16 +2248,34 @@ 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);
 
+       if (session && (prop = node.property ("playhead"))) {
+               nframes_t pos = atol (prop->value().c_str());
+               playhead_cursor->set_position (pos);
+       } else {
+               playhead_cursor->set_position (0);
+
+               /* reset_x_origin() doesn't work right here, since the old
+                  position may be zero already, and it does nothing in such
+                  circumstances.
+               */
+               
+               leftmost_frame = 0;
+               horizontal_adjustment.set_value (0);
+       }
+
+       if ((prop = node.property ("mixer-width"))) {
+               editor_mixer_strip_width = Width (string_2_enum (prop->value(), editor_mixer_strip_width));
+       }
+
        if ((prop = node.property ("zoom-focus"))) {
                set_zoom_focus ((ZoomFocus) atoi (prop->value()));
        }
 
        if ((prop = node.property ("zoom"))) {
-               set_frames_per_unit (atof (prop->value()));
+               reset_zoom (PBD::atof (prop->value()));
        }
 
        if ((prop = node.property ("snap-to"))) {
@@ -2176,6 +2286,10 @@ Editor::set_state (const XMLNode& node)
                set_snap_mode ((SnapMode) atoi (prop->value()));
        }
 
+       if ((prop = node.property ("edit-point"))) {
+               set_edit_point_preference ((EditPoint) string_2_enum (prop->value(), _edit_point));
+       }
+
        if ((prop = node.property ("mouse-mode"))) {
                MouseMode m = str2mousemode(prop->value());
                mouse_mode = MouseMode ((int) m + 1); /* lie, force mode switch */
@@ -2223,16 +2337,16 @@ Editor::set_state (const XMLNode& node)
 
        if ((prop = node.property ("follow-playhead"))) {
                bool yn = (prop->value() == "yes");
+               set_follow_playhead (yn);
                RefPtr<Action> act = ActionManager::get_action (X_("Editor"), X_("toggle-follow-playhead"));
                if (act) {
                        RefPtr<ToggleAction> tact = RefPtr<ToggleAction>::cast_dynamic(act);
-                       /* do it twice to force the change */
-                       tact->set_active (!yn);
-                       tact->set_active (yn);
+                       if (tact->get_active() != yn) {
+                               tact->set_active (yn);
+                       }
                }
        }
 
-
        if ((prop = node.property ("region-list-sort-type"))) {
                region_list_sort_type = (Editing::RegionListSortType) -1; // force change 
                reset_region_list_sort_type(str2regionlistsorttype(prop->value()));
@@ -2248,6 +2362,23 @@ Editor::set_state (const XMLNode& node)
 
                Glib::RefPtr<Action> act = ActionManager::get_action (X_("Editor"), X_("show-editor-mixer"));
                if (act) {
+
+                       Glib::RefPtr<ToggleAction> tact = Glib::RefPtr<ToggleAction>::cast_dynamic(act);
+                       bool yn = (prop->value() == X_("yes"));
+
+                       /* do it twice to force the change */
+                       
+                       tact->set_active (!yn);
+                       tact->set_active (yn);
+               }
+       }
+       
+       if ((prop = node.property ("show-editor-list"))) {
+
+               Glib::RefPtr<Action> act = ActionManager::get_action (X_("Editor"), X_("show-editor-list"));
+               assert(act);
+               if (act) {
+
                        Glib::RefPtr<ToggleAction> tact = Glib::RefPtr<ToggleAction>::cast_dynamic(act);
                        bool yn = (prop->value() == X_("yes"));
 
@@ -2258,6 +2389,7 @@ Editor::set_state (const XMLNode& node)
                }
        }
 
+
        return 0;
 }
 
@@ -2267,8 +2399,11 @@ Editor::get_state ()
        XMLNode* node = new XMLNode ("Editor");
        char buf[32];
 
+       _id.print (buf, sizeof (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);
@@ -2276,7 +2411,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);
@@ -2295,6 +2430,8 @@ Editor::get_state ()
                node->add_child_nocopy (*geometry);
        }
 
+       maybe_add_mixer_strip_width (*node);
+       
        snprintf (buf, sizeof(buf), "%d", (int) zoom_focus);
        node->add_property ("zoom-focus", buf);
        snprintf (buf, sizeof(buf), "%f", frames_per_unit);
@@ -2304,6 +2441,11 @@ Editor::get_state ()
        snprintf (buf, sizeof(buf), "%d", (int) snap_mode);
        node->add_property ("snap-mode", buf);
 
+       node->add_property ("edit-point", enum_2_string (_edit_point));
+
+       snprintf (buf, sizeof (buf), "%" PRIu32, playhead_cursor->current_frame);
+       node->add_property ("playhead", buf);
+
        node->add_property ("show-waveforms", _show_waveforms ? "yes" : "no");
        node->add_property ("show-waveforms-recording", _show_waveforms_recording ? "yes" : "no");
        node->add_property ("show-measures", _show_measures ? "yes" : "no");
@@ -2317,6 +2459,12 @@ Editor::get_state ()
                Glib::RefPtr<ToggleAction> tact = Glib::RefPtr<ToggleAction>::cast_dynamic(act);
                node->add_property (X_("show-editor-mixer"), tact->get_active() ? "yes" : "no");
        }
+       
+       act = ActionManager::get_action (X_("Editor"), X_("show-editor-list"));
+       if (act) {
+               Glib::RefPtr<ToggleAction> tact = Glib::RefPtr<ToggleAction>::cast_dynamic(act);
+               node->add_property (X_("show-editor-list"), tact->get_active() ? "yes" : "no");
+       }
 
        return *node;
 }
@@ -2326,18 +2474,11 @@ Editor::get_state ()
 TimeAxisView *
 Editor::trackview_by_y_position (double y)
 {
-       TrackViewList::iterator iter;
-       TimeAxisView *tv;
-
-       for (iter = track_views.begin(); iter != track_views.end(); ++iter) {
-
-               tv = *iter;
+       for (TrackViewList::iterator iter = track_views.begin(); iter != track_views.end(); ++iter) {
 
-               if (tv->hidden()) {
-                       continue;
-               }
+               TimeAxisView *tv;
 
-               if (tv->y_position <= y && y < ((tv->y_position + tv->height + track_spacing))) {
+               if ((tv = (*iter)->covers_y_position (y)) != 0) {
                        return tv;
                }
        }
@@ -2346,36 +2487,35 @@ Editor::trackview_by_y_position (double y)
 }
 
 void
-Editor::snap_to (jack_nframes_t& start, int32_t direction, bool for_mark)
+Editor::snap_to (nframes64_t& start, int32_t direction, bool for_mark)
 {
        Location* before = 0;
        Location* after = 0;
 
-       if (!session) {
+       if (!session || snap_mode == SnapOff) {
                return;
        }
 
-       const jack_nframes_t one_second = session->frame_rate();
-       const jack_nframes_t one_minute = session->frame_rate() * 60;
-
-       jack_nframes_t presnap = start;
+       const nframes64_t one_second = session->frame_rate();
+       const nframes64_t one_minute = session->frame_rate() * 60;
+       const nframes64_t one_smpte_second = (nframes64_t)(rint(session->smpte_frames_per_second()) * session->frames_per_smpte_frame());
+       nframes64_t one_smpte_minute = (nframes64_t)(rint(session->smpte_frames_per_second()) * session->frames_per_smpte_frame() * 60);
+       nframes64_t presnap = start;
 
        switch (snap_type) {
-       case SnapToFrame:
-               break;
-
        case SnapToCDFrame:
                if (direction) {
-                       start = (jack_nframes_t) ceil ((double) start / (one_second / 75)) * (one_second / 75);
+                       start = (nframes_t) ceil ((double) start / (one_second / 75)) * (one_second / 75);
                } else {
-                       start = (jack_nframes_t) floor ((double) start / (one_second / 75)) * (one_second / 75);
+                       start = (nframes_t) floor ((double) start / (one_second / 75)) * (one_second / 75);
                }
                break;
+
        case SnapToSMPTEFrame:
-               if (direction) {
-                       start = (jack_nframes_t) (ceil ((double) start / session->frames_per_smpte_frame()) * session->frames_per_smpte_frame());
+               if (fmod((double)start, (double)session->frames_per_smpte_frame()) > (session->frames_per_smpte_frame() / 2)) {
+                       start = (nframes_t) (ceil ((double) start / session->frames_per_smpte_frame()) * session->frames_per_smpte_frame());
                } else {
-                       start = (jack_nframes_t) (floor ((double) start / session->frames_per_smpte_frame()) *  session->frames_per_smpte_frame());
+                       start = (nframes_t) (floor ((double) start / session->frames_per_smpte_frame()) *  session->frames_per_smpte_frame());
                }
                break;
 
@@ -2386,10 +2526,10 @@ Editor::snap_to (jack_nframes_t& start, int32_t direction, bool for_mark)
                } else {
                        start -= session->smpte_offset ();
                }    
-               if (direction > 0) {
-                       start = (jack_nframes_t) ceil ((double) start / one_second) * one_second;
+               if (start % one_smpte_second > one_smpte_second / 2) {
+                       start = (nframes_t) ceil ((double) start / one_smpte_second) * one_smpte_second;
                } else {
-                       start = (jack_nframes_t) floor ((double) start / one_second) * one_second;
+                       start = (nframes_t) floor ((double) start / one_smpte_second) * one_smpte_second;
                }
                
                if (session->smpte_offset_negative())
@@ -2407,10 +2547,10 @@ Editor::snap_to (jack_nframes_t& start, int32_t direction, bool for_mark)
                } else {
                        start -= session->smpte_offset ();
                }
-               if (direction) {
-                       start = (jack_nframes_t) ceil ((double) start / one_minute) * one_minute;
+               if (start % one_smpte_minute > one_smpte_minute / 2) {
+                       start = (nframes_t) ceil ((double) start / one_smpte_minute) * one_smpte_minute;
                } else {
-                       start = (jack_nframes_t) floor ((double) start / one_minute) * one_minute;
+                       start = (nframes_t) floor ((double) start / one_smpte_minute) * one_smpte_minute;
                }
                if (session->smpte_offset_negative())
                {
@@ -2421,18 +2561,18 @@ Editor::snap_to (jack_nframes_t& start, int32_t direction, bool for_mark)
                break;
                
        case SnapToSeconds:
-               if (direction) {
-                       start = (jack_nframes_t) ceil ((double) start / one_second) * one_second;
+               if (start % one_second > one_second / 2) {
+                       start = (nframes_t) ceil ((double) start / one_second) * one_second;
                } else {
-                       start = (jack_nframes_t) floor ((double) start / one_second) * one_second;
+                       start = (nframes_t) floor ((double) start / one_second) * one_second;
                }
                break;
                
        case SnapToMinutes:
-               if (direction) {
-                       start = (jack_nframes_t) ceil ((double) start / one_minute) * one_minute;
+               if (start % one_minute > one_minute / 2) {
+                       start = (nframes_t) ceil ((double) start / one_minute) * one_minute;
                } else {
-                       start = (jack_nframes_t) floor ((double) start / one_minute) * one_minute;
+                       start = (nframes_t) floor ((double) start / one_minute) * one_minute;
                }
                break;
 
@@ -2445,27 +2585,23 @@ Editor::snap_to (jack_nframes_t& start, int32_t direction, bool for_mark)
                break;
 
        case SnapToAThirtysecondBeat:
-                start = session->tempo_map().round_to_beat_subdivision (start, 32);
-                break;
+               start = session->tempo_map().round_to_beat_subdivision (start, 32);
+               break;
 
        case SnapToASixteenthBeat:
-                start = session->tempo_map().round_to_beat_subdivision (start, 16);
-                break;
+               start = session->tempo_map().round_to_beat_subdivision (start, 16);
+               break;
 
        case SnapToAEighthBeat:
-                start = session->tempo_map().round_to_beat_subdivision (start, 8);
-                break;
+               start = session->tempo_map().round_to_beat_subdivision (start, 8);
+               break;
 
        case SnapToAQuarterBeat:
-                start = session->tempo_map().round_to_beat_subdivision (start, 4);
-                break;
-
-        case SnapToAThirdBeat:
-                start = session->tempo_map().round_to_beat_subdivision (start, 3);
-                break;
+               start = session->tempo_map().round_to_beat_subdivision (start, 4);
+               break;
 
-       case SnapToEditCursor:
-               start = edit_cursor->current_frame;
+       case SnapToAThirdBeat:
+               start = session->tempo_map().round_to_beat_subdivision (start, 3);
                break;
 
        case SnapToMark:
@@ -2513,7 +2649,7 @@ Editor::snap_to (jack_nframes_t& start, int32_t direction, bool for_mark)
        case SnapToRegionSync:
        case SnapToRegionBoundary:
                if (!region_boundary_cache.empty()) {
-                       vector<jack_nframes_t>::iterator i;
+                       vector<nframes_t>::iterator i;
 
                        if (direction > 0) {
                                i = std::upper_bound (region_boundary_cache.begin(), region_boundary_cache.end(), start);
@@ -2548,39 +2684,126 @@ Editor::snap_to (jack_nframes_t& start, int32_t direction, bool for_mark)
                }
                
        default:
+               /* handled at entry */
                return;
                
        }
 }
 
+double
+Editor::snap_length_beats (nframes_t start)
+{
+       if (!session) {
+               return 1.0;
+       }
+
+       /* FIXME: This could/should also work with non-tempo based snap settings (ie seconds) */
+
+       switch (snap_type) {
+       case SnapToBar:
+               return session->tempo_map().meter_at(start).beats_per_bar();
+
+       case SnapToBeat:
+               return 1.0;
+
+       case SnapToAThirtysecondBeat:
+               return 1.0 / (double)32.0;
+               break;
+
+       case SnapToASixteenthBeat:
+               return 1.0 / (double)16.0;
+               break;
+
+       case SnapToAEighthBeat:
+               return 1.0 / (double)8.0;
+               break;
+
+       case SnapToAQuarterBeat:
+               return 1.0 / (double)4.0;
+               break;
+
+       case SnapToAThirdBeat:
+               return 1.0 / (double)3.0;
+
+       default:
+               return 1.0;
+       }
+}
+
 void
 Editor::setup_toolbar ()
 {
        string pixmap_path;
+
+#ifdef GTKOSX
+       const guint32 FUDGE = 38; // Combo's are stupid - they steal space from the entry for the button
+#else
+       const guint32 FUDGE = 18; // Combo's are stupid - they steal space from the entry for the button
+#endif
+
+       /* Mode Buttons (tool selection) */
+
        vector<ToggleButton *> mouse_mode_buttons;
 
+       mouse_move_button.add (*(manage (new Image (::get_icon("tool_object")))));
+       mouse_move_button.set_relief(Gtk::RELIEF_NONE);
        mouse_mode_buttons.push_back (&mouse_move_button);
-       mouse_mode_buttons.push_back (&mouse_select_button);
-       mouse_mode_buttons.push_back (&mouse_gain_button);
+
+       if (!Profile->get_sae()) {
+               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_icon("tool_gain")))));
+               mouse_gain_button.set_relief(Gtk::RELIEF_NONE);
+               mouse_mode_buttons.push_back (&mouse_gain_button);
+       }
+
+       mouse_zoom_button.add (*(manage (new Image (::get_icon("tool_zoom")))));
+       mouse_zoom_button.set_relief(Gtk::RELIEF_NONE);
        mouse_mode_buttons.push_back (&mouse_zoom_button);
+       mouse_timefx_button.add (*(manage (new Image (::get_icon("tool_stretch")))));
+       mouse_timefx_button.set_relief(Gtk::RELIEF_NONE);
        mouse_mode_buttons.push_back (&mouse_timefx_button);
+       mouse_audition_button.add (*(manage (new Image (::get_icon("tool_audition")))));
+       mouse_audition_button.set_relief(Gtk::RELIEF_NONE);
+       mouse_note_button.add (*(manage (new Image (::get_icon("tool_note")))));
+       mouse_note_button.set_relief(Gtk::RELIEF_NONE);
+       mouse_mode_buttons.push_back (&mouse_note_button);
        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);
+       if (!Profile->get_sae()) {
+               mouse_mode_button_box.pack_start(mouse_select_button, true, true);
+       }
+       mouse_mode_button_box.pack_start(mouse_zoom_button, true, true);
+       if (!Profile->get_sae()) {
+               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.pack_start(mouse_note_button, true, true);
+       mouse_mode_button_box.set_homogeneous(true);
+
+       vector<string> edit_mode_strings;
+       edit_mode_strings.push_back (edit_mode_to_string (Splice));
+       edit_mode_strings.push_back (edit_mode_to_string (Slide));
 
-       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, longest (edit_mode_strings).c_str(), 2+FUDGE, 10);
+       set_popdown_strings (edit_mode_selector, 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), 
@@ -2598,13 +2821,15 @@ Editor::setup_toolbar ()
        mouse_zoom_button.set_name ("MouseModeButton");
        mouse_timefx_button.set_name ("MouseModeButton");
        mouse_audition_button.set_name ("MouseModeButton");
+       mouse_note_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"));
+       ARDOUR_UI::instance()->tooltips().set_tip (mouse_note_button, _("Edit MIDI Notes"));
 
        mouse_move_button.unset_flags (CAN_FOCUS);
        mouse_select_button.unset_flags (CAN_FOCUS);
@@ -2612,6 +2837,7 @@ Editor::setup_toolbar ()
        mouse_zoom_button.unset_flags (CAN_FOCUS);
        mouse_timefx_button.unset_flags (CAN_FOCUS);
        mouse_audition_button.unset_flags (CAN_FOCUS);
+       mouse_note_button.unset_flags (CAN_FOCUS);
 
        mouse_select_button.signal_toggled().connect (bind (mem_fun(*this, &Editor::mouse_mode_toggled), Editing::MouseRange));
        mouse_select_button.signal_button_release_event().connect (mem_fun(*this, &Editor::mouse_select_button_release));
@@ -2621,164 +2847,91 @@ Editor::setup_toolbar ()
        mouse_zoom_button.signal_toggled().connect (bind (mem_fun(*this, &Editor::mouse_mode_toggled), Editing::MouseZoom));
        mouse_timefx_button.signal_toggled().connect (bind (mem_fun(*this, &Editor::mouse_mode_toggled), Editing::MouseTimeFX));
        mouse_audition_button.signal_toggled().connect (bind (mem_fun(*this, &Editor::mouse_mode_toggled), Editing::MouseAudition));
+       mouse_note_button.signal_toggled().connect (bind (mem_fun(*this, &Editor::mouse_mode_toggled), Editing::MouseNote));
 
        // 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");
-
-       edit_mode_box.set_spacing (3);
-       edit_mode_box.set_border_width (3);
+       /* Zoom */
+       
+       zoom_box.set_spacing (1);
+       zoom_box.set_border_width (0);
 
-       /* XXX another disgusting hack because of the way combo boxes size themselves */
+       zoom_in_button.set_name ("EditorTimeButton");
+       zoom_in_button.set_size_request(-1,16);
+       zoom_in_button.add (*(manage (new Image (::get_icon("zoom_in")))));
+       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.set_size_request(-1,16);
+       zoom_out_button.add (*(manage (new Image (::get_icon("zoom_out")))));
+       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"));
 
-       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_out_full_button.set_name ("EditorTimeButton");
+       zoom_out_full_button.set_size_request(-1,16);
+       zoom_out_full_button.add (*(manage (new Image (::get_icon("zoom_full")))));
+       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"));
 
-       edit_mode_selector.signal_changed().connect (mem_fun(*this, &Editor::edit_mode_selection_done));
+       zoom_focus_selector.set_name ("ZoomFocusSelector");
+       Gtkmm2ext::set_size_request_to_display_given_text (zoom_focus_selector, _("Playhead"), FUDGE, 0);
+       set_popdown_strings (zoom_focus_selector, zoom_focus_strings);
+       zoom_focus_selector.signal_changed().connect (mem_fun(*this, &Editor::zoom_focus_selection_done));
+       ARDOUR_UI::instance()->tooltips().set_tip (zoom_focus_selector, _("Zoom focus"));
 
-       /* Snap Type */
+       zoom_box.pack_start (zoom_focus_selector, true, true);
+       zoom_box.pack_start (zoom_out_button, false, false);
+       zoom_box.pack_start (zoom_in_button, false, false);
+       zoom_box.pack_start (zoom_out_full_button, false, false);
 
-       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);
-
+       Gtkmm2ext::set_size_request_to_display_given_text (snap_type_selector, _("SMPTE Seconds"), 2+FUDGE, 10);
+       set_popdown_strings (snap_type_selector, snap_type_strings);
        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, _("Snap/Grid Units"));
 
        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);
-       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);
-
+       Gtkmm2ext::set_size_request_to_display_given_text (snap_mode_selector, _("Magnetic Snap"), 2+FUDGE, 10);
+       set_popdown_strings (snap_mode_selector, snap_mode_strings);
        snap_mode_selector.signal_changed().connect (mem_fun(*this, &Editor::snap_mode_selection_done));
+       ARDOUR_UI::instance()->tooltips().set_tip (snap_mode_selector, _("Snap/Grid Mode"));
 
-       /* 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);
-
-       /* 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));
-
-       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));
-
-       /* 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");
+       edit_point_selector.set_name ("SnapModeSelector");
+       Gtkmm2ext::set_size_request_to_display_given_text (edit_point_selector, _("Playhead"), 2+FUDGE, 10);
+       set_popdown_strings (edit_point_selector, edit_point_strings);
+       edit_point_selector.signal_changed().connect (mem_fun(*this, &Editor::edit_point_selection_done));
+       ARDOUR_UI::instance()->tooltips().set_tip (edit_point_selector, _("Edit point"));
 
-       selection_start_clock_label.set_text (_("Start:"));
-       selection_end_clock_label.set_text (_("End:"));
-       edit_cursor_clock_label.set_text (_("Edit"));
+       snap_box.pack_start (edit_point_clock, false, false);
+       snap_box.pack_start (snap_mode_selector, false, false);
+       snap_box.pack_start (snap_type_selector, false, false);
+       snap_box.pack_start (edit_point_selector, false, false);
 
-       /* 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.
+       /* Nudge */
 
-          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);
+       HBox *nudge_box = manage (new HBox);
+       nudge_box->set_spacing(1);
+       nudge_box->set_border_width (2);
 
-       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_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);
 
-       nudge_label.set_name ("ToolBarLabel");
 
-       vbox->pack_start (nudge_label, false, false);
-       vbox->pack_start (*nbox, false, false);
+       /* Pack everything in... */
 
-       hbox->pack_start (*vbox, false, false);
-
-       hbox->show_all ();
+       HBox* hbox = manage (new HBox);
+       hbox->set_spacing(10);
 
-       tools_tearoff = new TearOff (*hbox);
+       tools_tearoff = manage (new TearOff (*hbox));
        tools_tearoff->set_name ("MouseModeBase");
 
        tools_tearoff->Detach.connect (bind (mem_fun(*this, &Editor::detach_tearoff), static_cast<Box*>(&toolbar_hbox), 
@@ -2790,13 +2943,20 @@ 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);
+
        
-       toolbar_base.set_name ("ToolBarBase");
+       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);
 
        toolbar_frame.set_shadow_type (SHADOW_OUT);
@@ -2804,99 +2964,88 @@ Editor::setup_toolbar ()
        toolbar_frame.add (toolbar_base);
 }
 
-gint
-Editor::_autoscroll_canvas (void *arg)
-{
-       return ((Editor *) arg)->autoscroll_canvas ();
-}
 
-gint
-Editor::autoscroll_canvas ()
+void
+Editor::setup_midi_toolbar ()
 {
-       jack_nframes_t new_frame;
-       bool keep_calling = true;
-
-       if (autoscroll_direction < 0) {
-               if (leftmost_frame < autoscroll_distance) {
-                       new_frame = 0;
-               } else {
-                       new_frame = leftmost_frame - autoscroll_distance;
-               }
-       } else {
-               if (leftmost_frame > max_frames - autoscroll_distance) {
-                       new_frame = max_frames;
-               } else {
-                       new_frame = leftmost_frame + autoscroll_distance;
-               }
-       }
-
-       if (new_frame != leftmost_frame) {
-               reposition_x_origin (new_frame);
-       }
-
-       if (new_frame == 0 || new_frame == max_frames) {
-               /* we are done */
-               return FALSE;
-       }
-
-       autoscroll_cnt++;
-
-       if (autoscroll_cnt == 1) {
-
-               /* connect the timeout so that we get called repeatedly */
-               
-               autoscroll_timeout_tag = gtk_timeout_add (100, _autoscroll_canvas, this);
-               keep_calling = false;
-
-       } else if (autoscroll_cnt > 10 && autoscroll_cnt < 20) {
-               
-               /* after about a while, speed up a bit by changing the timeout interval */
-
-               autoscroll_timeout_tag = gtk_timeout_add (50, _autoscroll_canvas, this);
-               keep_calling = false;
-               
-       } else if (autoscroll_cnt >= 20 && autoscroll_cnt < 30) {
-
-               /* after about another while, speed up some more */
-
-               autoscroll_timeout_tag = gtk_timeout_add (25, _autoscroll_canvas, this);
-               keep_calling = false;
+       string pixmap_path;
 
-       } else if (autoscroll_cnt >= 30) {
+       /* Mode Buttons (tool selection) */
 
-               /* we've been scrolling for a while ... crank it up */
+       vector<ToggleButton *> midi_tool_buttons;
 
-               autoscroll_distance = 10 * (jack_nframes_t) floor (canvas_width * frames_per_unit);
-       }
+       midi_tool_pencil_button.add (*(manage (new Image (::get_icon("midi_tool_pencil")))));
+       midi_tool_pencil_button.set_relief(Gtk::RELIEF_NONE);
+       midi_tool_buttons.push_back (&midi_tool_pencil_button);
+       midi_tool_select_button.add (*(manage (new Image (::get_icon("midi_tool_select")))));
+       midi_tool_select_button.set_relief(Gtk::RELIEF_NONE);
+       midi_tool_buttons.push_back (&midi_tool_select_button);
+       midi_tool_erase_button.add (*(manage (new Image (::get_icon("midi_tool_erase")))));
+       midi_tool_erase_button.set_relief(Gtk::RELIEF_NONE);
+       midi_tool_buttons.push_back (&midi_tool_erase_button);
 
-       return keep_calling;
-}
+       midi_tool_pencil_button.set_active(true);
+       
+       midi_tool_button_set = new GroupedButtons (midi_tool_buttons);
+
+       midi_tool_button_box.set_border_width (2);
+       midi_tool_button_box.set_spacing(4);
+       midi_tool_button_box.set_spacing(1);
+       midi_tool_button_box.pack_start(midi_tool_pencil_button, true, true);
+       midi_tool_button_box.pack_start(midi_tool_select_button, true, true);
+       midi_tool_button_box.pack_start(midi_tool_erase_button, true, true);
+       midi_tool_button_box.set_homogeneous(true);
+
+       midi_tool_pencil_button.set_name ("MouseModeButton");
+       midi_tool_select_button.set_name ("MouseModeButton");
+       midi_tool_erase_button.set_name ("MouseModeButton");
+
+       ARDOUR_UI::instance()->tooltips().set_tip (midi_tool_pencil_button, _("Add/Move/Stretch Notes"));
+       ARDOUR_UI::instance()->tooltips().set_tip (midi_tool_select_button, _("Select/Move Notes"));
+       ARDOUR_UI::instance()->tooltips().set_tip (midi_tool_erase_button, _("Erase Notes"));
+
+       midi_tool_pencil_button.unset_flags (CAN_FOCUS);
+       midi_tool_select_button.unset_flags (CAN_FOCUS);
+       midi_tool_erase_button.unset_flags (CAN_FOCUS);
+       
+       midi_tool_pencil_button.signal_toggled().connect (bind (mem_fun(*this,
+                               &Editor::midi_edit_mode_toggled), Editing::MidiEditPencil));
+       midi_tool_select_button.signal_toggled().connect (bind (mem_fun(*this,
+                               &Editor::midi_edit_mode_toggled), Editing::MidiEditSelect));
+       midi_tool_erase_button.signal_toggled().connect (bind (mem_fun(*this,
+                               &Editor::midi_edit_mode_toggled), Editing::MidiEditErase));
+       
+       /* Pack everything in... */
+
+       midi_tools_tearoff = manage (new TearOff (midi_tool_button_box));
+       midi_tools_tearoff->set_name ("MouseModeBase");
+
+       /*
+       midi_tools_tearoff->Detach.connect (bind (mem_fun(*this, &Editor::detach_tearoff), static_cast<Box*>(&midi_toolbar_hbox), 
+                                            &midi_tools_tearoff->tearoff_window()));
+       midi_tools_tearoff->Attach.connect (bind (mem_fun(*this, &Editor::reattach_tearoff), static_cast<Box*> (&midi_toolbar_hbox), 
+                                            &midi_tools_tearoff->tearoff_window(), 0));
+       midi_tools_tearoff->Hidden.connect (bind (mem_fun(*this, &Editor::detach_tearoff), static_cast<Box*>(&midi_toolbar_hbox), 
+                                            &midi_tools_tearoff->tearoff_window()));
+       midi_tools_tearoff->Visible.connect (bind (mem_fun(*this, &Editor::reattach_tearoff), static_cast<Box*> (&midi_toolbar_hbox), 
+                                             &midi_tools_tearoff->tearoff_window(), 0));
+       */
 
-void
-Editor::start_canvas_autoscroll (int dir)
-{
-       if (!session) {
-               return;
-       }
+       midi_toolbar_hbox.set_spacing (10);
+       midi_toolbar_hbox.set_border_width (1);
 
-       stop_canvas_autoscroll ();
+       midi_toolbar_hbox.pack_start (*midi_tools_tearoff, false, true);
 
-       autoscroll_direction = dir;
-       autoscroll_distance = (jack_nframes_t) floor ((canvas_width * frames_per_unit)/10.0);
-       autoscroll_cnt = 0;
-       
-       /* do it right now, which will start the repeated callbacks */
+       midi_tool_button_box.show_all ();
+       midi_toolbar_hbox.show_all();
+       midi_tools_tearoff->show_all();
        
-       autoscroll_canvas ();
-}
+       midi_toolbar_base.set_name ("ToolBarBase");
+       midi_toolbar_base.add (midi_toolbar_hbox);
 
-void
-Editor::stop_canvas_autoscroll ()
-{
-       if (autoscroll_timeout_tag >= 0) {
-               gtk_timeout_remove (autoscroll_timeout_tag);
-               autoscroll_timeout_tag = -1;
-       }
+       midi_toolbar_frame.set_shadow_type (SHADOW_OUT);
+       midi_toolbar_frame.set_name ("BaseFrame");
+       midi_toolbar_frame.add (midi_toolbar_base);
 }
 
 int
@@ -2916,7 +3065,7 @@ Editor::convert_drop_to_paths (vector<ustring>& paths,
        vector<ustring> uris = data.get_uris();
 
        if (uris.empty()) {
-               
+
                /* This is seriously fucked up. Nautilus doesn't say that its URI lists
                   are actually URI lists. So do it by hand.
                */
@@ -2966,10 +3115,34 @@ 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);
-                       paths.push_back (p.substr (7));
+                       
+                       ustring p = *i;
+                        PBD::url_decode (p);
+
+                       // scan forward past three slashes
+                       
+                       ustring::size_type slashcnt = 0;
+                       ustring::size_type n = 0;
+                       ustring::iterator x = p.begin();
+
+                       while (slashcnt < 3 && x != p.end()) {
+                               if ((*x) == '/') {
+                                       slashcnt++;
+                               } else if (slashcnt == 3) {
+                                       break;
+                               }
+                               ++n;
+                               ++x;
+                       }
+
+                       if (slashcnt != 3 || x == p.end()) {
+                               error << _("malformed URL passed to drag-n-drop code") << endmsg;
+                               continue;
+                       }
+
+                       paths.push_back (p.substr (n - 1));
                }
        }
 
@@ -2990,13 +3163,15 @@ Editor::map_transport_state ()
        if (session->transport_stopped()) {
                have_pending_keyboard_selection = false;
        }
+
+       update_loop_range_view (true);
 }
 
 /* UNDO/REDO */
 
-Editor::State::State ()
+Editor::State::State (PublicEditor const * e)
 {
-       selection = new Selection;
+       selection = new Selection (e);
 }
 
 Editor::State::~State ()
@@ -3007,7 +3182,7 @@ Editor::State::~State ()
 UndoAction
 Editor::get_memento () const
 {
-       State *state = new State;
+       State *state = new State (this);
 
        store_state (*state);
        return bind (mem_fun (*(const_cast<Editor*>(this)), &Editor::restore_state), state);
@@ -3037,8 +3212,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);
        }
 }
 
@@ -3046,545 +3221,215 @@ 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)
+Editor::set_edit_group_solo (Route& route, bool yn)
 {
-       if (!clicked_trackview) {
-               return;
-       }
-
-       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);
-                       }
-               } else {
-                       selection->toggle (clicked_trackview);
-               }
-               break;
-       case Selection::Set:
-               if (selection->selected (clicked_trackview) && selection->tracks.size() == 1) {
-                       /* no commit necessary */
-                       return;
-               } 
+       RouteGroup *edit_group;
 
-               selection->set (clicked_trackview);
-               break;
-               
-       case Selection::Extend:
-               /* not defined yet */
-               break;
-       }
-       
-       if (with_undo) {
-               commit_reversible_command ();
+       if ((edit_group = route.edit_group()) != 0) {
+               edit_group->apply (&Route::set_solo, yn, this);
+       } else {
+               route.set_solo (yn, this);
        }
 }
 
 void
-Editor::set_selected_control_point_from_click (Selection::Operation op, bool with_undo, bool no_remove)
+Editor::set_edit_group_mute (Route& route, bool yn)
 {
-       if (!clicked_control_point) {
-               return;
-       }
-
-       if (with_undo) {
-               begin_reversible_command (_("set selected control point"));
-       }
+       RouteGroup *edit_group = 0;
 
-       switch (op) {
-       case Selection::Set:
-               break;
-       case Selection::Toggle:
-               break;
-       case Selection::Extend:
-               break;
-       }
-       if (with_undo) {
-               commit_reversible_command ();
+       if ((edit_group == route.edit_group()) != 0) {
+               edit_group->apply (&Route::set_mute, yn, this);
+       } else {
+               route.set_mute (yn, this);
        }
 }
-
+               
 void
-Editor::mapover_audio_tracks (slot<void,AudioTimeAxisView&,uint32_t> sl)
+Editor::history_changed ()
 {
-       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) {
-
-               AudioTimeAxisView* atv = dynamic_cast<AudioTimeAxisView*>(*ti);
-
-               if (!atv) {
-                       continue;
-               }
-
-               RouteGroup* group = atv->route().edit_group();
-
-               if (group && group->is_active()) {
-                       
-                       /* active group for this track, loop over all tracks and get every member of the group */
-
-                       for (TrackViewList::iterator i = track_views.begin(); i != track_views.end(); ++i) {
-                               
-                               AudioTimeAxisView* tatv;
-                               
-                               if ((tatv = dynamic_cast<AudioTimeAxisView*> (*i)) != 0) {
-                                       
-                                       if (tatv->route().edit_group() == group) {
-                                               relevant_tracks.insert (tatv);
-                                       }
-                               }
-                       }
+       string label;
 
+       if (undo_action && session) {
+               if (session->undo_depth() == 0) {
+                       label = _("Undo");
                } else {
-
-                       /* no active group, or no group */
-
-                       relevant_tracks.insert (atv);
+                       label = string_compose(_("Undo (%1)"), session->next_undo());
                }
-
-       }
-
-       /* step two: apply operation to each track */
-
-       uint32_t sz = relevant_tracks.size();
-       
-       for (set<AudioTimeAxisView*>::iterator ati = relevant_tracks.begin(); ati != relevant_tracks.end(); ++ati) {
-               sl (**ati, sz);
+               undo_action->property_label() = label;
        }
-}
 
-void
-Editor::mapped_set_selected_regionview_from_click (AudioTimeAxisView& atv, uint32_t ignored, 
-                                                 AudioRegionView* basis, vector<AudioRegionView*>* all_equivs)
-{
-       AudioPlaylist* pl;
-       vector<AudioRegion*> results;
-       AudioRegionView* marv;
-       DiskStream* ds;
-       
-       if ((ds = atv.get_diskstream()) == 0) {
-               /* bus */
-               return;
-       }
-       
-       if ((pl = 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) {
-                       all_equivs->push_back (marv);
+       if (redo_action && session) {
+               if (session->redo_depth() == 0) {
+                       label = _("Redo");
+               } else {
+                       label = string_compose(_("Redo (%1)"), session->next_redo());
                }
+               redo_action->property_label() = label;
        }
 }
 
 void
-Editor::set_selected_regionview_from_click (Selection::Operation op, bool no_track_remove)
+Editor::duplicate_dialog (bool with_dialog)
 {
-       cerr << "In SSRfC\n";
+       float times = 1.0f;
 
-       vector<AudioRegionView*> all_equivalent_regions;
-
-       if (!clicked_regionview) {
-               return;
+       if (mouse_mode == MouseRange) {
+               if (selection->time.length() == 0) {
+                       return;
+               }
        }
 
-       mapover_audio_tracks (bind (mem_fun (*this, &Editor::mapped_set_selected_regionview_from_click), 
-                                   clicked_regionview, &all_equivalent_regions));
        
+       if (mouse_mode != MouseRange) {
 
-       cerr << "mapover done\n";
-
-       begin_reversible_command (_("set selected regionview"));
-
-       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);
-                       }
-               } else {
-                       selection->add (all_equivalent_regions);
-               }
-#endif
-               set_selected_track_from_click (op, false, no_track_remove);
-               break;
+               ensure_entered_region_selected (true);
 
-       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. */
+               if (selection->regions.empty()) {
                        return;
-
-               } else {
-                       
-                       selection->set (all_equivalent_regions);
-                       set_selected_track_from_click (op, false, false);
                }
-               break;
-
-       case Selection::Extend:
-               /* not defined yet */
-               break;
        }
-       cerr << "case done\n";
-
-       commit_reversible_command () ;
-}
-
-void
-Editor::set_selected_regionview_from_region_list (Region& r, Selection::Operation op)
-{
-       vector<AudioRegionView*> all_equivalent_regions;
-       AudioRegion* region;
 
-       if ((region = dynamic_cast<AudioRegion*>(&r)) == 0) {
-               return;
-       }
+       if (with_dialog) {
 
-       for (TrackViewList::iterator i = track_views.begin(); i != track_views.end(); ++i) {
+               ArdourDialog win ("duplicate dialog");
+               Label  label (_("Duplicate how many times?"));
+               Adjustment adjustment (1.0, 1.0, 1000000.0, 1.0, 5.0);
+               SpinButton spinner (adjustment);
                
-               AudioTimeAxisView* tatv;
+               win.get_vbox()->set_spacing (12);
+               win.get_vbox()->pack_start (label);
                
-               if ((tatv = dynamic_cast<AudioTimeAxisView*> (*i)) != 0) {
-                       
-                       AudioPlaylist* pl;
-                       vector<AudioRegion*> results;
-                       AudioRegionView* marv;
-                       DiskStream* ds;
-                       
-                       if ((ds = tatv->get_diskstream()) == 0) {
-                               /* bus */
-                               continue;
-                       }
-
-                       if ((pl = 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) {
-                                       all_equivalent_regions.push_back (marv);
-                               }
-                       }
-                       
+               /* dialogs have ::add_action_widget() but that puts the spinner in the wrong
+                  place, visually. so do this by hand.
+               */
+               
+               win.get_vbox()->pack_start (spinner);
+               spinner.signal_activate().connect (sigc::bind (mem_fun (win, &ArdourDialog::response), RESPONSE_ACCEPT));
+               
+               label.show ();
+               spinner.show ();
+               
+               win.add_button (Stock::OK, RESPONSE_ACCEPT);
+               win.add_button (Stock::CANCEL, RESPONSE_CANCEL);
+               
+               win.set_position (WIN_POS_MOUSE);
+               
+               spinner.grab_focus ();
+               
+               switch (win.run ()) {
+               case RESPONSE_ACCEPT:
+                       break;
+               default:
+                       return;
                }
-       }
-       
-       begin_reversible_command (_("set selected regions"));
-       
-       switch (op) {
-       case Selection::Toggle:
-               /* XXX this is not correct */
-               selection->add (all_equivalent_regions);
-               break;
-       case Selection::Set:
-               selection->set (all_equivalent_regions);
-               break;
-       case Selection::Extend:
-               /* not defined yet */
-               break;
+               
+               times = adjustment.get_value();
        }
 
-       commit_reversible_command () ;
+       if (mouse_mode == MouseRange) {
+               duplicate_selection (times);
+       } else {
+               duplicate_some_regions (selection->regions, times);
+       }
 }
 
-bool
-Editor::set_selected_regionview_from_map_event (GdkEventAny* ev, StreamView* sv, Region* r)
+void
+Editor::show_verbose_canvas_cursor ()
 {
-       AudioRegionView* rv;
-       AudioRegion* ar;
-
-       if ((ar = dynamic_cast<AudioRegion*> (r)) == 0) {
-               return TRUE;
-       }
+       verbose_canvas_cursor->raise_to_top();
+       verbose_canvas_cursor->show();
+       verbose_cursor_visible = true;
+}
 
-       if ((rv = sv->find_view (*ar)) == 0) {
-               return TRUE;
-       }
+void
+Editor::hide_verbose_canvas_cursor ()
+{
+       verbose_canvas_cursor->hide();
+       verbose_cursor_visible = false;
+}
 
-       /* don't reset the selection if its something other than 
-          a single other region.
+void
+Editor::set_verbose_canvas_cursor (const string & txt, double x, double y)
+{
+       /* XXX get origin of canvas relative to root window,
+          add x and y and check compared to gdk_screen_{width,height}
        */
-
-       if (selection->audio_regions.size() > 1) {
-               return TRUE;
-       }
-       
-       begin_reversible_command (_("set selected regions"));
-       
-       selection->set (rv);
-
-       commit_reversible_command () ;
-
-       return TRUE;
+       verbose_canvas_cursor->property_text() = txt.c_str();
+       verbose_canvas_cursor->property_x() = x;
+       verbose_canvas_cursor->property_y() = y;
 }
 
 void
-Editor::set_edit_group_solo (Route& route, bool yn)
+Editor::set_verbose_canvas_cursor_text (const string & txt)
 {
-       RouteGroup *edit_group;
-
-       if ((edit_group = route.edit_group()) != 0) {
-               edit_group->apply (&Route::set_solo, yn, this);
-       } else {
-               route.set_solo (yn, this);
-       }
+       verbose_canvas_cursor->property_text() = txt.c_str();
 }
 
 void
-Editor::set_edit_group_mute (Route& route, bool yn)
+Editor::set_edit_mode (EditMode m)
 {
-       RouteGroup *edit_group = 0;
-
-       if ((edit_group == route.edit_group()) != 0) {
-               edit_group->apply (&Route::set_mute, yn, this);
-       } else {
-               route.set_mute (yn, this);
-       }
+       Config->set_edit_mode (m);
 }
-               
+
 void
-Editor::set_edit_menu (Menu& menu)
+Editor::cycle_edit_mode ()
 {
-       edit_menu = &menu;
-       edit_menu->signal_map_event().connect (mem_fun(*this, &Editor::edit_menu_map_handler));
+       switch (Config->get_edit_mode()) {
+       case Slide:
+               Config->set_edit_mode (Splice);
+               break;
+       case Splice:
+               Config->set_edit_mode (Slide);
+               break;
+       }
 }
 
-bool
-Editor::edit_menu_map_handler (GdkEventAny* ev)
+void
+Editor::edit_mode_selection_done ()
 {
-       using namespace Menu_Helpers;
-       MenuList& edit_items = edit_menu->items();
-       string label;
-
-       /* Nuke all the old items */
-               
-       edit_items.clear ();
-
        if (session == 0) {
-               return false;
-       }
-
-       if (session->undo_depth() == 0) {
-               label = _("Undo");
-       } else {
-               label = string_compose(_("Undo (%1)"), session->next_undo());
-       }
-       
-       edit_items.push_back (MenuElem (label, bind (mem_fun(*this, &Editor::undo), 1U)));
-       
-       if (session->undo_depth() == 0) {
-               edit_items.back().set_sensitive (false);
-       }
-       
-       if (session->redo_depth() == 0) {
-               label = _("Redo");
-       } else {
-               label = string_compose(_("Redo (%1)"), session->next_redo());
-       }
-       
-       edit_items.push_back (MenuElem (label, bind (mem_fun(*this, &Editor::redo), 1U)));
-       if (session->redo_depth() == 0) {
-               edit_items.back().set_sensitive (false);
-       }
-
-       vector<MenuItem*> mitems;
-
-       edit_items.push_back (SeparatorElem());
-       edit_items.push_back (MenuElem (_("Cut"), mem_fun(*this, &Editor::cut)));
-       mitems.push_back (&edit_items.back());
-       edit_items.push_back (MenuElem (_("Copy"), mem_fun(*this, &Editor::copy)));
-       mitems.push_back (&edit_items.back());
-       edit_items.push_back (MenuElem (_("Paste"), bind (mem_fun(*this, &Editor::paste), 1.0f)));
-       mitems.push_back (&edit_items.back());
-       edit_items.push_back (SeparatorElem());
-       edit_items.push_back (MenuElem (_("Align"), bind (mem_fun(*this, &Editor::align), ARDOUR::SyncPoint)));
-       mitems.push_back (&edit_items.back());
-       edit_items.push_back (MenuElem (_("Align Relative"), bind (mem_fun(*this, &Editor::align_relative), ARDOUR::SyncPoint)));
-       mitems.push_back (&edit_items.back());
-       edit_items.push_back (SeparatorElem());
-
-       if (selection->empty()) {
-               for (vector<MenuItem*>::iterator i = mitems.begin(); i != mitems.end(); ++i) {
-                       (*i)->set_sensitive (false);
-               }
-       }
-
-       Menu* import_menu = manage (new Menu());
-       import_menu->set_name ("ArdourContextMenu");
-       MenuList& import_items = import_menu->items();
-       
-       import_items.push_back (MenuElem (_("... as new track"), bind (mem_fun(*this, &Editor::add_external_audio_action), ImportAsTrack)));
-       import_items.push_back (MenuElem (_("... as new region"), bind (mem_fun(*this, &Editor::add_external_audio_action), ImportAsRegion)));
-
-       edit_items.push_back (MenuElem (_("Import audio (copy)"), *import_menu));
-       edit_items.push_back (SeparatorElem());
-
-       edit_items.push_back (MenuElem (_("Remove last capture"), mem_fun(*this, &Editor::remove_last_capture)));
-
-       if (!session->have_captured()) {
-               edit_items.back().set_sensitive (false);
-       }
-
-       return false;
-}
-
-void
-Editor::duplicate_dialog (bool dup_region)
-{
-       if (dup_region) {
-               if (clicked_regionview == 0) {
-                       return;
-               }
-       } else {
-               if (selection->time.length() == 0) {
-                       return;
-               }
-       }
-
-       ArdourDialog win ("duplicate dialog");
-       Entry  entry;
-       Label  label (_("Duplicate how many times?"));
-
-       win.get_vbox()->pack_start (label);
-       win.add_action_widget (entry, RESPONSE_ACCEPT);
-       win.add_button (Stock::OK, RESPONSE_ACCEPT);
-       win.add_button (Stock::CANCEL, RESPONSE_CANCEL);
-
-       win.set_position (WIN_POS_MOUSE);
-
-       entry.set_text ("1");
-       set_size_request_to_display_given_text (entry, X_("12345678"), 20, 15);
-       entry.select_region (0, entry.get_text_length());
-       entry.grab_focus ();
-
-
-       switch (win.run ()) {
-       case RESPONSE_ACCEPT:
-               break;
-       default:
-               return;
-       }
-
-       string text = entry.get_text();
-       float times;
-
-       if (sscanf (text.c_str(), "%f", &times) == 1) {
-               if (dup_region) {
-                       AudioRegionSelection regions;
-                       regions.add (clicked_regionview);
-                       duplicate_some_regions (regions, times);
-               } else {
-                       duplicate_selection (times);
-               }
-       }
-}
-
-void
-Editor::show_verbose_canvas_cursor ()
-{
-        verbose_canvas_cursor->raise_to_top();
-        verbose_canvas_cursor->show();
-       verbose_cursor_visible = true;
-}
-
-void
-Editor::hide_verbose_canvas_cursor ()
-{
-        verbose_canvas_cursor->hide();
-       verbose_cursor_visible = false;
-}
-
-void
-Editor::set_verbose_canvas_cursor (const string & txt, double x, double y)
-{
-       /* XXX get origin of canvas relative to root window,
-          add x and y and check compared to gdk_screen_{width,height}
-       */
-       verbose_canvas_cursor->property_text() = txt.c_str();
-       verbose_canvas_cursor->property_x() = x;
-       verbose_canvas_cursor->property_y() = y;
-}
-
-void
-Editor::set_verbose_canvas_cursor_text (const string & txt)
-{
-       verbose_canvas_cursor->property_text() = txt.c_str();
-}
-
-void
-Editor::edit_mode_selection_done ()
-{
-       if (session == 0) {
-               return;
+               return;
        }
 
        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;
        }
 
-       session->set_edit_mode (mode);
+       Config->set_edit_mode (mode);
 }      
 
 void
 Editor::snap_type_selection_done ()
 {
-       if (session == 0) {
-               return;
-       }
-
        string choice = snap_type_selector.get_active_text();
-       SnapType snaptype = SnapToFrame;
-       
+       SnapType snaptype = SnapToBeat;
+
        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;
        } else if (choice == _("Marks")) {
                snaptype = SnapToMark;
-       } else if (choice == _("Edit Cursor")) {
-               snaptype = SnapToEditCursor;
        } else if (choice == _("Region starts")) {
                snaptype = SnapToRegionStart;
        } else if (choice == _("Region ends")) {
@@ -3605,39 +3450,79 @@ Editor::snap_type_selection_done ()
                snaptype = SnapToSeconds;
        } else if (choice == _("Minutes")) {
                snaptype = SnapToMinutes;
-        } else if (choice == _("None")) {
-               snaptype = SnapToFrame;
        }
-       
-       set_snap_to (snaptype);
+
+       RefPtr<RadioAction> ract = snap_type_action (snaptype);
+       if (ract) {
+               ract->set_active ();
+       }
 }      
 
 void
 Editor::snap_mode_selection_done ()
 {
-       if(session == 0) {
-               return;
-       }
-
        string choice = snap_mode_selector.get_active_text();
        SnapMode mode = SnapNormal;
 
-       if (choice == _("Normal")) {
+       if (choice == _("No Grid")) {
+               mode = SnapOff;
+       } else if (choice == _("Grid")) {
                mode = SnapNormal;
        } else if (choice == _("Magnetic")) {
                mode = SnapMagnetic;
        }
 
-       set_snap_mode (mode);
+       RefPtr<RadioAction> ract = snap_mode_action (mode);
+
+       if (ract) {
+               ract->set_active (true);
+       }
 }
 
 void
-Editor::zoom_focus_selection_done ()
+Editor::cycle_edit_point (bool with_marker)
 {
-       if (session == 0) {
-               return;
+       switch (_edit_point) {
+       case EditAtMouse:
+               set_edit_point_preference (EditAtPlayhead);
+               break;
+       case EditAtPlayhead:
+               if (with_marker) {
+                       set_edit_point_preference (EditAtSelectedMarker);
+               } else {
+                       set_edit_point_preference (EditAtMouse);
+               }
+               break;
+       case EditAtSelectedMarker:
+               set_edit_point_preference (EditAtMouse);
+               break;
+       }
+}
+
+void
+Editor::edit_point_selection_done ()
+{
+       string choice = edit_point_selector.get_active_text();
+       EditPoint ep = EditAtSelectedMarker;
+
+       if (choice == _("Marker")) {
+               set_edit_point_preference (EditAtSelectedMarker);
+       } else if (choice == _("Playhead")) {
+               set_edit_point_preference (EditAtPlayhead);
+       } else {
+               set_edit_point_preference (EditAtMouse);
        }
 
+       RefPtr<RadioAction> ract = edit_point_action (ep);
+
+       if (ract) {
+               ract->set_active (true);
+       }
+}
+
+void
+Editor::zoom_focus_selection_done ()
+{
        string choice = zoom_focus_selector.get_active_text();
        ZoomFocus focus_type = ZoomFocusLeft;
 
@@ -3647,100 +3532,42 @@ Editor::zoom_focus_selection_done ()
                focus_type = ZoomFocusRight;
        } else if (choice == _("Center")) {
                focus_type = ZoomFocusCenter;
-        } else if (choice == _("Playhead")) {
+       } else if (choice == _("Play")) {
                focus_type = ZoomFocusPlayhead;
-        } else if (choice == _("Edit Cursor")) {
+       } else if (choice == _("Edit")) {
                focus_type = ZoomFocusEdit;
+       } else if (choice == _("Edit Point")) {
+               focus_type = ZoomFocusEdit;
+       } else {
+               focus_type = ZoomFocusMouse;
        } 
+       
+       RefPtr<RadioAction> ract = zoom_focus_action (focus_type);
 
-       set_zoom_focus (focus_type);
+       if (ract) {
+               ract->set_active ();
+       }
 }      
 
 gint
 Editor::edit_controls_button_release (GdkEventButton* ev)
 {
        if (Keyboard::is_context_menu_event (ev)) {
-               ARDOUR_UI::instance()->add_route ();
+               ARDOUR_UI::instance()->add_route (this);
        }
        return TRUE;
 }
 
-void
-Editor::track_selection_changed ()
-{
-       switch (selection->tracks.size()){
-       case 0:
-               break;
-       default:
-               set_selected_mixer_strip (*(selection->tracks.front()));
-               break;
-       }
-
-       for (TrackViewList::iterator i = track_views.begin(); i != track_views.end(); ++i) {
-               (*i)->set_selected (false);
-               if (mouse_mode == MouseRange) {
-                       (*i)->hide_selection ();
-               }
-       }
-
-       for (TrackSelection::iterator i = selection->tracks.begin(); i != selection->tracks.end(); ++i) {
-               (*i)->set_selected (true);
-               if (mouse_mode == MouseRange) {
-                       (*i)->show_selection (selection->time);
-               }
-       }
-}
-
-void
-Editor::time_selection_changed ()
-{
-       for (TrackViewList::iterator i = track_views.begin(); i != track_views.end(); ++i) {
-               (*i)->hide_selection ();
-       }
-
-       if (selection->tracks.empty()) {
-               for (TrackViewList::iterator i = track_views.begin(); i != track_views.end(); ++i) {
-                       (*i)->show_selection (selection->time);
-               }
-       } else {
-               for (TrackSelection::iterator i = selection->tracks.begin(); i != selection->tracks.end(); ++i) {
-                       (*i)->show_selection (selection->time);
-               }
-       }
-
-       if (selection->time.empty()) {
-               ActionManager::set_sensitive (ActionManager::time_selection_sensitive_actions, false);
-       } else {
-               ActionManager::set_sensitive (ActionManager::time_selection_sensitive_actions, true);
-       }
-}
-
-void
-Editor::region_selection_changed ()
-{
-       for (TrackViewList::iterator i = track_views.begin(); i != track_views.end(); ++i) {
-               (*i)->set_selected_regionviews (selection->audio_regions);
-       }
-}
-
-void
-Editor::point_selection_changed ()
-{
-       for (TrackViewList::iterator i = track_views.begin(); i != track_views.end(); ++i) {
-               (*i)->set_selected_points (selection->points);
-       }
-}
-
 gint
 Editor::mouse_select_button_release (GdkEventButton* ev)
 {
        /* this handles just right-clicks */
 
        if (ev->button != 3) {
-               return FALSE;
+               return false;
        }
 
-       return TRUE;
+       return true;
 }
 
 Editor::TrackViewList *
@@ -3784,10 +3611,15 @@ Editor::get_valid_views (TimeAxisView* track, RouteGroup* group)
 void
 Editor::set_zoom_focus (ZoomFocus f)
 {
+       string str = zoom_focus_strings[(int)f];
+
+       if (str != zoom_focus_selector.get_active_text()) {
+               zoom_focus_selector.set_active_text (str);
+       }
+       
        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 ();
@@ -3870,7 +3702,6 @@ Editor::set_show_measures (bool yn)
                if ((_show_measures = yn) == true) {
                        draw_measures ();
                }
-               DisplayControlChanged (ShowMeasures);
                instant_save ();
        }
 }
@@ -3893,27 +3724,38 @@ Editor::set_follow_playhead (bool yn)
                        /* catch up */
                        update_current_screen ();
                }
-               DisplayControlChanged (FollowPlayhead);
                instant_save ();
        }
 }
 
 void
-Editor::toggle_xfade_active (Crossfade* xfade)
+Editor::toggle_xfade_active (boost::weak_ptr<Crossfade> wxfade)
 {
-       xfade->set_active (!xfade->active());
+       boost::shared_ptr<Crossfade> xfade (wxfade.lock());
+       if (xfade) {
+               xfade->set_active (!xfade->active());
+       }
 }
 
 void
-Editor::toggle_xfade_length (Crossfade* xfade)
+Editor::toggle_xfade_length (boost::weak_ptr<Crossfade> wxfade)
 {
-       xfade->set_follow_overlap (!xfade->following_overlap());
+       boost::shared_ptr<Crossfade> xfade (wxfade.lock());
+       if (xfade) {
+               xfade->set_follow_overlap (!xfade->following_overlap());
+       }
 }
 
 void
-Editor::edit_xfade (Crossfade* xfade)
+Editor::edit_xfade (boost::weak_ptr<Crossfade> wxfade)
 {
-       CrossfadeEditor cew (*session, *xfade, xfade->fade_in().get_min_y(), 1.0);
+       boost::shared_ptr<Crossfade> xfade (wxfade.lock());
+
+       if (!xfade) {
+               return;
+       }
+
+       CrossfadeEditor cew (*session, xfade, xfade->fade_in().get_min_y(), 1.0);
                
        ensure_float (cew);
        
@@ -3934,10 +3776,10 @@ Editor::playlist_selector () const
        return *_playlist_selector;
 }
 
-jack_nframes_t
-Editor::get_nudge_distance (jack_nframes_t pos, jack_nframes_t& next)
+nframes_t
+Editor::get_nudge_distance (nframes_t pos, nframes_t& next)
 {
-       jack_nframes_t ret;
+       nframes_t ret;
 
        ret = nudge_clock.current_duration (pos);
        next = ret + 1; /* XXXX fix me */
@@ -3949,23 +3791,25 @@ void
 Editor::end_location_changed (Location* location)
 {
        ENSURE_GUI_THREAD (bind (mem_fun(*this, &Editor::end_location_changed), location));
-       horizontal_adjustment.set_upper (location->end() / frames_per_unit);
+       reset_scrolling_region ();
 }
 
 int
-Editor::playlist_deletion_dialog (Playlist* pl)
+Editor::playlist_deletion_dialog (boost::shared_ptr<Playlist> pl)
 {
        ArdourDialog dialog ("playlist deletion dialog");
        Label  label (string_compose (_("Playlist %1 is currently unused.\n"
-                                "If left alone, no audio files used by it will be cleaned.\n"
-                                "If deleted, audio files used by it alone by will cleaned."),
-                              pl->name()));
-
+                                       "If left alone, no audio files used by it will be cleaned.\n"
+                                       "If deleted, audio files used by it alone by will cleaned."),
+                                     pl->name()));
+       
        dialog.set_position (WIN_POS_CENTER);
        dialog.get_vbox()->pack_start (label);
 
+       label.show ();
+
        dialog.add_button (_("Delete playlist"), RESPONSE_ACCEPT);
-       dialog.add_button (_("Keep playlist"), RESPONSE_CANCEL);
+       dialog.add_button (_("Keep playlist"), RESPONSE_REJECT);
        dialog.add_button (_("Cancel"), RESPONSE_CANCEL);
 
        switch (dialog.run ()) {
@@ -3987,10 +3831,10 @@ Editor::playlist_deletion_dialog (Playlist* pl)
 }
 
 bool
-Editor::audio_region_selection_covers (jack_nframes_t where)
+Editor::audio_region_selection_covers (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;
                }
        }
@@ -4001,10 +3845,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 ();
 }
 
@@ -4049,6 +3893,9 @@ Editor::control_layout_scroll (GdkEventScroll* ev)
        return false;
 }
 
+
+/** A new snapshot has been selected.
+ */
 void
 Editor::snapshot_display_selection_changed ()
 {
@@ -4073,7 +3920,87 @@ Editor::snapshot_display_selection_changed ()
 bool
 Editor::snapshot_display_button_press (GdkEventButton* ev)
 {
-        return false;
+       if (ev->button == 3) {
+               /* Right-click on the snapshot list. Work out which snapshot it
+                  was over. */
+               Gtk::TreeModel::Path path;
+               Gtk::TreeViewColumn* col;
+               int cx;
+               int cy;
+               snapshot_display.get_path_at_pos ((int) ev->x, (int) ev->y, path, col, cx, cy);
+               Gtk::TreeModel::iterator iter = snapshot_display_model->get_iter (path);
+               if (iter) {
+                       Gtk::TreeModel::Row row = *iter;
+                       popup_snapshot_context_menu (ev->button, ev->time, row[snapshot_display_columns.real_name]);
+               }
+               return true;
+       }
+
+       return false;
+}
+
+
+/** Pop up the snapshot display context menu.
+ * @param button Button used to open the menu.
+ * @param time Menu open time.
+ * @snapshot_name Name of the snapshot that the menu click was over.
+ */
+
+void
+Editor::popup_snapshot_context_menu (int button, int32_t time, Glib::ustring snapshot_name)
+{
+       using namespace Menu_Helpers;
+
+       MenuList& items (snapshot_context_menu.items());
+       items.clear ();
+
+       const bool modification_allowed = (session->snap_name() != snapshot_name && session->name() != snapshot_name);
+
+       add_item_with_sensitivity (items, MenuElem (_("Remove"), bind (mem_fun (*this, &Editor::remove_snapshot), snapshot_name)), modification_allowed);
+
+       add_item_with_sensitivity (items, MenuElem (_("Rename"), bind (mem_fun (*this, &Editor::rename_snapshot), snapshot_name)), modification_allowed);
+
+       snapshot_context_menu.popup (button, time);
+}
+
+void
+Editor::rename_snapshot (Glib::ustring old_name)
+{
+       ArdourPrompter prompter(true);
+
+       string new_name;
+
+       prompter.set_name ("Prompter");
+       prompter.add_button (Gtk::Stock::SAVE, Gtk::RESPONSE_ACCEPT);
+       prompter.set_prompt (_("New name of snapshot"));
+       prompter.set_initial_text (old_name);
+       
+       if (prompter.run() == RESPONSE_ACCEPT) {
+               prompter.get_result (new_name);
+               if (new_name.length()) {
+                       session->rename_state (old_name, new_name);
+                       redisplay_snapshots ();
+               }
+       }
+}
+
+
+void
+Editor::remove_snapshot (Glib::ustring name)
+{
+       vector<string> choices;
+
+       std::string prompt  = string_compose (_("Do you really want to remove snapshot \"%1\" ?\n(cannot be undone)"), name);
+
+       choices.push_back (_("No, do nothing."));
+       choices.push_back (_("Yes, remove it."));
+
+       Gtkmm2ext::Choice prompter (prompt, choices);
+
+       if (prompter.run () == 1) {
+               session->remove_state (name);
+               redisplay_snapshots ();
+       }
 }
 
 void
@@ -4083,22 +4010,37 @@ Editor::redisplay_snapshots ()
                return;
        }
 
-       vector<string*>* states;
+       vector<sys::path> state_file_paths;
 
-       if ((states = session->possible_states()) == 0) {
-               return;
-       }
+       get_state_files_in_directory (session->session_directory().root_path(),
+                       state_file_paths);
+
+       if (state_file_paths.empty()) return;
+
+       vector<string> state_file_names(get_file_names_no_extension(state_file_paths));
 
        snapshot_display_model->clear ();
 
-       for (vector<string*>::iterator i = states->begin(); i != states->end(); ++i) {
-               string statename = *(*i);
+       for (vector<string>::iterator i = state_file_names.begin();
+                       i != state_file_names.end(); ++i)
+       {
+               string statename = (*i);
                TreeModel::Row row = *(snapshot_display_model->append());
-               row[snapshot_display_columns.visible_name] = statename;
+               
+               /* this lingers on in case we ever want to change the visible
+                  name of the snapshot.
+               */
+               
+               string display_name;
+               display_name = statename;
+
+               if (statename == session->snap_name()) {
+                       snapshot_display.get_selection()->select(row);
+               } 
+               
+               row[snapshot_display_columns.visible_name] = display_name;
                row[snapshot_display_columns.real_name] = statename;
        }
-
-       delete states;
 }
 
 void
@@ -4111,63 +4053,106 @@ Editor::session_state_saved (string snap_name)
 void
 Editor::maximise_editing_space ()
 {
+       initial_ruler_update_required = true;
+
        mouse_mode_tearoff->set_visible (false);
        tools_tearoff->set_visible (false);
 
        pre_maximal_pane_position = edit_pane.get_position();
-       edit_pane.set_position (edit_pane.get_width());
+       pre_maximal_editor_width = this->get_width();
+
+       if(post_maximal_pane_position == 0) {
+               post_maximal_pane_position = edit_pane.get_width();
+       }
+
 
        fullscreen();
+       if(post_maximal_editor_width) {
+               edit_pane.set_position (post_maximal_pane_position - 
+                       abs(post_maximal_editor_width - pre_maximal_editor_width));
+       } else {
+               edit_pane.set_position (post_maximal_pane_position);
+       }
 }
 
 void
 Editor::restore_editing_space ()
 {
+       initial_ruler_update_required = true;
+
+       // user changed width of pane during fullscreen
+       if(post_maximal_pane_position != edit_pane.get_position()) {
+               post_maximal_pane_position = edit_pane.get_position();
+       }
+
+       unfullscreen();
+
        mouse_mode_tearoff->set_visible (true);
        tools_tearoff->set_visible (true);
-       edit_pane.set_position (pre_maximal_pane_position);
+       post_maximal_editor_width = this->get_width();
 
-       unfullscreen();
+
+       edit_pane.set_position (
+               pre_maximal_pane_position + abs(this->get_width() - pre_maximal_editor_width)
+       );
 }
 
+/**
+ *  Make new playlists for a given track and also any others that belong
+ *  to the same active edit group.
+ *  @param v Track.
+ */
+
 void 
-Editor::new_playlists ()
+Editor::new_playlists (TimeAxisView* v)
 {
        begin_reversible_command (_("new playlists"));
-       mapover_audio_tracks (mem_fun (*this, &Editor::mapped_use_new_playlist));
+       mapover_tracks (mem_fun (*this, &Editor::mapped_use_new_playlist), v);
        commit_reversible_command ();
 }
 
+/**
+ *  Use a copy of the current playlist for a given track and also any others that belong
+ *  to the same active edit group.
+ *  @param v Track.
+ */
+
 void
-Editor::copy_playlists ()
+Editor::copy_playlists (TimeAxisView* v)
 {
        begin_reversible_command (_("copy playlists"));
-       mapover_audio_tracks (mem_fun (*this, &Editor::mapped_use_copy_playlist));
+       mapover_tracks (mem_fun (*this, &Editor::mapped_use_copy_playlist), v);
        commit_reversible_command ();
 }
 
+/**
+ *  Clear the current playlist for a given track and also any others that belong
+ *  to the same active edit group.
+ *  @param v Track.
+ */
+
 void 
-Editor::clear_playlists ()
+Editor::clear_playlists (TimeAxisView* v)
 {
        begin_reversible_command (_("clear playlists"));
-       mapover_audio_tracks (mem_fun (*this, &Editor::mapped_clear_playlist));
+       mapover_tracks (mem_fun (*this, &Editor::mapped_clear_playlist), v);
        commit_reversible_command ();
 }
 
 void 
-Editor::mapped_use_new_playlist (AudioTimeAxisView& atv, uint32_t sz)
+Editor::mapped_use_new_playlist (RouteTimeAxisView& atv, uint32_t sz)
 {
        atv.use_new_playlist (sz > 1 ? false : true);
 }
 
 void
-Editor::mapped_use_copy_playlist (AudioTimeAxisView& atv, uint32_t sz)
+Editor::mapped_use_copy_playlist (RouteTimeAxisView& atv, uint32_t sz)
 {
        atv.use_copy_playlist (sz > 1 ? false : true);
 }
 
 void 
-Editor::mapped_clear_playlist (AudioTimeAxisView& atv, uint32_t sz)
+Editor::mapped_clear_playlist (RouteTimeAxisView& atv, uint32_t sz)
 {
        atv.clear_playlist ();
 }
@@ -4178,51 +4163,426 @@ Editor::on_key_press_event (GdkEventKey* ev)
        return key_press_focus_accelerator_handler (*this, ev);
 }
 
+bool
+Editor::on_key_release_event (GdkEventKey* ev)
+{
+       return Gtk::Window::on_key_release_event (ev);
+       // return key_press_focus_accelerator_handler (*this, ev);
+}
+
 void
-Editor::update_layering_model ()
+Editor::reset_x_origin (nframes_t frame)
 {
-       RefPtr<Action> act;
+       queue_visual_change (frame);
+}
 
-       switch (session->get_layer_model()) {
-       case Session::LaterHigher:
-               act = ActionManager::get_action (X_("Editor"), X_("LayerLaterHigher"));
-               break;
-       case Session::MoveAddHigher:
-               act = ActionManager::get_action (X_("Editor"), X_("LayerMoveAddHigher"));
-               break;
-       case Session::AddHigher:
-               act = ActionManager::get_action (X_("Editor"), X_("LayerAddHigher"));
-               break;
+void
+Editor::reset_zoom (double fpu)
+{
+       queue_visual_change (fpu);
+}
+
+void
+Editor::reposition_and_zoom (nframes_t frame, double fpu)
+{
+       reset_x_origin (frame);
+       reset_zoom (fpu);
+}
+
+void
+Editor::swap_visual_state ()
+{
+       if (last_visual_state.frames_per_unit == 0) {
+               // never set
+               return;
        }
 
-       if (act) {
-               RefPtr<RadioAction> ract = RefPtr<RadioAction>::cast_dynamic(act);
-               if (ract && !ract->get_active()) {
-                       ract->set_active (true);
+       /* note: the correct functionality here is very dependent on the ordering of 
+          setting zoom focus, horizontal position and finally zoom. this is because
+          it is set_frames_per_unit() that overwrites last_visual_state.
+       */
+
+       set_zoom_focus (last_visual_state.zoom_focus);
+       reposition_and_zoom (last_visual_state.leftmost_frame, last_visual_state.frames_per_unit);
+       zoomed_to_region = false;
+}
+
+void
+Editor::set_frames_per_unit (double fpu)
+{
+       /* this is the core function that controls the zoom level of the canvas. it is called
+          whenever one or more calls are made to reset_zoom(). it executes in an idle handler.
+       */
+
+       if (fpu == frames_per_unit) {
+               return;
+       }
+
+       if (fpu < 2.0) {
+               fpu = 2.0;
+       }
+
+       
+       /* don't allow zooms that fit more than the maximum number
+          of frames into an 800 pixel wide space.
+       */
+
+       if (max_frames / fpu < 800.0) {
+               return;
+       }
+
+       if (fpu == frames_per_unit) {
+               return;
+       }
+       
+       last_visual_state.frames_per_unit = frames_per_unit;
+       last_visual_state.leftmost_frame = leftmost_frame;
+       last_visual_state.zoom_focus = zoom_focus;
+
+       frames_per_unit = fpu;
+       post_zoom ();
+}
+
+void
+Editor::post_zoom ()
+{
+       // convert fpu to frame count
+
+       nframes_t frames = (nframes_t) floor (frames_per_unit * canvas_width);
+
+       if (frames_per_unit != zoom_range_clock.current_duration()) {
+               zoom_range_clock.set (frames);
+       }
+
+       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) {
+                               (*i)->reshow_selection (selection->time);
+                       }
+               } else {
+                       for (TrackViewList::iterator i = track_views.begin(); i != track_views.end(); ++i) {
+                               (*i)->reshow_selection (selection->time);
+                       }
                }
        }
+
+       ZoomChanged (); /* EMIT_SIGNAL */
+
+       reset_hscrollbar_stepping ();
+       reset_scrolling_region ();
+
+       if (playhead_cursor) playhead_cursor->set_position (playhead_cursor->current_frame);
+
+       instant_save ();
 }
 
+void
+Editor::queue_visual_change (nframes_t where)
+{
+       pending_visual_change.pending = VisualChange::Type (pending_visual_change.pending | VisualChange::TimeOrigin);
+       pending_visual_change.time_origin = where;
+       
+       if (pending_visual_change.idle_handler_id < 0) {
+               pending_visual_change.idle_handler_id = g_idle_add (_idle_visual_changer, this);
+       }
+}
 
 void
-Editor::update_crossfade_model ()
+Editor::queue_visual_change (double fpu)
 {
-       RefPtr<Action> act;
+       pending_visual_change.pending = VisualChange::Type (pending_visual_change.pending | VisualChange::ZoomLevel);
+       pending_visual_change.frames_per_unit = fpu;
+
+       if (pending_visual_change.idle_handler_id < 0) {
+               pending_visual_change.idle_handler_id = g_idle_add ( _idle_visual_changer, this);
+       }
+       
+}
+
+int
+Editor::_idle_visual_changer (void* arg)
+{
+       return static_cast<Editor*>(arg)->idle_visual_changer ();
+}
+
+int
+Editor::idle_visual_changer ()
+{
+       VisualChange::Type p = pending_visual_change.pending;
+
+       pending_visual_change.pending = (VisualChange::Type) 0;
+
+       if (p & VisualChange::ZoomLevel) {
+               set_frames_per_unit (pending_visual_change.frames_per_unit);
+
+               compute_fixed_ruler_scale ();
+               compute_current_bbt_points(pending_visual_change.time_origin, pending_visual_change.time_origin + (nframes_t)(canvas_width * pending_visual_change.frames_per_unit));
+               compute_bbt_ruler_scale (pending_visual_change.time_origin, pending_visual_change.time_origin + (nframes_t)(canvas_width * pending_visual_change.frames_per_unit));
+               update_tempo_based_rulers ();
+       }
+       if (p & VisualChange::TimeOrigin) {
+               
+               nframes_t time_origin = (nframes_t) floor (horizontal_adjustment.get_value() * frames_per_unit);
+
+               if (time_origin != pending_visual_change.time_origin) {
+                       horizontal_adjustment.set_value (pending_visual_change.time_origin/frames_per_unit);
+               } else {
+                       update_fixed_rulers();
+                       redisplay_tempo (true);
+               }
+       }
+       pending_visual_change.idle_handler_id = -1;
+
+       return 0; /* this is always a one-shot call */
+}
 
-       switch (session->get_xfade_model()) {
-       case FullCrossfade:
-               act = ActionManager::get_action (X_("Editor"), X_("CrossfadesFull"));
+struct EditorOrderTimeAxisSorter {
+    bool operator() (const TimeAxisView* a, const TimeAxisView* b) const {
+           return a->order < b->order;
+    }
+};
+       
+void
+Editor::sort_track_selection (TrackSelection* sel)
+{
+       EditorOrderTimeAxisSorter cmp;
+
+       if (sel) {
+               sel->sort (cmp);
+       } else {
+               selection->tracks.sort (cmp);
+       }
+}
+
+nframes64_t
+Editor::get_preferred_edit_position (bool ignore_playhead)
+{
+       bool ignored;
+       nframes64_t where = 0;
+       EditPoint ep = _edit_point;
+
+       if (entered_marker) {
+               return entered_marker->position();
+       }
+
+       if (ignore_playhead && ep == EditAtPlayhead) {
+               ep = EditAtSelectedMarker;
+       }
+
+       switch (ep) {
+       case EditAtPlayhead:
+               where = session->audible_frame();
                break;
-       case ShortCrossfade:
-               act = ActionManager::get_action (X_("Editor"), X_("CrossfadesShort"));
+               
+       case EditAtSelectedMarker:
+               if (!selection->markers.empty()) {
+                       bool is_start;
+                       Location* loc = find_location_from_marker (selection->markers.front(), is_start);
+                       if (loc) {
+                               if (is_start) {
+                                       where =  loc->start();
+                               } else {
+                                       where = loc->end();
+                               }
+                               break;
+                       }
+               } 
+               /* fallthru */
+               
+       default:
+       case EditAtMouse:
+               if (!mouse_frame (where, ignored)) {
+                       /* XXX not right but what can we do ? */
+                       return 0;
+               }
+               snap_to (where);
                break;
        }
 
-       if (act) {
-               RefPtr<RadioAction> ract = RefPtr<RadioAction>::cast_dynamic(act);
-               if (ract && !ract->get_active()) {
-                       ract->set_active (true);
+       return where;
+}
+
+void
+Editor::set_loop_range (nframes_t start, nframes_t end, string cmd)
+{
+       if (!session) return;
+
+       begin_reversible_command (cmd);
+       
+       Location* tll;
+
+       if ((tll = transport_loop_location()) == 0) {
+               Location* loc = new Location (start, end, _("Loop"),  Location::IsAutoLoop);
+                XMLNode &before = session->locations()->get_state();
+               session->locations()->add (loc, true);
+               session->set_auto_loop_location (loc);
+                XMLNode &after = session->locations()->get_state();
+               session->add_command (new MementoCommand<Locations>(*(session->locations()), &before, &after));
+       }
+       else {
+                XMLNode &before = tll->get_state();
+               tll->set_hidden (false, this);
+               tll->set (start, end);
+                XMLNode &after = tll->get_state();
+                session->add_command (new MementoCommand<Location>(*tll, &before, &after));
+       }
+       
+       commit_reversible_command ();
+}
+
+void
+Editor::set_punch_range (nframes_t start, nframes_t end, string cmd)
+{
+       if (!session) return;
+
+       begin_reversible_command (cmd);
+       
+       Location* tpl;
+
+       if ((tpl = transport_punch_location()) == 0) {
+               Location* loc = new Location (start, end, _("Loop"),  Location::IsAutoPunch);
+                XMLNode &before = session->locations()->get_state();
+               session->locations()->add (loc, true);
+               session->set_auto_loop_location (loc);
+                XMLNode &after = session->locations()->get_state();
+               session->add_command (new MementoCommand<Locations>(*(session->locations()), &before, &after));
+       }
+       else {
+                XMLNode &before = tpl->get_state();
+               tpl->set_hidden (false, this);
+               tpl->set (start, end);
+                XMLNode &after = tpl->get_state();
+                session->add_command (new MementoCommand<Location>(*tpl, &before, &after));
+       }
+       
+       commit_reversible_command ();
+}
+
+RegionSelection
+Editor::get_regions_at (nframes64_t where, const TrackSelection& ts) const
+{
+       RegionSelection rs;
+       const TrackSelection* tracks;
+
+       if (ts.empty()) {
+               tracks = &track_views;
+       } else {
+               tracks = &ts;
+       }
+
+       for (TrackSelection::const_iterator t = tracks->begin(); t != tracks->end(); ++t) {
+       
+               AudioTimeAxisView* atv = dynamic_cast<AudioTimeAxisView*>(*t);
+
+               if (atv) {
+                       boost::shared_ptr<Diskstream> ds;
+                       boost::shared_ptr<Playlist> pl;
+                       
+                       if ((ds = atv->get_diskstream()) && ((pl = ds->playlist()))) {
+
+                               Playlist::RegionList* regions = pl->regions_at ((nframes_t) floor ( (double)where * ds->speed()));
+
+                               for (Playlist::RegionList::iterator i = regions->begin(); i != regions->end(); ++i) {
+
+                                       RegionView* rv = atv->audio_view()->find_view (*i);
+
+                                       if (rv) {
+                                               rs.push_back (rv);
+                                       }
+                               }
+
+                               delete regions;
+                       }
                }
        }
+
+       return rs;
 }
 
+
+RegionSelection
+Editor::get_regions_after (nframes64_t where, const TrackSelection& ts) const
+{
+       RegionSelection rs;
+       const TrackSelection* tracks;
+
+       if (ts.empty()) {
+               tracks = &track_views;
+       } else {
+               tracks = &ts;
+       }
+
+       for (TrackSelection::const_iterator t = tracks->begin(); t != tracks->end(); ++t) {
+       
+               AudioTimeAxisView* atv = dynamic_cast<AudioTimeAxisView*>(*t);
+
+               if (atv) {
+                       boost::shared_ptr<Diskstream> ds;
+                       boost::shared_ptr<Playlist> pl;
+                       
+                       if ((ds = atv->get_diskstream()) && ((pl = ds->playlist()))) {
+
+                               Playlist::RegionList* regions = pl->regions_touched ((nframes_t) floor ( (double)where * ds->speed()), max_frames);
+
+                               for (Playlist::RegionList::iterator i = regions->begin(); i != regions->end(); ++i) {
+
+                                       RegionView* rv = atv->audio_view()->find_view (*i);
+
+                                       if (rv) {
+                                               rs.push_back (rv);
+                                       }
+                               }
+
+                               delete regions;
+                       }
+               }
+       }
+
+       return rs;
+}
+
+RegionSelection&
+Editor::get_regions_for_action ()
+{
+       if (!selection->regions.empty()) {
+               return selection->regions;
+       } 
+
+       nframes64_t where = get_preferred_edit_position();
+       tmp_regions = get_regions_at (where, selection->tracks);
+       return tmp_regions;
+}
+
+void
+Editor::get_regions_corresponding_to (boost::shared_ptr<Region> region, vector<RegionView*>& regions)
+{
+
+       for (TrackViewList::iterator i = track_views.begin(); i != track_views.end(); ++i) {
+               
+               RouteTimeAxisView* tatv;
+               
+               if ((tatv = dynamic_cast<RouteTimeAxisView*> (*i)) != 0) {
+                       
+                       boost::shared_ptr<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);
+                       }
+                       
+                       for (vector<boost::shared_ptr<Region> >::iterator ir = results.begin(); ir != results.end(); ++ir) {
+                               if ((marv = tatv->view()->find_view (*ir)) != 0) {
+                                       regions.push_back (marv);
+                               }
+                       }
+                       
+               }
+       }
+}