update drobilla's fascistic dir-locals.el to force emacs users into whitespace submis...
[ardour.git] / gtk2_ardour / editor.cc
index 7f19e1e415000ce077b3c725a8d38a7e0d473a86..050f9b6a1e97108aed50012a494a6f1899e9d473 100644 (file)
@@ -19,7 +19,6 @@
 
 /* Note: public Editor methods are documented in public_editor.h */
 
-#define __STDC_LIMIT_MACROS 1
 #include <stdint.h>
 #include <unistd.h>
 #include <cstdlib>
 #include <gdkmm/color.h>
 #include <gdkmm/bitmap.h>
 
-#include <gtkmm2ext/grouped_buttons.h>
-#include <gtkmm2ext/gtk_ui.h>
-#include <gtkmm2ext/tearoff.h>
-#include <gtkmm2ext/utils.h>
-#include <gtkmm2ext/window_title.h>
-#include <gtkmm2ext/choice.h>
-#include <gtkmm2ext/cell_renderer_pixbuf_toggle.h>
+#include "gtkmm2ext/bindings.h"
+#include "gtkmm2ext/grouped_buttons.h"
+#include "gtkmm2ext/gtk_ui.h"
+#include "gtkmm2ext/tearoff.h"
+#include "gtkmm2ext/utils.h"
+#include "gtkmm2ext/window_title.h"
+#include "gtkmm2ext/choice.h"
+#include "gtkmm2ext/cell_renderer_pixbuf_toggle.h"
 
 #include "ardour/audio_diskstream.h"
 #include "ardour/audio_track.h"
@@ -78,6 +78,7 @@
 #include "control_protocol/control_protocol.h"
 
 #include "editor.h"
+#include "debug.h"
 #include "keyboard.h"
 #include "marker.h"
 #include "playlist_selector.h"
 #include "editor_locations.h"
 #include "editor_snapshots.h"
 #include "editor_summary.h"
+#include "region_layering_order_editor.h"
+#include "mouse_cursors.h"
+#include "editor_cursors.h"
 
 #include "i18n.h"
 
@@ -136,8 +140,6 @@ using Gtkmm2ext::Keyboard;
 
 const double Editor::timebar_height = 15.0;
 
-#include "editor_xpms"
-
 static const gchar *_snap_type_strings[] = {
        N_("CD Frames"),
        N_("Timecode Frames"),
@@ -148,6 +150,7 @@ static const gchar *_snap_type_strings[] = {
        N_("Beats/32"),
        N_("Beats/28"),
        N_("Beats/24"),
+       N_("Beats/20"),
        N_("Beats/16"),
        N_("Beats/14"),
        N_("Beats/12"),
@@ -201,34 +204,11 @@ static const gchar *_rb_opt_strings[] = {
        N_("Unpitched percussion with stable notes"),
        N_("Crisp monophonic instrumental"),
        N_("Unpitched solo percussion"),
+       N_("Resample without preserving pitch"),
        0
 };
 #endif
 
-/* Soundfile  drag-n-drop */
-
-Gdk::Cursor* Editor::cross_hair_cursor = 0;
-Gdk::Cursor* Editor::selector_cursor = 0;
-Gdk::Cursor* Editor::trimmer_cursor = 0;
-Gdk::Cursor* Editor::left_side_trim_cursor = 0;
-Gdk::Cursor* Editor::right_side_trim_cursor = 0;
-Gdk::Cursor* Editor::fade_in_cursor = 0;
-Gdk::Cursor* Editor::fade_out_cursor = 0;
-Gdk::Cursor* Editor::grabber_cursor = 0;
-Gdk::Cursor* Editor::grabber_note_cursor = 0;
-Gdk::Cursor* Editor::grabber_edit_point_cursor = 0;
-Gdk::Cursor* Editor::zoom_cursor = 0;
-Gdk::Cursor* Editor::time_fx_cursor = 0;
-Gdk::Cursor* Editor::fader_cursor = 0;
-Gdk::Cursor* Editor::speaker_cursor = 0;
-Gdk::Cursor* Editor::midi_pencil_cursor = 0;
-Gdk::Cursor* Editor::midi_select_cursor = 0;
-Gdk::Cursor* Editor::midi_resize_cursor = 0;
-Gdk::Cursor* Editor::midi_erase_cursor = 0;
-Gdk::Cursor* Editor::wait_cursor = 0;
-Gdk::Cursor* Editor::timebar_cursor = 0;
-Gdk::Cursor* Editor::transparent_cursor = 0;
-
 void
 show_me_the_size (Requisition* r, const char* what)
 {
@@ -239,20 +219,20 @@ show_me_the_size (Requisition* r, const char* what)
 static void
 pane_size_watcher (Paned* pane)
 {
-       /* if the handle of a pane vanishes into (at least) the tabs of a notebook,
-          it is no longer accessible. so stop that. this doesn't happen on X11,
-          just the quartz backend.
+       /* if the handle of a pane vanishes into (at least) the tabs of a notebook,
+          it is no longer accessible. so stop that. this doesn't happen on X11,
+          just the quartz backend.
 
-          ugh.
-       */
+          ugh.
+       */
 
-       int max_width_of_lhs = GTK_WIDGET(pane->gobj())->allocation.width - 25;
+       int max_width_of_lhs = GTK_WIDGET(pane->gobj())->allocation.width - 25;
 
-       gint pos = pane->get_position ();
+       gint pos = pane->get_position ();
 
-       if (pos > max_width_of_lhs) {
-               pane->set_position (max_width_of_lhs);
-       }
+       if (pos > max_width_of_lhs) {
+               pane->set_position (max_width_of_lhs);
+       }
 }
 #endif
 
@@ -263,7 +243,7 @@ Editor::Editor ()
        , minsec_label (_("Mins:Secs"))
        , bbt_label (_("Bars:Beats"))
        , timecode_label (_("Timecode"))
-       , frame_label (_("Samples"))
+       , samples_label (_("Samples"))
        , tempo_label (_("Tempo"))
        , meter_label (_("Meter"))
        , mark_label (_("Location Markers"))
@@ -287,6 +267,7 @@ Editor::Editor ()
        , automation_mode_button (_("mode"))
        , global_automation_button (_("automation"))
 
+       , _toolbar_viewport (*manage (new Gtk::Adjustment (0, 0, 1e10)), *manage (new Gtk::Adjustment (0, 0, 1e10)))
        , midi_panic_button (_("Panic"))
 
 #ifdef WITH_CMT
@@ -299,7 +280,9 @@ Editor::Editor ()
        , meters_running(false)
        , _pending_locate_request (false)
        , _pending_initial_locate (false)
+       , _last_cut_copy_source_track (0)
 
+       , _region_selection_change_updates_region_list (true)
 {
        constructed = false;
 
@@ -318,6 +301,7 @@ Editor::Editor ()
        clicked_crossfadeview = 0;
        clicked_control_point = 0;
        last_update_frame = 0;
+        pre_press_cursor = 0;
        _drags = new DragManager (this);
        current_mixer_strip = 0;
        current_bbt_points = 0;
@@ -329,6 +313,7 @@ Editor::Editor ()
        edit_point_strings = I18N (_edit_point_strings);
 #ifdef USE_RUBBERBAND
        rb_opt_strings = I18N (_rb_opt_strings);
+       rb_current_opt = 4;
 #endif
 
        snap_threshold = 5.0;
@@ -348,7 +333,6 @@ Editor::Editor ()
        show_gain_after_trim = false;
        verbose_cursor_on = true;
        last_item_entered = 0;
-       last_item_entered_n = 0;
 
        have_pending_keyboard_selection = false;
        _follow_playhead = true;
@@ -357,7 +341,6 @@ Editor::Editor ()
        editor_ruler_menu = 0;
        no_ruler_shown_update = false;
        marker_menu = 0;
-       session_range_marker_menu = 0;
        range_marker_menu = 0;
        marker_menu_item = 0;
        tempo_or_meter_marker_menu = 0;
@@ -381,10 +364,8 @@ Editor::Editor ()
        _dragging_edit_point = false;
        select_new_marker = false;
        rhythm_ferret = 0;
+       layering_order_editor = 0;
        _bundle_manager = 0;
-       for (ARDOUR::DataType::iterator i = ARDOUR::DataType::begin(); i != ARDOUR::DataType::end(); ++i) {
-               _global_port_matrix[*i] = 0;
-       }
        no_save_visual = false;
        resize_idle_id = -1;
 
@@ -428,12 +409,12 @@ Editor::Editor ()
        timecode_label.set_padding (5,0);
        timecode_label.hide ();
        timecode_label.set_no_show_all();
-       frame_label.set_name ("EditorTimeButton");
-       frame_label.set_size_request (-1, (int)timebar_height);
-       frame_label.set_alignment (1.0, 0.5);
-       frame_label.set_padding (5,0);
-       frame_label.hide ();
-       frame_label.set_no_show_all();
+       samples_label.set_name ("EditorTimeButton");
+       samples_label.set_size_request (-1, (int)timebar_height);
+       samples_label.set_alignment (1.0, 0.5);
+       samples_label.set_padding (5,0);
+       samples_label.hide ();
+       samples_label.set_no_show_all();
 
        tempo_label.set_name ("EditorTimeButton");
        tempo_label.set_size_request (-1, (int)timebar_height);
@@ -441,30 +422,35 @@ Editor::Editor ()
        tempo_label.set_padding (5,0);
        tempo_label.hide();
        tempo_label.set_no_show_all();
+       
        meter_label.set_name ("EditorTimeButton");
        meter_label.set_size_request (-1, (int)timebar_height);
        meter_label.set_alignment (1.0, 0.5);
        meter_label.set_padding (5,0);
        meter_label.hide();
        meter_label.set_no_show_all();
+       
        mark_label.set_name ("EditorTimeButton");
        mark_label.set_size_request (-1, (int)timebar_height);
        mark_label.set_alignment (1.0, 0.5);
        mark_label.set_padding (5,0);
        mark_label.hide();
        mark_label.set_no_show_all();
+       
        cd_mark_label.set_name ("EditorTimeButton");
        cd_mark_label.set_size_request (-1, (int)timebar_height);
        cd_mark_label.set_alignment (1.0, 0.5);
        cd_mark_label.set_padding (5,0);
        cd_mark_label.hide();
        cd_mark_label.set_no_show_all();
+       
        range_mark_label.set_name ("EditorTimeButton");
        range_mark_label.set_size_request (-1, (int)timebar_height);
        range_mark_label.set_alignment (1.0, 0.5);
        range_mark_label.set_padding (5,0);
        range_mark_label.hide();
        range_mark_label.set_no_show_all();
+       
        transport_mark_label.set_name ("EditorTimeButton");
        transport_mark_label.set_size_request (-1, (int)timebar_height);
        transport_mark_label.set_alignment (1.0, 0.5);
@@ -474,11 +460,14 @@ Editor::Editor ()
 
        initialize_rulers ();
        initialize_canvas ();
+       
        _summary = new EditorSummary (this);
 
        selection->TimeChanged.connect (sigc::mem_fun(*this, &Editor::time_selection_changed));
        selection->TracksChanged.connect (sigc::mem_fun(*this, &Editor::track_selection_changed));
+       
        editor_regions_selection_changed_connection = selection->RegionsChanged.connect (sigc::mem_fun(*this, &Editor::region_selection_changed));
+       
        selection->PointsChanged.connect (sigc::mem_fun(*this, &Editor::point_selection_changed));
        selection->MarkersChanged.connect (sigc::mem_fun(*this, &Editor::marker_selection_changed));
 
@@ -498,15 +487,16 @@ Editor::Editor ()
 
        controls_layout.add_events (Gdk::BUTTON_PRESS_MASK|Gdk::BUTTON_RELEASE_MASK|Gdk::ENTER_NOTIFY_MASK|Gdk::LEAVE_NOTIFY_MASK);
        controls_layout.signal_button_release_event().connect (sigc::mem_fun(*this, &Editor::edit_controls_button_release));
-       controls_layout_size_request_connection = controls_layout.signal_size_request().connect (sigc::mem_fun (*this, &Editor::controls_layout_size_request));
 
-       build_cursors ();
+       _cursors = new MouseCursors;
 
        ArdourCanvas::Canvas* time_pad = manage(new ArdourCanvas::Canvas());
        ArdourCanvas::SimpleLine* pad_line_1 = manage(new ArdourCanvas::SimpleLine(*time_pad->root(),
                        0.0, 1.0, 100.0, 1.0));
+       
        pad_line_1->property_color_rgba() = 0xFF0000FF;
        pad_line_1->show();
+       
        time_pad->show();
 
        time_canvas_vbox.set_size_request (-1, (int)(timebar_height * visible_timebars) + 2);
@@ -555,63 +545,57 @@ Editor::Editor ()
        _snapshots = new EditorSnapshots (this);
        _locations = new EditorLocations (this);
 
-       Gtk::Label* nlabel;
-
-       nlabel = manage (new Label (_("Regions")));
-       nlabel->set_angle (-90);
-       the_notebook.append_page (_regions->widget (), *nlabel);
-       nlabel = manage (new Label (_("Tracks & Busses")));
-       nlabel->set_angle (-90);
-       the_notebook.append_page (_routes->widget (), *nlabel);
-       nlabel = manage (new Label (_("Snapshots")));
-       nlabel->set_angle (-90);
-       the_notebook.append_page (_snapshots->widget (), *nlabel);
-       nlabel = manage (new Label (_("Route Groups")));
-       nlabel->set_angle (-90);
-       the_notebook.append_page (_route_groups->widget (), *nlabel);
-       nlabel = manage (new Label (_("Ranges & Marks")));
-       nlabel->set_angle (-90);
-       the_notebook.append_page (_locations->widget (), *nlabel);
-
-       the_notebook.set_show_tabs (true);
-       the_notebook.set_scrollable (true);
-       the_notebook.popup_disable ();
-       the_notebook.set_tab_pos (Gtk::POS_RIGHT);
-       the_notebook.show_all ();
-       
+       add_notebook_page (_("Regions"), _regions->widget ());
+       add_notebook_page (_("Tracks & Busses"), _routes->widget ());
+       add_notebook_page (_("Snapshots"), _snapshots->widget ());
+       add_notebook_page (_("Route Groups"), _route_groups->widget ());
+       add_notebook_page (_("Ranges & Marks"), _locations->widget ());
+
+       _the_notebook.set_show_tabs (true);
+       _the_notebook.set_scrollable (true);
+       _the_notebook.popup_disable ();
+       _the_notebook.set_tab_pos (Gtk::POS_RIGHT);
+       _the_notebook.show_all ();
+
        post_maximal_editor_width = 0;
        post_maximal_horizontal_pane_position = 0;
        post_maximal_editor_height = 0;
        post_maximal_vertical_pane_position = 0;
+       _notebook_shrunk = false;
 
        editor_summary_pane.pack1(edit_packer);
 
        Button* summary_arrows_left_left = manage (new Button);
        summary_arrows_left_left->add (*manage (new Arrow (ARROW_LEFT, SHADOW_NONE)));
-       summary_arrows_left_left->signal_pressed().connect (sigc::hide_return (sigc::mem_fun (*this, &Editor::horizontal_scroll_left_press)));
-       summary_arrows_left_left->signal_released().connect (sigc::mem_fun (*this, &Editor::horizontal_scroll_left_release));
+       summary_arrows_left_left->signal_pressed().connect (sigc::hide_return (sigc::bind (sigc::mem_fun (*this, &Editor::scroll_press), LEFT)));
+       summary_arrows_left_left->signal_released().connect (sigc::mem_fun (*this, &Editor::scroll_release));
+       
        Button* summary_arrows_left_right = manage (new Button);
        summary_arrows_left_right->add (*manage (new Arrow (ARROW_RIGHT, SHADOW_NONE)));
-       summary_arrows_left_right->signal_pressed().connect (sigc::hide_return (sigc::mem_fun (*this, &Editor::horizontal_scroll_right_press)));
-       summary_arrows_left_right->signal_released().connect (sigc::mem_fun (*this, &Editor::horizontal_scroll_right_release));
+       summary_arrows_left_right->signal_pressed().connect (sigc::hide_return (sigc::bind (sigc::mem_fun (*this, &Editor::scroll_press), RIGHT)));
+       summary_arrows_left_right->signal_released().connect (sigc::mem_fun (*this, &Editor::scroll_release));
+       
        VBox* summary_arrows_left = manage (new VBox);
        summary_arrows_left->pack_start (*summary_arrows_left_left);
        summary_arrows_left->pack_start (*summary_arrows_left_right);
 
-       Button* summary_arrows_right_left = manage (new Button);
-       summary_arrows_right_left->add (*manage (new Arrow (ARROW_LEFT, SHADOW_NONE)));
-       summary_arrows_right_left->signal_pressed().connect (sigc::hide_return (sigc::mem_fun (*this, &Editor::horizontal_scroll_left_press)));
-       summary_arrows_right_left->signal_released().connect (sigc::mem_fun (*this, &Editor::horizontal_scroll_left_release));
-       Button* summary_arrows_right_right = manage (new Button);
-       summary_arrows_right_right->add (*manage (new Arrow (ARROW_RIGHT, SHADOW_NONE)));
-       summary_arrows_right_right->signal_pressed().connect (sigc::hide_return (sigc::mem_fun (*this, &Editor::horizontal_scroll_right_press)));
-       summary_arrows_right_right->signal_released().connect (sigc::mem_fun (*this, &Editor::horizontal_scroll_right_release));
+       Button* summary_arrows_right_up = manage (new Button);
+       summary_arrows_right_up->add (*manage (new Arrow (ARROW_UP, SHADOW_NONE)));
+       summary_arrows_right_up->signal_pressed().connect (sigc::hide_return (sigc::bind (sigc::mem_fun (*this, &Editor::scroll_press), UP)));
+       summary_arrows_right_up->signal_released().connect (sigc::mem_fun (*this, &Editor::scroll_release));
+       
+       Button* summary_arrows_right_down = manage (new Button);
+       summary_arrows_right_down->add (*manage (new Arrow (ARROW_DOWN, SHADOW_NONE)));
+       summary_arrows_right_down->signal_pressed().connect (sigc::hide_return (sigc::bind (sigc::mem_fun (*this, &Editor::scroll_press), DOWN)));
+       summary_arrows_right_down->signal_released().connect (sigc::mem_fun (*this, &Editor::scroll_release));
+       
        VBox* summary_arrows_right = manage (new VBox);
-       summary_arrows_right->pack_start (*summary_arrows_right_left);
-       summary_arrows_right->pack_start (*summary_arrows_right_right);
+       summary_arrows_right->pack_start (*summary_arrows_right_up);
+       summary_arrows_right->pack_start (*summary_arrows_right_down);
 
        Frame* summary_frame = manage (new Frame);
        summary_frame->set_shadow_type (Gtk::SHADOW_ETCHED_IN);
+       
        summary_frame->add (*_summary);
        summary_frame->show ();
 
@@ -622,7 +606,7 @@ Editor::Editor ()
        editor_summary_pane.pack2 (_summary_hbox);
 
        edit_pane.pack1 (editor_summary_pane, true, true);
-       edit_pane.pack2 (the_notebook, false, true);
+       edit_pane.pack2 (_the_notebook, false, true);
 
        editor_summary_pane.signal_size_allocate().connect (sigc::bind (sigc::mem_fun (*this, &Editor::pane_allocation_handler), static_cast<Paned*> (&editor_summary_pane)));
 
@@ -630,10 +614,10 @@ Editor::Editor ()
 
        edit_pane.signal_size_allocate().connect (sigc::bind (sigc::mem_fun(*this, &Editor::pane_allocation_handler), static_cast<Paned*> (&edit_pane)));
 #ifdef GTKOSX
-        Glib::PropertyProxy<int> proxy = edit_pane.property_position();
-        proxy.signal_changed().connect (bind (sigc::ptr_fun (pane_size_watcher), static_cast<Paned*> (&edit_pane)));
+       Glib::PropertyProxy<int> proxy = edit_pane.property_position();
+       proxy.signal_changed().connect (bind (sigc::ptr_fun (pane_size_watcher), static_cast<Paned*> (&edit_pane)));
 #endif
-       top_hbox.pack_start (toolbar_frame, false, true);
+       top_hbox.pack_start (toolbar_frame);
 
        HBox *hbox = manage (new HBox);
        hbox->pack_start (edit_pane, true, true);
@@ -663,11 +647,9 @@ Editor::Editor ()
        _snap_mode = SnapOff;
        set_snap_mode (_snap_mode);
        set_mouse_mode (MouseObject, true);
+        pre_internal_mouse_mode = MouseObject;
        set_edit_point_preference (EditAtMouse, true);
 
-       XMLNode* node = ARDOUR_UI::instance()->editor_settings();
-       set_state (*node, Stateful::loading_state_version);
-
        _playlist_selector = new PlaylistSelector();
        _playlist_selector->signal_delete_event().connect (sigc::bind (sigc::ptr_fun (just_hide_it), static_cast<Window *> (_playlist_selector)));
 
@@ -701,14 +683,14 @@ Editor::Editor ()
                window_icons.push_back (icon);
        }
        if (!window_icons.empty()) {
-               set_icon_list (window_icons);
+               // set_icon_list (window_icons);
                set_default_icon_list (window_icons);
        }
 
        WindowTitle title(Glib::get_application_name());
        title += _("Editor");
        set_title (title.get_string());
-       set_wmclass (X_("ardour_editor"), "Ardour");
+       set_wmclass (X_("ardour_editor"), PROGRAM_NAME);
 
        add (vpacker);
        add_events (Gdk::KEY_PRESS_MASK|Gdk::KEY_RELEASE_MASK);
@@ -732,10 +714,28 @@ Editor::Editor ()
 
        TimeAxisView::CatchDeletion.connect (*this, invalidator (*this), ui_bind (&Editor::timeaxisview_deleted, this, _1), gui_context());
 
-       _last_normalization_value = 0;
+       _ignore_region_action = false;
+       _last_region_menu_was_main = false;
+       _popup_region_menu_item = 0;
+
+       _show_marker_lines = false;
+       _over_region_trim_target = false;
+
+        /* Button bindings */
+
+        button_bindings = new Bindings;
+
+       XMLNode* node = button_settings();
+        if (node) {
+                for (XMLNodeList::const_iterator i = node->children().begin(); i != node->children().end(); ++i) {
+                        button_bindings->load (**i);
+                }
+        }
 
        constructed = true;
        instant_save ();
+
+       setup_fade_images ();
 }
 
 Editor::~Editor()
@@ -752,12 +752,27 @@ Editor::~Editor()
        }
 #endif
 
+        delete button_bindings;
        delete _routes;
        delete _route_groups;
        delete track_canvas;
        delete _drags;
 }
 
+XMLNode*
+Editor::button_settings () const
+{
+       XMLNode* settings = ARDOUR_UI::instance()->editor_settings();
+       XMLNode* node = find_named_node (*settings, X_("Buttons"));
+
+       if (!node) {
+                cerr << "new empty Button node\n";
+               node = new XMLNode (X_("Buttons"));
+       }
+
+       return node;
+}
+
 void
 Editor::add_toplevel_controls (Container& cont)
 {
@@ -783,6 +798,10 @@ Editor::catch_vanishing_regionview (RegionView *rv)
        if (entered_regionview == rv) {
                set_entered_regionview (0);
        }
+
+       if (!_all_region_actions_sensitized) {
+               sensitize_all_region_actions (true);
+       }
 }
 
 void
@@ -799,6 +818,13 @@ Editor::set_entered_regionview (RegionView* rv)
        if ((entered_regionview = rv) != 0) {
                entered_regionview->entered (internal_editing ());
        }
+
+       if (!_all_region_actions_sensitized && _last_region_menu_was_main) {
+               /* This RegionView entry might have changed what region actions
+                  are allowed, so sensitize them all in case a key is pressed.
+               */
+               sensitize_all_region_actions (true);
+       }
 }
 
 void
@@ -816,16 +842,19 @@ Editor::set_entered_track (TimeAxisView* tav)
 void
 Editor::show_window ()
 {
-       if (! is_visible ()) {
+       if (!is_visible ()) {
                show_all ();
 
-               /* re-hide editor list if necessary */
-               editor_list_button_toggled ();
+               /* XXX: this is a bit unfortunate; it would probably
+                  be nicer if we could just call show () above rather
+                  than needing the show_all ()
+               */
 
-               /* re-hide summary widget if necessary */
+               /* re-hide stuff if necessary */
+               editor_list_button_toggled ();
                parameter_changed ("show-summary");
-
-               parameter_changed ("show-edit-group-tabs");
+               parameter_changed ("show-group-tabs");
+               parameter_changed ("show-zoom-tools");
 
                /* now reset all audio_time_axis heights, because widgets might need
                   to be re-hidden
@@ -838,7 +867,9 @@ Editor::show_window ()
                        tv->reset_height ();
                }
 
-               reset_zoom (frames_per_unit);
+               if (current_mixer_strip) {
+                       current_mixer_strip->hide_things ();
+               }
        }
 
        present ();
@@ -869,10 +900,10 @@ Editor::zoom_adjustment_changed ()
 
        if (fpu < 1.0) {
                fpu = 1.0;
-               zoom_range_clock.set ((nframes64_t) floor (fpu * _canvas_width));
+               zoom_range_clock.set ((framepos_t) floor (fpu * _canvas_width));
        } else if (fpu > _session->current_end_frame() / _canvas_width) {
                fpu = _session->current_end_frame() / _canvas_width;
-               zoom_range_clock.set ((nframes64_t) floor (fpu * _canvas_width));
+               zoom_range_clock.set ((framepos_t) floor (fpu * _canvas_width));
        }
 
        temporal_zoom (fpu);
@@ -892,7 +923,7 @@ Editor::control_scroll (float fraction)
        /*
                _control_scroll_target is an optional<T>
 
-               it acts like a pointer to an nframes64_t, with
+               it acts like a pointer to an framepos_t, with
                a operator conversion to boolean to check
                that it has a value could possibly use
                playhead_cursor->current_frame to store the
@@ -905,12 +936,12 @@ Editor::control_scroll (float fraction)
                _dragging_playhead = true;
        }
 
-       if ((fraction < 0.0f) && (*_control_scroll_target < (nframes64_t) fabs(step))) {
+       if ((fraction < 0.0f) && (*_control_scroll_target < (framepos_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 if ((fraction > 0.0f) && (max_framepos - *_control_scroll_target < step)) {
+               *_control_scroll_target = max_framepos - (current_page_frames()*2); // allow room for slop in where the PH is on the screen
        } else {
-               *_control_scroll_target += (nframes64_t) floor (step);
+               *_control_scroll_target += (framepos_t) floor (step);
        }
 
        /* move visuals, we'll catch up with it later */
@@ -941,7 +972,7 @@ Editor::control_scroll (float fraction)
 }
 
 bool
-Editor::deferred_control_scroll (nframes64_t /*target*/)
+Editor::deferred_control_scroll (framepos_t /*target*/)
 {
        _session->request_locate (*_control_scroll_target, _session->transport_rolling());
        // reset for next stream
@@ -975,7 +1006,7 @@ Editor::on_realize ()
 }
 
 void
-Editor::map_position_change (nframes64_t frame)
+Editor::map_position_change (framepos_t frame)
 {
        ENSURE_GUI_THREAD (*this, &Editor::map_position_change, frame)
 
@@ -991,7 +1022,7 @@ Editor::map_position_change (nframes64_t frame)
 }
 
 void
-Editor::center_screen (nframes64_t frame)
+Editor::center_screen (framepos_t frame)
 {
        double page = _canvas_width * frames_per_unit;
 
@@ -1004,12 +1035,12 @@ Editor::center_screen (nframes64_t frame)
 }
 
 void
-Editor::center_screen_internal (nframes64_t frame, float page)
+Editor::center_screen_internal (framepos_t frame, float page)
 {
        page /= 2;
 
        if (frame > page) {
-               frame -= (nframes64_t) page;
+               frame -= (framepos_t) page;
        } else {
                frame = 0;
        }
@@ -1076,17 +1107,8 @@ Editor::set_session (Session *t)
                sfbrowser->set_session (_session);
        }
 
-       for (ARDOUR::DataType::iterator i = ARDOUR::DataType::begin(); i != ARDOUR::DataType::end(); ++i) {
-               if (_global_port_matrix[*i]) {
-                       _global_port_matrix[*i]->set_session (_session);
-               }
-       }
-
        compute_fixed_ruler_scale ();
 
-       /* there are never any selected regions at startup */
-       sensitize_the_right_region_actions (false);
-
        XMLNode* node = ARDOUR_UI::instance()->editor_settings();
        set_state (*node, Stateful::loading_state_version);
 
@@ -1102,12 +1124,11 @@ Editor::set_session (Session *t)
           but use Gtkmm2ext::UI::instance()->call_slot();
        */
 
-        _session->StepEditStatusChange.connect (_session_connections, invalidator (*this), ui_bind(&Editor::step_edit_status_change, this, _1), gui_context());
+       _session->StepEditStatusChange.connect (_session_connections, invalidator (*this), ui_bind(&Editor::step_edit_status_change, this, _1), gui_context());
        _session->TransportStateChange.connect (_session_connections, invalidator (*this), boost::bind (&Editor::map_transport_state, this), gui_context());
        _session->PositionChanged.connect (_session_connections, invalidator (*this), ui_bind (&Editor::map_position_change, this, _1), gui_context());
        _session->RouteAdded.connect (_session_connections, invalidator (*this), ui_bind (&Editor::handle_new_route, this, _1), gui_context());
        _session->DirtyChanged.connect (_session_connections, invalidator (*this), boost::bind (&Editor::update_title, this), gui_context());
-       _session->TimecodeOffsetChanged.connect (_session_connections, invalidator (*this), boost::bind (&Editor::update_just_timecode, this), gui_context());
        _session->tempo_map().PropertyChanged.connect (_session_connections, invalidator (*this), ui_bind (&Editor::tempo_map_changed, this, _1), gui_context());
        _session->Located.connect (_session_connections, invalidator (*this), boost::bind (&Editor::located, this), gui_context());
        _session->config.ParameterChanged.connect (_session_connections, invalidator (*this), ui_bind (&Editor::parameter_changed, this, _1), gui_context());
@@ -1119,11 +1140,11 @@ Editor::set_session (Session *t)
        _session->history().Changed.connect (_session_connections, invalidator (*this), boost::bind (&Editor::history_changed, this), gui_context());
 
        if (Profile->get_sae()) {
-               BBT_Time bbt;
+               Timecode::BBT_Time bbt;
                bbt.bars = 0;
                bbt.beats = 0;
                bbt.ticks = 120;
-               nframes_t pos = _session->tempo_map().bbt_duration_at (0, bbt, 1);
+               framepos_t pos = _session->tempo_map().bbt_duration_at (0, bbt, 1);
                nudge_clock.set_mode(AudioClock::BBT);
                nudge_clock.set (pos, true, 0, AudioClock::BBT);
 
@@ -1135,10 +1156,12 @@ Editor::set_session (Session *t)
 
        Location* loc = _session->locations()->auto_loop_location();
        if (loc == 0) {
-               loc = new Location (0, _session->current_end_frame(), _("Loop"),(Location::Flags) (Location::IsAutoLoop | Location::IsHidden));
+               loc = new Location (*_session, 0, _session->current_end_frame(), _("Loop"),(Location::Flags) (Location::IsAutoLoop | Location::IsHidden));
+               
                if (loc->start() == loc->end()) {
                        loc->set_end (loc->start() + 1);
                }
+               
                _session->locations()->add (loc, false);
                _session->set_auto_loop_location (loc);
        } else {
@@ -1147,11 +1170,14 @@ Editor::set_session (Session *t)
        }
 
        loc = _session->locations()->auto_punch_location();
+       
        if (loc == 0) {
-               loc = new Location (0, _session->current_end_frame(), _("Punch"), (Location::Flags) (Location::IsAutoPunch | Location::IsHidden));
+               loc = new Location (*_session, 0, _session->current_end_frame(), _("Punch"), (Location::Flags) (Location::IsAutoPunch | Location::IsHidden));
+               
                if (loc->start() == loc->end()) {
                        loc->set_end (loc->start() + 1);
                }
+               
                _session->locations()->add (loc, false);
                _session->set_auto_punch_location (loc);
        } else {
@@ -1192,102 +1218,26 @@ Editor::set_session (Session *t)
        /* register for undo history */
        _session->register_with_memento_command_factory(_id, this);
 
+       ActionManager::ui_manager->signal_pre_activate().connect (sigc::mem_fun (*this, &Editor::action_pre_activated));
+
        start_updating_meters ();
 }
 
 void
-Editor::build_cursors ()
+Editor::action_pre_activated (Glib::RefPtr<Action> const & a)
 {
-       using namespace Gdk;
+       if (a->get_name() == "RegionMenu") {
+               /* When the main menu's region menu is opened, we setup the actions so that they look right
+                  in the menu.  I can't find a way of getting a signal when this menu is subsequently closed,
+                  so we resensitize all region actions when the entered regionview or the region selection
+                  changes.  HOWEVER we can't always resensitize on entered_regionview change because that
+                  happens after the region context menu is opened.  So we set a flag here, too.
 
-       Gdk::Color mbg ("#000000" ); /* Black */
-       Gdk::Color mfg ("#0000ff" ); /* Blue. */
-
-       {
-               RefPtr<Bitmap> source, mask;
-               source = Bitmap::create (mag_bits, mag_width, mag_height);
-               mask = Bitmap::create (magmask_bits, mag_width, mag_height);
-               zoom_cursor = new Gdk::Cursor (source, mask, mfg, mbg, mag_x_hot, mag_y_hot);
-       }
-
-       Gdk::Color fbg ("#ffffff" );
-       Gdk::Color ffg  ("#000000" );
-
-       {
-               RefPtr<Bitmap> source, mask;
-
-               source = Bitmap::create (fader_cursor_bits, fader_cursor_width, fader_cursor_height);
-               mask = Bitmap::create (fader_cursor_mask_bits, fader_cursor_width, fader_cursor_height);
-               fader_cursor = new Gdk::Cursor (source, mask, ffg, fbg, fader_cursor_x_hot, fader_cursor_y_hot);
-       }
-
-       {
-               RefPtr<Bitmap> source, mask;
-               source = Bitmap::create (speaker_cursor_bits, speaker_cursor_width, speaker_cursor_height);
-               mask = Bitmap::create (speaker_cursor_mask_bits, speaker_cursor_width, speaker_cursor_height);
-               speaker_cursor = new Gdk::Cursor (source, mask, ffg, fbg, speaker_cursor_x_hot, speaker_cursor_y_hot);
-       }
-
-       {
-               RefPtr<Bitmap> bits;
-               char pix[4] = { 0, 0, 0, 0 };
-               bits = Bitmap::create (pix, 2, 2);
-               Gdk::Color c;
-               transparent_cursor = new Gdk::Cursor (bits, bits, c, c, 0, 0);
-       }
-
-       {
-               RefPtr<Bitmap> bits;
-               char pix[4] = { 0, 0, 0, 0 };
-               bits = Bitmap::create (pix, 2, 2);
-               Gdk::Color c;
-               transparent_cursor = new Gdk::Cursor (bits, bits, c, c, 0, 0);
-       }
-
-
-       grabber_cursor = new Gdk::Cursor (HAND2);
-
-       {
-               Glib::RefPtr<Gdk::Pixbuf> grabber_note_pixbuf (::get_icon ("grabber_note"));
-               grabber_note_cursor = new Gdk::Cursor (Gdk::Display::get_default(), grabber_note_pixbuf, 5, 10);
-       }
-
-       {
-               Glib::RefPtr<Gdk::Pixbuf> grabber_edit_point_pixbuf (::get_icon ("grabber_edit_point"));
-               grabber_edit_point_cursor = new Gdk::Cursor (Gdk::Display::get_default(), grabber_edit_point_pixbuf, 5, 17);
-       }
-
-       cross_hair_cursor = new Gdk::Cursor (CROSSHAIR);
-       trimmer_cursor =  new Gdk::Cursor (SB_H_DOUBLE_ARROW);
-
-       {
-               Glib::RefPtr<Gdk::Pixbuf> apixbuf (::get_icon ("trim_left_cursor"));
-               left_side_trim_cursor = new Gdk::Cursor (Gdk::Display::get_default(), apixbuf, 5, 11);
-       }
-
-       {
-               Glib::RefPtr<Gdk::Pixbuf> apixbuf (::get_icon ("trim_right_cursor"));
-               right_side_trim_cursor = new Gdk::Cursor (Gdk::Display::get_default(), apixbuf, 23, 11);
-       }
-
-       {
-               Glib::RefPtr<Gdk::Pixbuf> apixbuf (::get_icon ("fade_in_cursor"));
-               fade_in_cursor = new Gdk::Cursor (Gdk::Display::get_default(), apixbuf, 0, 40);
-       }
-
-       {
-               Glib::RefPtr<Gdk::Pixbuf> apixbuf (::get_icon ("fade_out_cursor"));
-               fade_out_cursor = new Gdk::Cursor (Gdk::Display::get_default(), apixbuf, 27, 40);
+                  What a carry on :(
+               */
+               sensitize_the_right_region_actions ();
+               _last_region_menu_was_main = true;
        }
-
-       selector_cursor = new Gdk::Cursor (XTERM);
-       time_fx_cursor = new Gdk::Cursor (SIZING);
-       wait_cursor = new Gdk::Cursor  (WATCH);
-       timebar_cursor = new Gdk::Cursor(LEFT_PTR);
-       midi_pencil_cursor = new Gdk::Cursor (PENCIL);
-       midi_select_cursor = new Gdk::Cursor (CENTER_PTR);
-       midi_resize_cursor = new Gdk::Cursor (SIZING);
-       midi_erase_cursor = new Gdk::Cursor (DRAPED_BOX);
 }
 
 /** Pop up a context menu for when the user clicks on a fade in or fade out */
@@ -1318,14 +1268,57 @@ Editor::popup_fade_context_menu (int button, int32_t time, ArdourCanvas::Item* i
                items.push_back (SeparatorElem());
 
                if (Profile->get_sae()) {
-                       items.push_back (MenuElem (_("Linear"), sigc::bind (sigc::mem_fun (*this, &Editor::set_fade_in_shape), AudioRegion::Linear)));
-                       items.push_back (MenuElem (_("Slowest"), sigc::bind (sigc::mem_fun (*this, &Editor::set_fade_in_shape), AudioRegion::Fast)));
+                       
+                       items.push_back (MenuElem (_("Linear"), sigc::bind (sigc::mem_fun (*this, &Editor::set_fade_in_shape), FadeLinear)));
+                       items.push_back (MenuElem (_("Slowest"), sigc::bind (sigc::mem_fun (*this, &Editor::set_fade_in_shape), FadeFast)));
+                       
                } else {
-                       items.push_back (MenuElem (_("Linear"), sigc::bind (sigc::mem_fun (*this, &Editor::set_fade_in_shape), AudioRegion::Linear)));
-                       items.push_back (MenuElem (_("Slowest"), sigc::bind (sigc::mem_fun (*this, &Editor::set_fade_in_shape), AudioRegion::Fast)));
-                       items.push_back (MenuElem (_("Slow"), sigc::bind (sigc::mem_fun (*this, &Editor::set_fade_in_shape), AudioRegion::LogB)));
-                       items.push_back (MenuElem (_("Fast"), sigc::bind (sigc::mem_fun (*this, &Editor::set_fade_in_shape), AudioRegion::LogA)));
-                       items.push_back (MenuElem (_("Fastest"), sigc::bind (sigc::mem_fun (*this, &Editor::set_fade_in_shape), AudioRegion::Slow)));
+
+                       items.push_back (
+                               ImageMenuElem (
+                                       _("Linear"),
+                                       *_fade_in_images[FadeLinear],
+                                       sigc::bind (sigc::mem_fun (*this, &Editor::set_fade_in_shape), FadeLinear)
+                                       )
+                               );
+
+                       dynamic_cast<ImageMenuItem*>(&items.back())->set_always_show_image ();
+
+                       items.push_back (
+                               ImageMenuElem (
+                                       _("Slowest"),
+                                       *_fade_in_images[FadeFast],
+                                       sigc::bind (sigc::mem_fun (*this, &Editor::set_fade_in_shape), FadeFast)
+                                       ));
+                       
+                       dynamic_cast<ImageMenuItem*>(&items.back())->set_always_show_image ();
+                       
+                       items.push_back (
+                               ImageMenuElem (
+                                       _("Slow"),
+                                       *_fade_in_images[FadeLogB],
+                                       sigc::bind (sigc::mem_fun (*this, &Editor::set_fade_in_shape), FadeLogB)
+                                       ));
+                       
+                       dynamic_cast<ImageMenuItem*>(&items.back())->set_always_show_image ();
+                       
+                       items.push_back (
+                               ImageMenuElem (
+                                       _("Fast"),
+                                       *_fade_in_images[FadeLogA],
+                                       sigc::bind (sigc::mem_fun (*this, &Editor::set_fade_in_shape), FadeLogA)
+                                       ));
+                       
+                       dynamic_cast<ImageMenuItem*>(&items.back())->set_always_show_image ();
+                       
+                       items.push_back (
+                               ImageMenuElem (
+                                       _("Fastest"),
+                                       *_fade_in_images[FadeSlow],
+                                       sigc::bind (sigc::mem_fun (*this, &Editor::set_fade_in_shape), FadeSlow)
+                                       ));
+                       
+                       dynamic_cast<ImageMenuItem*>(&items.back())->set_always_show_image ();
                }
 
                break;
@@ -1341,14 +1334,55 @@ Editor::popup_fade_context_menu (int button, int32_t time, ArdourCanvas::Item* i
                items.push_back (SeparatorElem());
 
                if (Profile->get_sae()) {
-                       items.push_back (MenuElem (_("Linear"), sigc::bind (sigc::mem_fun (*this, &Editor::set_fade_out_shape), AudioRegion::Linear)));
-                       items.push_back (MenuElem (_("Slowest"), sigc::bind (sigc::mem_fun (*this, &Editor::set_fade_out_shape), AudioRegion::Slow)));
+                       items.push_back (MenuElem (_("Linear"), sigc::bind (sigc::mem_fun (*this, &Editor::set_fade_out_shape), FadeLinear)));
+                       items.push_back (MenuElem (_("Slowest"), sigc::bind (sigc::mem_fun (*this, &Editor::set_fade_out_shape), FadeSlow)));
                } else {
-                       items.push_back (MenuElem (_("Linear"), sigc::bind (sigc::mem_fun (*this, &Editor::set_fade_out_shape), AudioRegion::Linear)));
-                       items.push_back (MenuElem (_("Slowest"), sigc::bind (sigc::mem_fun (*this, &Editor::set_fade_out_shape), AudioRegion::Slow)));
-                       items.push_back (MenuElem (_("Slow"), sigc::bind (sigc::mem_fun (*this, &Editor::set_fade_out_shape), AudioRegion::LogA)));
-                       items.push_back (MenuElem (_("Fast"), sigc::bind (sigc::mem_fun (*this, &Editor::set_fade_out_shape), AudioRegion::LogB)));
-                       items.push_back (MenuElem (_("Fastest"), sigc::bind (sigc::mem_fun (*this, &Editor::set_fade_out_shape), AudioRegion::Fast)));
+
+                       items.push_back (
+                               ImageMenuElem (
+                                       _("Linear"),
+                                       *_fade_out_images[FadeLinear],
+                                       sigc::bind (sigc::mem_fun (*this, &Editor::set_fade_out_shape), FadeLinear)
+                                       )
+                               );
+
+                       dynamic_cast<ImageMenuItem*>(&items.back())->set_always_show_image ();
+
+                       items.push_back (
+                               ImageMenuElem (
+                                       _("Slowest"),
+                                       *_fade_out_images[FadeFast],
+                                       sigc::bind (sigc::mem_fun (*this, &Editor::set_fade_out_shape), FadeSlow)
+                                       ));
+                       
+                       dynamic_cast<ImageMenuItem*>(&items.back())->set_always_show_image ();
+                       
+                       items.push_back (
+                               ImageMenuElem (
+                                       _("Slow"),
+                                       *_fade_out_images[FadeLogB],
+                                       sigc::bind (sigc::mem_fun (*this, &Editor::set_fade_out_shape), FadeLogA)
+                                       ));
+                       
+                       dynamic_cast<ImageMenuItem*>(&items.back())->set_always_show_image ();
+                       
+                       items.push_back (
+                               ImageMenuElem (
+                                       _("Fast"),
+                                       *_fade_out_images[FadeLogA],
+                                       sigc::bind (sigc::mem_fun (*this, &Editor::set_fade_out_shape), FadeLogB)
+                                       ));
+                       
+                       dynamic_cast<ImageMenuItem*>(&items.back())->set_always_show_image ();
+                       
+                       items.push_back (
+                               ImageMenuElem (
+                                       _("Fastest"),
+                                       *_fade_out_images[FadeSlow],
+                                       sigc::bind (sigc::mem_fun (*this, &Editor::set_fade_out_shape), FadeFast)
+                                       ));
+                       
+                       dynamic_cast<ImageMenuItem*>(&items.back())->set_always_show_image ();
                }
 
                break;
@@ -1364,10 +1398,10 @@ Editor::popup_fade_context_menu (int button, int32_t time, ArdourCanvas::Item* i
 }
 
 void
-Editor::popup_track_context_menu (int button, int32_t time, ItemType item_type, bool with_selection, nframes64_t frame)
+Editor::popup_track_context_menu (int button, int32_t time, ItemType item_type, bool with_selection)
 {
        using namespace Menu_Helpers;
-       Menu* (Editor::*build_menu_function)(nframes64_t);
+       Menu* (Editor::*build_menu_function)();
        Menu *menu;
 
        switch (item_type) {
@@ -1408,7 +1442,7 @@ Editor::popup_track_context_menu (int button, int32_t time, ItemType item_type,
                return;
        }
 
-       menu = (this->*build_menu_function)(frame);
+       menu = (this->*build_menu_function)();
        menu->set_name ("ArdourContextMenu");
 
        /* now handle specific situations */
@@ -1417,8 +1451,8 @@ Editor::popup_track_context_menu (int button, int32_t time, ItemType item_type,
        case RegionItem:
        case RegionViewName:
        case RegionViewNameHighlight:
-        case LeftFrameHandle:
-        case RightFrameHandle:
+       case LeftFrameHandle:
+       case RightFrameHandle:
                if (!with_selection) {
                        if (region_edit_menu_split_item) {
                                if (clicked_regionview && clicked_regionview->region()->covers (get_preferred_edit_position())) {
@@ -1480,11 +1514,18 @@ Editor::popup_track_context_menu (int button, int32_t time, ItemType item_type,
                clicked_routeview->build_underlay_menu(menu);
        }
 
+       /* When the region menu is opened, we setup the actions so that they look right
+          in the menu.
+       */
+       sensitize_the_right_region_actions ();
+       _last_region_menu_was_main = false;
+
+       menu->signal_hide().connect (sigc::bind (sigc::mem_fun (*this, &Editor::sensitize_all_region_actions), true));
        menu->popup (button, time);
 }
 
 Menu*
-Editor::build_track_context_menu (nframes64_t)
+Editor::build_track_context_menu ()
 {
        using namespace Menu_Helpers;
 
@@ -1496,7 +1537,7 @@ Editor::build_track_context_menu (nframes64_t)
 }
 
 Menu*
-Editor::build_track_bus_context_menu (nframes64_t)
+Editor::build_track_bus_context_menu ()
 {
        using namespace Menu_Helpers;
 
@@ -1508,7 +1549,7 @@ Editor::build_track_bus_context_menu (nframes64_t)
 }
 
 Menu*
-Editor::build_track_region_context_menu (nframes64_t frame)
+Editor::build_track_region_context_menu ()
 {
        using namespace Menu_Helpers;
        MenuList& edit_items  = track_region_context_menu.items();
@@ -1520,31 +1561,19 @@ Editor::build_track_region_context_menu (nframes64_t frame)
        region_edit_menu_split_item = 0;
        region_edit_menu_split_multichannel_item = 0;
 
+       /* we might try to use items that are currently attached to a crossfade menu,
+          so clear that, too.
+       */
+       track_crossfade_context_menu.items().clear ();
+
        RouteTimeAxisView* rtv = dynamic_cast<RouteTimeAxisView*> (clicked_axisview);
 
        if (rtv) {
                boost::shared_ptr<Track> tr;
                boost::shared_ptr<Playlist> pl;
 
-               /* Don't offer a region submenu if we are in internal edit mode, as we don't select regions in this
-                  mode and so offering region context is somewhat confusing.
-               */
-               if ((tr = rtv->track()) && ((pl = tr->playlist())) && !internal_editing()) {
-                       Playlist::RegionList* regions = pl->regions_at ((nframes64_t) floor ( (double)frame * tr->speed()));
-
-                       if (selection->regions.size() > 1) {
-                               // there's already a multiple selection: just add a
-                               // single region context menu that will act on all
-                               // selected regions
-                               boost::shared_ptr<Region> dummy_region; // = NULL
-                               add_region_context_items (rtv->view(), dummy_region, edit_items);
-                       } else {
-                               for (Playlist::RegionList::reverse_iterator i = regions->rbegin(); i != regions->rend(); ++i) {
-                                       add_region_context_items (rtv->view(), (*i), edit_items);
-                               }
-                       }
-
-                       delete regions;
+               if ((tr = rtv->track())) {
+                       add_region_context_items (edit_items, tr);
                }
        }
 
@@ -1554,12 +1583,17 @@ Editor::build_track_region_context_menu (nframes64_t frame)
 }
 
 Menu*
-Editor::build_track_crossfade_context_menu (nframes64_t frame)
+Editor::build_track_crossfade_context_menu ()
 {
        using namespace Menu_Helpers;
        MenuList& edit_items  = track_crossfade_context_menu.items();
        edit_items.clear ();
 
+       /* we might try to use items that are currently attached to a crossfade menu,
+          so clear that, too.
+       */
+       track_region_context_menu.items().clear ();
+
        AudioTimeAxisView* atv = dynamic_cast<AudioTimeAxisView*> (clicked_axisview);
 
        if (atv) {
@@ -1569,29 +1603,25 @@ Editor::build_track_crossfade_context_menu (nframes64_t frame)
 
                if ((tr = atv->track()) && ((pl = tr->playlist()) != 0) && ((apl = boost::dynamic_pointer_cast<AudioPlaylist> (pl)) != 0)) {
 
-                       Playlist::RegionList* regions = pl->regions_at (frame);
                        AudioPlaylist::Crossfades xfades;
+                       framepos_t where;
+                       bool ignored;
 
-                       apl->crossfades_at (frame, xfades);
+                       /* The xfade menu is a bit of a special case, as we always use the mouse position
+                          to decide whether or not to display it (rather than the edit point).  No particularly
+                          strong reasons for this, other than it is a bit surprising to right-click on a xfade
+                          and not get a menu.
+                       */
+                       mouse_frame (where, ignored);
+                       apl->crossfades_at (where, xfades);
 
-                       bool many = xfades.size() > 1;
+                       bool const many = xfades.size() > 1;
 
                        for (AudioPlaylist::Crossfades::iterator i = xfades.begin(); i != xfades.end(); ++i) {
                                add_crossfade_context_items (atv->audio_view(), (*i), edit_items, many);
                        }
 
-                       if (selection->regions.size() > 1) {
-                               // there's already a multiple selection: just add a
-                               // single region context menu that will act on all
-                               // selected regions
-                               boost::shared_ptr<Region> dummy_region; // = NULL
-                               add_region_context_items (atv->audio_view(), dummy_region, edit_items);
-                       } else {
-                               for (Playlist::RegionList::reverse_iterator i = regions->rbegin(); i != regions->rend(); ++i) {
-                                       add_region_context_items (atv->audio_view(), (*i), edit_items);
-                               }
-                       }
-                       delete regions;
+                       add_region_context_items (edit_items, tr);
                }
        }
 
@@ -1601,7 +1631,7 @@ Editor::build_track_crossfade_context_menu (nframes64_t frame)
 }
 
 void
-Editor::analyze_region_selection()
+Editor::analyze_region_selection ()
 {
        if (analysis_window == 0) {
                analysis_window = new AnalysisWindow();
@@ -1637,7 +1667,7 @@ Editor::analyze_range_selection()
 }
 
 Menu*
-Editor::build_track_selection_context_menu (nframes64_t)
+Editor::build_track_selection_context_menu ()
 {
        using namespace Menu_Helpers;
        MenuList& edit_items  = track_selection_context_menu.items();
@@ -1711,237 +1741,49 @@ Editor::xfade_edit_right_region ()
 }
 
 void
-Editor::add_region_context_items (StreamView* sv, boost::shared_ptr<Region> region, Menu_Helpers::MenuList& edit_items)
+Editor::add_region_context_items (Menu_Helpers::MenuList& edit_items, boost::shared_ptr<Track> track)
 {
        using namespace Menu_Helpers;
-       Gtk::MenuItem* foo_item;
-       Menu     *region_menu = manage (new Menu);
-       MenuList& items       = region_menu->items();
-       region_menu->set_name ("ArdourContextMenu");
-
-       boost::shared_ptr<AudioRegion> ar;
-       boost::shared_ptr<MidiRegion>  mr;
-
-       if (region) {
-               ar = boost::dynamic_pointer_cast<AudioRegion> (region);
-               mr = boost::dynamic_pointer_cast<MidiRegion> (region);
-
-               /* when this particular menu pops up, make the relevant region
-                  become selected.
-               */
-
-               region_menu->signal_map_event().connect (
-                       sigc::bind (sigc::mem_fun(*this, &Editor::set_selected_regionview_from_map_event), sv, boost::weak_ptr<Region>(region)));
-
-               items.push_back (MenuElem (_("Rename..."), sigc::mem_fun(*this, &Editor::rename_region)));
-               if (mr) {
-                       items.push_back (MenuElem (_("List Editor..."), sigc::mem_fun(*this, &Editor::show_midi_list_editor)));
-               }
-               items.push_back (MenuElem (_("Region Properties..."), sigc::mem_fun(*this, &Editor::edit_region)));
-       }
-
-       items.push_back (MenuElem (_("Raise to Top Layer"), sigc::mem_fun(*this, &Editor::raise_region_to_top)));
-       items.push_back (MenuElem (_("Lower to Bottom Layer"), sigc::mem_fun  (*this, &Editor::lower_region_to_bottom)));
-       items.push_back (SeparatorElem());
-       items.push_back (MenuElem (_("Define Sync Point"), sigc::mem_fun(*this, &Editor::set_region_sync_from_edit_point)));
-       if (_edit_point == EditAtMouse) {
-               items.back ().set_sensitive (false);
-       }
-       items.push_back (MenuElem (_("Remove Sync Point"), sigc::mem_fun(*this, &Editor::remove_region_sync)));
-       items.push_back (SeparatorElem());
-
-       items.push_back (MenuElem (_("Audition"), sigc::mem_fun(*this, &Editor::play_selected_region)));
-       items.push_back (MenuElem (_("Export..."), sigc::mem_fun(*this, &Editor::export_region)));
-       items.push_back (MenuElem (_("Bounce"), sigc::mem_fun(*this, &Editor::bounce_region_selection)));
-
-       if (ar) {
-               items.push_back (MenuElem (_("Spectral Analysis..."), sigc::mem_fun(*this, &Editor::analyze_region_selection)));
-       }
-
-       items.push_back (SeparatorElem());
-
-       sigc::connection fooc;
-       boost::shared_ptr<Region> region_to_check;
-
-       if (region) {
-               region_to_check = region;
-       } else {
-               region_to_check = selection->regions.front()->region();
-       }
-
-       items.push_back (CheckMenuElem (_("Lock")));
-       CheckMenuItem* region_lock_item = static_cast<CheckMenuItem*>(&items.back());
-       if (region_to_check->locked()) {
-               region_lock_item->set_active();
-       }
-       region_lock_item->signal_activate().connect (sigc::mem_fun(*this, &Editor::toggle_region_lock));
-
-       items.push_back (CheckMenuElem (_("Glue to Bars and Beats")));
-       CheckMenuItem* bbt_glue_item = static_cast<CheckMenuItem*>(&items.back());
-
-       switch (region_to_check->position_lock_style()) {
-       case MusicTime:
-               bbt_glue_item->set_active (true);
-               break;
-       default:
-               bbt_glue_item->set_active (false);
-               break;
-       }
-
-       bbt_glue_item->signal_activate().connect (sigc::mem_fun (*this, &Editor::toggle_region_lock_style));
-
-       items.push_back (CheckMenuElem (_("Mute")));
-       CheckMenuItem* region_mute_item = static_cast<CheckMenuItem*>(&items.back());
-       fooc = region_mute_item->signal_activate().connect (sigc::mem_fun(*this, &Editor::toggle_region_mute));
-       if (region_to_check->muted()) {
-               fooc.block (true);
-               region_mute_item->set_active();
-               fooc.block (false);
-       }
-        
-        items.push_back (MenuElem (_("Transpose..."), mem_fun(*this, &Editor::pitch_shift_regions)));
-
-       if (!Profile->get_sae()) {
-               items.push_back (CheckMenuElem (_("Opaque")));
-               CheckMenuItem* region_opaque_item = static_cast<CheckMenuItem*>(&items.back());
-               fooc = region_opaque_item->signal_activate().connect (sigc::mem_fun(*this, &Editor::toggle_region_opaque));
-               if (region_to_check->opaque()) {
-                       fooc.block (true);
-                       region_opaque_item->set_active();
-                       fooc.block (false);
-               }
-       }
-
-       items.push_back (CheckMenuElem (_("Original Position"), sigc::mem_fun(*this, &Editor::naturalize)));
-       if (region_to_check->at_natural_position()) {
-               items.back().set_sensitive (false);
-       }
-
-       items.push_back (SeparatorElem());
-
-       if (ar) {
-
-               RegionView* rv = sv->find_view (ar);
-               AudioRegionView* arv = dynamic_cast<AudioRegionView*>(rv);
-
-               if (!Profile->get_sae()) {
-                       items.push_back (MenuElem (_("Reset Envelope"), sigc::mem_fun(*this, &Editor::reset_region_gain_envelopes)));
-
-                       items.push_back (CheckMenuElem (_("Envelope Visible")));
-                       CheckMenuItem* region_envelope_visible_item = static_cast<CheckMenuItem*> (&items.back());
-                       fooc = region_envelope_visible_item->signal_activate().connect (sigc::mem_fun(*this, &Editor::toggle_gain_envelope_visibility));
-                       if (arv->envelope_visible()) {
-                               fooc.block (true);
-                               region_envelope_visible_item->set_active (true);
-                               fooc.block (false);
-                       }
-
-                       items.push_back (CheckMenuElem (_("Envelope Active")));
-                       CheckMenuItem* region_envelope_active_item = static_cast<CheckMenuItem*> (&items.back());
-                       fooc = region_envelope_active_item->signal_activate().connect (sigc::mem_fun(*this, &Editor::toggle_gain_envelope_active));
-
-                       if (ar->envelope_active()) {
-                               fooc.block (true);
-                               region_envelope_active_item->set_active (true);
-                               fooc.block (false);
-                       }
-
-                       items.push_back (SeparatorElem());
-               }
-
-               items.push_back (MenuElem (_("Normalize..."), sigc::mem_fun(*this, &Editor::normalize_region)));
-               if (ar->scale_amplitude() != 1) {
-                       items.push_back (MenuElem (_("Reset Gain"), sigc::mem_fun(*this, &Editor::reset_region_scale_amplitude)));
-               }
-
-       } else if (mr) {
-               items.push_back (MenuElem (_("Quantize"), sigc::mem_fun(*this, &Editor::quantize_region)));
-               items.push_back (MenuElem (_("Fork"), sigc::mem_fun(*this, &Editor::fork_region)));
-               items.push_back (SeparatorElem());
-       }
-
-       items.push_back (MenuElem (_("Strip Silence..."), sigc::mem_fun (*this, &Editor::strip_region_silence)));
-       items.push_back (MenuElem (_("Reverse"), sigc::mem_fun(*this, &Editor::reverse_region)));
-       items.push_back (SeparatorElem());
-
-       /* range related stuff */
-
-       items.push_back (MenuElem (_("Add Single Range"), sigc::mem_fun (*this, &Editor::add_location_from_audio_region)));
-       items.push_back (MenuElem (_("Add Range Markers"), sigc::mem_fun (*this, &Editor::add_locations_from_audio_region)));
-       if (selection->regions.size() < 2) {
-               items.back().set_sensitive (false);
-       }
-
-       items.push_back (MenuElem (_("Set Range Selection"), sigc::mem_fun (*this, &Editor::set_selection_from_region)));
-       items.push_back (SeparatorElem());
-
-       /* Nudge region */
-
-       Menu *nudge_menu = manage (new Menu());
-       MenuList& nudge_items = nudge_menu->items();
-       nudge_menu->set_name ("ArdourContextMenu");
-
-       nudge_items.push_back (MenuElem (_("Nudge Forward"), (sigc::bind (sigc::mem_fun(*this, &Editor::nudge_forward), false, false))));
-       nudge_items.push_back (MenuElem (_("Nudge Backward"), (sigc::bind (sigc::mem_fun(*this, &Editor::nudge_backward), false, false))));
-       nudge_items.push_back (MenuElem (_("Nudge Forward by Capture Offset"), (sigc::mem_fun(*this, &Editor::nudge_forward_capture_offset))));
-       nudge_items.push_back (MenuElem (_("Nudge Backward by Capture Offset"), (sigc::mem_fun(*this, &Editor::nudge_backward_capture_offset))));
-
-       items.push_back (MenuElem (_("Nudge"), *nudge_menu));
-       items.push_back (SeparatorElem());
-
-       Menu *trim_menu = manage (new Menu);
-       MenuList& trim_items = trim_menu->items();
-       trim_menu->set_name ("ArdourContextMenu");
-
-       trim_items.push_back (MenuElem (_("Start to Edit Point"), sigc::mem_fun(*this, &Editor::trim_region_from_edit_point)));
-       foo_item = &trim_items.back();
-       if (_edit_point == EditAtMouse) {
-               foo_item->set_sensitive (false);
-       }
-       trim_items.push_back (MenuElem (_("Edit Point to End"), sigc::mem_fun(*this, &Editor::trim_region_to_edit_point)));
-       foo_item = &trim_items.back();
-       if (_edit_point == EditAtMouse) {
-               foo_item->set_sensitive (false);
-       }
-       trim_items.push_back (MenuElem (_("Trim to Loop"), sigc::mem_fun(*this, &Editor::trim_region_to_loop)));
-       trim_items.push_back (MenuElem (_("Trim to Punch"), sigc::mem_fun(*this, &Editor::trim_region_to_punch)));
-
-       items.push_back (MenuElem (_("Trim"), *trim_menu));
-       items.push_back (SeparatorElem());
-
-       items.push_back (MenuElem (_("Split"), (sigc::mem_fun(*this, &Editor::split))));
-       region_edit_menu_split_item = &items.back();
-
-       if (_edit_point == EditAtMouse) {
-               region_edit_menu_split_item->set_sensitive (false);
-       }
-
-       items.push_back (MenuElem (_("Make Mono Regions"), (sigc::mem_fun(*this, &Editor::split_multichannel_region))));
-       region_edit_menu_split_multichannel_item = &items.back();
-
-       items.push_back (MenuElem (_("Duplicate"), (sigc::bind (sigc::mem_fun(*this, &Editor::duplicate_dialog), false))));
-       items.push_back (MenuElem (_("Multi-Duplicate..."), (sigc::bind (sigc::mem_fun(*this, &Editor::duplicate_dialog), true))));
-       items.push_back (MenuElem (_("Fill Track"), (sigc::mem_fun(*this, &Editor::region_fill_track))));
-       items.push_back (SeparatorElem());
-       items.push_back (MenuElem (_("Remove"), sigc::mem_fun(*this, &Editor::remove_selected_regions)));
-
+       
        /* OK, stick the region submenu at the top of the list, and then add
           the standard items.
        */
 
+       RegionSelection rs = get_regions_from_selection_and_entered ();
+       
+       string::size_type pos = 0;
+       string menu_item_name = (rs.size() == 1) ? rs.front()->region()->name() : _("Selected Regions");
+
        /* we have to hack up the region name because "_" has a special
           meaning for menu titles.
        */
 
-       string::size_type pos = 0;
-       string menu_item_name = (region) ? region->name() : _("Selected Regions");
-
        while ((pos = menu_item_name.find ("_", pos)) != string::npos) {
                menu_item_name.replace (pos, 1, "__");
                pos += 2;
        }
 
-       edit_items.push_back (MenuElem (menu_item_name, *region_menu));
+       if (_popup_region_menu_item == 0) {
+               _popup_region_menu_item = new MenuItem (menu_item_name);
+               _popup_region_menu_item->set_submenu (*dynamic_cast<Menu*> (ActionManager::get_widget (X_("/PopupRegionMenu"))));
+               _popup_region_menu_item->show ();
+       } else {
+               _popup_region_menu_item->set_label (menu_item_name);
+       }
+
+       /* Use the mouse position rather than the edit point to decide whether to show the `choose top region'
+          dialogue.  If we use the edit point it gets a bit messy because the user still has to click over
+          *some* region in order to get the region context menu stuff to be displayed at all.
+       */
+       
+       framepos_t mouse;
+       bool ignored;
+       mouse_frame (mouse, ignored);
+
+       edit_items.push_back (*_popup_region_menu_item);
+       if (track->playlist()->count_regions_at (mouse) > 1 && (layering_order_editor == 0 || !layering_order_editor->is_visible ())) {
+               edit_items.push_back (*manage (_region_actions->get_action ("choose-top-region")->create_menu_item ()));
+       }
        edit_items.push_back (SeparatorElem());
 }
 
@@ -1966,7 +1808,7 @@ Editor::add_selection_context_items (Menu_Helpers::MenuList& edit_items)
        }
 
        edit_items.push_back (SeparatorElem());
-       edit_items.push_back (MenuElem (_("Convert to region in-place"), mem_fun(*this, &Editor::separate_region_from_selection)));
+       edit_items.push_back (MenuElem (_("Convert to Region In-Place"), mem_fun(*this, &Editor::separate_region_from_selection)));
        edit_items.push_back (MenuElem (_("Convert to Region in Region List"), sigc::mem_fun(*this, &Editor::new_region_from_selection)));
 
        edit_items.push_back (SeparatorElem());
@@ -2008,7 +1850,7 @@ Editor::add_dstream_context_items (Menu_Helpers::MenuList& edit_items)
        play_items.push_back (MenuElem (_("Play From Start"), sigc::mem_fun(*this, &Editor::play_from_start)));
        play_items.push_back (MenuElem (_("Play Region"), sigc::mem_fun(*this, &Editor::play_selected_region)));
        play_items.push_back (SeparatorElem());
-       play_items.push_back (MenuElem (_("Loop Region"), sigc::mem_fun(*this, &Editor::loop_selected_region)));
+       play_items.push_back (MenuElem (_("Loop Region"), sigc::bind (sigc::mem_fun (*this, &Editor::set_loop_from_region), true)));
 
        edit_items.push_back (MenuElem (_("Play"), *play_menu));
 
@@ -2048,8 +1890,8 @@ Editor::add_dstream_context_items (Menu_Helpers::MenuList& edit_items)
 
        cutnpaste_items.push_back (SeparatorElem());
 
-       cutnpaste_items.push_back (MenuElem (_("Align"), sigc::bind (sigc::mem_fun(*this, &Editor::align), ARDOUR::SyncPoint)));
-       cutnpaste_items.push_back (MenuElem (_("Align Relative"), sigc::bind (sigc::mem_fun(*this, &Editor::align_relative), ARDOUR::SyncPoint)));
+       cutnpaste_items.push_back (MenuElem (_("Align"), sigc::bind (sigc::mem_fun (*this, &Editor::align_regions), ARDOUR::SyncPoint)));
+       cutnpaste_items.push_back (MenuElem (_("Align Relative"), sigc::bind (sigc::mem_fun (*this, &Editor::align_regions_relative), ARDOUR::SyncPoint)));
 
        edit_items.push_back (MenuElem (_("Edit"), *cutnpaste_menu));
 
@@ -2166,6 +2008,7 @@ Editor::set_snap_to (SnapType st)
        case SnapToBeatDiv32:
        case SnapToBeatDiv28:
        case SnapToBeatDiv24:
+       case SnapToBeatDiv20:
        case SnapToBeatDiv16:
        case SnapToBeatDiv14:
        case SnapToBeatDiv12:
@@ -2245,7 +2088,7 @@ Editor::set_edit_point_preference (EditPoint ep, bool force)
                Glib::RefPtr<RadioAction>::cast_dynamic(act)->set_active (true);
        }
 
-       nframes64_t foo;
+       framepos_t foo;
        bool in_track_canvas;
 
        if (!mouse_frame (foo, in_track_canvas)) {
@@ -2323,10 +2166,10 @@ Editor::set_state (const XMLNode& node, int /*version*/)
 
        set_default_size (g.base_width, g.base_height);
        move (x, y);
-
+        
        if (_session && (prop = node.property ("playhead"))) {
-               nframes64_t pos;
-                sscanf (prop->value().c_str(), "%" PRIi64, &pos);
+               framepos_t pos;
+               sscanf (prop->value().c_str(), "%" PRIi64, &pos);
                playhead_cursor->set_position (pos);
        } else {
                playhead_cursor->set_position (0);
@@ -2342,6 +2185,8 @@ Editor::set_state (const XMLNode& node, int /*version*/)
 
        if ((prop = node.property ("zoom"))) {
                reset_zoom (PBD::atof (prop->value()));
+       } else {
+               reset_zoom (frames_per_unit);
        }
 
        if ((prop = node.property ("snap-to"))) {
@@ -2359,8 +2204,8 @@ Editor::set_state (const XMLNode& node, int /*version*/)
                set_mouse_mode (MouseObject, true);
        }
 
-       if ((prop = node.property ("left-frame")) != 0){
-               nframes64_t pos;
+       if ((prop = node.property ("left-frame")) != 0) {
+               framepos_t pos;
                if (sscanf (prop->value().c_str(), "%" PRId64, &pos) == 1) {
                        reset_x_origin (pos);
                }
@@ -2390,7 +2235,7 @@ Editor::set_state (const XMLNode& node, int /*version*/)
 
        if ((prop = node.property ("show-measures"))) {
                bool yn = string_is_affirmative (prop->value());
-               _show_measures = !yn;
+               _show_measures = yn;
                RefPtr<Action> act = ActionManager::get_action (X_("Editor"), X_("ToggleMeasureVisibility"));
                if (act) {
                        RefPtr<ToggleAction> tact = RefPtr<ToggleAction>::cast_dynamic(act);
@@ -2412,17 +2257,17 @@ Editor::set_state (const XMLNode& node, int /*version*/)
                }
        }
 
-        if ((prop = node.property ("stationary-playhead"))) {
-                bool yn = (prop->value() == "yes");
-                set_stationary_playhead (yn);
-                RefPtr<Action> act = ActionManager::get_action (X_("Editor"), X_("toggle-stationary-playhead"));
-                if (act) {
-                        RefPtr<ToggleAction> tact = RefPtr<ToggleAction>::cast_dynamic(act);
-                        if (tact->get_active() != yn) {
-                                tact->set_active (yn);
-                        }
-                }
-        }
+       if ((prop = node.property ("stationary-playhead"))) {
+               bool yn = (prop->value() == "yes");
+               set_stationary_playhead (yn);
+               RefPtr<Action> act = ActionManager::get_action (X_("Editor"), X_("toggle-stationary-playhead"));
+               if (act) {
+                       RefPtr<ToggleAction> tact = RefPtr<ToggleAction>::cast_dynamic(act);
+                       if (tact->get_active() != yn) {
+                               tact->set_active (yn);
+                       }
+               }
+       }
         
        if ((prop = node.property ("region-list-sort-type"))) {
                RegionListSortType st;
@@ -2438,36 +2283,49 @@ Editor::set_state (const XMLNode& node, int /*version*/)
        if ((prop = node.property ("show-editor-mixer"))) {
 
                Glib::RefPtr<Action> act = ActionManager::get_action (X_("Editor"), X_("show-editor-mixer"));
-               if (act) {
-
-                       Glib::RefPtr<ToggleAction> tact = Glib::RefPtr<ToggleAction>::cast_dynamic(act);
-                       bool yn = string_is_affirmative (prop->value());
+               assert (act);
 
-                       /* do it twice to force the change */
-
-                       tact->set_active (!yn);
-                       tact->set_active (yn);
-               }
+               Glib::RefPtr<ToggleAction> tact = Glib::RefPtr<ToggleAction>::cast_dynamic(act);
+               bool yn = string_is_affirmative (prop->value());
+               
+               /* do it twice to force the change */
+               
+               tact->set_active (!yn);
+               tact->set_active (yn);
        }
 
        if ((prop = node.property ("show-editor-list"))) {
 
                Glib::RefPtr<Action> act = ActionManager::get_action (X_("Editor"), X_("show-editor-list"));
-               assert(act);
-               if (act) {
+               assert (act);
+
+               Glib::RefPtr<ToggleAction> tact = Glib::RefPtr<ToggleAction>::cast_dynamic(act);
+               bool yn = string_is_affirmative (prop->value());
+               
+               /* do it twice to force the change */
+               
+               tact->set_active (!yn);
+               tact->set_active (yn);
+       }
 
-                       Glib::RefPtr<ToggleAction> tact = Glib::RefPtr<ToggleAction>::cast_dynamic(act);
-                       bool yn = string_is_affirmative (prop->value());
+       if ((prop = node.property (X_("editor-list-page")))) {
+               _the_notebook.set_current_page (atoi (prop->value ()));
+       }
 
-                       /* do it twice to force the change */
+       if ((prop = node.property (X_("show-marker-lines")))) {
+               Glib::RefPtr<Action> act = ActionManager::get_action (X_("Editor"), X_("show-marker-lines"));
+               assert (act);
+               Glib::RefPtr<ToggleAction> tact = Glib::RefPtr<ToggleAction>::cast_dynamic (act);
+               bool yn = string_is_affirmative (prop->value ());
 
-                       tact->set_active (!yn);
-                       tact->set_active (yn);
-               }
+               tact->set_active (!yn);
+               tact->set_active (yn);
        }
 
-       if ((prop = node.property (X_("editor-list-page")))) {
-               the_notebook.set_current_page (atoi (prop->value ()));
+       XMLNodeList children = node.children ();
+       for (XMLNodeList::const_iterator i = children.begin(); i != children.end(); ++i) {
+               selection->set_state (**i, Stateful::current_state_version);
+               _regions->set_state (**i);
        }
 
        return 0;
@@ -2506,6 +2364,9 @@ Editor::get_state ()
                geometry->add_property("y-off", string(buf));
                snprintf(buf,sizeof(buf), "%d",gtk_paned_get_position (static_cast<Paned*>(&edit_pane)->gobj()));
                geometry->add_property("edit-horizontal-pane-pos", string(buf));
+               geometry->add_property("notebook-shrunk", _notebook_shrunk ? "1" : "0");
+               snprintf(buf,sizeof(buf), "%d",pre_maximal_horizontal_pane_position);
+               geometry->add_property("pre-maximal-horizontal-pane-position", string(buf));
                snprintf(buf,sizeof(buf), "%d",gtk_paned_get_position (static_cast<Paned*>(&editor_summary_pane)->gobj()));
                geometry->add_property("edit-vertical-pane-pos", string(buf));
 
@@ -2534,7 +2395,7 @@ Editor::get_state ()
 
        node->add_property ("show-measures", _show_measures ? "yes" : "no");
        node->add_property ("follow-playhead", _follow_playhead ? "yes" : "no");
-        node->add_property ("stationary-playhead", _stationary_playhead ? "yes" : "no");
+       node->add_property ("stationary-playhead", _stationary_playhead ? "yes" : "no");
        node->add_property ("xfades-visible", _xfade_visibility ? "yes" : "no");
        node->add_property ("region-list-sort-type", enum_2_string (_regions->sort_type ()));
        node->add_property ("mouse-mode", enum2str(mouse_mode));
@@ -2553,9 +2414,20 @@ Editor::get_state ()
                node->add_property (X_("show-editor-list"), tact->get_active() ? "yes" : "no");
        }
 
-       snprintf (buf, sizeof (buf), "%d", the_notebook.get_current_page ());
+       snprintf (buf, sizeof (buf), "%d", _the_notebook.get_current_page ());
        node->add_property (X_("editor-list-page"), buf);
 
+        if (button_bindings) {
+                XMLNode* bb = new XMLNode (X_("Buttons"));
+                button_bindings->save (*bb);
+                node->add_child_nocopy (*bb);
+        } 
+
+       node->add_property (X_("show-marker-lines"), _show_marker_lines ? "yes" : "no");
+
+       node->add_child_nocopy (selection->get_state ());
+       node->add_child_nocopy (_regions->get_state ());
+       
        return *node;
 }
 
@@ -2586,7 +2458,7 @@ Editor::trackview_by_y_position (double y)
  *  @param event Event to get current key modifier information from, or 0.
  */
 void
-Editor::snap_to_with_modifier (nframes64_t& start, GdkEvent const * event, int32_t direction, bool for_mark)
+Editor::snap_to_with_modifier (framepos_t& start, GdkEvent const * event, int32_t direction, bool for_mark)
 {
        if (!_session || !event) {
                return;
@@ -2604,7 +2476,7 @@ Editor::snap_to_with_modifier (nframes64_t& start, GdkEvent const * event, int32
 }
 
 void
-Editor::snap_to (nframes64_t& start, int32_t direction, bool for_mark)
+Editor::snap_to (framepos_t& start, int32_t direction, bool for_mark)
 {
        if (!_session || _snap_mode == SnapOff) {
                return;
@@ -2614,58 +2486,54 @@ Editor::snap_to (nframes64_t& start, int32_t direction, bool for_mark)
 }
 
 void
-Editor::timecode_snap_to_internal (nframes64_t& start, int32_t direction, bool /*for_mark*/)
+Editor::timecode_snap_to_internal (framepos_t& start, int32_t direction, bool /*for_mark*/)
 {
-       const nframes64_t one_timecode_second = (nframes64_t)(rint(_session->timecode_frames_per_second()) * _session->frames_per_timecode_frame());
-       nframes64_t one_timecode_minute = (nframes64_t)(rint(_session->timecode_frames_per_second()) * _session->frames_per_timecode_frame() * 60);
+       const framepos_t one_timecode_second = (framepos_t)(rint(_session->timecode_frames_per_second()) * _session->frames_per_timecode_frame());
+       framepos_t one_timecode_minute = (framepos_t)(rint(_session->timecode_frames_per_second()) * _session->frames_per_timecode_frame() * 60);
 
        switch (_snap_type) {
        case SnapToTimecodeFrame:
                if (((direction == 0) && (fmod((double)start, (double)_session->frames_per_timecode_frame()) > (_session->frames_per_timecode_frame() / 2))) || (direction > 0)) {
-                       start = (nframes64_t) (ceil ((double) start / _session->frames_per_timecode_frame()) * _session->frames_per_timecode_frame());
+                       start = (framepos_t) (ceil ((double) start / _session->frames_per_timecode_frame()) * _session->frames_per_timecode_frame());
                } else {
-                       start = (nframes64_t) (floor ((double) start / _session->frames_per_timecode_frame()) *  _session->frames_per_timecode_frame());
+                       start = (framepos_t) (floor ((double) start / _session->frames_per_timecode_frame()) *  _session->frames_per_timecode_frame());
                }
                break;
 
        case SnapToTimecodeSeconds:
-               if (_session->timecode_offset_negative())
-               {
-                       start += _session->timecode_offset ();
+               if (_session->config.get_timecode_offset_negative()) {
+                       start += _session->config.get_timecode_offset ();
                } else {
-                       start -= _session->timecode_offset ();
+                       start -= _session->config.get_timecode_offset ();
                }
                if (((direction == 0) && (start % one_timecode_second > one_timecode_second / 2)) || direction > 0) {
-                       start = (nframes64_t) ceil ((double) start / one_timecode_second) * one_timecode_second;
+                       start = (framepos_t) ceil ((double) start / one_timecode_second) * one_timecode_second;
                } else {
-                       start = (nframes64_t) floor ((double) start / one_timecode_second) * one_timecode_second;
+                       start = (framepos_t) floor ((double) start / one_timecode_second) * one_timecode_second;
                }
 
-               if (_session->timecode_offset_negative())
-               {
-                       start -= _session->timecode_offset ();
+               if (_session->config.get_timecode_offset_negative()) {
+                       start -= _session->config.get_timecode_offset ();
                } else {
-                       start += _session->timecode_offset ();
+                       start += _session->config.get_timecode_offset ();
                }
                break;
 
        case SnapToTimecodeMinutes:
-               if (_session->timecode_offset_negative())
-               {
-                       start += _session->timecode_offset ();
+               if (_session->config.get_timecode_offset_negative()) {
+                       start += _session->config.get_timecode_offset ();
                } else {
-                       start -= _session->timecode_offset ();
+                       start -= _session->config.get_timecode_offset ();
                }
                if (((direction == 0) && (start % one_timecode_minute > one_timecode_minute / 2)) || direction > 0) {
-                       start = (nframes64_t) ceil ((double) start / one_timecode_minute) * one_timecode_minute;
+                       start = (framepos_t) ceil ((double) start / one_timecode_minute) * one_timecode_minute;
                } else {
-                       start = (nframes64_t) floor ((double) start / one_timecode_minute) * one_timecode_minute;
+                       start = (framepos_t) floor ((double) start / one_timecode_minute) * one_timecode_minute;
                }
-               if (_session->timecode_offset_negative())
-               {
-                       start -= _session->timecode_offset ();
+               if (_session->config.get_timecode_offset_negative()) {
+                       start -= _session->config.get_timecode_offset ();
                } else {
-                       start += _session->timecode_offset ();
+                       start += _session->config.get_timecode_offset ();
                }
                break;
        default:
@@ -2675,13 +2543,13 @@ Editor::timecode_snap_to_internal (nframes64_t& start, int32_t direction, bool /
 }
 
 void
-Editor::snap_to_internal (nframes64_t& start, int32_t direction, bool for_mark)
+Editor::snap_to_internal (framepos_t& start, int32_t direction, bool for_mark)
 {
-       const nframes64_t one_second = _session->frame_rate();
-       const nframes64_t one_minute = _session->frame_rate() * 60;
-       nframes64_t presnap = start;
-       nframes64_t before;
-       nframes64_t after;
+       const framepos_t one_second = _session->frame_rate();
+       const framepos_t one_minute = _session->frame_rate() * 60;
+       framepos_t presnap = start;
+       framepos_t before;
+       framepos_t after;
 
        switch (_snap_type) {
        case SnapToTimecodeFrame:
@@ -2691,25 +2559,25 @@ Editor::snap_to_internal (nframes64_t& start, int32_t direction, bool for_mark)
 
        case SnapToCDFrame:
                if (((direction == 0) && (start % (one_second/75) > (one_second/75) / 2)) || (direction > 0)) {
-                       start = (nframes64_t) ceil ((double) start / (one_second / 75)) * (one_second / 75);
+                       start = (framepos_t) ceil ((double) start / (one_second / 75)) * (one_second / 75);
                } else {
-                       start = (nframes64_t) floor ((double) start / (one_second / 75)) * (one_second / 75);
+                       start = (framepos_t) floor ((double) start / (one_second / 75)) * (one_second / 75);
                }
                break;
 
        case SnapToSeconds:
                if (((direction == 0) && (start % one_second > one_second / 2)) || (direction > 0)) {
-                       start = (nframes64_t) ceil ((double) start / one_second) * one_second;
+                       start = (framepos_t) ceil ((double) start / one_second) * one_second;
                } else {
-                       start = (nframes64_t) floor ((double) start / one_second) * one_second;
+                       start = (framepos_t) floor ((double) start / one_second) * one_second;
                }
                break;
 
        case SnapToMinutes:
                if (((direction == 0) && (start % one_minute > one_minute / 2)) || (direction > 0)) {
-                       start = (nframes64_t) ceil ((double) start / one_minute) * one_minute;
+                       start = (framepos_t) ceil ((double) start / one_minute) * one_minute;
                } else {
-                       start = (nframes64_t) floor ((double) start / one_minute) * one_minute;
+                       start = (framepos_t) floor ((double) start / one_minute) * one_minute;
                }
                break;
 
@@ -2730,6 +2598,9 @@ Editor::snap_to_internal (nframes64_t& start, int32_t direction, bool for_mark)
        case SnapToBeatDiv24:
                start = _session->tempo_map().round_to_beat_subdivision (start, 24, direction);
                break;
+       case SnapToBeatDiv20:
+               start = _session->tempo_map().round_to_beat_subdivision (start, 20, direction);
+               break;
        case SnapToBeatDiv16:
                start = _session->tempo_map().round_to_beat_subdivision (start, 16, direction);
                break;
@@ -2771,11 +2642,11 @@ Editor::snap_to_internal (nframes64_t& start, int32_t direction, bool for_mark)
 
                _session->locations()->marks_either_side (start, before, after);
 
-               if (before == max_frames) {
+               if (before == max_framepos) {
                        start = after;
-               } else if (after == max_frames) {
+               } else if (after == max_framepos) {
                        start = before;
-               } else if (before != max_frames && after != max_frames) {
+               } else if (before != max_framepos && after != max_framepos) {
                        /* have before and after */
                        if ((start - before) < (after - start)) {
                                start = before;
@@ -2792,8 +2663,8 @@ Editor::snap_to_internal (nframes64_t& start, int32_t direction, bool for_mark)
        case SnapToRegionBoundary:
                if (!region_boundary_cache.empty()) {
 
-                       vector<nframes64_t>::iterator prev = region_boundary_cache.end ();
-                       vector<nframes64_t>::iterator next = region_boundary_cache.end ();
+                       vector<framepos_t>::iterator prev = region_boundary_cache.end ();
+                       vector<framepos_t>::iterator next = region_boundary_cache.end ();
 
                        if (direction > 0) {
                                next = std::upper_bound (region_boundary_cache.begin(), region_boundary_cache.end(), start);
@@ -2806,8 +2677,8 @@ Editor::snap_to_internal (nframes64_t& start, int32_t direction, bool for_mark)
                                prev--;
                        }
 
-                       nframes64_t const p = (prev == region_boundary_cache.end()) ? region_boundary_cache.front () : *prev;
-                       nframes64_t const n = (next == region_boundary_cache.end()) ? region_boundary_cache.back () : *next;
+                       framepos_t const p = (prev == region_boundary_cache.end()) ? region_boundary_cache.front () : *prev;
+                       framepos_t const n = (next == region_boundary_cache.end()) ? region_boundary_cache.back () : *next;
 
                        if (start > (p + n) / 2) {
                                start = n;
@@ -2885,7 +2756,6 @@ Editor::setup_toolbar ()
        mouse_mode_button_box->pack_start (mouse_audition_button);
        mouse_mode_button_box->pack_start (internal_edit_button);
 
-       vector<string> edit_mode_strings;
        edit_mode_strings.push_back (edit_mode_to_string (Slide));
        if (!Profile->get_sae()) {
                edit_mode_strings.push_back (edit_mode_to_string (Splice));
@@ -2908,13 +2778,13 @@ Editor::setup_toolbar ()
        }
 
        _mouse_mode_tearoff->Detach.connect (sigc::bind (sigc::mem_fun(*this, &Editor::detach_tearoff), static_cast<Box*>(&toolbar_hbox),
-                                                         &_mouse_mode_tearoff->tearoff_window()));
+                                                        &_mouse_mode_tearoff->tearoff_window()));
        _mouse_mode_tearoff->Attach.connect (sigc::bind (sigc::mem_fun(*this, &Editor::reattach_tearoff), static_cast<Box*> (&toolbar_hbox),
-                                                         &_mouse_mode_tearoff->tearoff_window(), 1));
+                                                        &_mouse_mode_tearoff->tearoff_window(), 1));
        _mouse_mode_tearoff->Hidden.connect (sigc::bind (sigc::mem_fun(*this, &Editor::detach_tearoff), static_cast<Box*>(&toolbar_hbox),
-                                                         &_mouse_mode_tearoff->tearoff_window()));
+                                                        &_mouse_mode_tearoff->tearoff_window()));
        _mouse_mode_tearoff->Visible.connect (sigc::bind (sigc::mem_fun(*this, &Editor::reattach_tearoff), static_cast<Box*> (&toolbar_hbox),
-                                                          &_mouse_mode_tearoff->tearoff_window(), 1));
+                                                         &_mouse_mode_tearoff->tearoff_window(), 1));
 
        mouse_move_button.set_mode (false);
        mouse_select_button.set_mode (false);
@@ -2948,15 +2818,15 @@ Editor::setup_toolbar ()
        _zoom_box.set_border_width (0);
 
        zoom_in_button.set_name ("EditorTimeButton");
-       zoom_in_button.set_image (*(manage (new Image (Stock::ZOOM_IN, Gtk::ICON_SIZE_MENU))));
+       zoom_in_button.set_image (*(manage (new Image (::get_icon ("zoom_in")))));
        zoom_in_button.signal_clicked().connect (sigc::bind (sigc::mem_fun(*this, &Editor::temporal_zoom_step), false));
 
        zoom_out_button.set_name ("EditorTimeButton");
-       zoom_out_button.set_image (*(manage (new Image (Stock::ZOOM_OUT, Gtk::ICON_SIZE_MENU))));
+       zoom_out_button.set_image (*(manage (new Image (::get_icon ("zoom_out")))));
        zoom_out_button.signal_clicked().connect (sigc::bind (sigc::mem_fun(*this, &Editor::temporal_zoom_step), true));
 
        zoom_out_full_button.set_name ("EditorTimeButton");
-       zoom_out_full_button.set_image (*(manage (new Image (Stock::ZOOM_100, Gtk::ICON_SIZE_MENU))));
+       zoom_out_full_button.set_image (*(manage (new Image (::get_icon ("zoom_full")))));
        zoom_out_full_button.signal_clicked().connect (sigc::mem_fun(*this, &Editor::temporal_zoom_session));
 
        zoom_focus_selector.set_name ("ZoomFocusSelector");
@@ -2967,21 +2837,23 @@ Editor::setup_toolbar ()
        _zoom_box.pack_start (zoom_in_button, false, false);
        _zoom_box.pack_start (zoom_out_full_button, false, false);
 
+       _zoom_box.pack_start (zoom_focus_selector);
+       
        /* Track zoom buttons */
        tav_expand_button.set_name ("TrackHeightButton");
-       tav_expand_button.set_size_request(-1,20);
-       tav_expand_button.add (*(manage (new Image (::get_icon("tav_exp")))));
-       tav_expand_button.signal_clicked().connect (sigc::bind (sigc::mem_fun(*this, &Editor::tav_zoom_step), true));
+       tav_expand_button.set_size_request (-1, 20);
+       tav_expand_button.add (*(manage (new Image (::get_icon ("tav_exp")))));
+       RefPtr<Action> act = ActionManager::get_action (X_("Editor"), X_("expand-tracks"));
+       act->connect_proxy (tav_expand_button);
 
        tav_shrink_button.set_name ("TrackHeightButton");
-       tav_shrink_button.set_size_request(-1,20);
-       tav_shrink_button.add (*(manage (new Image (::get_icon("tav_shrink")))));
-       tav_shrink_button.signal_clicked().connect (sigc::bind (sigc::mem_fun(*this, &Editor::tav_zoom_step), false));
+       tav_shrink_button.set_size_request (-1, 20);
+       tav_shrink_button.add (*(manage (new Image (::get_icon ("tav_shrink")))));
+       act = ActionManager::get_action (X_("Editor"), X_("shrink-tracks"));
+       act->connect_proxy (tav_shrink_button);
 
-       _zoom_box.pack_start (tav_expand_button);
        _zoom_box.pack_start (tav_shrink_button);
-
-       _zoom_box.pack_start (zoom_focus_selector);
+       _zoom_box.pack_start (tav_expand_button);
        
        _zoom_tearoff = manage (new TearOff (_zoom_box));
 
@@ -3041,13 +2913,13 @@ Editor::setup_toolbar ()
        }
 
        _tools_tearoff->Detach.connect (sigc::bind (sigc::mem_fun(*this, &Editor::detach_tearoff), static_cast<Box*>(&toolbar_hbox),
-                                                    &_tools_tearoff->tearoff_window()));
+                                                   &_tools_tearoff->tearoff_window()));
        _tools_tearoff->Attach.connect (sigc::bind (sigc::mem_fun(*this, &Editor::reattach_tearoff), static_cast<Box*> (&toolbar_hbox),
-                                                    &_tools_tearoff->tearoff_window(), 0));
+                                                   &_tools_tearoff->tearoff_window(), 0));
        _tools_tearoff->Hidden.connect (sigc::bind (sigc::mem_fun(*this, &Editor::detach_tearoff), static_cast<Box*>(&toolbar_hbox),
-                                                    &_tools_tearoff->tearoff_window()));
+                                                   &_tools_tearoff->tearoff_window()));
        _tools_tearoff->Visible.connect (sigc::bind (sigc::mem_fun(*this, &Editor::reattach_tearoff), static_cast<Box*> (&toolbar_hbox),
-                                                     &_tools_tearoff->tearoff_window(), 0));
+                                                    &_tools_tearoff->tearoff_window(), 0));
 
        toolbar_hbox.set_spacing (10);
        toolbar_hbox.set_border_width (1);
@@ -3057,7 +2929,11 @@ Editor::setup_toolbar ()
        toolbar_hbox.pack_start (*_tools_tearoff, false, false);
 
        hbox->pack_start (snap_box, false, false);
-       hbox->pack_start (*nudge_box, false, false);
+        if (!Profile->get_small_screen()) {
+                hbox->pack_start (*nudge_box, false, false);
+        } else {
+                ARDOUR_UI::instance()->editor_transport_box().pack_start (*nudge_box, false, false);
+        }
        hbox->pack_start (panic_box, false, false);
 
        hbox->show_all ();
@@ -3065,22 +2941,28 @@ Editor::setup_toolbar ()
        toolbar_base.set_name ("ToolBarBase");
        toolbar_base.add (toolbar_hbox);
 
+       _toolbar_viewport.add (toolbar_base);
+       /* stick to the required height but allow width to vary if there's not enough room */
+       _toolbar_viewport.set_size_request (1, -1);
+
        toolbar_frame.set_shadow_type (SHADOW_OUT);
        toolbar_frame.set_name ("BaseFrame");
-       toolbar_frame.add (toolbar_base);
+       toolbar_frame.add (_toolbar_viewport);
+        
+        DPIReset.connect (sigc::mem_fun (*this, &Editor::resize_text_widgets));
 }
 
 void
 Editor::setup_tooltips ()
 {
        ARDOUR_UI::instance()->set_tip (mouse_move_button, _("Select/Move Objects"));
-       ARDOUR_UI::instance()->set_tip (mouse_gain_button, _("Draw Gain Automation"));
+       ARDOUR_UI::instance()->set_tip (mouse_gain_button, _("Draw Region Gain"));
        ARDOUR_UI::instance()->set_tip (mouse_zoom_button, _("Select Zoom Range"));
        ARDOUR_UI::instance()->set_tip (mouse_timefx_button, _("Stretch/Shrink Regions and MIDI Notes"));
        ARDOUR_UI::instance()->set_tip (mouse_audition_button, _("Listen to Specific Regions"));
        ARDOUR_UI::instance()->set_tip (join_object_range_button, _("Select/Move Objects or Ranges"));
        ARDOUR_UI::instance()->set_tip (internal_edit_button, _("Edit Region Contents (e.g. notes)"));
-       ARDOUR_UI::instance()->set_tip (*_group_tabs, _("Groups: context-click for possible operations"));
+       ARDOUR_UI::instance()->set_tip (*_group_tabs, _("Groups: click to (de)activate\nContext-click for other operations"));
        ARDOUR_UI::instance()->set_tip (nudge_forward_button, _("Nudge Region/Selection Forwards"));
        ARDOUR_UI::instance()->set_tip (nudge_backward_button, _("Nudge Region/Selection Backwards"));
        ARDOUR_UI::instance()->set_tip (zoom_in_button, _("Zoom In"));
@@ -3094,6 +2976,7 @@ Editor::setup_tooltips ()
        ARDOUR_UI::instance()->set_tip (edit_point_selector, _("Edit point"));
        ARDOUR_UI::instance()->set_tip (midi_sound_notes, _("Sound Notes"));
        ARDOUR_UI::instance()->set_tip (midi_panic_button, _("Send note off and reset controller messages on all MIDI channels"));
+       ARDOUR_UI::instance()->set_tip (edit_mode_selector, _("Edit Mode"));
 }
 
 void
@@ -3128,7 +3011,7 @@ Editor::setup_midi_toolbar ()
 
 int
 Editor::convert_drop_to_paths (
-               vector<ustring>&                paths,
+               vector<string>&                paths,
                const RefPtr<Gdk::DragContext>& /*context*/,
                gint                            /*x*/,
                gint                            /*y*/,
@@ -3139,8 +3022,8 @@ Editor::convert_drop_to_paths (
        if (_session == 0) {
                return -1;
        }
-
-       vector<ustring> uris = data.get_uris();
+        
+       vector<string> uris = data.get_uris();
 
        if (uris.empty()) {
 
@@ -3158,7 +3041,7 @@ Editor::convert_drop_to_paths (
                   THERE MAY BE NO NULL TERMINATING CHAR!!!
                */
 
-               ustring txt = data.get_text();
+               string txt = data.get_text();
                const char* p;
                const char* q;
 
@@ -3186,7 +3069,7 @@ Editor::convert_drop_to_paths (
 
                                        if (q > p)
                                        {
-                                               uris.push_back (ustring (p, q - p + 1));
+                                               uris.push_back (string (p, q - p + 1));
                                        }
                                }
                        }
@@ -3202,18 +3085,18 @@ Editor::convert_drop_to_paths (
                }
        }
 
-       for (vector<ustring>::iterator i = uris.begin(); i != uris.end(); ++i) {
+       for (vector<string>::iterator i = uris.begin(); i != uris.end(); ++i) {
 
                if ((*i).substr (0,7) == "file://") {
 
-                       ustring p = *i;
+                       string p = *i;
                        PBD::url_decode (p);
 
                        // scan forward past three slashes
 
-                       ustring::size_type slashcnt = 0;
-                       ustring::size_type n = 0;
-                       ustring::iterator x = p.begin();
+                       string::size_type slashcnt = 0;
+                       string::size_type n = 0;
+                       string::iterator x = p.begin();
 
                        while (slashcnt < 3 && x != p.end()) {
                                if ((*x) == '/') {
@@ -3276,34 +3159,18 @@ Editor::begin_reversible_command (string name)
 }
 
 void
-Editor::commit_reversible_command ()
+Editor::begin_reversible_command (GQuark q)
 {
        if (_session) {
-               _session->commit_reversible_command ();
-       }
-}
-
-void
-Editor::set_route_group_solo (Route& route, bool yn)
-{
-       RouteGroup *route_group;
-
-       if ((route_group = route.route_group()) != 0) {
-               route_group->apply (&Route::set_solo, yn, this);
-       } else {
-               route.set_solo (yn, this);
+               _session->begin_reversible_command (q);
        }
 }
 
 void
-Editor::set_route_group_mute (Route& route, bool yn)
+Editor::commit_reversible_command ()
 {
-       RouteGroup *route_group = 0;
-
-       if ((route_group = route.route_group()) != 0) {
-               route_group->apply (&Route::set_mute, yn, this);
-       } else {
-               route.set_mute (yn, this);
+       if (_session) {
+               _session->commit_reversible_command ();
        }
 }
 
@@ -3342,14 +3209,10 @@ Editor::duplicate_dialog (bool with_dialog)
                }
        }
 
-       RegionSelection rs;
-       get_regions_for_action (rs);
-
-       if (mouse_mode != MouseRange) {
+       RegionSelection rs = get_regions_from_selection_and_entered ();
 
-               if (rs.empty()) {
-                       return;
-               }
+       if (mouse_mode != MouseRange && rs.empty()) {
+               return;
        }
 
        if (with_dialog) {
@@ -3440,7 +3303,7 @@ Editor::clamp_verbose_cursor_y (double y)
 }
 
 void
-Editor::show_verbose_canvas_cursor_with (const string & txt)
+Editor::show_verbose_canvas_cursor_with (const string & txt, int32_t xoffset, int32_t yoffset)
 {
        verbose_canvas_cursor->property_text() = txt.c_str();
 
@@ -3450,6 +3313,9 @@ Editor::show_verbose_canvas_cursor_with (const string & txt)
        track_canvas->get_pointer (x, y);
        track_canvas->window_to_world (x, y, wx, wy);
 
+       wx += xoffset;
+       wy += yoffset;
+
        /* don't get too close to the edge */
        verbose_canvas_cursor->property_x() = clamp_verbose_cursor_x (wx);
        verbose_canvas_cursor->property_y() = clamp_verbose_cursor_y (wy);
@@ -3501,22 +3367,11 @@ Editor::cycle_edit_mode ()
 void
 Editor::edit_mode_selection_done ()
 {
-       if (_session == 0) {
-               return;
-       }
+        string s = edit_mode_selector.get_active_text ();
 
-       string choice = edit_mode_selector.get_active_text();
-       EditMode mode = Slide;
-
-       if (choice == _("Splice Edit")) {
-               mode = Splice;
-       } else if (choice == _("Slide Edit")) {
-               mode = Slide;
-       } else if (choice == _("Lock Edit")) {
-               mode = Lock;
-       }
-
-       Config->set_edit_mode (mode);
+        if (!s.empty()) {
+                Config->set_edit_mode (string_to_edit_mode (s));
+        }
 }
 
 void
@@ -3547,6 +3402,8 @@ Editor::snap_type_selection_done ()
                snaptype = SnapToBeatDiv14;
        } else if (choice == _("Beats/16")) {
                snaptype = SnapToBeatDiv16;
+       } else if (choice == _("Beats/20")) {
+               snaptype = SnapToBeatDiv20;
        } else if (choice == _("Beats/24")) {
                snaptype = SnapToBeatDiv24;
        } else if (choice == _("Beats/28")) {
@@ -3764,6 +3621,14 @@ Editor::pane_allocation_handler (Allocation &alloc, Paned* which)
                        return;
                }
 
+               if (geometry && (prop = geometry->property ("notebook-shrunk"))) {
+                       _notebook_shrunk = string_is_affirmative (prop->value ());
+               }
+
+               if (geometry && (prop = geometry->property ("pre-maximal-horizontal-pane-position"))) {
+                       pre_maximal_horizontal_pane_position = atoi (prop->value ());
+               }
+
                if (!geometry || (prop = geometry->property ("edit-horizontal-pane-pos")) == 0) {
                        /* initial allocation is 90% to canvas, 10% to notebook */
                        pos = (int) floor (alloc.get_width() * 0.90f);
@@ -3774,7 +3639,9 @@ Editor::pane_allocation_handler (Allocation &alloc, Paned* which)
 
                if (GTK_WIDGET(edit_pane.gobj())->allocation.width > pos) {
                        edit_pane.set_position (pos);
-                       pre_maximal_horizontal_pane_position = pos;
+                       if (pre_maximal_horizontal_pane_position == 0) {
+                               pre_maximal_horizontal_pane_position = pos;
+                       }
                }
 
                done = (Pane) (done | Horizontal);
@@ -3843,11 +3710,14 @@ Editor::toggle_follow_playhead ()
        }
 }
 
+/** @param yn true to follow playhead, otherwise false.
+ *  @param catch_up true to reset the editor view to show the playhead (if yn == true), otherwise false.
+ */
 void
-Editor::set_follow_playhead (bool yn)
+Editor::set_follow_playhead (bool yn, bool catch_up)
 {
        if (_follow_playhead != yn) {
-               if ((_follow_playhead = yn) == true) {
+               if ((_follow_playhead = yn) == true && catch_up) {
                        /* catch up */
                        reset_x_origin_to_follow_playhead ();
                }
@@ -3858,24 +3728,24 @@ Editor::set_follow_playhead (bool yn)
 void
 Editor::toggle_stationary_playhead ()
 {
-       RefPtr<Action> act = ActionManager::get_action (X_("Editor"), X_("toggle-stationary-playhead"));
-       if (act) {
-               RefPtr<ToggleAction> tact = RefPtr<ToggleAction>::cast_dynamic(act);
-               set_stationary_playhead (tact->get_active());
-       }
+       RefPtr<Action> act = ActionManager::get_action (X_("Editor"), X_("toggle-stationary-playhead"));
+       if (act) {
+               RefPtr<ToggleAction> tact = RefPtr<ToggleAction>::cast_dynamic(act);
+               set_stationary_playhead (tact->get_active());
+       }
 }
 
 void
 Editor::set_stationary_playhead (bool yn)
 {
-       if (_stationary_playhead != yn) {
-               if ((_stationary_playhead = yn) == true) {
-                       /* catch up */
-                       // FIXME need a 3.0 equivalent of this 2.X call
-                       // update_current_screen ();
-               }
-               instant_save ();
-       }
+       if (_stationary_playhead != yn) {
+               if ((_stationary_playhead = yn) == true) {
+                       /* catch up */
+                       // FIXME need a 3.0 equivalent of this 2.X call
+                       // update_current_screen ();
+               }
+               instant_save ();
+       }
 }
 
 void
@@ -3930,7 +3800,7 @@ Editor::playlist_selector () const
 }
 
 Evoral::MusicalTime
-Editor::get_grid_type_as_beats (bool& success, nframes64_t position)
+Editor::get_grid_type_as_beats (bool& success, framepos_t position)
 {
        success = true;
 
@@ -3948,6 +3818,9 @@ Editor::get_grid_type_as_beats (bool& success, nframes64_t position)
        case SnapToBeatDiv24:
                return 1.0/24.0;
                break;
+       case SnapToBeatDiv20:
+               return 1.0/20.0;
+               break;
        case SnapToBeatDiv16:
                return 1.0/16.0;
                break;
@@ -4006,10 +3879,10 @@ Editor::get_grid_type_as_beats (bool& success, nframes64_t position)
        return 0.0;
 }
 
-nframes64_t
-Editor::get_nudge_distance (nframes64_t pos, nframes64_t& next)
+framecnt_t
+Editor::get_nudge_distance (framepos_t pos, framecnt_t& next)
 {
-       nframes64_t ret;
+       framecnt_t ret;
 
        ret = nudge_clock.current_duration (pos);
        next = ret + 1; /* XXXX fix me */
@@ -4054,7 +3927,7 @@ Editor::playlist_deletion_dialog (boost::shared_ptr<Playlist> pl)
 }
 
 bool
-Editor::audio_region_selection_covers (nframes64_t where)
+Editor::audio_region_selection_covers (framepos_t where)
 {
        for (RegionSelection::iterator a = selection->regions.begin(); a != selection->regions.end(); ++a) {
                if ((*a)->region()->covers (where)) {
@@ -4129,10 +4002,8 @@ Editor::control_layout_scroll (GdkEventScroll* ev)
 }
 
 void
-Editor::session_state_saved (string snap_name)
+Editor::session_state_saved (string)
 {
-       ENSURE_GUI_THREAD (*this, &Editor::session_state_saved, snap_name);
-       
        update_title ();        
        _snapshots->redisplay ();
 }
@@ -4176,8 +4047,11 @@ Editor::maximise_editing_space ()
        if (Config->get_keep_tearoffs()) {
                _mouse_mode_tearoff->set_visible (true);
                _tools_tearoff->set_visible (true);
-               _zoom_tearoff->set_visible (true);
+               if (Config->get_show_zoom_tools ()) {
+                       _zoom_tearoff->set_visible (true);
+               }
        }
+
 }
 
 void
@@ -4197,7 +4071,9 @@ Editor::restore_editing_space ()
 
        _mouse_mode_tearoff->set_visible (true);
        _tools_tearoff->set_visible (true);
-       _zoom_tearoff->set_visible (true);
+       if (Config->get_show_zoom_tools ()) {
+               _zoom_tearoff->set_visible (true);
+       }
        post_maximal_editor_width = this->get_width();
        post_maximal_editor_height = this->get_height();
 
@@ -4287,7 +4163,7 @@ Editor::on_key_release_event (GdkEventKey* ev)
  *  @param frame New x origin.
  */
 void
-Editor::reset_x_origin (nframes64_t frame)
+Editor::reset_x_origin (framepos_t frame)
 {
        queue_visual_change (frame);
 }
@@ -4305,7 +4181,7 @@ Editor::reset_zoom (double fpu)
 }
 
 void
-Editor::reposition_and_zoom (nframes64_t frame, double fpu)
+Editor::reposition_and_zoom (framepos_t frame, double fpu)
 {
        reset_x_origin (frame);
        reset_zoom (fpu);
@@ -4423,7 +4299,7 @@ Editor::set_frames_per_unit (double fpu)
           of frames into an 800 pixel wide space.
        */
 
-       if (max_frames / fpu < 800.0) {
+       if (max_framepos / fpu < 800.0) {
                return;
        }
 
@@ -4439,7 +4315,7 @@ Editor::post_zoom ()
 {
        // convert fpu to frame count
 
-       nframes64_t frames = (nframes64_t) floor (frames_per_unit * _canvas_width);
+       framepos_t frames = (framepos_t) floor (frames_per_unit * _canvas_width);
 
        if (frames_per_unit != zoom_range_clock.current_duration()) {
                zoom_range_clock.set (frames);
@@ -4462,11 +4338,13 @@ Editor::post_zoom ()
        refresh_location_display();
        _summary->set_overlays_dirty ();
 
+       update_marker_labels ();
+
        instant_save ();
 }
 
 void
-Editor::queue_visual_change (nframes64_t where)
+Editor::queue_visual_change (framepos_t where)
 {
        pending_visual_change.add (VisualChange::TimeOrigin);
        pending_visual_change.time_origin = where;
@@ -4511,7 +4389,19 @@ Editor::idle_visual_changer ()
        VisualChange::Type p = pending_visual_change.pending;
        pending_visual_change.pending = (VisualChange::Type) 0;
 
-       double last_time_origin = horizontal_position ();
+       double const last_time_origin = horizontal_position ();
+
+       if (p & VisualChange::TimeOrigin) {
+               /* This is a bit of a hack, but set_frames_per_unit
+                  below will (if called) end up with the
+                  CrossfadeViews looking at Editor::leftmost_frame,
+                  and if we're changing origin and zoom in the same
+                  operation it will be the wrong value unless we
+                  update it here.
+               */
+
+               leftmost_frame = pending_visual_change.time_origin;
+       }
 
        if (p & VisualChange::ZoomLevel) {
                set_frames_per_unit (pending_visual_change.frames_per_unit);
@@ -4558,14 +4448,15 @@ Editor::sort_track_selection (TrackViewList* sel)
        }
 }
 
-nframes64_t
+framepos_t
 Editor::get_preferred_edit_position (bool ignore_playhead)
 {
        bool ignored;
-       nframes64_t where = 0;
+       framepos_t where = 0;
        EditPoint ep = _edit_point;
 
        if (entered_marker) {
+                DEBUG_TRACE (DEBUG::CutNPaste, string_compose ("GPEP: use entered marker @ %1\n", entered_marker->position()));
                return entered_marker->position();
        }
 
@@ -4576,6 +4467,7 @@ Editor::get_preferred_edit_position (bool ignore_playhead)
        switch (ep) {
        case EditAtPlayhead:
                where = _session->audible_frame();
+                DEBUG_TRACE (DEBUG::CutNPaste, string_compose ("GPEP: use playhead @ %1\n", where));
                break;
 
        case EditAtSelectedMarker:
@@ -4588,6 +4480,7 @@ Editor::get_preferred_edit_position (bool ignore_playhead)
                                } else {
                                        where = loc->end();
                                }
+                                DEBUG_TRACE (DEBUG::CutNPaste, string_compose ("GPEP: use selected marker @ %1\n", where));
                                break;
                        }
                }
@@ -4600,6 +4493,7 @@ Editor::get_preferred_edit_position (bool ignore_playhead)
                        return 0;
                }
                snap_to (where);
+                DEBUG_TRACE (DEBUG::CutNPaste, string_compose ("GPEP: use mouse @ %1\n", where));
                break;
        }
 
@@ -4607,7 +4501,7 @@ Editor::get_preferred_edit_position (bool ignore_playhead)
 }
 
 void
-Editor::set_loop_range (nframes64_t start, nframes64_t end, string cmd)
+Editor::set_loop_range (framepos_t start, framepos_t end, string cmd)
 {
        if (!_session) return;
 
@@ -4616,7 +4510,7 @@ Editor::set_loop_range (nframes64_t start, nframes64_t end, string cmd)
        Location* tll;
 
        if ((tll = transport_loop_location()) == 0) {
-               Location* loc = new Location (start, end, _("Loop"),  Location::IsAutoLoop);
+               Location* loc = new Location (*_session, start, end, _("Loop"),  Location::IsAutoLoop);
                XMLNode &before = _session->locations()->get_state();
                _session->locations()->add (loc, true);
                _session->set_auto_loop_location (loc);
@@ -4634,7 +4528,7 @@ Editor::set_loop_range (nframes64_t start, nframes64_t end, string cmd)
 }
 
 void
-Editor::set_punch_range (nframes64_t start, nframes64_t end, string cmd)
+Editor::set_punch_range (framepos_t start, framepos_t end, string cmd)
 {
        if (!_session) return;
 
@@ -4643,7 +4537,7 @@ Editor::set_punch_range (nframes64_t start, nframes64_t end, string cmd)
        Location* tpl;
 
        if ((tpl = transport_punch_location()) == 0) {
-               Location* loc = new Location (start, end, _("Loop"),  Location::IsAutoPunch);
+               Location* loc = new Location (*_session, start, end, _("Loop"),  Location::IsAutoPunch);
                XMLNode &before = _session->locations()->get_state();
                _session->locations()->add (loc, true);
                _session->set_auto_loop_location (loc);
@@ -4667,7 +4561,7 @@ Editor::set_punch_range (nframes64_t start, nframes64_t end, string cmd)
  *  @param ts Tracks to look on; if this is empty, all tracks are examined.
  */
 void
-Editor::get_regions_at (RegionSelection& rs, nframes64_t where, const TrackViewList& ts) const
+Editor::get_regions_at (RegionSelection& rs, framepos_t where, const TrackViewList& ts) const
 {
        const TrackViewList* tracks;
 
@@ -4678,7 +4572,9 @@ Editor::get_regions_at (RegionSelection& rs, nframes64_t where, const TrackViewL
        }
 
        for (TrackViewList::const_iterator t = tracks->begin(); t != tracks->end(); ++t) {
+
                RouteTimeAxisView* rtv = dynamic_cast<RouteTimeAxisView*>(*t);
+
                if (rtv) {
                        boost::shared_ptr<Track> tr;
                        boost::shared_ptr<Playlist> pl;
@@ -4686,7 +4582,7 @@ Editor::get_regions_at (RegionSelection& rs, nframes64_t where, const TrackViewL
                        if ((tr = rtv->track()) && ((pl = tr->playlist()))) {
 
                                Playlist::RegionList* regions = pl->regions_at (
-                                               (nframes64_t) floor ( (double)where * tr->speed()));
+                                               (framepos_t) floor ( (double) where * tr->speed()));
 
                                for (Playlist::RegionList::iterator i = regions->begin(); i != regions->end(); ++i) {
                                        RegionView* rv = rtv->view()->find_view (*i);
@@ -4702,7 +4598,7 @@ Editor::get_regions_at (RegionSelection& rs, nframes64_t where, const TrackViewL
 }
 
 void
-Editor::get_regions_after (RegionSelection& rs, nframes64_t where, const TrackViewList& ts) const
+Editor::get_regions_after (RegionSelection& rs, framepos_t where, const TrackViewList& ts) const
 {
        const TrackViewList* tracks;
 
@@ -4721,7 +4617,7 @@ Editor::get_regions_after (RegionSelection& rs, nframes64_t where, const TrackVi
                        if ((tr = rtv->track()) && ((pl = tr->playlist()))) {
 
                                Playlist::RegionList* regions = pl->regions_touched (
-                                               (nframes64_t) floor ( (double)where * tr->speed()), max_frames);
+                                       (framepos_t) floor ( (double)where * tr->speed()), max_framepos);
 
                                for (Playlist::RegionList::iterator i = regions->begin(); i != regions->end(); ++i) {
 
@@ -4738,47 +4634,96 @@ Editor::get_regions_after (RegionSelection& rs, nframes64_t where, const TrackVi
        }
 }
 
-/** Find all regions which are either:
- *      - selected or
- *      - the entered_regionview (if allow_entered == true) or
- *      - under the preferred edit position AND on a selected track, or on a track
- *        which is in the same active edit-enable route group as a selected region (if allow_edit_position == true)
- *  @param rs Returned region list.
- *  @param allow_entered true to include the entered_regionview in the list.
+/** Start with regions that are selected.  Then add equivalent regions
+ *  on tracks in the same active edit-enabled route group as any of
+ *  the regions that we started with.
  */
-void
-Editor::get_regions_for_action (RegionSelection& rs, bool allow_entered, bool allow_edit_position)
-{
-       /* Start with selected regions */
-       rs = selection->regions;
 
-       /* Add the entered_regionview, if requested */
-       if (allow_entered && entered_regionview) {
-               rs.add (entered_regionview);
-       }
+RegionSelection
+Editor::get_regions_from_selection ()
+{
+       return get_equivalent_regions (selection->regions, ARDOUR::Properties::edit.property_id);
+}
+
+/** Get regions using the following method:
+ *
+ *  Make an initial region list using the selected regions, unless
+ *  the edit point is `mouse' and the mouse is over an unselected
+ *  region.  In this case, start with just that region.
+ *
+ *  Then, make an initial track list of the tracks that these
+ *  regions are on, and if the edit point is not `mouse', add the
+ *  selected tracks.
+ *
+ *  Look at this track list and add any other tracks that are on the
+ *  same active edit-enabled route group as one of the initial tracks.
+ *
+ *  Finally take the initial region list and add any regions that are
+ *  under the edit point on one of the tracks on the track list to get
+ *  the returned region list.
+ *
+ *  The rationale here is that the mouse edit point is special in that
+ *  its position describes both a time and a track; the other edit
+ *  modes only describe a time.  Hence if the edit point is `mouse' we
+ *  ignore selected tracks, as we assume the user means something by
+ *  pointing at a particular track.  Also in this case we take note of
+ *  the region directly under the edit point, as there is always just one
+ *  (rather than possibly several with non-mouse edit points).
+ */
 
-       if (allow_edit_position) {
+RegionSelection
+Editor::get_regions_from_selection_and_edit_point ()
+{
+       RegionSelection regions;
+       
+       if (_edit_point == EditAtMouse && entered_regionview && !selection->regions.contains (entered_regionview)) {
+               regions.add (entered_regionview);
+       } else {
+               regions = selection->regions;
+       }
 
-               TrackViewList tracks = selection->tracks;
+       TrackViewList tracks;
 
-               /* tracks is currently the set of selected tracks; add any other tracks that
-                * have regions that are in the same edit-activated route group as one of
-                * our regions */
-               for (RegionSelection::iterator i = rs.begin (); i != rs.end(); ++i) {
+       if (_edit_point != EditAtMouse) {
+               tracks = selection->tracks;
+       }
 
-                       RouteGroup* g = (*i)->get_time_axis_view().route_group ();
-                       if (g && g->is_active() && g->is_edit()) {
-                               tracks.add (axis_views_from_routes (g->route_list()));
-                       }
-                       
+       /* Add any other tracks that have regions that are in the same
+          edit-activated route group as one of our regions.
+        */
+       for (RegionSelection::iterator i = regions.begin (); i != regions.end(); ++i) {
+               
+               RouteGroup* g = (*i)->get_time_axis_view().route_group ();
+               
+               if (g && g->is_active() && g->is_edit()) {
+                       tracks.add (axis_views_from_routes (g->route_list()));
                }
+       }
+       
+       if (!tracks.empty()) {
+               /* now find regions that are at the edit position on those tracks */
+               framepos_t const where = get_preferred_edit_position ();
+               get_regions_at (regions, where, tracks);
+       }
 
-               if (!tracks.empty()) {
-                       /* now find regions that are at the edit position on those tracks */
-                       nframes64_t const where = get_preferred_edit_position ();
-                       get_regions_at (rs, where, tracks);
-               }
+       return regions;
+}
+
+/** Start with regions that are selected, or the entered regionview if none are selected.
+ *  Then add equivalent regions on tracks in the same active edit-enabled route group as any
+ *  of the regions that we started with.
+ */
+
+RegionSelection
+Editor::get_regions_from_selection_and_entered ()
+{
+       RegionSelection regions = selection->regions;
+       
+       if (regions.empty() && entered_regionview) {
+               regions.add (entered_regionview);
        }
+
+       return get_equivalent_regions (regions, ARDOUR::Properties::edit.property_id);
 }
 
 void
@@ -4826,16 +4771,6 @@ Editor::show_rhythm_ferret ()
        rhythm_ferret->present ();
 }
 
-void
-Editor::show_global_port_matrix (ARDOUR::DataType t)
-{
-       if (_global_port_matrix[t] == 0) {
-               _global_port_matrix[t] = new GlobalPortMatrixWindow (_session, t);
-       }
-
-       _global_port_matrix[t]->show ();
-}
-
 void
 Editor::first_idle ()
 {
@@ -4919,7 +4854,7 @@ Editor::idle_resize ()
                }
        }
 
-        _pending_resize_amount = 0;
+       _pending_resize_amount = 0;
        flush_canvas ();
        _group_tabs->set_dirty ();
        resize_idle_id = -1;
@@ -4947,6 +4882,12 @@ Editor::region_view_added (RegionView *)
        _summary->set_dirty ();
 }
 
+void
+Editor::region_view_removed ()
+{
+       _summary->set_dirty ();
+}
+
 TimeAxisView*
 Editor::axis_view_from_route (boost::shared_ptr<Route> r) const
 {
@@ -5009,18 +4950,24 @@ Editor::handle_new_route (RouteList& routes)
 
                rtv->effective_gain_display ();
 
+                if (internal_editing()) {
+                        rtv->enter_internal_edit_mode ();
+                } else {
+                        rtv->leave_internal_edit_mode ();
+                }
+
                rtv->view()->RegionViewAdded.connect (sigc::mem_fun (*this, &Editor::region_view_added));
+               rtv->view()->RegionViewRemoved.connect (sigc::mem_fun (*this, &Editor::region_view_removed));
        }
 
        _routes->routes_added (new_views);
+       _summary->routes_added (new_views);
 
        if (show_editor_mixer_when_tracks_arrive) {
                show_editor_mixer (true);
        }
 
        editor_list_button.set_sensitive (true);
-
-       _summary->set_dirty ();
 }
 
 void
@@ -5032,14 +4979,22 @@ Editor::timeaxisview_deleted (TimeAxisView *tv)
        }
 
        ENSURE_GUI_THREAD (*this, &Editor::timeaxisview_deleted, tv);
-               
 
+       RouteTimeAxisView* rtav = dynamic_cast<RouteTimeAxisView*> (tv);
+       
        _routes->route_removed (tv);
 
        if (tv == entered_track) {
                entered_track = 0;
        }
-       
+
+       TimeAxisView::Children c = tv->get_child_list ();
+       for (TimeAxisView::Children::const_iterator i = c.begin(); i != c.end(); ++i) {
+               if (entered_track == i->get()) {
+                       entered_track = 0;
+               }
+       }
+
        /* remove it from the list of track views */
 
        TrackViewList::iterator i;
@@ -5051,7 +5006,6 @@ Editor::timeaxisview_deleted (TimeAxisView *tv)
        /* update whatever the current mixer strip is displaying, if revelant */
 
        boost::shared_ptr<Route> route;
-       RouteTimeAxisView* rtav = dynamic_cast<RouteTimeAxisView*> (tv);
 
        if (rtav) {
                route = rtav->route ();
@@ -5083,16 +5037,28 @@ Editor::timeaxisview_deleted (TimeAxisView *tv)
 }
 
 void
-Editor::hide_track_in_display (TimeAxisView* tv, bool /*temponly*/)
+Editor::hide_track_in_display (TimeAxisView* tv, bool apply_to_selection)
 {
-       RouteTimeAxisView* rtv = dynamic_cast<RouteTimeAxisView*> (tv);
+       if (apply_to_selection) {
+               for (TrackSelection::iterator i = selection->tracks.begin(); i != selection->tracks.end(); ) {
 
-       if (rtv && current_mixer_strip && (rtv->route() == current_mixer_strip->route())) {
-               // this will hide the mixer strip
-               set_selected_mixer_strip (*tv);
+                       TrackSelection::iterator j = i;
+                       ++j;
+                       
+                       hide_track_in_display (*i, false);
+                       
+                       i = j;
+               }
+       } else {
+               RouteTimeAxisView* rtv = dynamic_cast<RouteTimeAxisView*> (tv);
+               
+               if (rtv && current_mixer_strip && (rtv->route() == current_mixer_strip->route())) {
+                       // this will hide the mixer strip
+                       set_selected_mixer_strip (*tv);
+               }
+               
+               _routes->hide_track_in_display (*tv);
        }
-
-       _routes->hide_track_in_display (*tv);
 }
 
 bool
@@ -5114,12 +5080,13 @@ Editor::foreach_time_axis_view (sigc::slot<void,TimeAxisView&> theslot)
        }
 }
 
+/** Find a RouteTimeAxisView by the ID of its route */
 RouteTimeAxisView*
-Editor::get_route_view_by_id (PBD::ID& id)
+Editor::get_route_view_by_route_id (PBD::ID& id) const
 {
        RouteTimeAxisView* v;
 
-       for(TrackViewList::iterator i = track_views.begin(); i != track_views.end(); ++i) {
+       for (TrackViewList::const_iterator i = track_views.begin(); i != track_views.end(); ++i) {
                if((v = dynamic_cast<RouteTimeAxisView*>(*i)) != 0) {
                        if(v->route()->id() == id) {
                                return v;
@@ -5192,11 +5159,11 @@ Editor::show_region_in_region_list ()
 void
 Editor::step_edit_status_change (bool yn)
 {
-        if (yn) {
-                start_step_editing ();
-        } else {
-                stop_step_editing ();
-        }
+       if (yn) {
+               start_step_editing ();
+       } else {
+               stop_step_editing ();
+       }
 }
 
 void
@@ -5225,7 +5192,7 @@ Editor::check_step_edit ()
 }
 
 bool
-Editor::horizontal_scroll_left_press ()
+Editor::scroll_press (Direction dir)
 {
        ++_scroll_callbacks;
        
@@ -5233,44 +5200,32 @@ Editor::horizontal_scroll_left_press ()
                /* delay the first auto-repeat */
                return true;
        }
-               
-       double x = leftmost_position() - current_page_frames() / 5;
-       if (x < 0) {
-               x = 0;
-       }
-       
-       reset_x_origin (x);
 
-       /* do hacky auto-repeat */
-       if (!_scroll_connection.connected ()) {
-               _scroll_connection = Glib::signal_timeout().connect (sigc::mem_fun (*this, &Editor::horizontal_scroll_left_press), 100);
-               _scroll_callbacks = 0;
-       }
+       switch (dir) {
+       case LEFT:
+               scroll_backward (1);
+               break;
 
-       return true;
-}
+       case RIGHT:
+               scroll_forward (1);
+               break;
 
-void
-Editor::horizontal_scroll_left_release ()
-{
-       _scroll_connection.disconnect ();
-}
+       case UP:
+               scroll_tracks_up_line ();
+               break;
 
-bool
-Editor::horizontal_scroll_right_press ()
-{
-       ++_scroll_callbacks;
-       
-       if (_scroll_connection.connected() && _scroll_callbacks < 5) {
-               /* delay the first auto-repeat */
-               return true;
+       case DOWN:
+               scroll_tracks_down_line ();
+               break;
        }
 
-       reset_x_origin (leftmost_position() + current_page_frames() / 5);
-
        /* do hacky auto-repeat */
        if (!_scroll_connection.connected ()) {
-               _scroll_connection = Glib::signal_timeout().connect (sigc::mem_fun (*this, &Editor::horizontal_scroll_right_press), 100);
+
+               _scroll_connection = Glib::signal_timeout().connect (
+                       sigc::bind (sigc::mem_fun (*this, &Editor::scroll_press), dir), 100
+                       );
+               
                _scroll_callbacks = 0;
        }
 
@@ -5278,7 +5233,7 @@ Editor::horizontal_scroll_right_press ()
 }
 
 void
-Editor::horizontal_scroll_right_release ()
+Editor::scroll_release ()
 {
        _scroll_connection.disconnect ();
 }
@@ -5287,7 +5242,7 @@ Editor::horizontal_scroll_right_release ()
 void
 Editor::reset_x_origin_to_follow_playhead ()
 {
-       nframes64_t const frame = playhead_cursor->current_frame;
+       framepos_t const frame = playhead_cursor->current_frame;
 
        if (frame < leftmost_frame || frame > leftmost_frame + current_page_frames()) {
 
@@ -5303,7 +5258,7 @@ Editor::reset_x_origin_to_follow_playhead ()
                                                
                        if (frame < leftmost_frame) {
                                /* moving left */
-                               nframes64_t l = 0;
+                               framepos_t l = 0;
                                if (_session->transport_rolling()) {
                                        /* rolling; end up with the playhead at the right of the page */
                                        l = frame - current_page_frames ();
@@ -5357,7 +5312,7 @@ Editor::super_rapid_screen_update ()
 
        /* PLAYHEAD AND VIEWPORT */
 
-       nframes64_t const frame = _session->audible_frame();
+       framepos_t const frame = _session->audible_frame();
 
        /* There are a few reasons why we might not update the playhead / viewport stuff:
         *
@@ -5377,33 +5332,33 @@ Editor::super_rapid_screen_update ()
                        playhead_cursor->set_position (frame);
                }
 
-                if (!_stationary_playhead) {
+               if (!_stationary_playhead) {
 
-                        if (!_dragging_playhead && _follow_playhead && _session->requested_return_frame() < 0) {
-                                reset_x_origin_to_follow_playhead ();
-                        }
+                       if (!_dragging_playhead && _follow_playhead && _session->requested_return_frame() < 0) {
+                               reset_x_origin_to_follow_playhead ();
+                       }
 
-                } else {
+               } else {
                        
-                        /* don't do continuous scroll till the new position is in the rightmost quarter of the
-                           editor canvas
-                        */
+                       /* don't do continuous scroll till the new position is in the rightmost quarter of the
+                          editor canvas
+                       */
 #if 0                        
-                        // FIXME DO SOMETHING THAT WORKS HERE - this is 2.X code                         
-                        double target = ((double)frame - (double)current_page_frames()/2.0) / frames_per_unit;
-                        if (target <= 0.0) {
-                                target = 0.0;
-                        }
-                        if (fabs(target - current) < current_page_frames() / frames_per_unit) {
-                                target = (target * 0.15) + (current * 0.85);
-                        } else {
-                                /* relax */
-                        }
+                       // FIXME DO SOMETHING THAT WORKS HERE - this is 2.X code                         
+                       double target = ((double)frame - (double)current_page_frames()/2.0) / frames_per_unit;
+                       if (target <= 0.0) {
+                               target = 0.0;
+                       }
+                       if (fabs(target - current) < current_page_frames() / frames_per_unit) {
+                               target = (target * 0.15) + (current * 0.85);
+                       } else {
+                               /* relax */
+                       }
                         
-                        current = target;
-                        set_horizontal_position (current);
+                       current = target;
+                       set_horizontal_position (current);
 #endif
-                }
+               }
                
        }
 }
@@ -5486,9 +5441,122 @@ void
 Editor::show_editor_list (bool yn)
 {
        if (yn) {
-               the_notebook.show();
+               _the_notebook.show ();
        } else {
-               the_notebook.hide();
+               _the_notebook.hide ();
+       }
+}
+
+void
+Editor::change_region_layering_order ()
+{
+       framepos_t const position = get_preferred_edit_position ();
+       
+       if (!clicked_routeview) {
+               if (layering_order_editor) {
+                       layering_order_editor->hide ();
+               }
+               return;
+       }
+
+       boost::shared_ptr<Track> track = boost::dynamic_pointer_cast<Track> (clicked_routeview->route());
+
+       if (!track) {
+               return;
+       }
+
+       boost::shared_ptr<Playlist> pl = track->playlist();
+
+       if (!pl) {
+               return;
        }
+                
+       if (layering_order_editor == 0) {
+               layering_order_editor = new RegionLayeringOrderEditor(*this);
+       }
+
+       layering_order_editor->set_context (clicked_routeview->name(), _session, pl, position);
+       layering_order_editor->maybe_present ();
+}
+
+void
+Editor::update_region_layering_order_editor ()
+{
+       if (layering_order_editor && layering_order_editor->is_visible ()) {
+               change_region_layering_order ();
+       }
+}
+
+void
+Editor::setup_fade_images ()
+{
+       _fade_in_images[FadeLinear] = new Gtk::Image (get_icon_path (X_("crossfade-in-linear")));
+       _fade_in_images[FadeFast] = new Gtk::Image (get_icon_path (X_("crossfade-in-short-cut")));
+       _fade_in_images[FadeLogB] = new Gtk::Image (get_icon_path (X_("crossfade-in-slow-cut")));
+       _fade_in_images[FadeLogA] = new Gtk::Image (get_icon_path (X_("crossfade-in-fast-cut")));
+       _fade_in_images[FadeSlow] = new Gtk::Image (get_icon_path (X_("crossfade-in-long-cut")));
+
+       _fade_out_images[FadeLinear] = new Gtk::Image (get_icon_path (X_("crossfade-out-linear")));
+       _fade_out_images[FadeFast] = new Gtk::Image (get_icon_path (X_("crossfade-out-short-cut")));
+       _fade_out_images[FadeLogB] = new Gtk::Image (get_icon_path (X_("crossfade-out-slow-cut")));
+       _fade_out_images[FadeLogA] = new Gtk::Image (get_icon_path (X_("crossfade-out-fast-cut")));
+       _fade_out_images[FadeSlow] = new Gtk::Image (get_icon_path (X_("crossfade-out-long-cut")));
+}
+
+
+/** @return Gtk::manage()d menu item for a given action from `editor_actions' */
+Gtk::MenuItem&
+Editor::action_menu_item (std::string const & name)
+{
+       Glib::RefPtr<Action> a = editor_actions->get_action (name);
+       assert (a);
+       
+       return *manage (a->create_menu_item ());
+}
+
+void
+Editor::resize_text_widgets ()
+{
+        set_size_request_to_display_given_text (edit_mode_selector, edit_mode_strings, COMBO_FUDGE+10, 15);
+        set_size_request_to_display_given_text (zoom_focus_selector, zoom_focus_strings, COMBO_FUDGE+10, 15);
+        set_size_request_to_display_given_text (snap_type_selector, snap_type_strings, COMBO_FUDGE+10, 15);
+        set_size_request_to_display_given_text (snap_mode_selector, snap_mode_strings, COMBO_FUDGE+10, 15);
+        set_size_request_to_display_given_text (edit_point_selector, edit_point_strings, COMBO_FUDGE+10, 15);
+}
+        
+void
+Editor::add_notebook_page (string const & name, Gtk::Widget& widget)
+{
+       EventBox* b = manage (new EventBox);
+       b->signal_button_press_event().connect (sigc::bind (sigc::mem_fun (*this, &Editor::notebook_tab_clicked), &widget));
+       Label* l = manage (new Label (name));
+       l->set_angle (-90);
+       b->add (*l);
+       b->show_all ();
+       _the_notebook.append_page (widget, *b);
+}
+
+bool
+Editor::notebook_tab_clicked (GdkEventButton* ev, Gtk::Widget* page)
+{
+       if (ev->type == GDK_BUTTON_PRESS || ev->type == GDK_2BUTTON_PRESS) {
+               _the_notebook.set_current_page (_the_notebook.page_num (*page));
+       }
+
+       if (ev->type == GDK_2BUTTON_PRESS) {
+
+               /* double-click on a notebook tab shrinks or expands the notebook */
+
+               if (_notebook_shrunk) {
+                       edit_pane.set_position (pre_maximal_horizontal_pane_position);
+                       _notebook_shrunk = false;
+               } else {
+                       pre_maximal_horizontal_pane_position = edit_pane.get_position ();
+                       edit_pane.set_position (edit_pane.get_position() + page->get_width());
+                       _notebook_shrunk = true;
+               }
+       }
+
+       return true;
 }