introduce GUIObjectState; massive, pervasive changes in visibility and height managem...
[ardour.git] / gtk2_ardour / editor.cc
index cdbbb59105810b13021f236b79aaf5e33f3b83d8..814ea66b3dd78dffc8e5c47328dcdc08a8382105 100644 (file)
 #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"
 
 #include "control_protocol/control_protocol.h"
 
-#include "editor.h"
-#include "keyboard.h"
-#include "marker.h"
-#include "playlist_selector.h"
+#include "actions.h"
+#include "actions.h"
+#include "analysis_window.h"
+#include "audio_clock.h"
 #include "audio_region_view.h"
-#include "rgb_macros.h"
-#include "selection.h"
 #include "audio_streamview.h"
-#include "time_axis_view.h"
 #include "audio_time_axis.h"
-#include "utils.h"
-#include "crossfade_view.h"
+#include "automation_time_axis.h"
+#include "bundle_manager.h"
 #include "canvas-noevent-text.h"
-#include "editing.h"
-#include "public_editor.h"
-#include "crossfade_edit.h"
 #include "canvas_impl.h"
-#include "actions.h"
-#include "sfdb_ui.h"
-#include "gui_thread.h"
-#include "simpleline.h"
-#include "rhythm_ferret.h"
-#include "actions.h"
-#include "tempo_lines.h"
-#include "analysis_window.h"
-#include "bundle_manager.h"
-#include "global_port_matrix.h"
+#include "crossfade_edit.h"
+#include "crossfade_view.h"
+#include "debug.h"
+#include "editing.h"
+#include "editor.h"
+#include "editor_cursors.h"
 #include "editor_drag.h"
 #include "editor_group_tabs.h"
-#include "automation_time_axis.h"
-#include "editor_routes.h"
-#include "midi_time_axis.h"
-#include "mixer_strip.h"
-#include "editor_route_groups.h"
-#include "editor_regions.h"
 #include "editor_locations.h"
+#include "editor_regions.h"
+#include "editor_route_groups.h"
+#include "editor_routes.h"
 #include "editor_snapshots.h"
 #include "editor_summary.h"
-#include "region_layering_order_editor.h"
+#include "global_port_matrix.h"
+#include "gui_object.h"
+#include "gui_thread.h"
+#include "keyboard.h"
+#include "marker.h"
+#include "midi_time_axis.h"
+#include "mixer_strip.h"
 #include "mouse_cursors.h"
-#include "editor_cursors.h"
+#include "playlist_selector.h"
+#include "public_editor.h"
+#include "region_layering_order_editor.h"
+#include "rgb_macros.h"
+#include "rhythm_ferret.h"
+#include "selection.h"
+#include "sfdb_ui.h"
+#include "simpleline.h"
+#include "tempo_lines.h"
+#include "time_axis_view.h"
+#include "utils.h"
 
 #include "i18n.h"
 
@@ -258,7 +262,7 @@ Editor::Editor ()
 
          /* tool bar related */
 
-       , zoom_range_clock (X_("zoomrange"), false, X_("ZoomRangeClock"), true, false, true)
+       , zoom_range_clock (new AudioClock (X_("zoomrange"), false, X_("ZoomRangeClock"), true, false, true))
 
        , toolbar_selection_clock_table (2,3)
 
@@ -274,7 +278,7 @@ Editor::Editor ()
 
          /* nudge */
 
-       , nudge_clock (X_("nudge"), false, X_("NudgeClock"), true, false, true)
+       , nudge_clock (new AudioClock (X_("nudge"), false, X_("NudgeClock"), true, false, true))
        , meters_running(false)
        , _pending_locate_request (false)
        , _pending_initial_locate (false)
@@ -329,7 +333,6 @@ Editor::Editor ()
        current_interthread_info = 0;
        _show_measures = true;
        show_gain_after_trim = false;
-       verbose_cursor_on = true;
        last_item_entered = 0;
 
        have_pending_keyboard_selection = false;
@@ -363,7 +366,6 @@ Editor::Editor ()
        select_new_marker = false;
        rhythm_ferret = 0;
        layering_order_editor = 0;
-       _bundle_manager = 0;
        no_save_visual = false;
        resize_idle_id = -1;
 
@@ -387,7 +389,7 @@ Editor::Editor ()
 
        zoom_focus = ZoomFocusLeft;
        set_zoom_focus (ZoomFocusLeft);
-       zoom_range_clock.ValueChanged.connect (sigc::mem_fun(*this, &Editor::zoom_adjustment_changed));
+       zoom_range_clock->ValueChanged.connect (sigc::mem_fun(*this, &Editor::zoom_adjustment_changed));
 
        bbt_label.set_name ("EditorTimeButton");
        bbt_label.set_size_request (-1, (int)timebar_height);
@@ -420,35 +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);
@@ -458,14 +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));
 
@@ -491,10 +493,10 @@ Editor::Editor ()
        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);
@@ -565,42 +567,42 @@ Editor::Editor ()
 
        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 ();
 
        _summary_hbox.pack_start (*summary_arrows_left, false, false);
        _summary_hbox.pack_start (*summary_frame, true, true);
        _summary_hbox.pack_start (*summary_arrows_right, false, false);
-       
+
        editor_summary_pane.pack2 (_summary_hbox);
 
        edit_pane.pack1 (editor_summary_pane, true, true);
@@ -645,6 +647,7 @@ 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);
 
        _playlist_selector = new PlaylistSelector();
@@ -680,7 +683,7 @@ 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);
        }
 
@@ -702,7 +705,7 @@ Editor::Editor ()
        ControlProtocol::ZoomOut.connect (*this, invalidator (*this), boost::bind (&Editor::temporal_zoom_step, this, true), gui_context());
        ControlProtocol::ScrollTimeline.connect (*this, invalidator (*this), ui_bind (&Editor::control_scroll, this, _1), gui_context());
        BasicUI::AccessAction.connect (*this, invalidator (*this), ui_bind (&Editor::access_action, this, _1, _2), gui_context());
-       
+
        /* problematic: has to return a value and thus cannot be x-thread */
 
        Session::AskAboutPlaylistDeletion.connect_same_thread (*this, boost::bind (&Editor::playlist_deletion_dialog, this, _1));
@@ -718,6 +721,17 @@ Editor::Editor ()
        _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 ();
 
@@ -737,13 +751,28 @@ Editor::~Editor()
                image_socket_listener = 0 ;
        }
 #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)
 {
@@ -773,6 +802,8 @@ Editor::catch_vanishing_regionview (RegionView *rv)
        if (!_all_region_actions_sensitized) {
                sensitize_all_region_actions (true);
        }
+
+       _over_region_trim_target = false;
 }
 
 void
@@ -816,10 +847,15 @@ Editor::show_window ()
        if (!is_visible ()) {
                show_all ();
 
+               /* 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 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
@@ -832,6 +868,10 @@ Editor::show_window ()
                        tv = (static_cast<TimeAxisView*>(*i));
                        tv->reset_height ();
                }
+
+               if (current_mixer_strip) {
+                       current_mixer_strip->hide_things ();
+               }
        }
 
        present ();
@@ -858,14 +898,14 @@ Editor::zoom_adjustment_changed ()
                return;
        }
 
-       double fpu = zoom_range_clock.current_duration() / _canvas_width;
+       double fpu = zoom_range_clock->current_duration() / _canvas_width;
 
        if (fpu < 1.0) {
                fpu = 1.0;
-               zoom_range_clock.set ((framepos_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 ((framepos_t) floor (fpu * _canvas_width));
+               zoom_range_clock->set ((framepos_t) floor (fpu * _canvas_width));
        }
 
        temporal_zoom (fpu);
@@ -1046,9 +1086,9 @@ Editor::set_session (Session *t)
                return;
        }
 
-       zoom_range_clock.set_session (_session);
+       zoom_range_clock->set_session (_session);
        _playlist_selector->set_session (_session);
-       nudge_clock.set_session (_session);
+       nudge_clock->set_session (_session);
        _summary->set_session (_session);
        _group_tabs->set_session (_session);
        _route_groups->set_session (_session);
@@ -1098,7 +1138,7 @@ Editor::set_session (Session *t)
        _session->locations()->added.connect (_session_connections, invalidator (*this), ui_bind (&Editor::add_new_location, this, _1), gui_context());
        _session->locations()->removed.connect (_session_connections, invalidator (*this), ui_bind (&Editor::location_gone, this, _1), gui_context());
        _session->locations()->changed.connect (_session_connections, invalidator (*this), boost::bind (&Editor::refresh_location_display, this), gui_context());
-       _session->locations()->StateChanged.connect (_session_connections, invalidator (*this), ui_bind (&Editor::refresh_location_display_s, this, _1), gui_context());
+       _session->locations()->StateChanged.connect (_session_connections, invalidator (*this), ui_bind (&Editor::refresh_location_display, this), gui_context());
        _session->history().Changed.connect (_session_connections, invalidator (*this), boost::bind (&Editor::history_changed, this), gui_context());
 
        if (Profile->get_sae()) {
@@ -1107,11 +1147,11 @@ Editor::set_session (Session *t)
                bbt.beats = 0;
                bbt.ticks = 120;
                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);
+               nudge_clock->set_mode(AudioClock::BBT);
+               nudge_clock->set (pos, true, 0, AudioClock::BBT);
 
        } else {
-               nudge_clock.set (_session->frame_rate() * 5, true, 0, AudioClock::Timecode); // default of 5 seconds
+               nudge_clock->set (_session->frame_rate() * 5, true, 0, AudioClock::Timecode); // default of 5 seconds
        }
 
        playhead_cursor->canvas_item.show ();
@@ -1119,11 +1159,11 @@ Editor::set_session (Session *t)
        Location* loc = _session->locations()->auto_loop_location();
        if (loc == 0) {
                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 {
@@ -1132,14 +1172,14 @@ Editor::set_session (Session *t)
        }
 
        loc = _session->locations()->auto_punch_location();
-       
+
        if (loc == 0) {
                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 {
@@ -1164,7 +1204,7 @@ Editor::set_session (Session *t)
        super_rapid_screen_update_connection = ARDOUR_UI::instance()->SuperRapidScreenUpdate.connect (
                sigc::mem_fun (*this, &Editor::super_rapid_screen_update)
                );
-       
+
        switch (_snap_type) {
        case SnapToRegionStart:
        case SnapToRegionEnd:
@@ -1230,10 +1270,10 @@ 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), FadeLinear)));
                        items.push_back (MenuElem (_("Slowest"), sigc::bind (sigc::mem_fun (*this, &Editor::set_fade_in_shape), FadeFast)));
-                       
+
                } else {
 
                        items.push_back (
@@ -1252,34 +1292,34 @@ Editor::popup_fade_context_menu (int button, int32_t time, ArdourCanvas::Item* i
                                        *_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 ();
                }
 
@@ -1316,34 +1356,34 @@ Editor::popup_fade_context_menu (int button, int32_t time, ArdourCanvas::Item* i
                                        *_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 ();
                }
 
@@ -1534,10 +1574,8 @@ Editor::build_track_region_context_menu ()
                boost::shared_ptr<Track> tr;
                boost::shared_ptr<Playlist> pl;
 
-               if ((tr = rtv->track()) && ((pl = tr->playlist()))) {
-                       framepos_t const framepos = (framepos_t) floor ((double) get_preferred_edit_position() * tr->speed());
-                       uint32_t regions_at = pl->count_regions_at (framepos);
-                       add_region_context_items (edit_items, regions_at > 1);
+               if ((tr = rtv->track())) {
+                       add_region_context_items (edit_items, tr);
                }
        }
 
@@ -1568,18 +1606,24 @@ Editor::build_track_crossfade_context_menu ()
                if ((tr = atv->track()) && ((pl = tr->playlist()) != 0) && ((apl = boost::dynamic_pointer_cast<AudioPlaylist> (pl)) != 0)) {
 
                        AudioPlaylist::Crossfades xfades;
+                       framepos_t where;
+                       bool ignored;
 
-                       apl->crossfades_at (get_preferred_edit_position (), 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);
                        }
 
-                       framepos_t framepos = (framepos_t) floor ((double) get_preferred_edit_position() * tr->speed());
-                       uint32_t regions_at = pl->count_regions_at (framepos);
-                       add_region_context_items (edit_items, regions_at > 1);
+                       add_region_context_items (edit_items, tr);
                }
        }
 
@@ -1699,38 +1743,47 @@ Editor::xfade_edit_right_region ()
 }
 
 void
-Editor::add_region_context_items (Menu_Helpers::MenuList& edit_items, bool multiple_regions_at_position)
+Editor::add_region_context_items (Menu_Helpers::MenuList& edit_items, boost::shared_ptr<Track> track)
 {
        using namespace Menu_Helpers;
-       
+
        /* OK, stick the region submenu at the top of the list, and then add
           the standard items.
        */
 
-       /* we have to hack up the region name because "_" has a special
-          meaning for menu titles.
-       */
-
        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.
+       */
+
        while ((pos = menu_item_name.find ("_", pos)) != string::npos) {
                menu_item_name.replace (pos, 1, "__");
                pos += 2;
        }
 
        if (_popup_region_menu_item == 0) {
-               _popup_region_menu_item = manage (new MenuItem (menu_item_name));
+               _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 (multiple_regions_at_position && (layering_order_editor == 0 || !layering_order_editor->is_visible ())) {
+       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());
@@ -1750,11 +1803,35 @@ Editor::add_selection_context_items (Menu_Helpers::MenuList& edit_items)
        edit_items.push_back (SeparatorElem());
        edit_items.push_back (MenuElem (_("Spectral Analysis"), sigc::mem_fun(*this, &Editor::analyze_range_selection)));
 
-       if (!selection->regions.empty()) {
-               edit_items.push_back (SeparatorElem());
-               edit_items.push_back (MenuElem (_("Extend Range to End of Region"), sigc::bind (sigc::mem_fun(*this, &Editor::extend_selection_to_end_of_region), false)));
-               edit_items.push_back (MenuElem (_("Extend Range to Start of Region"), sigc::bind (sigc::mem_fun(*this, &Editor::extend_selection_to_start_of_region), false)));
-       }
+       edit_items.push_back (SeparatorElem());
+
+       edit_items.push_back (
+               MenuElem (
+                       _("Move Range Start to Previous Region Boundary"),
+                       sigc::bind (sigc::mem_fun (*this, &Editor::move_range_selection_start_or_end_to_region_boundary), false, false)
+                       )
+               );
+
+       edit_items.push_back (
+               MenuElem (
+                       _("Move Range Start to Next Region Boundary"),
+                       sigc::bind (sigc::mem_fun (*this, &Editor::move_range_selection_start_or_end_to_region_boundary), false, true)
+                       )
+               );
+
+       edit_items.push_back (
+               MenuElem (
+                       _("Move Range End to Previous Region Boundary"),
+                       sigc::bind (sigc::mem_fun (*this, &Editor::move_range_selection_start_or_end_to_region_boundary), true, false)
+                       )
+               );
+
+       edit_items.push_back (
+               MenuElem (
+                       _("Move Range End to Next Region Boundary"),
+                       sigc::bind (sigc::mem_fun (*this, &Editor::move_range_selection_start_or_end_to_region_boundary), true, true)
+                       )
+               );
 
        edit_items.push_back (SeparatorElem());
        edit_items.push_back (MenuElem (_("Convert to Region In-Place"), mem_fun(*this, &Editor::separate_region_from_selection)));
@@ -2054,7 +2131,7 @@ Editor::set_state (const XMLNode& node, int /*version*/)
 {
        const XMLProperty* prop;
        XMLNode* geometry;
-       int x, y, xoff, yoff;
+       int x, y;
        Gdk::Geometry g;
 
        if ((prop = node.property ("id")) != 0) {
@@ -2065,8 +2142,6 @@ Editor::set_state (const XMLNode& node, int /*version*/)
        g.base_height = default_height;
        x = 1;
        y = 1;
-       xoff = 0;
-       yoff = 21;
 
        if ((geometry = find_named_node (node, "geometry")) != 0) {
 
@@ -2098,24 +2173,11 @@ Editor::set_state (const XMLNode& node, int /*version*/)
                if (prop) {
                        y = atoi (prop->value());
                }
-
-               if ((prop = geometry->property ("x_off")) == 0) {
-                       prop = geometry->property ("x-off");
-               }
-               if (prop) {
-                       xoff = atoi (prop->value());
-               }
-               if ((prop = geometry->property ("y_off")) == 0) {
-                       prop = geometry->property ("y-off");
-               }
-               if (prop) {
-                       yoff = atoi (prop->value());
-               }
        }
 
-       //set_default_size (g.base_width, g.base_height);
+       set_default_size (g.base_width, g.base_height);
        move (x, y);
-        
+
        if (_session && (prop = node.property ("playhead"))) {
                framepos_t pos;
                sscanf (prop->value().c_str(), "%" PRIi64, &pos);
@@ -2123,7 +2185,7 @@ Editor::set_state (const XMLNode& node, int /*version*/)
        } else {
                playhead_cursor->set_position (0);
        }
-       
+
        if ((prop = node.property ("mixer-width"))) {
                editor_mixer_strip_width = Width (string_2_enum (prop->value(), editor_mixer_strip_width));
        }
@@ -2217,7 +2279,7 @@ Editor::set_state (const XMLNode& node, int /*version*/)
                        }
                }
        }
-        
+
        if ((prop = node.property ("region-list-sort-type"))) {
                RegionListSortType st;
                _regions->reset_sort_type ((RegionListSortType) string_2_enum (prop->value(), st), true);
@@ -2236,9 +2298,9 @@ Editor::set_state (const XMLNode& node, int /*version*/)
 
                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);
        }
@@ -2250,9 +2312,9 @@ Editor::set_state (const XMLNode& node, int /*version*/)
 
                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);
        }
@@ -2292,9 +2354,8 @@ Editor::get_state ()
        if (is_realized()) {
                Glib::RefPtr<Gdk::Window> win = get_window();
 
-               int x, y, xoff, yoff, width, height;
+               int x, y, width, height;
                win->get_root_origin(x, y);
-               win->get_position(xoff, yoff);
                win->get_size(width, height);
 
                XMLNode* geometry = new XMLNode ("geometry");
@@ -2307,10 +2368,6 @@ Editor::get_state ()
                geometry->add_property("x-pos", string(buf));
                snprintf(buf, sizeof(buf), "%d", y);
                geometry->add_property("y-pos", string(buf));
-               snprintf(buf, sizeof(buf), "%d", xoff);
-               geometry->add_property("x-off", string(buf));
-               snprintf(buf, sizeof(buf), "%d", yoff);
-               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");
@@ -2366,11 +2423,17 @@ Editor::get_state ()
        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;
 }
 
@@ -2660,19 +2723,6 @@ Editor::snap_to_internal (framepos_t& start, int32_t direction, bool for_mark)
 void
 Editor::setup_toolbar ()
 {
-       string pixmap_path;
-
-       /* Mode Buttons (tool selection) */
-
-       mouse_move_button.set_relief(Gtk::RELIEF_NONE);
-       mouse_select_button.set_relief(Gtk::RELIEF_NONE);
-       mouse_gain_button.set_relief(Gtk::RELIEF_NONE);
-       mouse_zoom_button.set_relief(Gtk::RELIEF_NONE);
-       mouse_timefx_button.set_relief(Gtk::RELIEF_NONE);
-       mouse_audition_button.set_relief(Gtk::RELIEF_NONE);
-       // internal_edit_button.set_relief(Gtk::RELIEF_NONE);
-       join_object_range_button.set_relief(Gtk::RELIEF_NONE);
-
        HBox* mode_box = manage(new HBox);
        mode_box->set_border_width (2);
        mode_box->set_spacing(4);
@@ -2761,15 +2811,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");
@@ -2781,21 +2831,23 @@ Editor::setup_toolbar ()
        _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_shrink_button);
        _zoom_box.pack_start (tav_expand_button);
-       
+
        _zoom_tearoff = manage (new TearOff (_zoom_box));
 
        _zoom_tearoff->Detach.connect (sigc::bind (sigc::mem_fun(*this, &Editor::detach_tearoff), static_cast<Box*>(&toolbar_hbox),
@@ -2806,7 +2858,7 @@ Editor::setup_toolbar ()
                                                   &_zoom_tearoff->tearoff_window()));
        _zoom_tearoff->Visible.connect (sigc::bind (sigc::mem_fun(*this, &Editor::reattach_tearoff), static_cast<Box*> (&toolbar_hbox),
                                                    &_zoom_tearoff->tearoff_window(), 0));
-       
+
        snap_box.set_spacing (1);
        snap_box.set_border_width (2);
 
@@ -2837,7 +2889,7 @@ Editor::setup_toolbar ()
 
        nudge_box->pack_start (nudge_backward_button, false, false);
        nudge_box->pack_start (nudge_forward_button, false, false);
-       nudge_box->pack_start (nudge_clock, false, false);
+       nudge_box->pack_start (*nudge_clock, false, false);
 
 
        /* Pack everything in... */
@@ -2848,7 +2900,7 @@ Editor::setup_toolbar ()
        _tools_tearoff = manage (new TearOff (*hbox));
        _tools_tearoff->set_name ("MouseModeBase");
        _tools_tearoff->tearoff_window().signal_key_press_event().connect (sigc::bind (sigc::ptr_fun (relay_key_press), &_tools_tearoff->tearoff_window()), false);
-        
+
        if (Profile->get_sae()) {
                _tools_tearoff->set_can_be_torn_off (false);
        }
@@ -2870,7 +2922,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 ();
@@ -2885,7 +2941,7 @@ Editor::setup_toolbar ()
        toolbar_frame.set_shadow_type (SHADOW_OUT);
        toolbar_frame.set_name ("BaseFrame");
        toolbar_frame.add (_toolbar_viewport);
-        
+
         DPIReset.connect (sigc::mem_fun (*this, &Editor::resize_text_widgets));
 }
 
@@ -2933,7 +2989,6 @@ Editor::setup_midi_toolbar ()
 
        /* Midi sound notes */
        midi_sound_notes.add (*(manage (new Image (::get_icon("midi_sound_notes")))));
-       midi_sound_notes.set_relief(Gtk::RELIEF_NONE);
        midi_sound_notes.unset_flags (CAN_FOCUS);
 
        /* Panic */
@@ -2959,7 +3014,7 @@ Editor::convert_drop_to_paths (
        if (_session == 0) {
                return -1;
        }
-        
+
        vector<string> uris = data.get_uris();
 
        if (uris.empty()) {
@@ -3202,79 +3257,6 @@ Editor::duplicate_dialog (bool with_dialog)
        }
 }
 
-void
-Editor::show_verbose_canvas_cursor ()
-{
-       verbose_canvas_cursor->raise_to_top();
-       verbose_canvas_cursor->show();
-       verbose_cursor_visible = true;
-}
-
-void
-Editor::hide_verbose_canvas_cursor ()
-{
-       verbose_canvas_cursor->hide();
-       verbose_cursor_visible = false;
-}
-
-double
-Editor::clamp_verbose_cursor_x (double x)
-{
-       if (x < 0) {
-               x = 0;
-       } else {
-               x = min (_canvas_width - 200.0, x);
-       }
-       return x;
-}
-
-double
-Editor::clamp_verbose_cursor_y (double y)
-{
-       if (y < canvas_timebars_vsize) {
-               y = canvas_timebars_vsize;
-       } else {
-               y = min (_canvas_height - 50, y);
-       }
-       return y;
-}
-
-void
-Editor::show_verbose_canvas_cursor_with (const string & txt, int32_t xoffset, int32_t yoffset)
-{
-       verbose_canvas_cursor->property_text() = txt.c_str();
-
-       int x, y;
-       double wx, wy;
-
-       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);
-
-       show_verbose_canvas_cursor ();
-}
-
-void
-Editor::set_verbose_canvas_cursor (const string & txt, double x, double y)
-{
-       verbose_canvas_cursor->property_text() = txt.c_str();
-       /* don't get too close to the edge */
-       verbose_canvas_cursor->property_x() = clamp_verbose_cursor_x (x);
-       verbose_canvas_cursor->property_y() = clamp_verbose_cursor_y (y);
-}
-
-void
-Editor::set_verbose_canvas_cursor_text (const string & txt)
-{
-       verbose_canvas_cursor->property_text() = txt.c_str();
-}
-
 void
 Editor::set_edit_mode (EditMode m)
 {
@@ -3470,16 +3452,19 @@ Editor::zoom_focus_selection_done ()
        }
 }
 
-gint
+bool
 Editor::edit_controls_button_release (GdkEventButton* ev)
 {
        if (Keyboard::is_context_menu_event (ev)) {
                ARDOUR_UI::instance()->add_route (this);
+       } else if (ev->button == 1) {
+               selection->clear_tracks ();
        }
-       return TRUE;
+
+       return true;
 }
 
-gint
+bool
 Editor::mouse_select_button_release (GdkEventButton* ev)
 {
        /* this handles just right-clicks */
@@ -3502,9 +3487,6 @@ Editor::set_zoom_focus (ZoomFocus f)
 
        if (zoom_focus != f) {
                zoom_focus = f;
-
-               ZoomFocusChanged (); /* EMIT_SIGNAL */
-
                instant_save ();
        }
 }
@@ -3526,7 +3508,6 @@ Editor::pane_allocation_handler (Allocation &alloc, Paned* which)
        XMLProperty* prop;
        char buf[32];
        XMLNode* node = ARDOUR_UI::instance()->editor_settings();
-       int width, height;
 
        enum Pane {
                Horizontal = 0x1,
@@ -3534,23 +3515,8 @@ Editor::pane_allocation_handler (Allocation &alloc, Paned* which)
        };
 
        static Pane done;
-       
-       XMLNode* geometry;
 
-       width = default_width;
-       height = default_height;
-
-       if ((geometry = find_named_node (*node, "geometry")) != 0) {
-
-               prop = geometry->property ("x-size");
-               if (prop) {
-                       width = atoi (prop->value());
-               }
-               prop = geometry->property ("y-size");
-               if (prop) {
-                       height = atoi (prop->value());
-               }
-       }
+       XMLNode* geometry = find_named_node (*node, "geometry");
 
        if (which == static_cast<Paned*> (&edit_pane)) {
 
@@ -3582,7 +3548,7 @@ Editor::pane_allocation_handler (Allocation &alloc, Paned* which)
                }
 
                done = (Pane) (done | Horizontal);
-               
+
        } else if (which == static_cast<Paned*> (&editor_summary_pane)) {
 
                if (done & Vertical) {
@@ -3821,7 +3787,7 @@ Editor::get_nudge_distance (framepos_t pos, framecnt_t& next)
 {
        framecnt_t ret;
 
-       ret = nudge_clock.current_duration (pos);
+       ret = nudge_clock->current_duration (pos);
        next = ret + 1; /* XXXX fix me */
 
        return ret;
@@ -3832,8 +3798,8 @@ Editor::playlist_deletion_dialog (boost::shared_ptr<Playlist> pl)
 {
        ArdourDialog dialog (_("Playlist Deletion"));
        Label  label (string_compose (_("Playlist %1 is currently unused.\n"
-                                       "If left alone, no audio files used by it will be cleaned.\n"
-                                       "If deleted, audio files used by it alone by will cleaned."),
+                                       "If it is kept, its audio files will not be cleaned.\n"
+                                       "If it is deleted, audio files used by it alone will be cleaned."),
                                      pl->name()));
 
        dialog.set_position (WIN_POS_CENTER);
@@ -3841,8 +3807,8 @@ Editor::playlist_deletion_dialog (boost::shared_ptr<Playlist> pl)
 
        label.show ();
 
-       dialog.add_button (_("Delete playlist"), RESPONSE_ACCEPT);
-       dialog.add_button (_("Keep playlist"), RESPONSE_REJECT);
+       dialog.add_button (_("Delete Playlist"), RESPONSE_ACCEPT);
+       dialog.add_button (_("Keep Playlist"), RESPONSE_REJECT);
        dialog.add_button (_("Cancel"), RESPONSE_CANCEL);
 
        switch (dialog.run ()) {
@@ -3941,7 +3907,7 @@ Editor::control_layout_scroll (GdkEventScroll* ev)
 void
 Editor::session_state_saved (string)
 {
-       update_title ();        
+       update_title ();
        _snapshots->redisplay ();
 }
 
@@ -3964,7 +3930,7 @@ Editor::maximise_editing_space ()
        if (post_maximal_vertical_pane_position == 0) {
                post_maximal_vertical_pane_position = editor_summary_pane.get_height();
        }
-       
+
        fullscreen ();
 
        if (post_maximal_editor_width) {
@@ -4003,7 +3969,7 @@ Editor::restore_editing_space ()
        if (post_maximal_vertical_pane_position != editor_summary_pane.get_position()) {
                post_maximal_vertical_pane_position = editor_summary_pane.get_position();
        }
-       
+
        unfullscreen();
 
        _mouse_mode_tearoff->set_visible (true);
@@ -4128,6 +4094,16 @@ Editor::reposition_and_zoom (framepos_t frame, double fpu)
        }
 }
 
+Editor::VisualState::VisualState ()
+       : gui_state (new GUIObjectState)
+{
+}
+
+Editor::VisualState::~VisualState ()
+{
+       delete gui_state;
+}
+
 Editor::VisualState*
 Editor::current_visual_state (bool with_tracks)
 {
@@ -4137,10 +4113,8 @@ Editor::current_visual_state (bool with_tracks)
        vs->leftmost_frame = leftmost_frame;
        vs->zoom_focus = zoom_focus;
 
-       if (with_tracks) {
-               for (TrackViewList::iterator i = track_views.begin(); i != track_views.end(); ++i) {
-                       vs->track_states.push_back (TAVState ((*i), &(*i)->get_state()));
-               }
+       if (with_tracks) {      
+               *(vs->gui_state) = *ARDOUR_UI::instance()->gui_object_state;
        }
 
        return vs;
@@ -4195,22 +4169,14 @@ Editor::use_visual_state (VisualState& vs)
 
        set_zoom_focus (vs.zoom_focus);
        reposition_and_zoom (vs.leftmost_frame, vs.frames_per_unit);
+       
+       *ARDOUR_UI::instance()->gui_object_state = *vs.gui_state;
 
-       for (list<TAVState>::iterator i = vs.track_states.begin(); i != vs.track_states.end(); ++i) {
-               TrackViewList::iterator t;
-
-               /* check if the track still exists - it could have been deleted */
-
-               if ((t = find (track_views.begin(), track_views.end(), i->first)) != track_views.end()) {
-                       (*t)->set_state (*(i->second), Stateful::loading_state_version);
-               }
-       }
-
-
-       if (!vs.track_states.empty()) {
-               _routes->update_visibility ();
+       for (TrackViewList::iterator i = track_views.begin(); i != track_views.end(); ++i) {    
+               (*i)->reset_visual_state ();
        }
 
+       _routes->update_visibility ();
        _routes->resume_redisplay ();
 
        no_save_visual = false;
@@ -4254,8 +4220,8 @@ Editor::post_zoom ()
 
        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);
+       if (frames_per_unit != zoom_range_clock->current_duration()) {
+               zoom_range_clock->set (frames);
        }
 
        if (mouse_mode == MouseRange && selection->time.start () != selection->time.end_frame ()) {
@@ -4393,6 +4359,7 @@ Editor::get_preferred_edit_position (bool ignore_playhead)
        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();
        }
 
@@ -4403,6 +4370,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:
@@ -4415,6 +4383,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;
                        }
                }
@@ -4427,6 +4396,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;
        }
 
@@ -4608,7 +4578,7 @@ 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 {
@@ -4625,14 +4595,14 @@ Editor::get_regions_from_selection_and_edit_point ()
           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 ();
@@ -4651,7 +4621,7 @@ RegionSelection
 Editor::get_regions_from_selection_and_entered ()
 {
        RegionSelection regions = selection->regions;
-       
+
        if (regions.empty() && entered_regionview) {
                regions.add (entered_regionview);
        }
@@ -4815,6 +4785,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
 {
@@ -4855,8 +4831,6 @@ Editor::handle_new_route (RouteList& routes)
        RouteTimeAxisView *rtv;
        list<RouteTimeAxisView*> new_views;
 
-        cerr << "Handle new route\n";
-
        for (RouteList::iterator x = routes.begin(); x != routes.end(); ++x) {
                boost::shared_ptr<Route> route = (*x);
 
@@ -4867,9 +4841,11 @@ Editor::handle_new_route (RouteList& routes)
                DataType dt = route->input()->default_type();
 
                if (dt == ARDOUR::DataType::AUDIO) {
-                       rtv = new AudioTimeAxisView (*this, _session, route, *track_canvas);
+                       rtv = new AudioTimeAxisView (*this, _session, *track_canvas);
+                       rtv->set_route (route);
                } else if (dt == ARDOUR::DataType::MIDI) {
-                       rtv = new MidiTimeAxisView (*this, _session, route, *track_canvas);
+                       rtv = new MidiTimeAxisView (*this, _session, *track_canvas);
+                       rtv->set_route (route);
                } else {
                        throw unknown_type();
                }
@@ -4879,7 +4855,14 @@ 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);
@@ -4903,7 +4886,7 @@ 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) {
@@ -4931,7 +4914,7 @@ Editor::timeaxisview_deleted (TimeAxisView *tv)
 
        if (rtav) {
                route = rtav->route ();
-       } 
+       }
 
        if (current_mixer_strip && current_mixer_strip->route() == route) {
 
@@ -4944,31 +4927,43 @@ Editor::timeaxisview_deleted (TimeAxisView *tv)
                } else {
                        next_tv = (*i);
                }
-               
-               
+
+
                if (next_tv) {
                        set_selected_mixer_strip (*next_tv);
                } else {
                        /* make the editor mixer strip go away setting the
                         * button to inactive (which also unticks the menu option)
                         */
-                       
+
                        ActionManager::uncheck_toggleaction ("<Actions>/Editor/show-editor-mixer");
                }
-       } 
+       }
 }
 
 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;
 
-       _routes->hide_track_in_display (*tv);
+                       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);
+       }
 }
 
 bool
@@ -5102,52 +5097,40 @@ Editor::check_step_edit ()
 }
 
 bool
-Editor::horizontal_scroll_left_press ()
+Editor::scroll_press (Direction dir)
 {
        ++_scroll_callbacks;
-       
+
        if (_scroll_connection.connected() && _scroll_callbacks < 5) {
                /* 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;
        }
 
@@ -5155,7 +5138,7 @@ Editor::horizontal_scroll_right_press ()
 }
 
 void
-Editor::horizontal_scroll_right_release ()
+Editor::scroll_release ()
 {
        _scroll_connection.disconnect ();
 }
@@ -5169,15 +5152,15 @@ Editor::reset_x_origin_to_follow_playhead ()
        if (frame < leftmost_frame || frame > leftmost_frame + current_page_frames()) {
 
                if (_session->transport_speed() < 0) {
-                       
+
                        if (frame > (current_page_frames() / 2)) {
                                center_screen (frame-(current_page_frames()/2));
                        } else {
                                center_screen (current_page_frames()/2);
                        }
-                       
+
                } else {
-                                               
+
                        if (frame < leftmost_frame) {
                                /* moving left */
                                framepos_t l = 0;
@@ -5188,11 +5171,11 @@ Editor::reset_x_origin_to_follow_playhead ()
                                        /* not rolling: end up with the playhead 3/4 of the way along the page */
                                        l = frame - (3 * current_page_frames() / 4);
                                }
-                               
+
                                if (l < 0) {
                                        l = 0;
                                }
-                               
+
                                center_screen_internal (l + (current_page_frames() / 2), current_page_frames ());
                        } else {
                                /* moving right */
@@ -5261,12 +5244,12 @@ Editor::super_rapid_screen_update ()
                        }
 
                } else {
-                       
+
                        /* 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                         
+#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;
@@ -5276,12 +5259,12 @@ Editor::super_rapid_screen_update ()
                        } else {
                                /* relax */
                        }
-                        
+
                        current = target;
                        set_horizontal_position (current);
 #endif
                }
-               
+
        }
 }
 
@@ -5294,7 +5277,7 @@ Editor::session_going_away ()
        _session_connections.drop_connections ();
 
        super_rapid_screen_update_connection.disconnect ();
-       
+
        selection->clear ();
        cut_buffer->clear ();
 
@@ -5334,8 +5317,8 @@ Editor::session_going_away ()
        }
        track_views.clear ();
 
-       zoom_range_clock.set_session (0);
-       nudge_clock.set_session (0);
+       zoom_range_clock->set_session (0);
+       nudge_clock->set_session (0);
 
        editor_list_button.set_active(false);
        editor_list_button.set_sensitive(false);
@@ -5373,7 +5356,7 @@ 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 ();
@@ -5392,7 +5375,7 @@ Editor::change_region_layering_order ()
        if (!pl) {
                return;
        }
-                
+
        if (layering_order_editor == 0) {
                layering_order_editor = new RegionLayeringOrderEditor(*this);
        }
@@ -5432,7 +5415,7 @@ 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 ());
 }
 
@@ -5445,7 +5428,7 @@ Editor::resize_text_widgets ()
         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)
 {