new mouse mode (Draw (notes)); switch into internal mode when draw is selected; notab...
[ardour.git] / gtk2_ardour / editor.cc
index 8a6e78d9aafc48c8e4c8bffd81920688aa9b1b68..f679eecca659551cffc405f432788d33b487e349 100644 (file)
@@ -58,7 +58,6 @@
 #include "gtkmm2ext/choice.h"
 #include "gtkmm2ext/cell_renderer_pixbuf_toggle.h"
 
-#include "ardour/audio_diskstream.h"
 #include "ardour/audio_track.h"
 #include "ardour/audioplaylist.h"
 #include "ardour/audioregion.h"
 
 #include "control_protocol/control_protocol.h"
 
-#include "editor.h"
-#include "debug.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 "mixer_ui.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"
 
@@ -260,15 +262,13 @@ 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_("zoom range"), true, false, true))
 
        , toolbar_selection_clock_table (2,3)
 
        , automation_mode_button (_("mode"))
-       , global_automation_button (_("automation"))
 
        , _toolbar_viewport (*manage (new Gtk::Adjustment (0, 0, 1e10)), *manage (new Gtk::Adjustment (0, 0, 1e10)))
-       , midi_panic_button (_("Panic"))
 
 #ifdef WITH_CMT
        , image_socket_listener(0)
@@ -276,13 +276,14 @@ Editor::Editor ()
 
          /* nudge */
 
-       , nudge_clock (X_("nudge"), false, X_("NudgeClock"), true, false, true)
+       , nudge_clock (new AudioClock (X_("nudge"), false, X_("nudge"), true, false, true))
        , meters_running(false)
        , _pending_locate_request (false)
        , _pending_initial_locate (false)
        , _last_cut_copy_source_track (0)
 
        , _region_selection_change_updates_region_list (true)
+       , _following_mixer_selection (false)
 {
        constructed = false;
 
@@ -304,7 +305,6 @@ Editor::Editor ()
         pre_press_cursor = 0;
        _drags = new DragManager (this);
        current_mixer_strip = 0;
-       current_bbt_points = 0;
        tempo_lines = 0;
 
        snap_type_strings =  I18N (_snap_type_strings);
@@ -331,7 +331,6 @@ Editor::Editor ()
        current_interthread_info = 0;
        _show_measures = true;
        show_gain_after_trim = false;
-       last_item_entered = 0;
 
        have_pending_keyboard_selection = false;
        _follow_playhead = true;
@@ -364,7 +363,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;
 
@@ -388,7 +386,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);
@@ -421,35 +419,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);
@@ -459,14 +457,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));
 
@@ -492,10 +490,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);
@@ -568,12 +566,12 @@ Editor::Editor ()
        summary_arrows_left_left->add (*manage (new Arrow (ARROW_LEFT, SHADOW_NONE)));
        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::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);
@@ -582,26 +580,26 @@ Editor::Editor ()
        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_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);
@@ -637,9 +635,12 @@ Editor::Editor ()
        /* register actions now so that set_state() can find them and set toggles/checks etc */
 
        register_actions ();
+       /* when we start using our own keybinding system for the editor, this
+        * will be uncommented
+        */
+       // load_bindings ();
 
        setup_toolbar ();
-       setup_midi_toolbar ();
 
        _snap_type = SnapToBeat;
        set_snap_to (_snap_type);
@@ -703,8 +704,9 @@ Editor::Editor ()
        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());
+       ControlProtocol::SelectByRID.connect (*this, invalidator (*this), ui_bind (&Editor::control_select, 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));
@@ -870,6 +872,7 @@ Editor::show_window ()
 
                if (current_mixer_strip) {
                        current_mixer_strip->hide_things ();
+                       current_mixer_strip->parameter_changed ("mixer-strip-visibility");
                }
        }
 
@@ -897,19 +900,45 @@ 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);
 }
 
+void
+Editor::control_select (uint32_t rid) 
+{
+       /* handles the (static) signal from the ControlProtocol class that
+        * requests setting the selected track to a given RID
+        */
+        
+       if (!_session) {
+               return;
+       }
+
+       boost::shared_ptr<Route> r = _session->route_by_remote_id (rid);
+
+       if (!r) {
+               return;
+       }
+
+       TimeAxisView* tav = axis_view_from_route (r);
+
+       if (tav) {
+               selection->set (tav);
+       } else {
+               selection->clear_tracks ();
+       }
+}
+
 void
 Editor::control_scroll (float fraction)
 {
@@ -1085,9 +1114,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);
@@ -1110,6 +1139,44 @@ Editor::set_session (Session *t)
 
        compute_fixed_ruler_scale ();
 
+       /* Make sure we have auto loop and auto punch ranges */
+
+       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 {
+               // force name
+               loc->set_name (_("Loop"));
+       }
+
+       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 {
+               // force name
+               loc->set_name (_("Punch"));
+       }
+
+       refresh_location_display ();
+
+       /* This must happen after refresh_location_display(), as (amongst other things) we restore
+          the selected Marker; this needs the LocationMarker list to be available.
+       */
        XMLNode* node = ARDOUR_UI::instance()->editor_settings();
        set_state (*node, Stateful::loading_state_version);
 
@@ -1146,52 +1213,20 @@ 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);
 
        } else {
-               nudge_clock.set (_session->frame_rate() * 5, true, 0, AudioClock::Timecode); // default of 5 seconds
+               nudge_clock->set_mode (AudioClock::Timecode);
+               nudge_clock->set (_session->frame_rate() * 5, true);
        }
 
        playhead_cursor->canvas_item.show ();
 
-       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 {
-               // force name
-               loc->set_name (_("Loop"));
-       }
-
-       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 {
-               // force name
-               loc->set_name (_("Punch"));
-       }
-
        boost::function<void (string)> pc (boost::bind (&Editor::parameter_changed, this, _1));
        Config->map_parameters (pc);
        _session->config.map_parameters (pc);
 
-       refresh_location_display ();
-
        restore_ruler_visibility ();
        //tempo_map_changed (PropertyChange (0));
        _session->tempo_map().apply_with_metrics (*this, &Editor::draw_metric_marks);
@@ -1203,7 +1238,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:
@@ -1217,7 +1252,7 @@ Editor::set_session (Session *t)
        }
 
        /* register for undo history */
-       _session->register_with_memento_command_factory(_id, this);
+       _session->register_with_memento_command_factory(id(), this);
 
        ActionManager::ui_manager->signal_pre_activate().connect (sigc::mem_fun (*this, &Editor::action_pre_activated));
 
@@ -1269,10 +1304,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 (
@@ -1291,34 +1326,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 ();
                }
 
@@ -1355,34 +1390,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 ();
                }
 
@@ -1745,13 +1780,13 @@ void
 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.
        */
 
        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");
 
@@ -1776,14 +1811,14 @@ Editor::add_region_context_items (Menu_Helpers::MenuList& edit_items, boost::sha
           dialogue.  If we use the edit point it gets a bit messy because the user still has to click over
           *some* region in order to get the region context menu stuff to be displayed at all.
        */
-       
+
        framepos_t mouse;
        bool ignored;
        mouse_frame (mouse, ignored);
 
        edit_items.push_back (*_popup_region_menu_item);
        if (track->playlist()->count_regions_at (mouse) > 1 && (layering_order_editor == 0 || !layering_order_editor->is_visible ())) {
-               edit_items.push_back (*manage (_region_actions->get_action ("choose-top-region")->create_menu_item ()));
+               edit_items.push_back (*manage (_region_actions->get_action ("choose-top-region-context-menu")->create_menu_item ()));
        }
        edit_items.push_back (SeparatorElem());
 }
@@ -1817,7 +1852,7 @@ Editor::add_selection_context_items (Menu_Helpers::MenuList& edit_items)
                        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"),
@@ -1831,7 +1866,7 @@ Editor::add_selection_context_items (Menu_Helpers::MenuList& edit_items)
                        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)));
        edit_items.push_back (MenuElem (_("Convert to Region in Region List"), sigc::mem_fun(*this, &Editor::new_region_from_selection)));
@@ -1856,7 +1891,7 @@ Editor::add_selection_context_items (Menu_Helpers::MenuList& edit_items)
        edit_items.push_back (MenuElem (_("Consolidate Range With Processing"), sigc::bind (sigc::mem_fun(*this, &Editor::bounce_range_selection), true, true)));
        edit_items.push_back (MenuElem (_("Bounce Range to Region List"), sigc::bind (sigc::mem_fun(*this, &Editor::bounce_range_selection), false, false)));
        edit_items.push_back (MenuElem (_("Bounce Range to Region List With Processing"), sigc::bind (sigc::mem_fun(*this, &Editor::bounce_range_selection), false, true)));
-       edit_items.push_back (MenuElem (_("Export Range"), sigc::mem_fun(*this, &Editor::export_selection)));
+       edit_items.push_back (MenuElem (_("Export Range..."), sigc::mem_fun(*this, &Editor::export_selection)));
 }
 
 
@@ -1911,7 +1946,7 @@ Editor::add_dstream_context_items (Menu_Helpers::MenuList& edit_items)
 
        cutnpaste_items.push_back (MenuElem (_("Cut"), sigc::mem_fun(*this, &Editor::cut)));
        cutnpaste_items.push_back (MenuElem (_("Copy"), sigc::mem_fun(*this, &Editor::copy)));
-       cutnpaste_items.push_back (MenuElem (_("Paste"), sigc::bind (sigc::mem_fun(*this, &Editor::paste), 1.0f)));
+       cutnpaste_items.push_back (MenuElem (_("Paste"), sigc::bind (sigc::mem_fun(*this, &Editor::paste), 1.0f, true)));
 
        cutnpaste_items.push_back (SeparatorElem());
 
@@ -1982,7 +2017,7 @@ Editor::add_bus_context_items (Menu_Helpers::MenuList& edit_items)
 
        cutnpaste_items.push_back (MenuElem (_("Cut"), sigc::mem_fun(*this, &Editor::cut)));
        cutnpaste_items.push_back (MenuElem (_("Copy"), sigc::mem_fun(*this, &Editor::copy)));
-       cutnpaste_items.push_back (MenuElem (_("Paste"), sigc::bind (sigc::mem_fun(*this, &Editor::paste), 1.0f)));
+       cutnpaste_items.push_back (MenuElem (_("Paste"), sigc::bind (sigc::mem_fun(*this, &Editor::paste), 1.0f, true)));
 
        Menu *nudge_menu = manage (new Menu());
        MenuList& nudge_items = nudge_menu->items();
@@ -2130,19 +2165,15 @@ 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) {
-               _id = prop->value ();
-       }
+       set_id (node);
 
        g.base_width = default_width;
        g.base_height = default_height;
        x = 1;
        y = 1;
-       xoff = 0;
-       yoff = 21;
 
        if ((geometry = find_named_node (node, "geometry")) != 0) {
 
@@ -2174,24 +2205,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);
        move (x, y);
-        
+
        if (_session && (prop = node.property ("playhead"))) {
                framepos_t pos;
                sscanf (prop->value().c_str(), "%" PRIi64, &pos);
@@ -2199,7 +2217,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));
        }
@@ -2283,7 +2301,7 @@ Editor::set_state (const XMLNode& node, int /*version*/)
        }
 
        if ((prop = node.property ("stationary-playhead"))) {
-               bool yn = (prop->value() == "yes");
+               bool yn = string_is_affirmative (prop->value());
                set_stationary_playhead (yn);
                RefPtr<Action> act = ActionManager::get_action (X_("Editor"), X_("toggle-stationary-playhead"));
                if (act) {
@@ -2293,7 +2311,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);
@@ -2312,9 +2330,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);
        }
@@ -2326,9 +2344,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);
        }
@@ -2362,15 +2380,14 @@ Editor::get_state ()
        XMLNode* node = new XMLNode ("Editor");
        char buf[32];
 
-       _id.print (buf, sizeof (buf));
+       id().print (buf, sizeof (buf));
        node->add_property ("id", buf);
 
        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");
@@ -2383,10 +2400,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");
@@ -2446,13 +2459,13 @@ Editor::get_state ()
                 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;
 }
 
@@ -2461,14 +2474,14 @@ Editor::get_state ()
 /** @param y y offset from the top of all trackviews.
  *  @return pair: TimeAxisView that y is over, layer index.
  *  TimeAxisView may be 0.  Layer index is the layer number if the TimeAxisView is valid and is
- *  in stacked region display mode, otherwise 0.
+ *  in stacked or expanded region display mode, otherwise 0.
  */
-std::pair<TimeAxisView *, layer_t>
+std::pair<TimeAxisView *, double>
 Editor::trackview_by_y_position (double y)
 {
        for (TrackViewList::iterator iter = track_views.begin(); iter != track_views.end(); ++iter) {
 
-               std::pair<TimeAxisView*, int> const r = (*iter)->covers_y_position (y);
+               std::pair<TimeAxisView*, double> const r = (*iter)->covers_y_position (y);
                if (r.first) {
                        return r;
                }
@@ -2749,6 +2762,7 @@ Editor::setup_toolbar ()
        /* table containing mode buttons */
 
        HBox* mouse_mode_button_box = manage (new HBox ());
+       mouse_mode_button_box->set_spacing (2);
 
        if (Profile->get_sae()) {
                mouse_mode_button_box->pack_start (mouse_move_button);
@@ -2766,6 +2780,7 @@ Editor::setup_toolbar ()
 
        mouse_mode_button_box->pack_start (mouse_timefx_button);
        mouse_mode_button_box->pack_start (mouse_audition_button);
+       mouse_mode_button_box->pack_start (mouse_draw_button);
        mouse_mode_button_box->pack_start (internal_edit_button);
 
        edit_mode_strings.push_back (edit_mode_to_string (Slide));
@@ -2775,11 +2790,11 @@ Editor::setup_toolbar ()
        edit_mode_strings.push_back (edit_mode_to_string (Lock));
 
        edit_mode_selector.set_name ("EditModeSelector");
-       set_popdown_strings (edit_mode_selector, edit_mode_strings, true);
+       set_popdown_strings (edit_mode_selector, edit_mode_strings);
        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_box);
+       mode_box->pack_start (edit_mode_selector, false, false);
+       mode_box->pack_start (*mouse_mode_button_box, false, false);
 
        _mouse_mode_tearoff = manage (new TearOff (*mode_box));
        _mouse_mode_tearoff->set_name ("MouseModeBase");
@@ -2798,64 +2813,46 @@ Editor::setup_toolbar ()
        _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);
-       mouse_gain_button.set_mode (false);
-       mouse_zoom_button.set_mode (false);
-       mouse_timefx_button.set_mode (false);
-       mouse_audition_button.set_mode (false);
-       join_object_range_button.set_mode (false);
-
-       mouse_move_button.set_name ("MouseModeButton");
-       mouse_select_button.set_name ("MouseModeButton");
-       mouse_gain_button.set_name ("MouseModeButton");
-       mouse_zoom_button.set_name ("MouseModeButton");
-       mouse_timefx_button.set_name ("MouseModeButton");
-       mouse_audition_button.set_name ("MouseModeButton");
-       internal_edit_button.set_name ("MouseModeButton");
-       join_object_range_button.set_name ("MouseModeButton");
-
-       mouse_move_button.unset_flags (CAN_FOCUS);
-       mouse_select_button.unset_flags (CAN_FOCUS);
-       mouse_gain_button.unset_flags (CAN_FOCUS);
-       mouse_zoom_button.unset_flags (CAN_FOCUS);
-       mouse_timefx_button.unset_flags (CAN_FOCUS);
-       mouse_audition_button.unset_flags (CAN_FOCUS);
-       internal_edit_button.unset_flags (CAN_FOCUS);
-       join_object_range_button.unset_flags (CAN_FOCUS);
-
        /* Zoom */
 
-       _zoom_box.set_spacing (1);
-       _zoom_box.set_border_width (0);
+       _zoom_box.set_spacing (2);
+       _zoom_box.set_border_width (2);
 
-       zoom_in_button.set_name ("EditorTimeButton");
-       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));
+       RefPtr<Action> act;
+
+       zoom_in_button.set_name ("zoom button");
+       zoom_in_button.set_image (::get_icon ("zoom_in"));
+       zoom_in_button.set_tweaks (ArdourButton::ShowClick);
+       act = ActionManager::get_action (X_("Editor"), X_("temporal-zoom-in"));
+       zoom_in_button.set_related_action (act);
 
-       zoom_out_button.set_name ("EditorTimeButton");
-       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_button.set_name ("zoom button");
+       zoom_out_button.set_image (::get_icon ("zoom_out"));
+       zoom_out_button.set_tweaks (ArdourButton::ShowClick);
+       act = ActionManager::get_action (X_("Editor"), X_("temporal-zoom-out"));
+       zoom_out_button.set_related_action (act);
 
-       zoom_out_full_button.set_name ("EditorTimeButton");
-       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_out_full_button.set_name ("zoom button");
+       zoom_out_full_button.set_image (::get_icon ("zoom_full"));
+       zoom_out_full_button.set_tweaks (ArdourButton::ShowClick);
+       act = ActionManager::get_action (X_("Editor"), X_("zoom-to-session"));
+       zoom_out_full_button.set_related_action (act);
 
        zoom_focus_selector.set_name ("ZoomFocusSelector");
-       set_popdown_strings (zoom_focus_selector, zoom_focus_strings, true);
+       set_popdown_strings (zoom_focus_selector, zoom_focus_strings);
        zoom_focus_selector.signal_changed().connect (sigc::mem_fun(*this, &Editor::zoom_focus_selection_done));
 
        _zoom_box.pack_start (zoom_out_button, false, false);
        _zoom_box.pack_start (zoom_in_button, false, false);
        _zoom_box.pack_start (zoom_out_full_button, false, false);
 
-       _zoom_box.pack_start (zoom_focus_selector);
-       
+       _zoom_box.pack_start (zoom_focus_selector, false, false);
+
        /* 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")))));
-       RefPtr<Action> act = ActionManager::get_action (X_("Editor"), X_("expand-tracks"));
+       act = ActionManager::get_action (X_("Editor"), X_("expand-tracks"));
        act->connect_proxy (tav_expand_button);
 
        tav_shrink_button.set_name ("TrackHeightButton");
@@ -2866,7 +2863,7 @@ Editor::setup_toolbar ()
 
        _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),
@@ -2877,20 +2874,20 @@ 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);
 
        snap_type_selector.set_name ("SnapTypeSelector");
-       set_popdown_strings (snap_type_selector, snap_type_strings, true);
+       set_popdown_strings (snap_type_selector, snap_type_strings);
        snap_type_selector.signal_changed().connect (sigc::mem_fun(*this, &Editor::snap_type_selection_done));
 
        snap_mode_selector.set_name ("SnapModeSelector");
-       set_popdown_strings (snap_mode_selector, snap_mode_strings, true);
+       set_popdown_strings (snap_mode_selector, snap_mode_strings);
        snap_mode_selector.signal_changed().connect (sigc::mem_fun(*this, &Editor::snap_mode_selection_done));
 
        edit_point_selector.set_name ("EditPointSelector");
-       set_popdown_strings (edit_point_selector, edit_point_strings, true);
+       set_popdown_strings (edit_point_selector, edit_point_strings);
        edit_point_selector.signal_changed().connect (sigc::mem_fun(*this, &Editor::edit_point_selection_done));
 
        snap_box.pack_start (snap_mode_selector, false, false);
@@ -2900,7 +2897,7 @@ Editor::setup_toolbar ()
        /* Nudge */
 
        HBox *nudge_box = manage (new HBox);
-       nudge_box->set_spacing(1);
+       nudge_box->set_spacing (2);
        nudge_box->set_border_width (2);
 
        nudge_forward_button.signal_button_release_event().connect (sigc::mem_fun(*this, &Editor::nudge_forward_release), false);
@@ -2908,7 +2905,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... */
@@ -2919,7 +2916,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);
        }
@@ -2960,14 +2957,14 @@ 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));
 }
 
 void
 Editor::setup_tooltips ()
 {
        ARDOUR_UI::instance()->set_tip (mouse_move_button, _("Select/Move Objects"));
+       ARDOUR_UI::instance()->set_tip (mouse_select_button, _("Select/Move Ranges"));
+       ARDOUR_UI::instance()->set_tip (mouse_draw_button, _("Draw/Edit MIDI Notes"));
        ARDOUR_UI::instance()->set_tip (mouse_gain_button, _("Draw Region Gain"));
        ARDOUR_UI::instance()->set_tip (mouse_zoom_button, _("Select Zoom Range"));
        ARDOUR_UI::instance()->set_tip (mouse_timefx_button, _("Stretch/Shrink Regions and MIDI Notes"));
@@ -2986,40 +2983,9 @@ Editor::setup_tooltips ()
        ARDOUR_UI::instance()->set_tip (snap_type_selector, _("Snap/Grid Units"));
        ARDOUR_UI::instance()->set_tip (snap_mode_selector, _("Snap/Grid Mode"));
        ARDOUR_UI::instance()->set_tip (edit_point_selector, _("Edit point"));
-       ARDOUR_UI::instance()->set_tip (midi_sound_notes, _("Sound Notes"));
-       ARDOUR_UI::instance()->set_tip (midi_panic_button, _("Send note off and reset controller messages on all MIDI channels"));
        ARDOUR_UI::instance()->set_tip (edit_mode_selector, _("Edit Mode"));
 }
 
-void
-Editor::midi_panic ()
-{
-       cerr << "MIDI panic\n";
-
-       if (_session) {
-               _session->midi_panic();
-       }
-}
-
-void
-Editor::setup_midi_toolbar ()
-{
-       RefPtr<Action> act;
-
-       /* Midi sound notes */
-       midi_sound_notes.add (*(manage (new Image (::get_icon("midi_sound_notes")))));
-       midi_sound_notes.unset_flags (CAN_FOCUS);
-
-       /* Panic */
-
-       act = ActionManager::get_action (X_("MIDI"), X_("panic"));
-       midi_panic_button.set_name("MidiPanicButton");
-       act->connect_proxy (midi_panic_button);
-
-       panic_box.pack_start (midi_sound_notes , true, true);
-       panic_box.pack_start (midi_panic_button, true, true);
-}
-
 int
 Editor::convert_drop_to_paths (
                vector<string>&                paths,
@@ -3033,7 +2999,7 @@ Editor::convert_drop_to_paths (
        if (_session == 0) {
                return -1;
        }
-        
+
        vector<string> uris = data.get_uris();
 
        if (uris.empty()) {
@@ -3192,9 +3158,9 @@ Editor::history_changed ()
 
        if (undo_action && _session) {
                if (_session->undo_depth() == 0) {
-                       label = _("Undo");
+                       label = S_("Command|Undo");
                } else {
-                       label = string_compose(_("Undo (%1)"), _session->next_undo());
+                       label = string_compose(S_("Command|Undo (%1)"), _session->next_undo());
                }
                undo_action->property_label() = label;
        }
@@ -3479,7 +3445,7 @@ Editor::edit_controls_button_release (GdkEventButton* ev)
        } else if (ev->button == 1) {
                selection->clear_tracks ();
        }
-       
+
        return true;
 }
 
@@ -3527,7 +3493,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,
@@ -3535,23 +3500,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)) {
 
@@ -3583,7 +3533,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) {
@@ -3610,7 +3560,9 @@ Editor::pane_allocation_handler (Allocation &alloc, Paned* which)
 void
 Editor::detach_tearoff (Box* /*b*/, Window* /*w*/)
 {
-       if (_tools_tearoff->torn_off() && _mouse_mode_tearoff->torn_off()) {
+       if ((_tools_tearoff->torn_off() || !_tools_tearoff->visible()) && 
+           (_mouse_mode_tearoff->torn_off() || !_mouse_mode_tearoff->visible()) && 
+           (_zoom_tearoff->torn_off() || !_zoom_tearoff->visible())) {
                top_hbox.remove (toolbar_frame);
        }
 }
@@ -3795,7 +3747,7 @@ Editor::get_grid_type_as_beats (bool& success, framepos_t position)
 
        case SnapToBar:
                if (_session) {
-                       return _session->tempo_map().meter_at (position).beats_per_bar();
+                       return _session->tempo_map().meter_at (position).divisions_per_bar();
                }
                break;
 
@@ -3822,7 +3774,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;
@@ -3942,13 +3894,16 @@ Editor::control_layout_scroll (GdkEventScroll* ev)
 void
 Editor::session_state_saved (string)
 {
-       update_title ();        
+       update_title ();
        _snapshots->redisplay ();
 }
 
 void
 Editor::maximise_editing_space ()
 {
+       /* these calls will leave each tearoff visible *if* it is torn off
+        */
+
        _mouse_mode_tearoff->set_visible (false);
        _tools_tearoff->set_visible (false);
        _zoom_tearoff->set_visible (false);
@@ -3965,7 +3920,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) {
@@ -3975,12 +3930,16 @@ Editor::maximise_editing_space ()
                edit_pane.set_position (post_maximal_horizontal_pane_position);
        }
 
-       if (post_maximal_editor_height) {
-               editor_summary_pane.set_position (post_maximal_vertical_pane_position -
-                       abs(post_maximal_editor_height - pre_maximal_editor_height));
-       } else {
-               editor_summary_pane.set_position (post_maximal_vertical_pane_position);
-       }
+       /* Hack: we must do this in an idle handler for it to work; see comment in
+          restore_editing_space()
+       */
+          
+       Glib::signal_idle().connect (
+               sigc::bind (
+                       sigc::mem_fun (*this, &Editor::idle_reset_vertical_pane_position),
+                       post_maximal_vertical_pane_position
+                       )
+               );
 
        if (Config->get_keep_tearoffs()) {
                _mouse_mode_tearoff->set_visible (true);
@@ -3992,6 +3951,13 @@ Editor::maximise_editing_space ()
 
 }
 
+bool
+Editor::idle_reset_vertical_pane_position (int p)
+{
+       editor_summary_pane.set_position (p);
+       return false;
+}
+
 void
 Editor::restore_editing_space ()
 {
@@ -4004,7 +3970,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);
@@ -4016,7 +3982,17 @@ Editor::restore_editing_space ()
        post_maximal_editor_height = this->get_height();
 
        edit_pane.set_position (pre_maximal_horizontal_pane_position + abs(this->get_width() - pre_maximal_editor_width));
-       editor_summary_pane.set_position (pre_maximal_vertical_pane_position + abs(this->get_height() - pre_maximal_editor_height));
+
+       /* This is a bit of a hack, but it seems that if you set the vertical pane position
+          here it gets reset to some wrong value after this method has finished.  Doing
+          the setup in an idle callback seems to work.
+       */
+       Glib::signal_idle().connect (
+               sigc::bind (
+                       sigc::mem_fun (*this, &Editor::idle_reset_vertical_pane_position),
+                       pre_maximal_vertical_pane_position
+                       )
+               );
 }
 
 /**
@@ -4129,6 +4105,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)
 {
@@ -4138,10 +4124,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;
@@ -4196,22 +4180,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;
@@ -4255,8 +4231,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 ()) {
@@ -4375,24 +4351,23 @@ struct EditorOrderTimeAxisSorter {
 };
 
 void
-Editor::sort_track_selection (TrackViewList* sel)
+Editor::sort_track_selection (TrackViewList& sel)
 {
        EditorOrderTimeAxisSorter cmp;
-
-       if (sel) {
-               sel->sort (cmp);
-       } else {
-               selection->tracks.sort (cmp);
-       }
+       sel.sort (cmp);
 }
 
 framepos_t
-Editor::get_preferred_edit_position (bool ignore_playhead)
+Editor::get_preferred_edit_position (bool ignore_playhead, bool from_context_menu)
 {
        bool ignored;
        framepos_t where = 0;
        EditPoint ep = _edit_point;
 
+       if (from_context_menu && (ep == EditAtMouse)) {
+               return  event_frame (&context_click_event, 0, 0);
+       }
+
        if (entered_marker) {
                 DEBUG_TRACE (DEBUG::CutNPaste, string_compose ("GPEP: use entered marker @ %1\n", entered_marker->position()));
                return entered_marker->position();
@@ -4519,7 +4494,7 @@ Editor::get_regions_at (RegionSelection& rs, framepos_t where, const TrackViewLi
 
                        if ((tr = rtv->track()) && ((pl = tr->playlist()))) {
 
-                               Playlist::RegionList* regions = pl->regions_at (
+                               boost::shared_ptr<Playlist::RegionList> regions = pl->regions_at (
                                                (framepos_t) floor ( (double) where * tr->speed()));
 
                                for (Playlist::RegionList::iterator i = regions->begin(); i != regions->end(); ++i) {
@@ -4528,8 +4503,6 @@ Editor::get_regions_at (RegionSelection& rs, framepos_t where, const TrackViewLi
                                                rs.add (rv);
                                        }
                                }
-
-                               delete regions;
                        }
                }
        }
@@ -4554,7 +4527,7 @@ Editor::get_regions_after (RegionSelection& rs, framepos_t where, const TrackVie
 
                        if ((tr = rtv->track()) && ((pl = tr->playlist()))) {
 
-                               Playlist::RegionList* regions = pl->regions_touched (
+                               boost::shared_ptr<Playlist::RegionList> regions = pl->regions_touched (
                                        (framepos_t) floor ( (double)where * tr->speed()), max_framepos);
 
                                for (Playlist::RegionList::iterator i = regions->begin(); i != regions->end(); ++i) {
@@ -4565,8 +4538,6 @@ Editor::get_regions_after (RegionSelection& rs, framepos_t where, const TrackVie
                                                rs.push_back (rv);
                                        }
                                }
-
-                               delete regions;
                        }
                }
        }
@@ -4613,7 +4584,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 {
@@ -4630,14 +4601,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 ();
@@ -4656,7 +4627,7 @@ RegionSelection
 Editor::get_regions_from_selection_and_entered ()
 {
        RegionSelection regions = selection->regions;
-       
+
        if (regions.empty() && entered_regionview) {
                regions.add (entered_regionview);
        }
@@ -4713,13 +4684,13 @@ void
 Editor::first_idle ()
 {
        MessageDialog* dialog = 0;
-
+       
        if (track_views.size() > 1) {
-               dialog = new MessageDialog (*this,
-                                           string_compose (_("Please wait while %1 loads visual data"), PROGRAM_NAME),
-                                           true,
-                                           Gtk::MESSAGE_INFO,
-                                           Gtk::BUTTONS_NONE);
+               dialog = new MessageDialog (
+                       *this,
+                       string_compose (_("Please wait while %1 loads visual data."), PROGRAM_NAME),
+                       true
+                       );
                dialog->present ();
                ARDOUR_UI::instance()->flush_pending ();
        }
@@ -4732,7 +4703,6 @@ Editor::first_idle ()
        _routes->redisplay ();
 
        delete dialog;
-
        _have_idled = true;
 }
 
@@ -4876,9 +4846,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();
                }
@@ -4919,7 +4891,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) {
@@ -4947,7 +4919,7 @@ Editor::timeaxisview_deleted (TimeAxisView *tv)
 
        if (rtav) {
                route = rtav->route ();
-       } 
+       }
 
        if (current_mixer_strip && current_mixer_strip->route() == route) {
 
@@ -4960,18 +4932,18 @@ 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
@@ -4982,19 +4954,19 @@ Editor::hide_track_in_display (TimeAxisView* tv, bool apply_to_selection)
 
                        TrackSelection::iterator j = i;
                        ++j;
-                       
+
                        hide_track_in_display (*i, false);
-                       
+
                        i = j;
                }
        } else {
                RouteTimeAxisView* rtv = dynamic_cast<RouteTimeAxisView*> (tv);
-               
+
                if (rtv && current_mixer_strip && (rtv->route() == current_mixer_strip->route())) {
                        // this will hide the mixer strip
                        set_selected_mixer_strip (*tv);
                }
-               
+
                _routes->hide_track_in_display (*tv);
        }
 }
@@ -5020,7 +4992,7 @@ Editor::foreach_time_axis_view (sigc::slot<void,TimeAxisView&> theslot)
 
 /** Find a RouteTimeAxisView by the ID of its route */
 RouteTimeAxisView*
-Editor::get_route_view_by_route_id (PBD::ID& id) const
+Editor::get_route_view_by_route_id (const PBD::ID& id) const
 {
        RouteTimeAxisView* v;
 
@@ -5133,7 +5105,7 @@ bool
 Editor::scroll_press (Direction dir)
 {
        ++_scroll_callbacks;
-       
+
        if (_scroll_connection.connected() && _scroll_callbacks < 5) {
                /* delay the first auto-repeat */
                return true;
@@ -5163,7 +5135,7 @@ Editor::scroll_press (Direction dir)
                _scroll_connection = Glib::signal_timeout().connect (
                        sigc::bind (sigc::mem_fun (*this, &Editor::scroll_press), dir), 100
                        );
-               
+
                _scroll_callbacks = 0;
        }
 
@@ -5185,15 +5157,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;
@@ -5204,11 +5176,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 */
@@ -5277,12 +5249,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;
@@ -5292,12 +5264,12 @@ Editor::super_rapid_screen_update ()
                        } else {
                                /* relax */
                        }
-                        
+
                        current = target;
                        set_horizontal_position (current);
 #endif
                }
-               
+
        }
 }
 
@@ -5310,7 +5282,7 @@ Editor::session_going_away ()
        _session_connections.drop_connections ();
 
        super_rapid_screen_update_connection.disconnect ();
-       
+
        selection->clear ();
        cut_buffer->clear ();
 
@@ -5350,8 +5322,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);
@@ -5361,8 +5333,7 @@ Editor::session_going_away ()
        hide_measures ();
        clear_marker_display ();
 
-       delete current_bbt_points;
-       current_bbt_points = 0;
+       current_bbt_points_begin = current_bbt_points_end;
 
        /* get rid of any existing editor mixer strip */
 
@@ -5386,10 +5357,10 @@ Editor::show_editor_list (bool yn)
 }
 
 void
-Editor::change_region_layering_order ()
+Editor::change_region_layering_order (bool from_context_menu)
 {
-       framepos_t const position = get_preferred_edit_position ();
-       
+       const framepos_t position = get_preferred_edit_position (false, from_context_menu);
+
        if (!clicked_routeview) {
                if (layering_order_editor) {
                        layering_order_editor->hide ();
@@ -5408,9 +5379,10 @@ Editor::change_region_layering_order ()
        if (!pl) {
                return;
        }
-                
+
        if (layering_order_editor == 0) {
-               layering_order_editor = new RegionLayeringOrderEditor(*this);
+               layering_order_editor = new RegionLayeringOrderEditor (*this);
+               layering_order_editor->set_position (WIN_POS_MOUSE);
        }
 
        layering_order_editor->set_context (clicked_routeview->name(), _session, pl, position);
@@ -5421,7 +5393,7 @@ void
 Editor::update_region_layering_order_editor ()
 {
        if (layering_order_editor && layering_order_editor->is_visible ()) {
-               change_region_layering_order ();
+               change_region_layering_order (true);
        }
 }
 
@@ -5448,20 +5420,10 @@ Editor::action_menu_item (std::string const & name)
 {
        Glib::RefPtr<Action> a = editor_actions->get_action (name);
        assert (a);
-       
+
        return *manage (a->create_menu_item ());
 }
 
-void
-Editor::resize_text_widgets ()
-{
-        set_size_request_to_display_given_text (edit_mode_selector, edit_mode_strings, COMBO_FUDGE+10, 15);
-        set_size_request_to_display_given_text (zoom_focus_selector, zoom_focus_strings, COMBO_FUDGE+10, 15);
-        set_size_request_to_display_given_text (snap_type_selector, snap_type_strings, COMBO_FUDGE+10, 15);
-        set_size_request_to_display_given_text (snap_mode_selector, snap_mode_strings, COMBO_FUDGE+10, 15);
-        set_size_request_to_display_given_text (edit_point_selector, edit_point_strings, COMBO_FUDGE+10, 15);
-}
-        
 void
 Editor::add_notebook_page (string const & name, Gtk::Widget& widget)
 {
@@ -5498,3 +5460,20 @@ Editor::notebook_tab_clicked (GdkEventButton* ev, Gtk::Widget* page)
        return true;
 }
 
+void
+Editor::popup_control_point_context_menu (ArdourCanvas::Item* item, GdkEvent* event)
+{
+       using namespace Menu_Helpers;
+       
+       MenuList& items = _control_point_context_menu.items ();
+       items.clear ();
+       
+       items.push_back (MenuElem (_("Edit..."), sigc::bind (sigc::mem_fun (*this, &Editor::edit_control_point), item)));
+       items.push_back (MenuElem (_("Delete"), sigc::bind (sigc::mem_fun (*this, &Editor::remove_control_point), item)));
+       if (!can_remove_control_point (item)) {
+               items.back().set_sensitive (false);
+       }
+
+       _control_point_context_menu.popup (event->button.button, event->button.time);
+}
+