Editor GUI touchups:
[ardour.git] / gtk2_ardour / editor.cc
index 95fa048f036de7861c14fdd316ab88eff565968a..39e3098fa48509619f6418d44cd616f628810226 100644 (file)
@@ -15,7 +15,6 @@
     along with this program; if not, write to the Free Software
     Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
 
-    $Id$
 */
 
 #include <unistd.h>
 
 #include <pbd/convert.h>
 #include <pbd/error.h>
+#include <pbd/stacktrace.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 <ardour/audio_track.h>
 #include <ardour/audio_diskstream.h>
@@ -53,7 +56,6 @@
 
 #include "ardour_ui.h"
 #include "editor.h"
-#include "grouped_buttons.h"
 #include "keyboard.h"
 #include "marker.h"
 #include "playlist_selector.h"
@@ -68,7 +70,6 @@
 #include "editing.h"
 #include "public_editor.h"
 #include "crossfade_edit.h"
-#include "audio_time_axis.h"
 #include "canvas_impl.h"
 #include "actions.h"
 #include "gui_thread.h"
@@ -170,9 +171,8 @@ check_adjustment (Gtk::Adjustment* adj)
 
 }
 
-Editor::Editor (AudioEngine& eng) 
-       : engine (eng),
-
+Editor::Editor ()
+       : 
          /* time display buttons */
 
          minsec_label (_("Mins:Secs")),
@@ -196,8 +196,8 @@ Editor::Editor (AudioEngine& eng)
 
          /* tool bar related */
 
-         edit_cursor_clock (X_("EditCursorClock"), true),
-         zoom_range_clock (X_("ZoomRangeClock"), true, true),
+         edit_cursor_clock (X_("editcursor"), false, X_("EditCursorClock"), true),
+         zoom_range_clock (X_("zoomrange"), false, X_("ZoomRangeClock"), true, true),
          
          toolbar_selection_clock_table (2,3),
          
@@ -210,7 +210,7 @@ Editor::Editor (AudioEngine& eng)
 
          /* nudge */
 
-         nudge_clock (X_("NudgeClock"), true, true)
+         nudge_clock (X_("nudge"), false, X_("NudgeClock"), true, true)
 
 {
        constructed = false;
@@ -271,7 +271,6 @@ Editor::Editor (AudioEngine& eng)
        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;
@@ -283,6 +282,7 @@ Editor::Editor (AudioEngine& eng)
        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;
@@ -290,7 +290,6 @@ Editor::Editor (AudioEngine& eng)
        new_transport_marker_menu = 0;
        editor_mixer_strip_width = Wide;
        show_editor_mixer_when_tracks_arrive = false;
-       repos_zoom_queued = false;
        region_edit_menu_split_item = 0;
        temp_location = 0;
        region_edit_menu_split_multichannel_item = 0;
@@ -309,6 +308,7 @@ Editor::Editor (AudioEngine& eng)
        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];
@@ -321,7 +321,7 @@ Editor::Editor (AudioEngine& eng)
        
        set_mouse_mode (MouseObject, true);
 
-       frames_per_unit = 2048; /* too early to use set_frames_per_unit */
+       frames_per_unit = 2048; /* too early to use reset_zoom () */
        reset_hscrollbar_stepping ();
        
        zoom_focus = ZoomFocusLeft;
@@ -354,21 +354,24 @@ 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 ();
 
        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);
@@ -421,6 +424,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.
        */
@@ -438,7 +444,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);
@@ -503,12 +509,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);
@@ -594,11 +599,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_release_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 */
@@ -606,7 +612,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);
@@ -707,7 +713,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);
@@ -823,65 +832,6 @@ Editor::tie_vertical_scrolling ()
        edit_cursor->set_y_axis(y1);
 }
 
-void
-Editor::set_frames_per_unit (double fpu)
-{
-       nframes_t frames;
-
-       if (fpu == frames_per_unit) {
-               return;
-       }
-
-       if (fpu < 2.0) {
-               fpu = 2.0;
-       }
-
-       // convert fpu to frame count
-
-       frames = (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) {
-               return;
-       }
-
-       if (fpu == frames_per_unit) {
-               return;
-       }
-
-       frames_per_unit = fpu;
-
-       if (frames != 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 (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 ()
 {
@@ -896,26 +846,6 @@ Editor::instant_save ()
        }
 }
 
-void
-Editor::reposition_x_origin (nframes_t frame)
-{
-       if (frame != leftmost_frame) {
-               leftmost_frame = frame;
-               
-               nframes_t rightmost_frame = leftmost_frame + current_page_frames ();
-
-               if (rightmost_frame > last_canvas_frame) {
-                       last_canvas_frame = rightmost_frame;
-                       reset_scrolling_region ();
-               }
-
-               horizontal_adjustment.set_value (frame/frames_per_unit);
-       } else {
-               update_fixed_rulers();
-               tempo_map_changed (Change (0));
-       }
-}
-
 void
 Editor::edit_cursor_clock_changed()
 {
@@ -971,9 +901,9 @@ Editor::control_scroll (float fraction)
 
        if (target > (current_page_frames() / 2)) {
                /* try to center PH in window */
-               reposition_x_origin (target - (current_page_frames()/2));
+               reset_x_origin (target - (current_page_frames()/2));
        } else {
-               reposition_x_origin (0);
+               reset_x_origin (0);
        }
 
        /* cancel the existing */
@@ -992,36 +922,6 @@ Editor::deferred_control_scroll (nframes_t target)
        return false;
 }
 
-void 
-Editor::canvas_horizontally_scrolled ()
-{
-
-       leftmost_frame = (nframes_t) floor (horizontal_adjustment.get_value() * frames_per_unit);
-       update_fixed_rulers ();
-       tempo_map_changed (Change (0));
-
-}
-
-void
-Editor::reposition_and_zoom (nframes_t frame, double nfpu)
-{
-       if (!repos_zoom_queued) {
-               repos_zoom_queued = true;
-               Glib::signal_idle().connect (bind (mem_fun(*this, &Editor::deferred_reposition_and_zoom), frame, nfpu));
-       }
-}
-
-gint
-Editor::deferred_reposition_and_zoom (nframes_t frame, double nfpu)
-{
-
-       set_frames_per_unit (nfpu);
-       reposition_x_origin  (frame);
-       repos_zoom_queued = false;
-       
-       return FALSE;
-}
-
 void
 Editor::on_realize ()
 {
@@ -1079,7 +979,7 @@ Editor::center_screen_internal (nframes_t frame, float page)
                frame = 0;
        }
 
-    reposition_x_origin (frame);
+       reset_x_origin (frame);
 }
 
 void
@@ -1091,6 +991,7 @@ Editor::handle_new_duration ()
                                  
        if (new_end > last_canvas_frame) {
                last_canvas_frame = new_end;
+               horizontal_adjustment.set_upper (last_canvas_frame / frames_per_unit);
                reset_scrolling_region ();
        }
 
@@ -1113,24 +1014,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());
        }
 }
 
@@ -1139,6 +1037,13 @@ 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();
        }
@@ -1170,7 +1075,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 (mem_fun(*this, &Editor::tempo_map_changed)));
+       session_connections.push_back (session->tempo_map().StateChanged.connect (bind (mem_fun(*this, &Editor::tempo_map_changed), false)));
 
        edit_groups_changed ();
 
@@ -1192,8 +1097,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"));
        }
@@ -1206,8 +1110,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"));
        }
@@ -1235,27 +1138,12 @@ 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_value (0);
-
        restore_ruler_visibility ();
        //tempo_map_changed (Change (0));
        session->tempo_map().apply_with_metrics (*this, &Editor::draw_metric_marks);
 
-       edit_cursor->set_position (0);
-       playhead_cursor->set_position (0);
-
        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 ()) {
@@ -1346,35 +1234,35 @@ Editor::popup_fade_context_menu (int button, int32_t time, ArdourCanvas::Item* i
        case FadeInItem:
        case FadeInHandleItem:
                if (arv->audio_region()->fade_in_active()) {
-                       items.push_back (MenuElem (_("Deactivate"), bind (mem_fun (*arv, &AudioRegionView::set_fade_in_active), false)));
+                       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, &AudioRegionView::set_fade_in_shape), AudioRegion::Linear)));
-               items.push_back (MenuElem (_("Slowest"), bind (mem_fun (*arv, &AudioRegionView::set_fade_in_shape), AudioRegion::LogB)));
-               items.push_back (MenuElem (_("Slow"), bind (mem_fun (*arv, &AudioRegionView::set_fade_in_shape), AudioRegion::Fast)));
-               items.push_back (MenuElem (_("Fast"), bind (mem_fun (*arv, &AudioRegionView::set_fade_in_shape), AudioRegion::LogA)));
-               items.push_back (MenuElem (_("Fastest"), bind (mem_fun (*arv, &AudioRegionView::set_fade_in_shape), AudioRegion::Slow)));
+               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->audio_region()->fade_out_active()) {
-                       items.push_back (MenuElem (_("Deactivate"), bind (mem_fun (*arv, &AudioRegionView::set_fade_out_active), false)));
+                       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, &AudioRegionView::set_fade_out_shape), AudioRegion::Linear)));
-               items.push_back (MenuElem (_("Slowest"), bind (mem_fun (*arv, &AudioRegionView::set_fade_out_shape), AudioRegion::Fast)));
-               items.push_back (MenuElem (_("Slow"), bind (mem_fun (*arv, &AudioRegionView::set_fade_out_shape), AudioRegion::LogB)));
-               items.push_back (MenuElem (_("Fast"), bind (mem_fun (*arv, &AudioRegionView::set_fade_out_shape), AudioRegion::LogA)));
-               items.push_back (MenuElem (_("Fastest"), bind (mem_fun (*arv, &AudioRegionView::set_fade_out_shape), AudioRegion::Slow)));
+               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;
 
@@ -1648,7 +1536,7 @@ Editor::build_track_selection_context_menu (nframes_t ignored)
 }
 
 void
-Editor::add_crossfade_context_items (AudioStreamView* 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);
@@ -1662,8 +1550,8 @@ Editor::add_crossfade_context_items (AudioStreamView* view, Crossfade* xfade, Me
                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()) {
 
@@ -1722,7 +1610,7 @@ Editor::add_region_context_items (AudioStreamView* sv, boost::shared_ptr<Region>
           become selected.
        */
 
-       // region_menu->signal_map_event().connect (bind (mem_fun(*this, &Editor::set_selected_regionview_from_map_event), sv, boost::weak_ptr<Region>(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 (_("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)));
@@ -1889,7 +1777,7 @@ Editor::add_selection_context_items (Menu_Helpers::MenuList& edit_items)
        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)));
@@ -2113,12 +2001,34 @@ Editor::set_state (const XMLNode& node)
        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 (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 ("zoom-focus"))) {
                set_zoom_focus ((ZoomFocus) atoi (prop->value()));
        }
 
        if ((prop = node.property ("zoom"))) {
-               set_frames_per_unit (PBD::atof (prop->value()));
+               reset_zoom (PBD::atof (prop->value()));
        }
 
        if ((prop = node.property ("snap-to"))) {
@@ -2262,6 +2172,11 @@ Editor::get_state ()
        snprintf (buf, sizeof(buf), "%d", (int) snap_mode);
        node->add_property ("snap-mode", buf);
 
+       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");
        node->add_property ("show-measures", _show_measures ? "yes" : "no");
@@ -2308,7 +2223,8 @@ Editor::snap_to (nframes_t& start, int32_t direction, bool for_mark)
 
        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;
 
        switch (snap_type) {
@@ -2322,8 +2238,9 @@ Editor::snap_to (nframes_t& start, int32_t direction, bool for_mark)
                        start = (nframes_t) floor ((double) start / (one_second / 75)) * (one_second / 75);
                }
                break;
+
        case SnapToSMPTEFrame:
-               if (direction) {
+               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 = (nframes_t) (floor ((double) start / session->frames_per_smpte_frame()) *  session->frames_per_smpte_frame());
@@ -2337,10 +2254,10 @@ Editor::snap_to (nframes_t& start, int32_t direction, bool for_mark)
                } else {
                        start -= session->smpte_offset ();
                }    
-               if (direction > 0) {
-                       start = (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 = (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())
@@ -2358,10 +2275,10 @@ Editor::snap_to (nframes_t& start, int32_t direction, bool for_mark)
                } else {
                        start -= session->smpte_offset ();
                }
-               if (direction) {
-                       start = (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 = (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())
                {
@@ -2372,7 +2289,7 @@ Editor::snap_to (nframes_t& start, int32_t direction, bool for_mark)
                break;
                
        case SnapToSeconds:
-               if (direction) {
+               if (start % one_second > one_second / 2) {
                        start = (nframes_t) ceil ((double) start / one_second) * one_second;
                } else {
                        start = (nframes_t) floor ((double) start / one_second) * one_second;
@@ -2380,7 +2297,7 @@ Editor::snap_to (nframes_t& start, int32_t direction, bool for_mark)
                break;
                
        case SnapToMinutes:
-               if (direction) {
+               if (start % one_minute > one_minute / 2) {
                        start = (nframes_t) ceil ((double) start / one_minute) * one_minute;
                } else {
                        start = (nframes_t) floor ((double) start / one_minute) * one_minute;
@@ -2612,22 +2529,25 @@ Editor::setup_toolbar ()
        zoom_box.set_border_width (2);
 
        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"));
 
        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"));
-       
+
        zoom_focus_selector.set_name ("ZoomFocusSelector");
-       Gtkmm2ext::set_size_request_to_display_given_text (zoom_focus_selector, "Edit Cursor", 2+FUDGE, 0);
+       Gtkmm2ext::set_size_request_to_display_given_text (zoom_focus_selector, "Edit Cursor", 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"));
@@ -2801,6 +2721,8 @@ Editor::map_transport_state ()
        if (session->transport_stopped()) {
                have_pending_keyboard_selection = false;
        }
+
+       update_loop_range_view (true);
 }
 
 /* UNDO/REDO */
@@ -2861,626 +2783,89 @@ Editor::commit_reversible_command ()
        }
 }
 
-struct TrackViewByPositionSorter
+void
+Editor::set_edit_group_solo (Route& route, bool yn)
 {
-    bool operator() (const TimeAxisView* a, const TimeAxisView *b) {
-           return a->y_position < b->y_position;
-    }
-};
+       RouteGroup *edit_group;
 
-bool
-Editor::extend_selection_to_track (TimeAxisView& view)
-{
-       if (selection->selected (&view)) {
-               /* already selected, do nothing */
-               return false;
+       if ((edit_group = route.edit_group()) != 0) {
+               edit_group->apply (&Route::set_solo, yn, this);
+       } else {
+               route.set_solo (yn, this);
        }
+}
 
-       if (selection->tracks.empty()) {
-
-               if (!selection->selected (&view)) {
-                       selection->set (&view);
-                       return true;
-               } else {
-                       return false;
-               }
-       } 
-
-       /* something is already selected, so figure out which range of things to add */
-       
-       TrackViewList to_be_added;
-       TrackViewList sorted = track_views;
-       TrackViewByPositionSorter cmp;
-       bool passed_clicked = false;
-       bool forwards;
-
-       sorted.sort (cmp);
+void
+Editor::set_edit_group_mute (Route& route, bool yn)
+{
+       RouteGroup *edit_group = 0;
 
-       if (!selection->selected (&view)) {
-               to_be_added.push_back (&view);
+       if ((edit_group == route.edit_group()) != 0) {
+               edit_group->apply (&Route::set_mute, yn, this);
+       } else {
+               route.set_mute (yn, this);
        }
+}
+               
+void
+Editor::history_changed ()
+{
+       string label;
 
-       /* figure out if we should go forward or backwards */
-
-       for (TrackViewList::iterator i = sorted.begin(); i != sorted.end(); ++i) {
-
-               if ((*i) == &view) {
-                       passed_clicked = true;
+       if (undo_action && session) {
+               if (session->undo_depth() == 0) {
+                       label = _("Undo");
+               } else {
+                       label = string_compose(_("Undo (%1)"), session->next_undo());
                }
+               undo_action->property_label() = label;
+       }
 
-               if (selection->selected (*i)) {
-                       if (passed_clicked) {
-                               forwards = true;
-                       } else {
-                               forwards = false;
-                       }
-                       break;
+       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;
        }
-                       
-       passed_clicked = false;
-
-       if (forwards) {
+}
 
-               for (TrackViewList::iterator i = sorted.begin(); i != sorted.end(); ++i) {
-                                       
-                       if ((*i) == &view) {
-                               passed_clicked = true;
-                               continue;
-                       }
-                                       
-                       if (passed_clicked) {
-                               if ((*i)->hidden()) {
-                                       continue;
-                               }
-                               if (selection->selected (*i)) {
-                                       break;
-                               } else if (!(*i)->hidden()) {
-                                       to_be_added.push_back (*i);
-                               }
-                       }
+void
+Editor::duplicate_dialog (bool dup_region)
+{
+       if (dup_region) {
+               if (clicked_regionview == 0) {
+                       return;
                }
-
        } else {
-
-               for (TrackViewList::reverse_iterator r = sorted.rbegin(); r != sorted.rend(); ++r) {
-                                       
-                       if ((*r) == &view) {
-                               passed_clicked = true;
-                               continue;
-                       }
-                                       
-                       if (passed_clicked) {
-                                               
-                               if ((*r)->hidden()) {
-                                       continue;
-                               }
-                                               
-                               if (selection->selected (*r)) {
-                                       break;
-                               } else if (!(*r)->hidden()) {
-                                       to_be_added.push_back (*r);
-                               }
-                       }
+               if (selection->time.length() == 0) {
+                       return;
                }
        }
-                       
-       if (!to_be_added.empty()) {
-               selection->add (to_be_added);
-               return true;
-       }
-       
-       return false;
-}
 
+       ArdourDialog win ("duplicate dialog");
+       Entry  entry;
+       Label  label (_("Duplicate how many times?"));
 
-bool
-Editor::set_selected_track (TimeAxisView& view, Selection::Operation op, bool no_remove)
-{
-       bool commit = false;
-
-       switch (op) {
-       case Selection::Toggle:
-               if (selection->selected (&view)) {
-                       if (!no_remove) {
-                               selection->remove (&view);
-                               commit = true;
-                       }
-               } else {
-                       selection->add (&view);
-                       commit = false;
-               }
-               break;
+       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);
 
-       case Selection::Add:
-               if (!selection->selected (&view)) {
-                       selection->add (&view);
-                       commit = true;
-               }
-               break;
+       win.set_position (WIN_POS_MOUSE);
 
-       case Selection::Set:
-               if (selection->selected (&view) && selection->tracks.size() == 1) {
-                       /* no commit necessary */
-               } else {
-                       selection->set (&view);
-                       commit = true;
-               }
-               break;
-               
-       case Selection::Extend:
-               commit = extend_selection_to_track (view);
+       entry.set_text ("1");
+       set_size_request_to_display_given_text (entry, X_("12345678"), 20, 15);
+       entry.select_region (0, -1);
+       entry.grab_focus ();
+
+
+       switch (win.run ()) {
+       case RESPONSE_ACCEPT:
                break;
-       }
-
-       return commit;
-}
-
-bool
-Editor::set_selected_track_from_click (Selection::Operation op, bool no_remove)
-{
-       if (!clicked_trackview) {
-               return false;
-       }
-       
-       return set_selected_track (*clicked_trackview, op, no_remove);
-}
-
-bool
-Editor::set_selected_control_point_from_click (Selection::Operation op, bool no_remove)
-{
-       if (!clicked_control_point) {
-               return false;
-       }
-
-       /* select this point and any others that it represents */
-
-       double y1, y2;
-       nframes_t x1, x2;
-
-       x1 = pixel_to_frame (clicked_control_point->get_x() - 10);
-       x2 = pixel_to_frame (clicked_control_point->get_x() + 10);
-       y1 = clicked_control_point->get_x() - 10;
-       y2 = clicked_control_point->get_y() + 10;
-
-       return select_all_within (x1, x2, y1, y2, op);
-}
-
-void
-Editor::get_relevant_audio_tracks (set<AudioTimeAxisView*>& relevant_tracks)
-{
-       /* step one: get all selected tracks and all tracks in the relevant edit groups */
-
-       for (TrackSelection::iterator ti = selection->tracks.begin(); ti != selection->tracks.end(); ++ti) {
-
-               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);
-                                       }
-                               }
-                       }
-               } else {
-                       relevant_tracks.insert (atv);
-               }
-       }
-}
-
-void
-Editor::mapover_audio_tracks (slot<void,AudioTimeAxisView&,uint32_t> sl)
-{
-       set<AudioTimeAxisView*> relevant_tracks;
-
-       get_relevant_audio_tracks (relevant_tracks);
-
-       uint32_t sz = relevant_tracks.size();
-
-       for (set<AudioTimeAxisView*>::iterator ati = relevant_tracks.begin(); ati != relevant_tracks.end(); ++ati) {
-               sl (**ati, sz);
-       }
-}
-
-void
-Editor::mapped_set_selected_regionview_from_click (RouteTimeAxisView& tv, uint32_t ignored, 
-                                                  RegionView* basis, vector<RegionView*>* all_equivs)
-{
-       boost::shared_ptr<Playlist> pl;
-       vector<boost::shared_ptr<Region> > results;
-       RegionView* marv;
-       boost::shared_ptr<Diskstream> ds;
-
-       if ((ds = tv.get_diskstream()) == 0) {
-               /* bus */
-               return;
-       }
-
-       if (&tv == &basis->get_time_axis_view()) {
-               /* looking in same track as the original */
-               return;
-       }
-
-       
-       if ((pl = ds->playlist()) != 0) {
-               pl->get_equivalent_regions (basis->region(), results);
-       }
-       
-       for (vector<boost::shared_ptr<Region> >::iterator ir = results.begin(); ir != results.end(); ++ir) {
-               if ((marv = tv.view()->find_view (*ir)) != 0) {
-                       all_equivs->push_back (marv);
-               }
-       }
-}
-
-bool
-Editor::set_selected_regionview_from_click (bool press, Selection::Operation op, bool no_track_remove)
-{
-       vector<RegionView*> all_equivalent_regions;
-       bool commit = false;
-
-       if (!clicked_regionview || !clicked_audio_trackview) {
-               return false;
-       }
-
-       if (op == Selection::Toggle || op == Selection::Set) {
-               
-               mapover_audio_tracks (bind (mem_fun (*this, &Editor::mapped_set_selected_regionview_from_click), 
-                                           clicked_regionview, &all_equivalent_regions));
-               
-               
-               /* add clicked regionview since we skipped all other regions in the same track as the one it was in */
-               
-               all_equivalent_regions.push_back (clicked_regionview);
-               
-               switch (op) {
-               case Selection::Toggle:
-                       
-                       if (clicked_regionview->get_selected()) {
-                               if (press) {
-
-                                       /* whatever was clicked was selected already; do nothing here but allow
-                                          the button release to deselect it
-                                       */
-
-                                       button_release_can_deselect = true;
-
-                               } else {
-
-                                       if (button_release_can_deselect) {
-
-                                               /* just remove this one region, but only on a permitted button release */
-
-                                               selection->remove (clicked_regionview);
-                                               commit = true;
-
-                                               /* no more deselect action on button release till a new press
-                                                  finds an already selected object.
-                                               */
-
-                                               button_release_can_deselect = false;
-                                       }
-                               } 
-
-                       } else {
-
-                               if (press) {
-                                       /* add all the equivalent regions, but only on button press */
-                                       
-                                       if (!all_equivalent_regions.empty()) {
-                                               commit = true;
-                                       }
-                                       
-                                       for (vector<RegionView*>::iterator i = all_equivalent_regions.begin(); i != all_equivalent_regions.end(); ++i) {
-                                               selection->add (*i);
-                                       }
-                               } 
-                       }
-                       break;
-                       
-               case Selection::Set:
-                       if (!clicked_regionview->get_selected()) {
-                               selection->set (all_equivalent_regions);
-                               commit = true;
-                       } else {
-                               /* no commit necessary: clicked on an already selected region */
-                               goto out;
-                       }
-                       break;
-
-               default:
-                       /* silly compiler */
-                       break;
-               }
-
-       } else if (op == Selection::Extend) {
-
-               list<Selectable*> results;
-               nframes_t last_frame;
-               nframes_t first_frame;
-
-               /* 1. find the last selected regionview in the track that was clicked in */
-
-               last_frame = 0;
-               first_frame = max_frames;
-
-               for (RegionSelection::iterator x = selection->regions.begin(); x != selection->regions.end(); ++x) {
-                       if (&(*x)->get_time_axis_view() == &clicked_regionview->get_time_axis_view()) {
-
-                               if ((*x)->region()->last_frame() > last_frame) {
-                                       last_frame = (*x)->region()->last_frame();
-                               }
-
-                               if ((*x)->region()->first_frame() < first_frame) {
-                                       first_frame = (*x)->region()->first_frame();
-                               }
-                       }
-               }
-
-               /* 2. figure out the boundaries for our search for new objects */
-
-               switch (clicked_regionview->region()->coverage (first_frame, last_frame)) {
-               case OverlapNone:
-                       if (last_frame < clicked_regionview->region()->first_frame()) {
-                               first_frame = last_frame;
-                               last_frame = clicked_regionview->region()->last_frame();
-                       } else {
-                               last_frame = first_frame;
-                               first_frame = clicked_regionview->region()->first_frame();
-                       }
-                       break;
-
-               case OverlapExternal:
-                       if (last_frame < clicked_regionview->region()->first_frame()) {
-                               first_frame = last_frame;
-                               last_frame = clicked_regionview->region()->last_frame();
-                       } else {
-                               last_frame = first_frame;
-                               first_frame = clicked_regionview->region()->first_frame();
-                       }
-                       break;
-
-               case OverlapInternal:
-                       if (last_frame < clicked_regionview->region()->first_frame()) {
-                               first_frame = last_frame;
-                               last_frame = clicked_regionview->region()->last_frame();
-                       } else {
-                               last_frame = first_frame;
-                               first_frame = clicked_regionview->region()->first_frame();
-                       }
-                       break;
-
-               case OverlapStart:
-               case OverlapEnd:
-                       /* nothing to do except add clicked region to selection, since it
-                          overlaps with the existing selection in this track.
-                       */
-                       break;
-               }
-
-               /* 2. find all selectable objects (regionviews in this case) between that one and the end of the
-                     one that was clicked.
-               */
-
-               set<AudioTimeAxisView*> relevant_tracks;
-               
-               get_relevant_audio_tracks (relevant_tracks);
-               
-               for (set<AudioTimeAxisView*>::iterator t = relevant_tracks.begin(); t != relevant_tracks.end(); ++t) {
-                       (*t)->get_selectables (first_frame, last_frame, -1.0, -1.0, results);
-               }
-               
-               /* 3. convert to a vector of audio regions */
-
-               vector<RegionView*> regions;
-               
-               for (list<Selectable*>::iterator x = results.begin(); x != results.end(); ++x) {
-                       RegionView* arv;
-
-                       if ((arv = dynamic_cast<RegionView*>(*x)) != 0) {
-                               regions.push_back (arv);
-                       }
-               }
-
-               if (!regions.empty()) {
-                       selection->add (regions);
-                       commit = true;
-               }
-       }
-
-  out:
-       return commit;
-}
-
-void
-Editor::set_selected_regionview_from_region_list (boost::shared_ptr<Region> region, Selection::Operation op)
-{
-       vector<RegionView*> all_equivalent_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) {
-                                       all_equivalent_regions.push_back (marv);
-                               }
-                       }
-                       
-               }
-       }
-       
-       begin_reversible_command (_("set selected regions"));
-       
-       switch (op) {
-       case Selection::Toggle:
-               /* XXX this is not correct */
-               selection->toggle (all_equivalent_regions);
-               break;
-       case Selection::Set:
-               selection->set (all_equivalent_regions);
-               break;
-       case Selection::Extend:
-               selection->add (all_equivalent_regions);
-               break;
-       case Selection::Add:
-               selection->add (all_equivalent_regions);
-               break;
-       }
-
-       commit_reversible_command () ;
-}
-
-bool
-Editor::set_selected_regionview_from_map_event (GdkEventAny* ev, StreamView* sv, boost::weak_ptr<Region> weak_r)
-{
-       RegionView* rv;
-       boost::shared_ptr<Region> r (weak_r.lock());
-
-       if (!r) {
-               return true;
-       }
-
-       boost::shared_ptr<AudioRegion> ar;
-
-       if ((ar = boost::dynamic_pointer_cast<AudioRegion> (r)) == 0) {
-               return true;
-       }
-
-       if ((rv = sv->find_view (ar)) == 0) {
-               return true;
-       }
-
-       /* don't reset the selection if its something other than 
-          a single other region.
-       */
-
-       if (selection->regions.size() > 1) {
-               return true;
-       }
-       
-       begin_reversible_command (_("set selected regions"));
-       
-       selection->set (rv);
-
-       commit_reversible_command () ;
-
-       return true;
-}
-
-void
-Editor::set_edit_group_solo (Route& route, bool yn)
-{
-       RouteGroup *edit_group;
-
-       if ((edit_group = route.edit_group()) != 0) {
-               edit_group->apply (&Route::set_solo, yn, this);
-       } else {
-               route.set_solo (yn, this);
-       }
-}
-
-void
-Editor::set_edit_group_mute (Route& route, bool yn)
-{
-       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);
-       }
-}
-               
-void
-Editor::history_changed ()
-{
-       string label;
-
-       if (undo_action && session) {
-               if (session->undo_depth() == 0) {
-                       label = _("Undo");
-               } else {
-                       label = string_compose(_("Undo (%1)"), session->next_undo());
-               }
-               undo_action->property_label() = label;
-       }
-
-       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::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, -1);
-       entry.grab_focus ();
-
-
-       switch (win.run ()) {
-       case RESPONSE_ACCEPT:
-               break;
-       default:
-               return;
+       default:
+               return;
        }
 
        string text = entry.get_text();
@@ -3650,87 +3035,21 @@ 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->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 *
@@ -3865,7 +3184,6 @@ Editor::set_show_measures (bool yn)
                if ((_show_measures = yn) == true) {
                        draw_measures ();
                }
-               DisplayControlChanged (ShowMeasures);
                instant_save ();
        }
 }
@@ -3888,27 +3206,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);
        
@@ -3962,7 +3291,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 ()) {
@@ -4092,15 +3421,16 @@ Editor::redisplay_snapshots ()
                string statename = *(*i);
                TreeModel::Row row = *(snapshot_display_model->append());
                
-               // we don't have a way of changing the rendering in just one TreeView 
-               // cell so just put an asterisk on each side of the name for now.
+               /* 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()) {
-                       display_name = "*"+statename+"*";
                        snapshot_display.get_selection()->select(row);
-               } else {
-                       display_name = statename;
-               }
+               } 
                
                row[snapshot_display_columns.visible_name] = display_name;
                row[snapshot_display_columns.real_name] = statename;
@@ -4211,3 +3541,159 @@ Editor::on_key_press_event (GdkEventKey* ev)
        return key_press_focus_accelerator_handler (*this, ev);
 }
 
+void
+Editor::reset_x_origin (nframes_t frame)
+{
+       queue_visual_change (frame);
+}
+
+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::set_frames_per_unit (double fpu)
+{
+       nframes_t frames;
+
+       /* 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;
+       }
+
+       // convert fpu to frame count
+
+       frames = (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) {
+               return;
+       }
+
+       if (fpu == frames_per_unit) {
+               return;
+       }
+
+       frames_per_unit = fpu;
+
+       if (frames != 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 (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::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::queue_visual_change (double fpu)
+{
+       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_full (G_PRIORITY_HIGH_IDLE, _idle_visual_changer, this, 0);
+       }
+}
+
+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;
+       pending_visual_change.idle_handler_id = -1;
+       
+       if (p & VisualChange::ZoomLevel) {
+               set_frames_per_unit (pending_visual_change.frames_per_unit);
+       }
+
+       if (p & VisualChange::TimeOrigin) {
+               if (pending_visual_change.time_origin != leftmost_frame) {
+                       horizontal_adjustment.set_value (pending_visual_change.time_origin/frames_per_unit);
+                       /* the signal handler will do the rest */
+               } else {
+                       update_fixed_rulers();
+                       tempo_map_changed (Change (0), true);
+               }
+       }
+
+       return 0;
+}
+
+struct EditorOrderTimeAxisSorter {
+    bool operator() (const TimeAxisView* a, const TimeAxisView* b) const {
+           return a->order < b->order;
+    }
+};
+       
+void
+Editor::sort_track_selection ()
+{
+       EditorOrderTimeAxisSorter cmp;
+       selection->tracks.sort (cmp);
+}
+
+nframes_t
+Editor::edit_cursor_position(bool sync)
+{
+       if (sync && edit_cursor->current_frame != edit_cursor_clock.current_time()) {
+               edit_cursor_clock.set(edit_cursor->current_frame, true);
+       }
+
+       return edit_cursor->current_frame;
+}