catch markers as they go away, to avoid selection corruption; add select-range-betwee...
[ardour.git] / gtk2_ardour / editor.cc
index 1e01e3e24355fde645695e336965c48bab3ed659..bde42378e9ddfb56bb8f32aaf589f11f00a0e02d 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$
 */
 
+#include <sys/time.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/stacktrace.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>
@@ -39,6 +42,8 @@
 #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/audio_diskstream.h>
@@ -50,6 +55,7 @@
 #include <ardour/session_route.h>
 #include <ardour/tempo.h>
 #include <ardour/utils.h>
+#include <ardour/profile.h>
 
 #include <control_protocol/control_protocol.h>
 
@@ -72,6 +78,7 @@
 #include "canvas_impl.h"
 #include "actions.h"
 #include "gui_thread.h"
+#include "sfdb_ui.h"
 
 #ifdef FFT_ANALYSIS
 #include "analysis_window.h"
@@ -114,7 +121,7 @@ static const gchar *_snap_type_strings[] = {
        N_("Beats"),
        N_("Bars"),
        N_("Marks"),
-       N_("Edit Cursor"),
+       N_("Edit Point"),
        N_("Region starts"),
        N_("Region ends"),
        N_("Region syncs"),
@@ -128,12 +135,20 @@ static const gchar *_snap_mode_strings[] = {
        0
 };
 
+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_("Marker"),
        0
 };
 
@@ -156,25 +171,9 @@ show_me_the_size (Requisition* r, const char* what)
        cerr << "size of " << what << " = " << r->width << " x " << r->height << endl;
 }
 
-void 
-check_adjustment (Gtk::Adjustment* adj)
-{
-       cerr << "CHANGE adj  = " 
-            << adj->get_lower () <<  ' '
-            << adj->get_upper () <<  ' '
-            << adj->get_value () <<  ' '
-            << adj->get_step_increment () <<  ' '
-            << adj->get_page_increment () <<  ' '
-            << adj->get_page_size () <<  ' '
-            << endl;
-
-}
-
-Editor::Editor (AudioEngine& eng) 
-       : engine (eng),
-
+Editor::Editor ()
+       : 
          /* time display buttons */
-
          minsec_label (_("Mins:Secs")),
          bbt_label (_("Bars:Beats")),
          smpte_label (_("Timecode")),
@@ -228,6 +227,7 @@ Editor::Editor (AudioEngine& eng)
        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;
@@ -239,15 +239,21 @@ Editor::Editor (AudioEngine& eng)
        drag_info.item = 0;
        current_mixer_strip = 0;
        current_bbt_points = 0;
-
-       snap_type_strings = I18N (_snap_type_strings);
-       snap_mode_strings = I18N (_snap_mode_strings);
-       zoom_focus_strings = I18N(_zoom_focus_strings);
+       
+       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);
 
        snap_type = SnapToFrame;
        set_snap_to (snap_type);
+
        snap_mode = SnapNormal;
        set_snap_mode (snap_mode);
+
+       _edit_point = EditAtMouse;
+       set_edit_point_preference (_edit_point);
+
        snap_threshold = 5.0;
        bbt_beat_subdivision = 4;
        canvas_width = 0;
@@ -255,6 +261,7 @@ Editor::Editor (AudioEngine& eng)
        autoscroll_active = false;
        autoscroll_timeout_tag = -1;
        interthread_progress_window = 0;
+       logo_item = 0;
 
 #ifdef FFT_ANALYSIS
        analysis_window = 0;
@@ -266,12 +273,12 @@ 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;
        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; 
        have_pending_keyboard_selection = false;
@@ -299,23 +306,28 @@ Editor::Editor (AudioEngine& eng)
        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;
        in_edit_group_row_change = false;
        last_canvas_frame = 0;
-       edit_cursor = 0;
        playhead_cursor = 0;
        button_release_can_deselect = true;
        canvas_idle_queued = false;
        _dragging_playhead = false;
        _dragging_hscrollbar = false;
 
-       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];
+       scrubbing_direction = 0;
+
+       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;
@@ -334,7 +346,7 @@ Editor::Editor (AudioEngine& eng)
 
        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));
+       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);
@@ -366,12 +378,13 @@ Editor::Editor (AudioEngine& eng)
 
        edit_cursor_clock.ValueChanged.connect (mem_fun(*this, &Editor::edit_cursor_clock_changed));
        
+       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) + 2);
 
        bbt_label.set_name ("EditorTimeButton");
        bbt_label.set_size_request (-1, (int)timebar_height);
@@ -424,6 +437,9 @@ Editor::Editor (AudioEngine& eng)
        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.property_shadow_type() = Gtk::SHADOW_OUT;
+
        /* these enable us to have a dedicated window (for cursor setting, etc.) 
           for the canvas areas.
        */
@@ -441,7 +457,7 @@ Editor::Editor (AudioEngine& eng)
        
        edit_packer.attach (edit_vscrollbar,         0, 1, 1, 3,    FILL,        FILL|EXPAND, 0, 0);
 
-       edit_packer.attach (time_button_event_box,   1, 2, 0, 1,    FILL,        FILL, 0, 0);
+       edit_packer.attach (time_button_frame,       0, 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 (controls_layout,         1, 2, 1, 2,    FILL,        FILL|EXPAND, 0, 0);
@@ -471,6 +487,7 @@ Editor::Editor (AudioEngine& eng)
        
        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));
+       route_display_model->signal_rows_reordered().connect (mem_fun (*this, &Editor::track_list_reorder));
 
        route_list_display.signal_button_press_event().connect (mem_fun (*this, &Editor::route_list_display_button_press), false);
 
@@ -510,7 +527,7 @@ Editor::Editor (AudioEngine& eng)
 
        edit_group_display.set_name ("EditGroupList");
        edit_group_display.get_selection()->set_mode (SELECTION_SINGLE);
-       edit_group_display.set_headers_visible (false);
+       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);
@@ -662,6 +679,9 @@ 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 */
@@ -710,7 +730,10 @@ Editor::Editor (AudioEngine& eng)
                set_icon_list (window_icons);
                set_default_icon_list (window_icons);
        }
-       set_title (_("ardour: editor"));
+
+       WindowTitle title(Glib::get_application_name());
+       title += _("Editor");
+       set_title (title.get_string());
        set_wmclass (X_("ardour_editor"), "Ardour");
 
        add (vpacker);
@@ -727,6 +750,7 @@ Editor::Editor (AudioEngine& eng)
        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 ();
@@ -821,9 +845,26 @@ void
 Editor::tie_vertical_scrolling ()
 {
        double y1 = vertical_adjustment.get_value();
+
+       playhead_cursor->set_y_axis (y1);
+       if (logo_item) {
+               logo_item->property_y() = y1;
+       }
+
        controls_layout.get_vadjustment()->set_value (y1);
-       playhead_cursor->set_y_axis(y1);
-       edit_cursor->set_y_axis(y1);
+
+#ifdef GTKOSX
+       /* the way idle updates and immediate window flushing work on GTK-Quartz
+          requires that we force an immediate redraw right here. The controls
+          layout will do the same all by itself, as does the canvas widget, but
+          most of the time, the canvas itself hasn't updated itself because its
+          idle handler hasn't run. consequently, the call that its layout makes
+          to gdk_window_process_updates() finds nothing to do. here, we force
+          the update to happen, then request a flush of the new window state.
+       */
+       track_canvas.update_now ();
+       gdk_window_process_updates (GTK_LAYOUT(track_canvas.gobj())->bin_window, true);
+#endif
 }
 
 void
@@ -843,12 +884,8 @@ Editor::instant_save ()
 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());
-       }
 }
 
-
 void
 Editor::zoom_adjustment_changed ()
 {
@@ -879,40 +916,65 @@ Editor::control_scroll (float fraction)
        }
 
        double step = fraction * current_page_frames();
-       nframes_t target;
 
-       if ((fraction < 0.0f) && (session->transport_frame() < (nframes_t) fabs(step))) {
-               target = 0;
-       } else if ((fraction > 0.0f) && (max_frames - session->transport_frame() < step)) {
-               target = (max_frames - (current_page_frames()*2)); // allow room for slop in where the PH is on the screen
+       /*
+               _control_scroll_target is an optional<T>
+       
+               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;
+       }
+
+       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 {
-               target = (session->transport_frame() + (nframes_t) floor ((fraction * current_page_frames())));
+               *_control_scroll_target += (nframes_t) floor (step);
        }
 
        /* move visuals, we'll catch up with it later */
 
-       playhead_cursor->set_position (target);
-
-       if (target > (current_page_frames() / 2)) {
+       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 (target - (current_page_frames()/2));
+               reset_x_origin (*_control_scroll_target - (current_page_frames()/2));
        } else {
                reset_x_origin (0);
        }
 
-       /* cancel the existing */
+       /*
+               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 */
 
        control_scroll_connection.disconnect ();
 
-       /* add the next one */
+       /* add the next timeout */
 
-       control_scroll_connection = Glib::signal_timeout().connect (bind (mem_fun (*this, &Editor::deferred_control_scroll), target), 50);
+       control_scroll_connection = Glib::signal_timeout().connect (bind (mem_fun (*this, &Editor::deferred_control_scroll), *_control_scroll_target), 250);
 }
 
 bool
 Editor::deferred_control_scroll (nframes_t target)
 {
-       session->request_locate (target);
+       session->request_locate (*_control_scroll_target, session->transport_rolling());
+       // reset for next stream
+       _control_scroll_target = boost::none;
+       _dragging_playhead = false;
        return false;
 }
 
@@ -1008,24 +1070,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());
        }
 }
 
@@ -1072,7 +1131,7 @@ Editor::connect_to_session (Session *t)
 
        session_connections.push_back (session->SMPTEOffsetChanged.connect (mem_fun(*this, &Editor::update_just_smpte)));
 
-       session_connections.push_back (session->tempo_map().StateChanged.connect (bind (mem_fun(*this, &Editor::tempo_map_changed), false)));
+       session_connections.push_back (session->tempo_map().StateChanged.connect (mem_fun(*this, &Editor::tempo_map_changed)));
 
        edit_groups_changed ();
 
@@ -1094,8 +1153,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"));
        }
@@ -1108,8 +1166,7 @@ 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"));
        }
@@ -1125,6 +1182,10 @@ 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));
 
+       if (sfbrowser) {
+               sfbrowser->set_session (session);
+       }
+
        handle_new_duration ();
 
        redisplay_regions ();
@@ -1239,12 +1300,17 @@ Editor::popup_fade_context_menu (int button, int32_t time, ArdourCanvas::Item* i
                }
                
                items.push_back (SeparatorElem());
-               
-               items.push_back (MenuElem (_("Linear"), bind (mem_fun (*this, &Editor::set_fade_in_shape), AudioRegion::Linear)));
-               items.push_back (MenuElem (_("Slowest"), bind (mem_fun (*this, &Editor::set_fade_in_shape), AudioRegion::LogB)));
-               items.push_back (MenuElem (_("Slow"), bind (mem_fun (*this, &Editor::set_fade_in_shape), AudioRegion::Fast)));
-               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)));
+
+               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:
@@ -1256,13 +1322,17 @@ Editor::popup_fade_context_menu (int button, int32_t time, ArdourCanvas::Item* i
                }
                
                items.push_back (SeparatorElem());
-               
-               items.push_back (MenuElem (_("Linear"), bind (mem_fun (*this, &Editor::set_fade_out_shape), AudioRegion::Linear)));
-               items.push_back (MenuElem (_("Slowest"), bind (mem_fun (*this, &Editor::set_fade_out_shape), AudioRegion::Fast)));
-               items.push_back (MenuElem (_("Slow"), bind (mem_fun (*this, &Editor::set_fade_out_shape), AudioRegion::LogB)));
-               items.push_back (MenuElem (_("Fast"), bind (mem_fun (*this, &Editor::set_fade_out_shape), AudioRegion::LogA)));
-               items.push_back (MenuElem (_("Fastest"), bind (mem_fun (*this, &Editor::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:
@@ -1329,10 +1399,10 @@ Editor::popup_track_context_menu (int button, int32_t time, ItemType item_type,
        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);
                                }
                        }
                        /*
@@ -1362,7 +1432,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_audio_trackview && clicked_audio_trackview->audio_track()) {
 
                /* Bounce to disk */
                
@@ -1529,7 +1599,8 @@ Editor::build_track_selection_context_menu (nframes_t ignored)
        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;
 }
@@ -1615,7 +1686,7 @@ Editor::add_region_context_items (AudioStreamView* sv, boost::shared_ptr<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());
 
@@ -1629,62 +1700,83 @@ Editor::add_region_context_items (AudioStreamView* sv, boost::shared_ptr<Region>
 
        items.push_back (SeparatorElem());
 
-       items.push_back (CheckMenuElem (_("Lock"), mem_fun(*this, &Editor::toggle_region_lock)));
+       sigc::connection fooc;
+
+       items.push_back (CheckMenuElem (_("Lock")));
        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"), mem_fun(*this, &Editor::toggle_region_mute)));
+       items.push_back (CheckMenuElem (_("Mute")));
        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()) {
+               fooc.block (true);
                region_mute_item->set_active();
+               fooc.block (false);
        }
-       items.push_back (CheckMenuElem (_("Opaque"), mem_fun(*this, &Editor::toggle_region_opaque)));
-       region_opaque_item = static_cast<CheckMenuItem*>(&items.back());
-       if (region->opaque()) {
-               region_opaque_item->set_active();
+       
+       if (!Profile->get_sae()) {
+               items.push_back (CheckMenuElem (_("Opaque")));
+               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 (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);
-
-               items.push_back (MenuElem (_("Reset Envelope"), mem_fun(*this, &Editor::reset_region_gain_envelopes)));
                
-               items.push_back (CheckMenuElem (_("Envelope Visible"), mem_fun(*this, &Editor::toggle_gain_envelope_visibility)));
-               region_envelope_visible_item = static_cast<CheckMenuItem*> (&items.back());
-
-               if (arv->envelope_visible()) {
-                       region_envelope_visible_item->set_active (true);
-               }
-
-               items.push_back (CheckMenuElem (_("Envelope Active"), mem_fun(*this, &Editor::toggle_gain_envelope_active)));
-               region_envelope_active_item = static_cast<CheckMenuItem*> (&items.back());
+               if (!Profile->get_sae()) {
+                       items.push_back (MenuElem (_("Reset Envelope"), mem_fun(*this, &Editor::reset_region_gain_envelopes)));
+
+                       items.push_back (CheckMenuElem (_("Envelope Visible")));
+                       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")));
+                       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);
+                       }
 
-               if (ar->envelope_active()) {
-                       region_envelope_active_item->set_active (true);
+                       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)));
                }
        }
+
        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)));
@@ -1709,8 +1801,8 @@ Editor::add_region_context_items (AudioStreamView* sv, boost::shared_ptr<Region>
        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)));
                             
        items.push_back (MenuElem (_("Trim"), *trim_menu));
        items.push_back (SeparatorElem());
@@ -1747,15 +1839,12 @@ Editor::add_region_context_items (AudioStreamView* sv, boost::shared_ptr<Region>
 }
 
 void
-Editor::add_selection_context_items (Menu_Helpers::MenuList& edit_items)
+Editor::add_selection_context_items (Menu_Helpers::MenuList& items)
 {
        using namespace Menu_Helpers;
-       Menu     *selection_menu = manage (new Menu);
-       MenuList& items       = selection_menu->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());
@@ -1763,15 +1852,22 @@ 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 (_("Add Range Markers"), mem_fun (*this, &Editor::add_location_from_selection)));
-       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 (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)));
@@ -1780,9 +1876,6 @@ Editor::add_selection_context_items (Menu_Helpers::MenuList& edit_items)
        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
@@ -1796,7 +1889,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"), mem_fun(*this, &Editor::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());
@@ -1818,11 +1911,15 @@ 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 between cursors"), bind (mem_fun(*this, &Editor::select_all_selectables_between_cursors), playhead_cursor, edit_cursor)));
+       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));
@@ -1835,8 +1932,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());
 
@@ -1863,9 +1959,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));
 }
@@ -1881,7 +1977,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"), mem_fun(*this, &Editor::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));
 
@@ -1896,8 +1992,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)));
 
@@ -1919,9 +2015,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));
 }
@@ -1965,6 +2061,18 @@ Editor::set_snap_mode (SnapMode mode)
 
        instant_save ();
 }
+void
+Editor::set_edit_point_preference (EditPoint 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);
+       }
+
+       instant_save ();
+}
 
 int
 Editor::set_state (const XMLNode& node)
@@ -2015,11 +2123,8 @@ Editor::set_state (const XMLNode& node)
                horizontal_adjustment.set_value (0);
        }
 
-       if (session && (prop = node.property ("edit-cursor"))) {
-               nframes_t pos = atol (prop->value().c_str());
-               edit_cursor->set_position (pos);
-       } else {
-               edit_cursor->set_position (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"))) {
@@ -2038,6 +2143,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 */
@@ -2085,16 +2194,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()));
@@ -2162,6 +2271,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);
@@ -2171,10 +2282,10 @@ 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);
-       snprintf (buf, sizeof (buf), "%" PRIu32, edit_cursor->current_frame);
-       node->add_property ("edit-cursor", buf);
 
        node->add_property ("show-waveforms", _show_waveforms ? "yes" : "no");
        node->add_property ("show-waveforms-recording", _show_waveforms_recording ? "yes" : "no");
@@ -2211,7 +2322,7 @@ Editor::trackview_by_y_position (double y)
 }
 
 void
-Editor::snap_to (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;
@@ -2220,11 +2331,11 @@ Editor::snap_to (nframes_t& start, int32_t direction, bool for_mark)
                return;
        }
 
-       const nframes_t one_second = session->frame_rate();
-       const nframes_t one_minute = session->frame_rate() * 60;
-       const nframes_t one_smpte_second = (nframes_t)(rint(session->smpte_frames_per_second()) * session->frames_per_smpte_frame());
-       nframes_t one_smpte_minute = (nframes_t)(rint(session->smpte_frames_per_second()) * session->frames_per_smpte_frame() * 60);
-       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:
@@ -2331,8 +2442,8 @@ Editor::snap_to (nframes_t& start, int32_t direction, bool for_mark)
                 start = session->tempo_map().round_to_beat_subdivision (start, 3);
                 break;
 
-       case SnapToEditCursor:
-               start = edit_cursor->current_frame;
+       case SnapToEditPoint:
+               start = get_preferred_edit_position ();
                break;
 
        case SnapToMark:
@@ -2546,7 +2657,7 @@ Editor::setup_toolbar ()
        ARDOUR_UI::instance()->tooltips().set_tip (zoom_out_full_button, _("Zoom to Session"));
 
        zoom_focus_selector.set_name ("ZoomFocusSelector");
-       Gtkmm2ext::set_size_request_to_display_given_text (zoom_focus_selector, "Edit Cursor", FUDGE, 0);
+       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"));
@@ -2556,8 +2667,6 @@ Editor::setup_toolbar ()
        zoom_box.pack_start (zoom_in_button, false, false);
        zoom_box.pack_start (zoom_out_full_button, false, false);
 
-       /* Edit Cursor / Snap */
-
        snap_box.set_spacing (1);
        snap_box.set_border_width (2);
 
@@ -2565,17 +2674,24 @@ Editor::setup_toolbar ()
        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));
-       ARDOUR_UI::instance()->tooltips().set_tip (snap_type_selector, _("Unit to snap cursors and ranges to"));
+       ARDOUR_UI::instance()->tooltips().set_tip (snap_type_selector, _("Snap/Grid Units"));
 
        snap_mode_selector.set_name ("SnapModeSelector");
        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"));
+
+       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"));
 
        snap_box.pack_start (edit_cursor_clock, false, false);
        snap_box.pack_start (snap_mode_selector, false, false);
        snap_box.pack_start (snap_type_selector, false, false);
-
+       snap_box.pack_start (edit_point_selector, false, false);
 
        /* Nudge */
 
@@ -2646,7 +2762,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.
                */
@@ -2696,10 +2812,35 @@ 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;
+
+                       
+                       ustring p = *i;
                         PBD::url_decode (p);
-                       paths.push_back (p.substr (7));
+
+                       // 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));
                }
        }
 
@@ -2720,6 +2861,8 @@ Editor::map_transport_state ()
        if (session->transport_stopped()) {
                have_pending_keyboard_selection = false;
        }
+
+       update_loop_range_view (true);
 }
 
 /* UNDO/REDO */
@@ -2767,7 +2910,7 @@ void
 Editor::begin_reversible_command (string name)
 {
        if (session) {
-                before = &get_state();
+                // before = &get_state();
                session->begin_reversible_command (name);
        }
 }
@@ -2776,7 +2919,8 @@ void
 Editor::commit_reversible_command ()
 {
        if (session) {
-               session->commit_reversible_command (new MementoCommand<Editor>(*this, before, &get_state()));
+               // session->commit_reversible_command (new MementoCommand<Editor>(*this, before, &get_state()));
+               session->commit_reversible_command ();
        }
 }
 
@@ -2831,32 +2975,34 @@ Editor::history_changed ()
 void
 Editor::duplicate_dialog (bool dup_region)
 {
-       if (dup_region) {
-               if (clicked_regionview == 0) {
-                       return;
-               }
-       } else {
-               if (selection->time.length() == 0) {
-                       return;
-               }
+       if (selection->regions.empty() && (selection->time.length() == 0)) {
+               return;
        }
 
        ArdourDialog win ("duplicate dialog");
-       Entry  entry;
        Label  label (_("Duplicate how many times?"));
+       Adjustment adjustment (1.0, 1.0, 1000000.0, 1.0, 5.0);
+       SpinButton spinner (adjustment);
 
+       win.get_vbox()->set_spacing (12);
        win.get_vbox()->pack_start (label);
-       win.add_action_widget (entry, RESPONSE_ACCEPT);
+
+       /* 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);
 
-       entry.set_text ("1");
-       set_size_request_to_display_given_text (entry, X_("12345678"), 20, 15);
-       entry.select_region (0, -1);
-       entry.grab_focus ();
-
+       spinner.grab_focus ();
 
        switch (win.run ()) {
        case RESPONSE_ACCEPT:
@@ -2865,17 +3011,12 @@ Editor::duplicate_dialog (bool dup_region)
                return;
        }
 
-       string text = entry.get_text();
-       float times;
+       float times = adjustment.get_value();
 
-       if (sscanf (text.c_str(), "%f", &times) == 1) {
-               if (dup_region) {
-                       RegionSelection regions;
-                       regions.add (clicked_regionview);
-                       duplicate_some_regions (regions, times);
-               } else {
-                       duplicate_selection (times);
-               }
+       if (!selection->regions.empty()) {
+               duplicate_some_regions (selection->regions, times);
+       } else {
+               duplicate_selection (times);
        }
 }
 
@@ -2952,8 +3093,8 @@ Editor::snap_type_selection_done ()
                snaptype = SnapToBar;
        } else if (choice == _("Marks")) {
                snaptype = SnapToMark;
-       } else if (choice == _("Edit Cursor")) {
-               snaptype = SnapToEditCursor;
+       } else if (choice == _("Edit Point")) {
+               snaptype = SnapToEditPoint;
        } else if (choice == _("Region starts")) {
                snaptype = SnapToRegionStart;
        } else if (choice == _("Region ends")) {
@@ -3003,6 +3144,27 @@ Editor::snap_mode_selection_done ()
        }
 }
 
+void
+Editor::edit_point_selection_done ()
+{
+       string choice = edit_point_selector.get_active_text();
+       EditPoint ep = EditAtSelectedMarker;
+
+       if (choice == _("Marker")) {
+               _edit_point = EditAtSelectedMarker;
+       } else if (choice == _("Playhead")) {
+               _edit_point = EditAtPlayhead;
+       } else {
+               _edit_point = EditAtMouse;
+       }
+
+       RefPtr<RadioAction> ract = edit_point_action (ep);
+
+       if (ract) {
+               ract->set_active (true);
+       }
+}
+
 void
 Editor::zoom_focus_selection_done ()
 {
@@ -3017,7 +3179,9 @@ Editor::zoom_focus_selection_done ()
                focus_type = ZoomFocusCenter;
        } else if (choice == _("Playhead")) {
                focus_type = ZoomFocusPlayhead;
-       } else if (choice == _("Edit Cursor")) {
+       } else if (choice == _("Mouse")) {
+               focus_type = ZoomFocusMouse;
+       } else if (choice == _("Edit Point")) {
                focus_type = ZoomFocusEdit;
        } 
        
@@ -3032,7 +3196,7 @@ 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;
 }
@@ -3288,7 +3452,7 @@ Editor::playlist_deletion_dialog (boost::shared_ptr<Playlist> pl)
        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 ()) {
@@ -3372,6 +3536,9 @@ Editor::control_layout_scroll (GdkEventScroll* ev)
        return false;
 }
 
+
+/** A new snapshot has been selected.
+ */
 void
 Editor::snapshot_display_selection_changed ()
 {
@@ -3396,7 +3563,93 @@ 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);
+
+       items.push_back (MenuElem (_("Remove"), bind (mem_fun (*this, &Editor::remove_snapshot), snapshot_name)));
+       if (!modification_allowed) {
+               items.back().set_sensitive (false);
+       }
+
+       items.push_back (MenuElem (_("Rename"), bind (mem_fun (*this, &Editor::rename_snapshot), snapshot_name)));
+       if (!modification_allowed) {
+               items.back().set_sensitive (false);
+       }
+
+       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
@@ -3490,27 +3743,47 @@ Editor::restore_editing_space ()
        );
 }
 
+/**
+ *  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_audio_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_audio_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_audio_tracks (mem_fun (*this, &Editor::mapped_clear_playlist), v);
        commit_reversible_command ();
 }
 
@@ -3613,7 +3886,6 @@ Editor::set_frames_per_unit (double fpu)
        reset_hscrollbar_stepping ();
        reset_scrolling_region ();
        
-       if (edit_cursor) edit_cursor->set_position (edit_cursor->current_frame);
        if (playhead_cursor) playhead_cursor->set_position (playhead_cursor->current_frame);
 
        instant_save ();
@@ -3665,11 +3937,11 @@ Editor::idle_visual_changer ()
                        /* the signal handler will do the rest */
                } else {
                        update_fixed_rulers();
-                       tempo_map_changed (Change (0), true);
+                       redisplay_tempo (true);
                }
        }
 
-       return 0;
+       return 0; /* this is always a one-shot call */
 }
 
 struct EditorOrderTimeAxisSorter {
@@ -3685,3 +3957,131 @@ Editor::sort_track_selection ()
        selection->tracks.sort (cmp);
 }
 
+nframes64_t
+Editor::get_preferred_edit_position() const
+{
+       bool ignored;
+       nframes64_t where;
+
+       // XXX EDIT CURSOR used to sync with edit cursor clock
+
+       switch (_edit_point) {
+       case EditAtPlayhead:
+               return playhead_cursor->current_frame;
+               
+       case EditAtSelectedMarker:
+               if (!selection->markers.empty()) {
+                       bool whocares;
+                       Location* loc = find_location_from_marker (selection->markers.front(), whocares);
+                       if (loc) {
+                               return loc->start();
+                       }
+               } 
+               /* fallthru */
+               
+       default:
+       case EditAtMouse:
+               if (mouse_frame (where, ignored)) {
+                       return where;
+               } 
+       }
+
+       return -1;
+}
+
+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;
+}