Fix track height menu.
[ardour.git] / gtk2_ardour / editor.cc
index ecefc1f3d1a3db47591847c42544f7b6e438202b..085767e50b1776a0a0f7c8ab6f3f970c1d7de1c6 100644 (file)
@@ -220,6 +220,27 @@ show_me_the_size (Requisition* r, const char* what)
        cerr << "size of " << what << " = " << r->width << " x " << r->height << endl;
 }
 
+#ifdef GTKOSX
+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.
+
+          ugh.
+       */
+
+       int max_width_of_lhs = GTK_WIDGET(pane->gobj())->allocation.width - 25;
+
+       gint pos = pane->get_position ();
+
+       if (pos > max_width_of_lhs) {
+               pane->set_position (max_width_of_lhs);
+       }
+}
+#endif
+
 Editor::Editor ()
        : _join_object_range_state (JOIN_OBJECT_RANGE_NONE)
 
@@ -241,7 +262,6 @@ Editor::Editor ()
          */
 
        , vertical_adjustment (0.0, 0.0, 10.0, 400.0)
-       , horizontal_adjustment (0.0, 0.0, 20.0, 1200.0)
 
          /* tool bar related */
 
@@ -304,14 +324,12 @@ Editor::Editor ()
        last_autoscroll_y = 0;
        autoscroll_active = false;
        autoscroll_timeout_tag = -1;
-       interthread_progress_window = 0;
        logo_item = 0;
 
        analysis_window = 0;
 
        current_interthread_info = 0;
        _show_measures = true;
-       _show_waveforms_recording = true;
        show_gain_after_trim = false;
        verbose_cursor_on = true;
        last_item_entered = 0;
@@ -319,14 +337,15 @@ Editor::Editor ()
 
        have_pending_keyboard_selection = false;
        _follow_playhead = true;
+        _stationary_playhead = false;
        _xfade_visibility = true;
        editor_ruler_menu = 0;
        no_ruler_shown_update = false;
        marker_menu = 0;
-       start_end_marker_menu = 0;
+       session_range_marker_menu = 0;
        range_marker_menu = 0;
        marker_menu_item = 0;
-       tm_marker_menu = 0;
+       tempo_or_meter_marker_menu = 0;
        transport_marker_menu = 0;
        new_transport_marker_menu = 0;
        editor_mixer_strip_width = Wide;
@@ -340,7 +359,6 @@ Editor::Editor ()
        entered_regionview = 0;
        entered_marker = 0;
        clear_entered_track = false;
-       _new_regionviews_show_envelope = false;
        current_timefx = 0;
        playhead_cursor = 0;
        button_release_can_deselect = true;
@@ -374,7 +392,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);
@@ -449,7 +467,6 @@ Editor::Editor ()
        selection->MarkersChanged.connect (sigc::mem_fun(*this, &Editor::marker_selection_changed));
 
        edit_controls_vbox.set_spacing (0);
-       horizontal_adjustment.signal_value_changed().connect (sigc::mem_fun(*this, &Editor::scroll_canvas_horizontally), false);
        vertical_adjustment.signal_value_changed().connect (sigc::mem_fun(*this, &Editor::tie_vertical_scrolling), true);
        track_canvas->signal_map_event().connect (sigc::mem_fun (*this, &Editor::track_canvas_map_handler));
 
@@ -528,7 +545,7 @@ Editor::Editor ()
        nlabel = manage (new Label (_("Regions")));
        nlabel->set_angle (-90);
        the_notebook.append_page (_regions->widget (), *nlabel);
-       nlabel = manage (new Label (_("Tracks/Busses")));
+       nlabel = manage (new Label (_("Tracks & Busses")));
        nlabel->set_angle (-90);
        the_notebook.append_page (_routes->widget (), *nlabel);
        nlabel = manage (new Label (_("Snapshots")));
@@ -588,7 +605,10 @@ Editor::Editor ()
        edit_pane.pack2 (the_notebook, false, true);
 
        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)));
+#endif
        top_hbox.pack_start (toolbar_frame, false, true);
 
        HBox *hbox = manage (new HBox);
@@ -627,7 +647,7 @@ Editor::Editor ()
        _playlist_selector = new PlaylistSelector();
        _playlist_selector->signal_delete_event().connect (sigc::bind (sigc::ptr_fun (just_hide_it), static_cast<Window *> (_playlist_selector)));
 
-       RegionView::RegionViewGoingAway.connect (*this, ui_bind (&Editor::catch_vanishing_regionview, this, _1), gui_context());
+       RegionView::RegionViewGoingAway.connect (*this, invalidator (*this),  ui_bind (&Editor::catch_vanishing_regionview, this, _1), gui_context());
 
        /* nudge stuff */
 
@@ -674,19 +694,19 @@ Editor::Editor ()
 
        /* allow external control surfaces/protocols to do various things */
 
-       ControlProtocol::ZoomToSession.connect (*this, boost::bind (&Editor::temporal_zoom_session, this), gui_context());
-       ControlProtocol::ZoomIn.connect (*this, boost::bind (&Editor::temporal_zoom_step, this, false), gui_context());
-       ControlProtocol::ZoomOut.connect (*this, boost::bind (&Editor::temporal_zoom_step, this, true), gui_context());
-       ControlProtocol::ScrollTimeline.connect (*this, ui_bind (&Editor::control_scroll, this, _1), gui_context());
-       BasicUI::AccessAction.connect (*this, ui_bind (&Editor::access_action, this, _1, _2), gui_context());
+       ControlProtocol::ZoomToSession.connect (*this, invalidator (*this), boost::bind (&Editor::temporal_zoom_session, this), gui_context());
+       ControlProtocol::ZoomIn.connect (*this, invalidator (*this), boost::bind (&Editor::temporal_zoom_step, this, false), gui_context());
+       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));
 
-       Config->ParameterChanged.connect (*this, ui_bind (&Editor::parameter_changed, this, _1), gui_context());
+       Config->ParameterChanged.connect (*this, invalidator (*this), ui_bind (&Editor::parameter_changed, this, _1), gui_context());
 
-       TimeAxisView::CatchDeletion.connect (*this, ui_bind (&Editor::timeaxisview_deleted, this, _1), gui_context());
+       TimeAxisView::CatchDeletion.connect (*this, invalidator (*this), ui_bind (&Editor::timeaxisview_deleted, this, _1), gui_context());
 
        _last_normalization_value = 0;
 
@@ -793,6 +813,8 @@ Editor::show_window ()
                        tv = (static_cast<TimeAxisView*>(*i));
                        tv->reset_height ();
                }
+
+               reset_zoom (frames_per_unit);
        }
 
        present ();
@@ -968,24 +990,6 @@ Editor::center_screen_internal (nframes64_t frame, float page)
        reset_x_origin (frame);
 }
 
-void
-Editor::handle_new_duration ()
-{
-       if (!_session) {
-               return;
-       }
-
-       ENSURE_GUI_THREAD (*this, &Editor::handle_new_duration)
-       nframes64_t new_end = _session->current_end_frame() + (nframes64_t) floorf (current_page_frames() * 0.10f);
-
-       horizontal_adjustment.set_upper (new_end / frames_per_unit);
-       horizontal_adjustment.set_page_size (current_page_frames()/frames_per_unit);
-
-       if (horizontal_adjustment.get_value() + _canvas_width > horizontal_adjustment.get_upper()) {
-               horizontal_adjustment.set_value (horizontal_adjustment.get_upper() - _canvas_width);
-       }
-       //cerr << "Editor::handle_new_duration () called ha v:l:u:ps:lcf = " << horizontal_adjustment.get_value() << ":" << horizontal_adjustment.get_lower() << ":" << horizontal_adjustment.get_upper() << ":" << horizontal_adjustment.get_page_size() << ":" << endl;//DEBUG
-}
 
 void
 Editor::update_title ()
@@ -1065,22 +1069,20 @@ Editor::set_session (Session *t)
           but use Gtkmm2ext::UI::instance()->call_slot();
        */
 
-       _session->TransportStateChange.connect (_session_connections, boost::bind (&Editor::map_transport_state, this), gui_context());
-       _session->PositionChanged.connect (_session_connections, ui_bind (&Editor::map_position_change, this, _1), gui_context());
-       _session->RouteAdded.connect (_session_connections, ui_bind (&Editor::handle_new_route, this, _1), gui_context());
-       _session->DurationChanged.connect (_session_connections, boost::bind (&Editor::handle_new_duration, this), gui_context());
-       _session->DirtyChanged.connect (_session_connections, boost::bind (&Editor::update_title, this), gui_context());
-       _session->TimecodeOffsetChanged.connect (_session_connections, boost::bind (&Editor::update_just_timecode, this), gui_context());
-       _session->tempo_map().StateChanged.connect (_session_connections, ui_bind (&Editor::tempo_map_changed, this, _1), gui_context());
-       _session->Located.connect (_session_connections, boost::bind (&Editor::located, this), gui_context());
-       _session->config.ParameterChanged.connect (_session_connections, ui_bind (&Editor::parameter_changed, this, _1), gui_context());
-       _session->StateSaved.connect (_session_connections, ui_bind (&Editor::session_state_saved, this, _1), gui_context());
-       _session->locations()->added.connect (_session_connections, ui_bind (&Editor::add_new_location, this, _1), gui_context());
-       _session->locations()->removed.connect (_session_connections, ui_bind (&Editor::location_gone, this, _1), gui_context());
-       _session->locations()->changed.connect (_session_connections, boost::bind (&Editor::refresh_location_display, this), gui_context());
-       _session->locations()->StateChanged.connect (_session_connections, ui_bind (&Editor::refresh_location_display_s, this, _1), gui_context());
-       _session->locations()->end_location()->changed.connect (_session_connections, ui_bind (&Editor::end_location_changed, this, _1), gui_context());
-       _session->history().Changed.connect (_session_connections, boost::bind (&Editor::history_changed, this), 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());
+       _session->StateSaved.connect (_session_connections, invalidator (*this), ui_bind (&Editor::session_state_saved, this, _1), gui_context());
+       _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->history().Changed.connect (_session_connections, invalidator (*this), boost::bind (&Editor::history_changed, this), gui_context());
 
        if (Profile->get_sae()) {
                BBT_Time bbt;
@@ -1128,10 +1130,9 @@ Editor::set_session (Session *t)
        _session->config.map_parameters (pc);
 
        refresh_location_display ();
-       handle_new_duration ();
 
        restore_ruler_visibility ();
-       //tempo_map_changed (Change (0));
+       //tempo_map_changed (PropertyChange (0));
        _session->tempo_map().apply_with_metrics (*this, &Editor::draw_metric_marks);
 
        for (TrackViewList::iterator i = track_views.begin(); i != track_views.end(); ++i) {
@@ -1333,7 +1334,7 @@ Editor::popup_track_context_menu (int button, int32_t time, ItemType item_type,
                break;
 
        case StreamItem:
-               if (clicked_routeview->get_diskstream()) {
+               if (clicked_routeview->track()) {
                        build_menu_function = &Editor::build_track_context_menu;
                } else {
                        build_menu_function = &Editor::build_track_bus_context_menu;
@@ -1362,16 +1363,13 @@ Editor::popup_track_context_menu (int button, int32_t time, ItemType item_type,
                                        ActionManager::set_sensitive (ActionManager::edit_point_in_region_sensitive_actions, false);
                                }
                        }
-                       /*
                        if (region_edit_menu_split_multichannel_item) {
-                               if (clicked_regionview && clicked_regionview->region().n_channels() > 1) {
-                                       // GTK2FIX find the action, change its sensitivity
-                                       // region_edit_menu_split_multichannel_item->set_sensitive (true);
+                               if (clicked_regionview && clicked_regionview->region()->n_channels() > 1) {
+                                       region_edit_menu_split_multichannel_item->set_sensitive (true);
                                } else {
-                                       // GTK2FIX see above
-                                       // region_edit_menu_split_multichannel_item->set_sensitive (false);
+                                       region_edit_menu_split_multichannel_item->set_sensitive (false);
                                }
-                       }*/
+                       }
                }
                break;
 
@@ -1455,11 +1453,11 @@ Editor::build_track_region_context_menu (nframes64_t frame)
        RouteTimeAxisView* rtv = dynamic_cast<RouteTimeAxisView*> (clicked_axisview);
 
        if (rtv) {
-               boost::shared_ptr<Diskstream> ds;
+               boost::shared_ptr<Track> tr;
                boost::shared_ptr<Playlist> pl;
 
-               if ((ds = rtv->get_diskstream()) && ((pl = ds->playlist()))) {
-                       Playlist::RegionList* regions = pl->regions_at ((nframes64_t) floor ( (double)frame * ds->speed()));
+               if ((tr = rtv->track()) && ((pl = tr->playlist()))) {
+                       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
@@ -1468,7 +1466,7 @@ Editor::build_track_region_context_menu (nframes64_t frame)
                                boost::shared_ptr<Region> dummy_region; // = NULL
                                add_region_context_items (rtv->view(), dummy_region, edit_items);
                        } else {
-                               for (Playlist::RegionList::iterator i = regions->begin(); i != regions->end(); ++i) {
+                               for (Playlist::RegionList::reverse_iterator i = regions->rbegin(); i != regions->rend(); ++i) {
                                        add_region_context_items (rtv->view(), (*i), edit_items);
                                }
                        }
@@ -1492,11 +1490,11 @@ Editor::build_track_crossfade_context_menu (nframes64_t frame)
        AudioTimeAxisView* atv = dynamic_cast<AudioTimeAxisView*> (clicked_axisview);
 
        if (atv) {
-               boost::shared_ptr<Diskstream> ds;
+               boost::shared_ptr<Track> tr;
                boost::shared_ptr<Playlist> pl;
                boost::shared_ptr<AudioPlaylist> apl;
 
-               if ((ds = atv->get_diskstream()) && ((pl = ds->playlist()) != 0) && ((apl = boost::dynamic_pointer_cast<AudioPlaylist> (pl)) != 0)) {
+               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;
@@ -1516,7 +1514,7 @@ Editor::build_track_crossfade_context_menu (nframes64_t frame)
                                boost::shared_ptr<Region> dummy_region; // = NULL
                                add_region_context_items (atv->audio_view(), dummy_region, edit_items);
                        } else {
-                               for (Playlist::RegionList::iterator i = regions->begin(); i != regions->end(); ++i) {
+                               for (Playlist::RegionList::reverse_iterator i = regions->rbegin(); i != regions->rend(); ++i) {
                                        add_region_context_items (atv->audio_view(), (*i), edit_items);
                                }
                        }
@@ -1681,11 +1679,11 @@ Editor::add_region_context_items (StreamView* sv, boost::shared_ptr<Region> regi
        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 (_("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 (MenuElem (_("Spectral Analysis..."), sigc::mem_fun(*this, &Editor::analyze_region_selection)));
        }
 
        items.push_back (SeparatorElem());
@@ -1718,7 +1716,7 @@ Editor::add_region_context_items (StreamView* sv, boost::shared_ptr<Region> regi
                break;
        }
 
-       bbt_glue_item->signal_activate().connect (sigc::bind (sigc::mem_fun (*this, &Editor::set_region_lock_style), Region::MusicTime));
+       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());
@@ -1728,6 +1726,8 @@ Editor::add_region_context_items (StreamView* sv, boost::shared_ptr<Region> regi
                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")));
@@ -1777,7 +1777,7 @@ Editor::add_region_context_items (StreamView* sv, boost::shared_ptr<Region> regi
                        items.push_back (SeparatorElem());
                }
 
-               items.push_back (MenuElem (_("Normalize"), sigc::mem_fun(*this, &Editor::normalize_region)));
+               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)));
                }
@@ -1847,7 +1847,7 @@ Editor::add_region_context_items (StreamView* sv, boost::shared_ptr<Region> regi
        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 (_("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)));
@@ -2241,7 +2241,8 @@ Editor::set_state (const XMLNode& node, int /*version*/)
        move (x, y);
 
        if (_session && (prop = node.property ("playhead"))) {
-               nframes64_t pos = atol (prop->value().c_str());
+               nframes64_t pos;
+                sscanf (prop->value().c_str(), "%" PRIi64, &pos);
                playhead_cursor->set_position (pos);
        } else {
                playhead_cursor->set_position (0);
@@ -2303,18 +2304,6 @@ Editor::set_state (const XMLNode& node, int /*version*/)
                set_edit_point_preference ((EditPoint) string_2_enum (prop->value(), _edit_point), true);
        }
 
-       if ((prop = node.property ("show-waveforms-recording"))) {
-               bool yn = string_is_affirmative (prop->value());
-               _show_waveforms_recording = !yn;
-               RefPtr<Action> act = ActionManager::get_action (X_("Editor"), X_("ToggleWaveformsWhileRecording"));
-               if (act) {
-                       RefPtr<ToggleAction> tact = RefPtr<ToggleAction>::cast_dynamic(act);
-                       /* do it twice to force the change */
-                       tact->set_active (!yn);
-                       tact->set_active (yn);
-               }
-       }
-
        if ((prop = node.property ("show-measures"))) {
                bool yn = string_is_affirmative (prop->value());
                _show_measures = !yn;
@@ -2339,6 +2328,18 @@ 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 ("region-list-sort-type"))) {
                RegionListSortType st;
                _regions->reset_sort_type ((RegionListSortType) string_2_enum (prop->value(), st), true);
@@ -2381,6 +2382,13 @@ Editor::set_state (const XMLNode& node, int /*version*/)
                }
        }
 
+       if ((prop = node.property (X_("editor-list-page")))) {
+               the_notebook.set_current_page (atoi (prop->value ()));
+       }
+
+       if ((prop = node.property (X_("editor-pane-position")))) {
+               edit_pane.set_position (atoi (prop->value ()));
+       }
 
        return 0;
 }
@@ -2442,9 +2450,9 @@ Editor::get_state ()
        snprintf (buf, sizeof (buf), "%f", vertical_adjustment.get_value ());
        node->add_property ("y-origin", buf);
 
-       node->add_property ("show-waveforms-recording", _show_waveforms_recording ? "yes" : "no");
        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 ("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));
@@ -2463,6 +2471,12 @@ 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 ());
+       node->add_property (X_("editor-list-page"), buf);
+
+       snprintf (buf, sizeof (buf), "%d", edit_pane.get_position ());
+       node->add_property (X_("editor-pane-position"), buf);
+
        return *node;
 }
 
@@ -2749,37 +2763,26 @@ Editor::setup_toolbar ()
 
        /* table containing mode buttons */
 
-       Table* mouse_mode_button_table = manage (new Table (Profile->get_sae() ? 4 : 6, 2));
-
-       int c = 0;
+       HBox* mouse_mode_button_box = manage (new HBox ());
 
        if (Profile->get_sae()) {
-               mouse_mode_button_table->attach (mouse_move_button, c, c + 1, 0, 1);
-               ++c;
+               mouse_mode_button_box->pack_start (mouse_move_button);
        } else {
-               mouse_mode_button_table->attach (mouse_move_button, c, c + 1, 0, 1);
-               mouse_mode_button_table->attach (mouse_select_button, c + 1, c + 2, 0, 1);
-               mouse_mode_button_table->attach (join_object_range_button, c, c + 2, 1, 2);
-               c += 2;
+               mouse_mode_button_box->pack_start (mouse_move_button);
+               mouse_mode_button_box->pack_start (join_object_range_button);
+               mouse_mode_button_box->pack_start (mouse_select_button);
        }
 
-       mouse_mode_button_table->attach (mouse_zoom_button, c, c + 1, 0, 1);
-       ++c;
-       
+       mouse_mode_button_box->pack_start (mouse_zoom_button);
+
        if (!Profile->get_sae()) {
-               mouse_mode_button_table->attach (mouse_gain_button, c, c + 1, 0, 1);
-               ++c;
+               mouse_mode_button_box->pack_start (mouse_gain_button);
        }
-       
-       mouse_mode_button_table->attach (mouse_timefx_button, c, c + 1, 0, 1);
-       ++c;
-       
-       mouse_mode_button_table->attach (mouse_audition_button, c, c + 1, 0, 1);
-       ++c;
-       
-       mouse_mode_button_table->attach (internal_edit_button, c, c + 1, 0, 1);
-       ++c;
-       
+
+       mouse_mode_button_box->pack_start (mouse_timefx_button);
+       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()) {
@@ -2792,24 +2795,24 @@ Editor::setup_toolbar ()
        edit_mode_selector.signal_changed().connect (sigc::mem_fun(*this, &Editor::edit_mode_selection_done));
 
        mode_box->pack_start (edit_mode_selector);
-       mode_box->pack_start (*mouse_mode_button_table);
+       mode_box->pack_start (*mouse_mode_button_box);
 
-       mouse_mode_tearoff = manage (new TearOff (*mode_box));
-       mouse_mode_tearoff->set_name ("MouseModeBase");
-       mouse_mode_tearoff->tearoff_window().signal_key_press_event().connect (sigc::bind (sigc::ptr_fun (relay_key_press), &mouse_mode_tearoff->tearoff_window()), false);
+       _mouse_mode_tearoff = manage (new TearOff (*mode_box));
+       _mouse_mode_tearoff->set_name ("MouseModeBase");
+       _mouse_mode_tearoff->tearoff_window().signal_key_press_event().connect (sigc::bind (sigc::ptr_fun (relay_key_press), &_mouse_mode_tearoff->tearoff_window()), false);
 
        if (Profile->get_sae()) {
-               mouse_mode_tearoff->set_can_be_torn_off (false);
+               _mouse_mode_tearoff->set_can_be_torn_off (false);
        }
 
-       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->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->Hidden.connect (sigc::bind (sigc::mem_fun(*this, &Editor::detach_tearoff), static_cast<Box*>(&toolbar_hbox),
-                                                 &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->Detach.connect (sigc::bind (sigc::mem_fun(*this, &Editor::detach_tearoff), static_cast<Box*>(&toolbar_hbox),
+                                                         &_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->Hidden.connect (sigc::bind (sigc::mem_fun(*this, &Editor::detach_tearoff), static_cast<Box*>(&toolbar_hbox),
+                                                         &_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_move_button.set_mode (false);
        mouse_select_button.set_mode (false);
@@ -2843,15 +2846,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_BUTTON))));
+       zoom_in_button.set_image (*(manage (new Image (Stock::ZOOM_IN, Gtk::ICON_SIZE_MENU))));
        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_BUTTON))));
+       zoom_out_button.set_image (*(manage (new Image (Stock::ZOOM_OUT, Gtk::ICON_SIZE_MENU))));
        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_BUTTON))));
+       zoom_out_full_button.set_image (*(manage (new Image (Stock::ZOOM_100, Gtk::ICON_SIZE_MENU))));
        zoom_out_full_button.signal_clicked().connect (sigc::mem_fun(*this, &Editor::temporal_zoom_session));
 
        zoom_focus_selector.set_name ("ZoomFocusSelector");
@@ -2923,28 +2926,28 @@ Editor::setup_toolbar ()
        HBox* hbox = manage (new HBox);
        hbox->set_spacing(10);
 
-       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);
-
+       _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);
+               _tools_tearoff->set_can_be_torn_off (false);
        }
 
-       tools_tearoff->Detach.connect (sigc::bind (sigc::mem_fun(*this, &Editor::detach_tearoff), static_cast<Box*>(&toolbar_hbox),
-                                            &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->Hidden.connect (sigc::bind (sigc::mem_fun(*this, &Editor::detach_tearoff), static_cast<Box*>(&toolbar_hbox),
-                                            &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->Detach.connect (sigc::bind (sigc::mem_fun(*this, &Editor::detach_tearoff), static_cast<Box*>(&toolbar_hbox),
+                                                    &_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->Hidden.connect (sigc::bind (sigc::mem_fun(*this, &Editor::detach_tearoff), static_cast<Box*>(&toolbar_hbox),
+                                                    &_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));
 
        toolbar_hbox.set_spacing (10);
        toolbar_hbox.set_border_width (1);
 
-       toolbar_hbox.pack_start (*mouse_mode_tearoff, false, false);
-       toolbar_hbox.pack_start (*tools_tearoff, false, false);
+       toolbar_hbox.pack_start (*_mouse_mode_tearoff, false, false);
+       toolbar_hbox.pack_start (*_tools_tearoff, false, false);
 
        hbox->pack_start (snap_box, false, false);
        hbox->pack_start (*nudge_box, false, false);
@@ -3139,7 +3142,7 @@ Editor::map_transport_state ()
 {
        ENSURE_GUI_THREAD (*this, &Editor::map_transport_state)
 
-       if (_session->transport_stopped()) {
+       if (_session && _session->transport_stopped()) {
                have_pending_keyboard_selection = false;
        }
 
@@ -3162,7 +3165,6 @@ void
 Editor::begin_reversible_command (string name)
 {
        if (_session) {
-               before = &get_state();
                _session->begin_reversible_command (name);
        }
 }
@@ -3171,7 +3173,7 @@ void
 Editor::commit_reversible_command ()
 {
        if (_session) {
-               _session->commit_reversible_command (new MementoCommand<Editor>(*this, before, &get_state()));
+               _session->commit_reversible_command ();
        }
 }
 
@@ -3246,8 +3248,8 @@ Editor::duplicate_dialog (bool with_dialog)
 
        if (with_dialog) {
 
-               ArdourDialog win ("Duplicate");
-               Label  label (_("Number of Duplications:"));
+               ArdourDialog win (_("Duplicate"));
+               Label label (_("Number of duplications:"));
                Adjustment adjustment (1.0, 1.0, 1000000.0, 1.0, 5.0);
                SpinButton spinner (adjustment, 0.0, 1);
                HBox hbox;
@@ -3653,10 +3655,7 @@ Editor::pane_allocation_handler (Allocation &alloc, Paned* which)
 void
 Editor::detach_tearoff (Box* /*b*/, Window* /*w*/)
 {
-       cerr << "remove tearoff\n";
-
-       if (tools_tearoff->torn_off() &&
-           mouse_mode_tearoff->torn_off()) {
+       if (_tools_tearoff->torn_off() && _mouse_mode_tearoff->torn_off()) {
                top_hbox.remove (toolbar_frame);
        }
 }
@@ -3664,7 +3663,6 @@ Editor::detach_tearoff (Box* /*b*/, Window* /*w*/)
 void
 Editor::reattach_tearoff (Box* /*b*/, Window* /*w*/, int32_t /*n*/)
 {
-       cerr << "reattach tearoff\n";
        if (toolbar_frame.get_parent() == 0) {
                top_hbox.pack_end (toolbar_frame);
        }
@@ -3707,6 +3705,29 @@ 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());
+       }
+}
+
+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 ();
+       }
+}
+
 void
 Editor::toggle_xfade_active (boost::weak_ptr<Crossfade> wxfade)
 {
@@ -3746,7 +3767,10 @@ Editor::edit_xfade (boost::weak_ptr<Crossfade> wxfade)
        }
 
        cew.apply ();
-       xfade->StateChanged (Change (~0));
+       PropertyChange all_crossfade_properties;
+       all_crossfade_properties.add (ARDOUR::Properties::active);
+       all_crossfade_properties.add (ARDOUR::Properties::follow_overlap);
+       xfade->PropertyChanged (all_crossfade_properties);
 }
 
 PlaylistSelector&
@@ -3820,19 +3844,10 @@ Editor::get_nudge_distance (nframes64_t pos, nframes64_t& next)
        return ret;
 }
 
-void
-Editor::end_location_changed (Location* location)
-{
-       ENSURE_GUI_THREAD (*this, &Editor::end_location_changed, location)
-       //reset_scrolling_region ();
-       nframes64_t session_span = location->start() + (nframes64_t) floorf (current_page_frames() * 0.10f);
-       horizontal_adjustment.set_upper (session_span / frames_per_unit);
-}
-
 int
 Editor::playlist_deletion_dialog (boost::shared_ptr<Playlist> pl)
 {
-       ArdourDialog dialog ("playlist deletion dialog");
+       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."),
@@ -3952,8 +3967,8 @@ Editor::session_state_saved (string snap_name)
 void
 Editor::maximise_editing_space ()
 {
-       mouse_mode_tearoff->set_visible (false);
-       tools_tearoff->set_visible (false);
+       _mouse_mode_tearoff->set_visible (false);
+       _tools_tearoff->set_visible (false);
 
        pre_maximal_pane_position = edit_pane.get_position();
        pre_maximal_editor_width = this->get_width();
@@ -3983,8 +3998,8 @@ Editor::restore_editing_space ()
 
        unfullscreen();
 
-       mouse_mode_tearoff->set_visible (true);
-       tools_tearoff->set_visible (true);
+       _mouse_mode_tearoff->set_visible (true);
+       _tools_tearoff->set_visible (true);
        post_maximal_editor_width = this->get_width();
 
        edit_pane.set_position (pre_maximal_pane_position + abs(this->get_width() - pre_maximal_editor_width));
@@ -4002,7 +4017,7 @@ Editor::new_playlists (TimeAxisView* v)
        begin_reversible_command (_("new playlists"));
        vector<boost::shared_ptr<ARDOUR::Playlist> > playlists;
        _session->playlists->get (playlists);
-       mapover_tracks (sigc::bind (sigc::mem_fun (*this, &Editor::mapped_use_new_playlist), playlists), v, RouteGroup::Edit);
+       mapover_tracks (sigc::bind (sigc::mem_fun (*this, &Editor::mapped_use_new_playlist), playlists), v, ARDOUR::Properties::edit.property_id);
        commit_reversible_command ();
 }
 
@@ -4018,7 +4033,7 @@ Editor::copy_playlists (TimeAxisView* v)
        begin_reversible_command (_("copy playlists"));
        vector<boost::shared_ptr<ARDOUR::Playlist> > playlists;
        _session->playlists->get (playlists);
-       mapover_tracks (sigc::bind (sigc::mem_fun (*this, &Editor::mapped_use_copy_playlist), playlists), v, RouteGroup::Edit);
+       mapover_tracks (sigc::bind (sigc::mem_fun (*this, &Editor::mapped_use_copy_playlist), playlists), v, ARDOUR::Properties::edit.property_id);
        commit_reversible_command ();
 }
 
@@ -4033,7 +4048,7 @@ Editor::clear_playlists (TimeAxisView* v)
        begin_reversible_command (_("clear playlists"));
        vector<boost::shared_ptr<ARDOUR::Playlist> > playlists;
        _session->playlists->get (playlists);
-       mapover_tracks (sigc::mem_fun (*this, &Editor::mapped_clear_playlist), v, RouteGroup::Edit);
+       mapover_tracks (sigc::mem_fun (*this, &Editor::mapped_clear_playlist), v, ARDOUR::Properties::edit.property_id);
        commit_reversible_command ();
 }
 
@@ -4092,7 +4107,6 @@ Editor::reset_zoom (double fpu)
 void
 Editor::reposition_and_zoom (nframes64_t frame, double fpu)
 {
-       //cerr << "Editor::reposition_and_zoom () called ha v:l:u:ps:fpu = " << horizontal_adjustment.get_value() << ":" << horizontal_adjustment.get_lower() << ":" << horizontal_adjustment.get_upper() << ":" << horizontal_adjustment.get_page_size() << ":" << frames_per_unit << endl;//DEBUG
        reset_x_origin (frame);
        reset_zoom (fpu);
 
@@ -4126,10 +4140,11 @@ Editor::undo_visual_state ()
                return;
        }
 
+       redo_visual_stack.push_back (current_visual_state());
+
        VisualState* vs = undo_visual_stack.back();
        undo_visual_stack.pop_back();
        use_visual_state (*vs);
-       redo_visual_stack.push_back (vs);
 }
 
 void
@@ -4139,10 +4154,11 @@ Editor::redo_visual_state ()
                return;
        }
 
+       undo_visual_stack.push_back (current_visual_state());
+
        VisualState* vs = redo_visual_stack.back();
        redo_visual_stack.pop_back();
        use_visual_state (*vs);
-       undo_visual_stack.push_back (vs);
 }
 
 void
@@ -4221,8 +4237,6 @@ Editor::set_frames_per_unit (double fpu)
 void
 Editor::post_zoom ()
 {
-       nframes64_t cef = 0;
-
        // convert fpu to frame count
 
        nframes64_t frames = (nframes64_t) floor (frames_per_unit * _canvas_width);
@@ -4237,15 +4251,8 @@ Editor::post_zoom ()
                }
        }
 
-       leftmost_frame = (nframes64_t) floor (horizontal_adjustment.get_value() * frames_per_unit);
-
        ZoomChanged (); /* EMIT_SIGNAL */
 
-       if (_session) {
-               cef = _session->current_end_frame() + (current_page_frames() / 10);// Add a little extra so we can see the end marker
-       }
-       horizontal_adjustment.set_upper (cef / frames_per_unit);
-
        //reset_scrolling_region ();
 
        if (playhead_cursor) {
@@ -4262,17 +4269,7 @@ void
 Editor::queue_visual_change (nframes64_t where)
 {
        pending_visual_change.add (VisualChange::TimeOrigin);
-
-       /* if we're moving beyond the end, make sure the upper limit of the horizontal adjustment
-          can reach.
-       */
-
-       if (_session && (where > _session->current_end_frame())) {
-               horizontal_adjustment.set_upper ((where + current_page_frames()) / frames_per_unit);
-       }
-
        pending_visual_change.time_origin = where;
-
        ensure_visual_change_idle_handler ();
 }
 
@@ -4314,7 +4311,7 @@ Editor::idle_visual_changer ()
        VisualChange::Type p = pending_visual_change.pending;
        pending_visual_change.pending = (VisualChange::Type) 0;
 
-       double last_time_origin = horizontal_adjustment.get_value();
+       double last_time_origin = horizontal_position ();
 
        if (p & VisualChange::ZoomLevel) {
                set_frames_per_unit (pending_visual_change.frames_per_unit);
@@ -4325,13 +4322,13 @@ Editor::idle_visual_changer ()
                update_tempo_based_rulers ();
        }
        if (p & VisualChange::TimeOrigin) {
-               horizontal_adjustment.set_value (pending_visual_change.time_origin / frames_per_unit);
+               set_horizontal_position (pending_visual_change.time_origin / frames_per_unit);
        }
        if (p & VisualChange::YOrigin) {
                vertical_adjustment.set_value (pending_visual_change.y_origin);
        }
 
-       if (last_time_origin == horizontal_adjustment.get_value()) {
+       if (last_time_origin == horizontal_position ()) {
                /* changed signal not emitted */
                update_fixed_rulers ();
                redisplay_tempo (true);
@@ -4339,7 +4336,6 @@ Editor::idle_visual_changer ()
 
        _summary->set_overlays_dirty ();
 
-       // cerr << "Editor::idle_visual_changer () called ha v:l:u:ps:fpu = " << horizontal_adjustment.get_value() << ":" << horizontal_adjustment.get_lower() << ":" << horizontal_adjustment.get_upper() << ":" << horizontal_adjustment.get_page_size() << ":" << frames_per_unit << endl;//DEBUG
        pending_visual_change.idle_handler_id = -1;
        return 0; /* this is always a one-shot call */
 }
@@ -4484,13 +4480,13 @@ 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<Diskstream> ds;
+                       boost::shared_ptr<Track> tr;
                        boost::shared_ptr<Playlist> pl;
 
-                       if ((ds = rtv->get_diskstream()) && ((pl = ds->playlist()))) {
+                       if ((tr = rtv->track()) && ((pl = tr->playlist()))) {
 
                                Playlist::RegionList* regions = pl->regions_at (
-                                               (nframes64_t) floor ( (double)where * ds->speed()));
+                                               (nframes64_t) floor ( (double)where * tr->speed()));
 
                                for (Playlist::RegionList::iterator i = regions->begin(); i != regions->end(); ++i) {
                                        RegionView* rv = rtv->view()->find_view (*i);
@@ -4519,13 +4515,13 @@ Editor::get_regions_after (RegionSelection& rs, nframes64_t where, const TrackVi
        for (TrackViewList::const_iterator t = tracks->begin(); t != tracks->end(); ++t) {
                RouteTimeAxisView* rtv = dynamic_cast<RouteTimeAxisView*>(*t);
                if (rtv) {
-                       boost::shared_ptr<Diskstream> ds;
+                       boost::shared_ptr<Track> tr;
                        boost::shared_ptr<Playlist> pl;
 
-                       if ((ds = rtv->get_diskstream()) && ((pl = ds->playlist()))) {
+                       if ((tr = rtv->track()) && ((pl = tr->playlist()))) {
 
                                Playlist::RegionList* regions = pl->regions_touched (
-                                               (nframes64_t) floor ( (double)where * ds->speed()), max_frames);
+                                               (nframes64_t) floor ( (double)where * tr->speed()), max_frames);
 
                                for (Playlist::RegionList::iterator i = regions->begin(); i != regions->end(); ++i) {
 
@@ -4571,7 +4567,7 @@ Editor::get_regions_for_action (RegionSelection& rs, bool allow_entered, bool al
                for (RegionSelection::iterator i = rs.begin (); i != rs.end(); ++i) {
 
                        RouteGroup* g = (*i)->get_time_axis_view().route_group ();
-                       if (g && g->active_property (RouteGroup::Edit)) {
+                       if (g && g->is_active() && g->is_edit()) {
                                tracks.add (axis_views_from_routes (g->route_list()));
                        }
                        
@@ -4597,14 +4593,14 @@ Editor::get_regions_corresponding_to (boost::shared_ptr<Region> region, vector<R
                        boost::shared_ptr<Playlist> pl;
                        vector<boost::shared_ptr<Region> > results;
                        RegionView* marv;
-                       boost::shared_ptr<Diskstream> ds;
+                       boost::shared_ptr<Track> tr;
 
-                       if ((ds = tatv->get_diskstream()) == 0) {
+                       if ((tr = tatv->track()) == 0) {
                                /* bus */
                                continue;
                        }
 
-                       if ((pl = (ds->playlist())) != 0) {
+                       if ((pl = (tr->playlist())) != 0) {
                                pl->get_region_list_equivalent_regions (region, results);
                        }
 
@@ -4647,7 +4643,7 @@ Editor::first_idle ()
 
        if (track_views.size() > 1) {
                dialog = new MessageDialog (*this,
-                                           _("Please wait while Ardour loads visual data"),
+                                           string_compose (_("Please wait while %1 loads visual data"), PROGRAM_NAME),
                                            true,
                                            Gtk::MESSAGE_INFO,
                                            Gtk::BUTTONS_NONE);
@@ -4667,8 +4663,8 @@ Editor::first_idle ()
        _have_idled = true;
 }
 
-static gboolean
-_idle_resizer (gpointer arg)
+gboolean
+Editor::_idle_resize (gpointer arg)
 {
        return ((Editor*)arg)->idle_resize ();
 }
@@ -4677,7 +4673,7 @@ void
 Editor::add_to_idle_resize (TimeAxisView* view, int32_t h)
 {
        if (resize_idle_id < 0) {
-               resize_idle_id = g_idle_add (_idle_resizer, this);
+               resize_idle_id = g_idle_add (_idle_resize, this);
                _pending_resize_amount = 0;
        }
 
@@ -4702,8 +4698,8 @@ Editor::add_to_idle_resize (TimeAxisView* view, int32_t h)
        }
 
        /* clamp */
-       if (uint32_t (min_resulting) < TimeAxisView::hSmall) {
-               _pending_resize_amount += TimeAxisView::hSmall - min_resulting;
+       if (uint32_t (min_resulting) < TimeAxisView::preset_height (HeightSmall)) {
+               _pending_resize_amount += TimeAxisView::preset_height (HeightSmall) - min_resulting;
        }
 }
 
@@ -4723,6 +4719,7 @@ Editor::idle_resize ()
                }
        }
 
+        _pending_resize_amount = 0;
        flush_canvas ();
        _group_tabs->set_dirty ();
        resize_idle_id = -1;
@@ -4799,7 +4796,7 @@ Editor::handle_new_route (RouteList& routes)
        for (RouteList::iterator x = routes.begin(); x != routes.end(); ++x) {
                boost::shared_ptr<Route> route = (*x);
 
-               if (route->is_hidden()) {
+               if (route->is_hidden() || route->is_monitor()) {
                        continue;
                }
 
@@ -4893,16 +4890,16 @@ Editor::timeaxisview_deleted (TimeAxisView *tv)
 }
 
 void
-Editor::hide_track_in_display (TimeAxisView& tv, bool /*temponly*/)
+Editor::hide_track_in_display (TimeAxisView* tv, bool /*temponly*/)
 {
-       RouteTimeAxisView* rtv = dynamic_cast<RouteTimeAxisView*> (&tv);
+       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);
+               set_selected_mixer_strip (*tv);
        }
 
-       _routes->hide_track_in_display (tv);
+       _routes->hide_track_in_display (*tv);
 }
 
 bool
@@ -4976,9 +4973,9 @@ Editor::hide_a_region (boost::shared_ptr<Region> r)
 }
 
 void
-Editor::remove_a_region (boost::shared_ptr<Region> r)
+Editor::show_a_region (boost::shared_ptr<Region> r)
 {
-       _session->remove_region_from_region_list (r);
+       r->set_hidden (false);
 }
 
 void
@@ -4993,6 +4990,12 @@ Editor::hide_region_from_region_list ()
        _regions->selection_mapover (sigc::mem_fun (*this, &Editor::hide_a_region));
 }
 
+void
+Editor::show_region_in_region_list ()
+{
+       _regions->selection_mapover (sigc::mem_fun (*this, &Editor::show_a_region));
+}
+
 void
 Editor::start_step_editing ()
 {
@@ -5116,6 +5119,7 @@ Editor::super_rapid_screen_update ()
         * 1.  we don't update things when there's a pending locate request, otherwise
         *     when the editor requests a locate there is a chance that this method
         *     will move the playhead before the locate request is processed, causing
+        *     a visual glitch.
         * 2.  if we're not rolling, there's nothing to do here (locates are handled elsewhere).
         * 3.  if we're still at the same frame that we were last time, there's nothing to do.
         */
@@ -5128,33 +5132,33 @@ Editor::super_rapid_screen_update ()
                        playhead_cursor->set_position (frame);
                }
 
-#undef CONTINUOUS_SCROLL
-#ifndef CONTINUOUS_SCROLL              
-
-               if (!_dragging_playhead && _follow_playhead && _session->requested_return_frame() < 0) {
-                       reset_x_origin_to_follow_playhead ();
-               }
+                if (!_stationary_playhead) {
 
-#else  // CONTINUOUS_SCROLL
-                               
-               /* don't do continuous scroll till the new position is in the rightmost quarter of the
-                  editor canvas
-               */
-               
-               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;
-               horizontal_adjustment.set_value (current);
+                        if (!_dragging_playhead && _follow_playhead && _session->requested_return_frame() < 0) {
+                                reset_x_origin_to_follow_playhead ();
+                        }
 
-#endif // CONTINUOUS_SCROLL
+                } 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                         
+                        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);
+#endif
+                }
                
        }
 }