the endless quest to plug memory leaks -- episode 379
[ardour.git] / gtk2_ardour / editor.cc
index aed4330c4687e188167e8a50c4ba3bac2c5aee53..757b767b7ed338897c0b33b0e85578b5abc1d21d 100644 (file)
 #include "gtkmm2ext/bindings.h"
 #include "gtkmm2ext/grouped_buttons.h"
 #include "gtkmm2ext/gtk_ui.h"
-#include "gtkmm2ext/tearoff.h"
+#include <gtkmm2ext/keyboard.h>
 #include "gtkmm2ext/utils.h"
 #include "gtkmm2ext/window_title.h"
 #include "gtkmm2ext/choice.h"
 #include "gtkmm2ext/cell_renderer_pixbuf_toggle.h"
 
+#include "ardour/analysis_graph.h"
 #include "ardour/audio_track.h"
 #include "ardour/audioengine.h"
 #include "ardour/audioregion.h"
 #include "ardour/lmath.h"
 #include "ardour/location.h"
 #include "ardour/profile.h"
+#include "ardour/route.h"
 #include "ardour/route_group.h"
 #include "ardour/session_playlists.h"
 #include "ardour/tempo.h"
 #include "ardour/utils.h"
+#include "ardour/vca_manager.h"
+#include "ardour/vca.h"
 
 #include "canvas/debug.h"
 #include "canvas/text.h"
 #include "editor_routes.h"
 #include "editor_snapshots.h"
 #include "editor_summary.h"
+#include "export_report.h"
 #include "global_port_matrix.h"
 #include "gui_object.h"
 #include "gui_thread.h"
 #include "keyboard.h"
+#include "keyeditor.h"
+#include "luainstance.h"
 #include "marker.h"
 #include "midi_region_view.h"
 #include "midi_time_axis.h"
 #include "region_layering_order_editor.h"
 #include "rgb_macros.h"
 #include "rhythm_ferret.h"
+#include "route_sorter.h"
 #include "selection.h"
+#include "simple_progress_dialog.h"
 #include "sfdb_ui.h"
 #include "tempo_lines.h"
 #include "time_axis_view.h"
 #include "tooltips.h"
 #include "ui_config.h"
 #include "utils.h"
+#include "vca_time_axis.h"
 #include "verbose_cursor.h"
 
-#include "i18n.h"
+#include "pbd/i18n.h"
 
 using namespace std;
 using namespace ARDOUR;
@@ -207,8 +217,8 @@ static const gchar *_zoom_focus_strings[] = {
        N_("Right"),
        N_("Center"),
        N_("Playhead"),
-       N_("Mouse"),
-       N_("Edit point"),
+       N_("Mouse"),
+       N_("Edit point"),
        0
 };
 
@@ -227,30 +237,6 @@ static const gchar *_rb_opt_strings[] = {
 
 #define COMBO_TRIANGLE_WIDTH 25 // ArdourButton _diameter (11) + 2 * arrow-padding (2*2) + 2 * text-padding (2*5)
 
-static void
-pane_size_watcher (Paned* pane)
-{
-       /* if the handle of a pane vanishes into (at least) the tabs of a notebook,
-          it is:
-
-             X: hard to access
-             Quartz: impossible to access
-
-          so stop that by preventing it from ever getting too narrow. 35
-          pixels is basically a rough guess at the tab width.
-
-          ugh.
-       */
-
-       int max_width_of_lhs = GTK_WIDGET(pane->gobj())->allocation.width - 35;
-
-       gint pos = pane->get_position ();
-
-       if (pos > max_width_of_lhs) {
-               pane->set_position (max_width_of_lhs);
-       }
-}
-
 Editor::Editor ()
        : PublicEditor (global_hpacker)
        , editor_mixer_strip_width (Wide)
@@ -287,12 +273,10 @@ Editor::Editor ()
        , track_edit_playlist_submenu (0)
        , track_selection_edit_playlist_submenu (0)
        , _popup_region_menu_item (0)
-       , global_vpacker (key_bindings)
        , _track_canvas (0)
        , _track_canvas_viewport (0)
        , within_track_canvas (false)
        , _verbose_cursor (0)
-       , logo_item (0)
        , tempo_group (0)
        , meter_group (0)
        , marker_group (0)
@@ -388,7 +372,8 @@ Editor::Editor ()
        , tempo_lines (0)
        , global_rect_group (0)
        , time_line_group (0)
-       , tempo_or_meter_marker_menu (0)
+       , tempo_marker_menu (0)
+       , meter_marker_menu (0)
        , marker_menu (0)
        , range_marker_menu (0)
        , transport_marker_menu (0)
@@ -398,10 +383,7 @@ Editor::Editor ()
        , bbt_beat_subdivision (4)
        , _visible_track_count (-1)
        ,  toolbar_selection_clock_table (2,3)
-       , _mouse_mode_tearoff (0)
        ,  automation_mode_button (_("mode"))
-       , _zoom_tearoff (0)
-       , _tools_tearoff (0)
        ,  _toolbar_viewport (*manage (new Gtk::Adjustment (0, 0, 1e10)), *manage (new Gtk::Adjustment (0, 0, 1e10)))
        , selection (new Selection (this))
        , cut_buffer (new Selection (this))
@@ -436,6 +418,7 @@ Editor::Editor ()
        , show_gain_after_trim (false)
        , selection_op_cmd_depth (0)
        , selection_op_history_it (0)
+       , no_save_instant (false)
        , current_timefx (0)
        , current_mixer_strip (0)
        , show_editor_mixer_when_tracks_arrive (false)
@@ -466,6 +449,7 @@ Editor::Editor ()
        , _stepping_axis_view (0)
        , quantize_dialog (0)
        , _main_menu_disabler (0)
+       , myactions (X_("editor"))
 {
        /* we are a singleton */
 
@@ -496,16 +480,17 @@ Editor::Editor ()
        build_snap_type_menu();
        build_edit_point_menu();
 
-       location_marker_color = ARDOUR_UI::config()->color ("location marker");
-       location_range_color = ARDOUR_UI::config()->color ("location range");
-       location_cd_marker_color = ARDOUR_UI::config()->color ("location cd marker");
-       location_loop_color = ARDOUR_UI::config()->color ("location loop");
-       location_punch_color = ARDOUR_UI::config()->color ("location punch");
+       location_marker_color = UIConfiguration::instance().color ("location marker");
+       location_range_color = UIConfiguration::instance().color ("location range");
+       location_cd_marker_color = UIConfiguration::instance().color ("location cd marker");
+       location_loop_color = UIConfiguration::instance().color ("location loop");
+       location_punch_color = UIConfiguration::instance().color ("location punch");
 
        timebar_height = std::max(12., ceil (15. * ARDOUR_UI::ui_scale));
 
        TimeAxisView::setup_sizes ();
        ArdourMarker::setup_sizes (timebar_height);
+       TempoCurve::setup_sizes (timebar_height);
 
        bbt_label.set_name ("EditorRulerLabel");
        bbt_label.set_size_request (-1, (int)timebar_height);
@@ -679,7 +664,18 @@ Editor::Editor ()
 
        _notebook_shrunk = false;
 
-       editor_summary_pane.pack1(edit_packer);
+
+       /* Pick up some settings we need to cache, early */
+
+       XMLNode* settings = ARDOUR_UI::instance()->editor_settings();
+       XMLProperty* prop;
+
+       if (settings && (prop = settings->property ("notebook-shrunk"))) {
+               _notebook_shrunk = string_is_affirmative (prop->value ());
+       }
+
+       editor_summary_pane.set_check_divider_position (true);
+       editor_summary_pane.add (edit_packer);
 
        Button* summary_arrows_left_left = manage (new Button);
        summary_arrows_left_left->add (*manage (new Arrow (ARROW_LEFT, SHADOW_NONE)));
@@ -720,22 +716,39 @@ Editor::Editor ()
        _summary_hbox.pack_start (*summary_arrows_right, false, false);
 
        if (!ARDOUR::Profile->get_trx()) {
-               editor_summary_pane.pack2 (_summary_hbox);
+               editor_summary_pane.add (_summary_hbox);
        }
 
-       edit_pane.pack1 (editor_summary_pane, true, true);
+       edit_pane.set_check_divider_position (true);
+       edit_pane.add (editor_summary_pane);
        if (!ARDOUR::Profile->get_trx()) {
-               edit_pane.pack2 (_the_notebook, false, true);
+               edit_pane.add (_the_notebook);
        }
 
-       editor_summary_pane.signal_size_allocate().connect (sigc::bind (sigc::mem_fun (*this, &Editor::pane_allocation_handler), static_cast<Paned*> (&editor_summary_pane)));
+       edit_pane.set_drag_cursor (*_cursors->expand_left_right);
+       edit_pane.set_child_minsize (_the_notebook, 30); /* rough guess at width of notebook tabs */
+       editor_summary_pane.set_drag_cursor (*_cursors->expand_up_down);
 
-       /* XXX: editor_summary_pane might need similar to the edit_pane */
+       float fract;
 
-       edit_pane.signal_size_allocate().connect (sigc::bind (sigc::mem_fun(*this, &Editor::pane_allocation_handler), static_cast<Paned*> (&edit_pane)));
+       {
+               LocaleGuard lg;
 
-       Glib::PropertyProxy<int> proxy = edit_pane.property_position();
-       proxy.signal_changed().connect (bind (sigc::ptr_fun (pane_size_watcher), static_cast<Paned*> (&edit_pane)));
+               if (!settings || ((prop = settings->property ("edit-horizontal-pane-pos")) == 0) || ((fract = atof (prop->value())) > 1.0)) {
+                       /* initial allocation is 90% to canvas, 10% to notebook */
+                       edit_pane.set_divider (0, 0.90);
+               } else {
+                       edit_pane.set_divider (0, fract);
+               }
+
+               if (!settings || ((prop = settings->property ("edit-vertical-pane-pos")) == 0) || ((fract = atof (prop->value())) > 1.0)) {
+                       /* initial allocation is 90% to canvas, 10% to summary */
+                       editor_summary_pane.set_divider (0, 0.90);
+               } else {
+
+                       editor_summary_pane.set_divider (0, fract);
+               }
+       }
 
        top_hbox.pack_start (toolbar_frame);
 
@@ -746,13 +759,15 @@ Editor::Editor ()
        global_vpacker.pack_start (*hbox, true, true);
        global_hpacker.pack_start (global_vpacker, true, true);
 
+       /* need to show the "contents" widget so that notebook will show if tab is switched to
+        */
+
+       global_hpacker.show ();
+
        /* 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 ();
+       load_bindings ();
 
        setup_toolbar ();
 
@@ -790,14 +805,18 @@ Editor::Editor ()
        ControlProtocol::VerticalZoomInSelected.connect (*this, invalidator (*this), boost::bind (&Editor::control_vertical_zoom_in_selected, this), gui_context());
        ControlProtocol::VerticalZoomOutSelected.connect (*this, invalidator (*this), boost::bind (&Editor::control_vertical_zoom_out_selected, this), gui_context());
 
-       ControlProtocol::AddRouteToSelection.connect (*this, invalidator (*this), boost::bind (&Editor::control_select, this, _1, Selection::Add), gui_context());
-       ControlProtocol::RemoveRouteFromSelection.connect (*this, invalidator (*this), boost::bind (&Editor::control_select, this, _1, Selection::Toggle), gui_context());
-       ControlProtocol::SetRouteSelection.connect (*this, invalidator (*this), boost::bind (&Editor::control_select, this, _1, Selection::Set), gui_context());
-       ControlProtocol::ToggleRouteSelection.connect (*this, invalidator (*this), boost::bind (&Editor::control_select, this, _1, Selection::Toggle), gui_context());
-       ControlProtocol::ClearRouteSelection.connect (*this, invalidator (*this), boost::bind (&Editor::control_unselect, this), gui_context());
+       ControlProtocol::AddStripableToSelection.connect (*this, invalidator (*this), boost::bind (&Editor::control_select, this, _1, Selection::Add), gui_context());
+       ControlProtocol::RemoveStripableFromSelection.connect (*this, invalidator (*this), boost::bind (&Editor::control_select, this, _1, Selection::Toggle), gui_context());
+       ControlProtocol::SetStripableSelection.connect (*this, invalidator (*this), boost::bind (&Editor::control_select, this, _1, Selection::Set), gui_context());
+       ControlProtocol::ToggleStripableSelection.connect (*this, invalidator (*this), boost::bind (&Editor::control_select, this, _1, Selection::Toggle), gui_context());
+       ControlProtocol::ClearStripableSelection.connect (*this, invalidator (*this), boost::bind (&Editor::control_unselect, this), gui_context());
 
        BasicUI::AccessAction.connect (*this, invalidator (*this), boost::bind (&Editor::access_action, this, _1, _2), gui_context());
 
+       /* handle escape */
+
+       ARDOUR_UI::instance()->Escape.connect (*this, invalidator (*this), boost::bind (&Editor::escape, this), 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));
@@ -817,12 +836,12 @@ Editor::Editor ()
 
         /* Button bindings */
 
-        button_bindings = new Bindings;
+       button_bindings = new Bindings ("editor-mouse");
 
        XMLNode* node = button_settings();
         if (node) {
                 for (XMLNodeList::const_iterator i = node->children().begin(); i != node->children().end(); ++i) {
-                        button_bindings->load (**i);
+                        button_bindings->load_operation (**i);
                 }
         }
 
@@ -834,17 +853,21 @@ Editor::Editor ()
 
        setup_fade_images ();
 
+       LuaInstance::instance(); // instantiate
+       LuaInstance::instance()->ActionChanged.connect (sigc::mem_fun (*this, &Editor::set_script_action_name));
+
        instant_save ();
 }
 
 Editor::~Editor()
 {
-        delete button_bindings;
+       delete button_bindings;
        delete _routes;
        delete _route_groups;
        delete _track_canvas_viewport;
        delete _drags;
        delete nudge_clock;
+       delete _verbose_cursor;
        delete quantize_dialog;
        delete _summary;
        delete _group_tabs;
@@ -943,7 +966,7 @@ Editor::set_entered_track (TimeAxisView* tav)
 void
 Editor::instant_save ()
 {
-       if (!constructed || !ARDOUR_UI::instance()->session_loaded) {
+       if (!constructed || !ARDOUR_UI::instance()->session_loaded || no_save_instant) {
                return;
        }
 
@@ -991,23 +1014,9 @@ Editor::control_unselect ()
 }
 
 void
-Editor::control_select (uint32_t rid, Selection::Operation op)
+Editor::control_select (boost::shared_ptr<Stripable> s, Selection::Operation op)
 {
-       /* 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);
+       TimeAxisView* tav = axis_view_from_stripable (s);
 
        if (tav) {
                switch (op) {
@@ -1173,7 +1182,7 @@ Editor::generic_event_handler (GdkEvent* ev)
                        /* leaving window, so reset focus, thus ending any and
                           all text entry operations.
                        */
-                       reset_focus();
+                       reset_focus (&contents());
                        break;
                }
                break;
@@ -1260,7 +1269,7 @@ Editor::update_title ()
        if (!own_window()) {
                return;
        }
-               
+
        if (_session) {
                bool dirty = _session->dirty();
 
@@ -1277,6 +1286,7 @@ Editor::update_title ()
                }
 
                WindowTitle title(session_name);
+               title += S_("Window|Editor");
                title += Glib::get_application_name();
                own_window()->set_title (title.get_string());
        } else {
@@ -1354,9 +1364,11 @@ Editor::set_session (Session *t)
        _session->StepEditStatusChange.connect (_session_connections, invalidator (*this), boost::bind (&Editor::step_edit_status_change, this, _1), gui_context());
        _session->TransportStateChange.connect (_session_connections, invalidator (*this), boost::bind (&Editor::map_transport_state, this), gui_context());
        _session->PositionChanged.connect (_session_connections, invalidator (*this), boost::bind (&Editor::map_position_change, this, _1), gui_context());
+       _session->vca_manager().VCAAdded.connect (_session_connections, invalidator (*this), boost::bind (&Editor::add_vcas, this, _1), gui_context());
        _session->RouteAdded.connect (_session_connections, invalidator (*this), boost::bind (&Editor::add_routes, this, _1), gui_context());
        _session->DirtyChanged.connect (_session_connections, invalidator (*this), boost::bind (&Editor::update_title, this), gui_context());
        _session->tempo_map().PropertyChanged.connect (_session_connections, invalidator (*this), boost::bind (&Editor::tempo_map_changed, this, _1), gui_context());
+       _session->tempo_map().MetricPositionChanged.connect (_session_connections, invalidator (*this), boost::bind (&Editor::marker_position_changed, this), gui_context());
        _session->Located.connect (_session_connections, invalidator (*this), boost::bind (&Editor::located, this), gui_context());
        _session->config.ParameterChanged.connect (_session_connections, invalidator (*this), boost::bind (&Editor::parameter_changed, this, _1), gui_context());
        _session->StateSaved.connect (_session_connections, invalidator (*this), boost::bind (&Editor::session_state_saved, this, _1), gui_context());
@@ -1395,12 +1407,33 @@ Editor::set_session (Session *t)
                break;
        }
 
+       /* catch up on selection of stripables (other selection state is lost
+        * when a session is closed
+        */
+
+       StripableList sl;
+       TrackViewList tl;
+       _session->get_stripables (sl);
+       for (StripableList::const_iterator s = sl.begin(); s != sl.end(); ++s) {
+               if ((*s)->presentation_info().selected()) {
+                       RouteTimeAxisView* rtav = get_route_view_by_route_id ((*s)->id());
+                       if (rtav) {
+                               tl.push_back (rtav);
+                       }
+               }
+       }
+       if (!tl.empty()) {
+               selection->set (tl);
+       }
+
        /* register for undo history */
        _session->register_with_memento_command_factory(id(), this);
        _session->register_with_memento_command_factory(_selection_memento->id(), _selection_memento);
 
        ActionManager::ui_manager->signal_pre_activate().connect (sigc::mem_fun (*this, &Editor::action_pre_activated));
 
+       LuaInstance::instance()->set_session(_session);
+
        start_updating_meters ();
 }
 
@@ -1660,7 +1693,7 @@ Editor::build_track_context_menu ()
 {
        using namespace Menu_Helpers;
 
-       MenuList& edit_items = track_context_menu.items();
+       MenuList& edit_items = track_context_menu.items();
        edit_items.clear();
 
        add_dstream_context_items (edit_items);
@@ -1672,7 +1705,7 @@ Editor::build_track_bus_context_menu ()
 {
        using namespace Menu_Helpers;
 
-       MenuList& edit_items = track_context_menu.items();
+       MenuList& edit_items = track_context_menu.items();
        edit_items.clear();
 
        add_bus_context_items (edit_items);
@@ -1709,7 +1742,103 @@ Editor::build_track_region_context_menu ()
 }
 
 void
-Editor::analyze_region_selection ()
+Editor::loudness_analyze_region_selection ()
+{
+       if (!_session) {
+               return;
+       }
+       Selection& s (PublicEditor::instance ().get_selection ());
+       RegionSelection ars = s.regions;
+       ARDOUR::AnalysisGraph ag (_session);
+       framecnt_t total_work = 0;
+
+       for (RegionSelection::iterator j = ars.begin (); j != ars.end (); ++j) {
+               AudioRegionView* arv = dynamic_cast<AudioRegionView*> (*j);
+               if (!arv) {
+                       continue;
+               }
+               if (!boost::dynamic_pointer_cast<AudioRegion> (arv->region ())) {
+                       continue;
+               }
+               assert (dynamic_cast<RouteTimeAxisView *> (&arv->get_time_axis_view ()));
+               total_work += arv->region ()->length ();
+       }
+
+       SimpleProgressDialog spd (_("Region Loudness Analysis"), sigc::mem_fun (ag, &AnalysisGraph::cancel));
+       ScopedConnection c;
+       ag.set_total_frames (total_work);
+       ag.Progress.connect_same_thread (c, boost::bind (&SimpleProgressDialog::update_progress, &spd, _1, _2));
+       spd.show();
+
+       for (RegionSelection::iterator j = ars.begin (); j != ars.end (); ++j) {
+               AudioRegionView* arv = dynamic_cast<AudioRegionView*> (*j);
+               if (!arv) {
+                       continue;
+               }
+               boost::shared_ptr<AudioRegion> ar = boost::dynamic_pointer_cast<AudioRegion> (arv->region ());
+               if (!ar) {
+                       continue;
+               }
+               ag.analyze_region (ar);
+       }
+       spd.hide();
+       if (!ag.canceled ()) {
+               ExportReport er (_("Audio Report/Analysis"), ag.results ());
+               er.run();
+       }
+}
+
+void
+Editor::loudness_analyze_range_selection ()
+{
+       if (!_session) {
+               return;
+       }
+       Selection& s (PublicEditor::instance ().get_selection ());
+       TimeSelection ts = s.time;
+       ARDOUR::AnalysisGraph ag (_session);
+       framecnt_t total_work = 0;
+
+       for (TrackSelection::iterator i = s.tracks.begin (); i != s.tracks.end (); ++i) {
+               boost::shared_ptr<AudioPlaylist> pl = boost::dynamic_pointer_cast<AudioPlaylist> ((*i)->playlist ());
+               if (!pl) {
+                       continue;
+               }
+               RouteUI *rui = dynamic_cast<RouteUI *> (*i);
+               if (!pl || !rui) {
+                       continue;
+               }
+               for (std::list<AudioRange>::iterator j = ts.begin (); j != ts.end (); ++j) {
+                       total_work += j->length ();
+               }
+       }
+
+       SimpleProgressDialog spd (_("Range Loudness Analysis"), sigc::mem_fun (ag, &AnalysisGraph::cancel));
+       ScopedConnection c;
+       ag.set_total_frames (total_work);
+       ag.Progress.connect_same_thread (c, boost::bind (&SimpleProgressDialog::update_progress, &spd, _1, _2));
+       spd.show();
+
+       for (TrackSelection::iterator i = s.tracks.begin (); i != s.tracks.end (); ++i) {
+               boost::shared_ptr<AudioPlaylist> pl = boost::dynamic_pointer_cast<AudioPlaylist> ((*i)->playlist ());
+               if (!pl) {
+                       continue;
+               }
+               RouteUI *rui = dynamic_cast<RouteUI *> (*i);
+               if (!pl || !rui) {
+                       continue;
+               }
+               ag.analyze_range (rui->route (), pl, ts);
+       }
+       spd.hide();
+       if (!ag.canceled ()) {
+               ExportReport er (_("Audio Report/Analysis"), ag.results ());
+               er.run();
+       }
+}
+
+void
+Editor::spectral_analyze_region_selection ()
 {
        if (analysis_window == 0) {
                analysis_window = new AnalysisWindow();
@@ -1727,7 +1856,7 @@ Editor::analyze_region_selection ()
 }
 
 void
-Editor::analyze_range_selection()
+Editor::spectral_analyze_range_selection()
 {
        if (analysis_window == 0) {
                analysis_window = new AnalysisWindow();
@@ -1821,7 +1950,8 @@ Editor::add_selection_context_items (Menu_Helpers::MenuList& edit_items)
        edit_items.push_back (MenuElem (_("Zoom to Range"), sigc::bind (sigc::mem_fun(*this, &Editor::temporal_zoom_selection), false)));
 
        edit_items.push_back (SeparatorElem());
-       edit_items.push_back (MenuElem (_("Spectral Analysis"), sigc::mem_fun(*this, &Editor::analyze_range_selection)));
+       edit_items.push_back (MenuElem (_("Loudness Analysis"), sigc::mem_fun(*this, &Editor::loudness_analyze_range_selection)));
+       edit_items.push_back (MenuElem (_("Spectral Analysis"), sigc::mem_fun(*this, &Editor::spectral_analyze_range_selection)));
 
        edit_items.push_back (SeparatorElem());
 
@@ -2028,6 +2158,37 @@ Editor::snap_type() const
        return _snap_type;
 }
 
+bool
+Editor::snap_musical() const
+{
+       switch (_snap_type) {
+       case SnapToBeatDiv128:
+       case SnapToBeatDiv64:
+       case SnapToBeatDiv32:
+       case SnapToBeatDiv28:
+       case SnapToBeatDiv24:
+       case SnapToBeatDiv20:
+       case SnapToBeatDiv16:
+       case SnapToBeatDiv14:
+       case SnapToBeatDiv12:
+       case SnapToBeatDiv10:
+       case SnapToBeatDiv8:
+       case SnapToBeatDiv7:
+       case SnapToBeatDiv6:
+       case SnapToBeatDiv5:
+       case SnapToBeatDiv4:
+       case SnapToBeatDiv3:
+       case SnapToBeatDiv2:
+       case SnapToBeat:
+       case SnapToBar:
+               return true;
+       default:
+               break;
+       }
+
+       return false;
+}
+
 SnapMode
 Editor::snap_mode() const
 {
@@ -2078,14 +2239,10 @@ Editor::set_snap_to (SnapType st)
        case SnapToBeatDiv4:
        case SnapToBeatDiv3:
        case SnapToBeatDiv2: {
-               ARDOUR::TempoMap::BBTPointList::const_iterator current_bbt_points_begin;
-               ARDOUR::TempoMap::BBTPointList::const_iterator current_bbt_points_end;
-
-               compute_current_bbt_points (leftmost_frame, leftmost_frame + current_page_samples(),
-                                           current_bbt_points_begin, current_bbt_points_end);
-               compute_bbt_ruler_scale (leftmost_frame, leftmost_frame + current_page_samples(),
-                                        current_bbt_points_begin, current_bbt_points_end);
-               update_tempo_based_rulers (current_bbt_points_begin, current_bbt_points_end);
+               std::vector<TempoMap::BBTPoint> grid;
+               compute_current_bbt_points (grid, leftmost_frame, leftmost_frame + current_page_samples());
+               compute_bbt_ruler_scale (grid, leftmost_frame, leftmost_frame + current_page_samples());
+               update_tempo_based_rulers (grid);
                break;
        }
 
@@ -2181,12 +2338,13 @@ Editor::set_edit_point_preference (EditPoint ep, bool force)
 int
 Editor::set_state (const XMLNode& node, int version)
 {
-       const XMLProperty* prop;
-
+       XMLProperty const * prop;
        set_id (node);
+       PBD::Unwinder<bool> nsi (no_save_instant, true);
+       LocaleGuard lg;
 
        Tabbable::set_state (node, version);
-       
+
        if (_session && (prop = node.property ("playhead"))) {
                framepos_t pos;
                sscanf (prop->value().c_str(), "%" PRIi64, &pos);
@@ -2206,6 +2364,8 @@ Editor::set_state (const XMLNode& node, int version)
 
        if ((prop = node.property ("zoom-focus"))) {
                zoom_focus_selection_done ((ZoomFocus) string_2_enum (prop->value(), zoom_focus));
+       } else {
+               zoom_focus_selection_done (zoom_focus);
        }
 
        if ((prop = node.property ("zoom"))) {
@@ -2222,10 +2382,20 @@ Editor::set_state (const XMLNode& node, int version)
 
        if ((prop = node.property ("snap-to"))) {
                snap_type_selection_done ((SnapType) string_2_enum (prop->value(), _snap_type));
+               set_snap_to ((SnapType) string_2_enum (prop->value(), _snap_type));
+       } else {
+               set_snap_to (_snap_type);
        }
 
        if ((prop = node.property ("snap-mode"))) {
                snap_mode_selection_done((SnapMode) string_2_enum (prop->value(), _snap_mode));
+               /* set text of Dropdown. in case _snap_mode == SnapOff (default)
+                * snap_mode_selection_done() will only mark an already active item as active
+                * which does not trigger set_text().
+                */
+               set_snap_mode ((SnapMode) string_2_enum (prop->value(), _snap_mode));
+       } else {
+               set_snap_mode (_snap_mode);
        }
 
        if ((prop = node.property ("internal-snap-to"))) {
@@ -2278,6 +2448,8 @@ Editor::set_state (const XMLNode& node, int version)
 
        if ((prop = node.property ("edit-point"))) {
                set_edit_point_preference ((EditPoint) string_2_enum (prop->value(), _edit_point), true);
+       } else {
+               set_edit_point_preference (_edit_point);
        }
 
        if ((prop = node.property ("show-measures"))) {
@@ -2404,88 +2576,91 @@ Editor::set_state (const XMLNode& node, int version)
                }
        }
 
-       return 0;
+       return LuaInstance::instance()->set_state(node);
 }
 
 XMLNode&
 Editor::get_state ()
 {
-       XMLNode& node (Tabbable::get_state());
+       XMLNode* node = new XMLNode (X_("Editor"));
        char buf[32];
+       LocaleGuard lg;
 
        id().print (buf, sizeof (buf));
-       node.add_property ("id", buf);
-
-#if 0
-       // need to save this somehow 
-       snprintf(buf,sizeof(buf), "%d",gtk_paned_get_position (static_cast<Paned*>(&edit_pane)->gobj()));
-       geometry->add_property("edit-horizontal-pane-pos", string(buf));
-       geometry->add_property("notebook-shrunk", _notebook_shrunk ? "1" : "0");
-       snprintf(buf,sizeof(buf), "%d",gtk_paned_get_position (static_cast<Paned*>(&editor_summary_pane)->gobj()));
-       geometry->add_property("edit-vertical-pane-pos", string(buf));
-#endif
-       
-       maybe_add_mixer_strip_width (node);
+       node->add_property ("id", buf);
+
+       node->add_child_nocopy (Tabbable::get_state());
+
+       snprintf(buf,sizeof(buf), "%f", edit_pane.get_divider ());
+       node->add_property("edit-horizontal-pane-pos", string(buf));
+       node->add_property("notebook-shrunk", _notebook_shrunk ? "1" : "0");
+       snprintf(buf,sizeof(buf), "%f", editor_summary_pane.get_divider());
+       node->add_property("edit-vertical-pane-pos", string(buf));
 
-       node.add_property ("zoom-focus", enum_2_string (zoom_focus));
+       maybe_add_mixer_strip_width (*node);
+
+       node->add_property ("zoom-focus", enum_2_string (zoom_focus));
 
        snprintf (buf, sizeof(buf), "%" PRId64, samples_per_pixel);
-       node.add_property ("zoom", buf);
-       node.add_property ("snap-to", enum_2_string (_snap_type));
-       node.add_property ("snap-mode", enum_2_string (_snap_mode));
-       node.add_property ("internal-snap-to", enum_2_string (internal_snap_type));
-       node.add_property ("internal-snap-mode", enum_2_string (internal_snap_mode));
-       node.add_property ("pre-internal-snap-to", enum_2_string (pre_internal_snap_type));
-       node.add_property ("pre-internal-snap-mode", enum_2_string (pre_internal_snap_mode));
-       node.add_property ("edit-point", enum_2_string (_edit_point));
+       node->add_property ("zoom", buf);
+       node->add_property ("snap-to", enum_2_string (_snap_type));
+       node->add_property ("snap-mode", enum_2_string (_snap_mode));
+       node->add_property ("internal-snap-to", enum_2_string (internal_snap_type));
+       node->add_property ("internal-snap-mode", enum_2_string (internal_snap_mode));
+       node->add_property ("pre-internal-snap-to", enum_2_string (pre_internal_snap_type));
+       node->add_property ("pre-internal-snap-mode", enum_2_string (pre_internal_snap_mode));
+       node->add_property ("edit-point", enum_2_string (_edit_point));
        snprintf (buf, sizeof(buf), "%d", _visible_track_count);
-       node.add_property ("visible-track-count", buf);
+       node->add_property ("visible-track-count", buf);
 
        snprintf (buf, sizeof (buf), "%" PRIi64, playhead_cursor->current_frame ());
-       node.add_property ("playhead", buf);
+       node->add_property ("playhead", buf);
        snprintf (buf, sizeof (buf), "%" PRIi64, leftmost_frame);
-       node.add_property ("left-frame", buf);
+       node->add_property ("left-frame", buf);
        snprintf (buf, sizeof (buf), "%f", vertical_adjustment.get_value ());
-       node.add_property ("y-origin", buf);
+       node->add_property ("y-origin", buf);
 
-       node.add_property ("show-measures", _show_measures ? "yes" : "no");
-       node.add_property ("maximised", _maximised ? "yes" : "no");
-       node.add_property ("follow-playhead", _follow_playhead ? "yes" : "no");
-       node.add_property ("stationary-playhead", _stationary_playhead ? "yes" : "no");
-       node.add_property ("region-list-sort-type", enum_2_string (_regions->sort_type ()));
-       node.add_property ("mouse-mode", enum2str(mouse_mode));
-       node.add_property ("join-object-range", smart_mode_action->get_active () ? "yes" : "no");
+       node->add_property ("show-measures", _show_measures ? "yes" : "no");
+       node->add_property ("maximised", _maximised ? "yes" : "no");
+       node->add_property ("follow-playhead", _follow_playhead ? "yes" : "no");
+       node->add_property ("stationary-playhead", _stationary_playhead ? "yes" : "no");
+       node->add_property ("region-list-sort-type", enum_2_string (_regions->sort_type ()));
+       node->add_property ("mouse-mode", enum2str(mouse_mode));
+       node->add_property ("join-object-range", smart_mode_action->get_active () ? "yes" : "no");
 
        Glib::RefPtr<Action> act = ActionManager::get_action (X_("Editor"), X_("show-editor-mixer"));
        if (act) {
                Glib::RefPtr<ToggleAction> tact = Glib::RefPtr<ToggleAction>::cast_dynamic(act);
-               node.add_property (X_("show-editor-mixer"), tact->get_active() ? "yes" : "no");
+               node->add_property (X_("show-editor-mixer"), tact->get_active() ? "yes" : "no");
        }
 
        act = ActionManager::get_action (X_("Editor"), X_("show-editor-list"));
        if (act) {
                Glib::RefPtr<ToggleAction> tact = Glib::RefPtr<ToggleAction>::cast_dynamic(act);
-               node.add_property (X_("show-editor-list"), tact->get_active() ? "yes" : "no");
+               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);
+       node->add_property (X_("editor-list-page"), buf);
 
         if (button_bindings) {
                 XMLNode* bb = new XMLNode (X_("Buttons"));
                 button_bindings->save (*bb);
-                node.add_child_nocopy (*bb);
+                node->add_child_nocopy (*bb);
         }
 
-       node.add_property (X_("show-marker-lines"), _show_marker_lines ? "yes" : "no");
+       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 ());
+       node->add_child_nocopy (selection->get_state ());
+       node->add_child_nocopy (_regions->get_state ());
 
        snprintf (buf, sizeof (buf), "%" PRId64, nudge_clock->current_duration());
-       node.add_property ("nudge-clock-value", buf);
+       node->add_property ("nudge-clock-value", buf);
 
-       return node;
+       node->add_child_nocopy (LuaInstance::instance()->get_action_state());
+       node->add_child_nocopy (LuaInstance::instance()->get_hook_state());
+
+       return *node;
 }
 
 /** if @param trackview_relative_offset is true, @param y y is an offset into the trackview area, in pixel units
@@ -2889,24 +3064,8 @@ Editor::setup_toolbar ()
        if (!ARDOUR::Profile->get_trx()) {
                mode_box->pack_start (edit_mode_selector, false, false);
        }
-       mode_box->pack_start (*mouse_mode_box, false, 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() || Profile->get_mixbus() ) {
-               _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));
+       mode_box->pack_start (*mouse_mode_box, false, false);
 
        /* Zoom */
 
@@ -2978,23 +3137,6 @@ Editor::setup_toolbar ()
                _zoom_box.pack_start (tav_expand_button);
        }
 
-       if (!ARDOUR::Profile->get_trx()) {
-               _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),
-                                                          &_zoom_tearoff->tearoff_window()));
-               _zoom_tearoff->Attach.connect (sigc::bind (sigc::mem_fun(*this, &Editor::reattach_tearoff), static_cast<Box*> (&toolbar_hbox),
-                                                          &_zoom_tearoff->tearoff_window(), 0));
-               _zoom_tearoff->Hidden.connect (sigc::bind (sigc::mem_fun(*this, &Editor::detach_tearoff), static_cast<Box*>(&toolbar_hbox),
-                                                          &_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));
-       }
-
-       if (Profile->get_sae() || Profile->get_mixbus() ) {
-               _zoom_tearoff->set_can_be_torn_off (false);
-       }
-
        snap_box.set_spacing (2);
        snap_box.set_border_width (2);
 
@@ -3027,37 +3169,19 @@ Editor::setup_toolbar ()
        HBox* hbox = manage (new HBox);
        hbox->set_spacing(2);
 
-       _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() || Profile->get_mixbus()) {
-               _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));
-
        toolbar_hbox.set_spacing (2);
        toolbar_hbox.set_border_width (1);
 
-       toolbar_hbox.pack_start (*_mouse_mode_tearoff, false, false);
+       toolbar_hbox.pack_start (*mode_box, false, false);
        if (!ARDOUR::Profile->get_trx()) {
-               toolbar_hbox.pack_start (*_zoom_tearoff, false, false);
-               toolbar_hbox.pack_start (*_tools_tearoff, false, false);
+               toolbar_hbox.pack_start (_zoom_box, false, false);
+               toolbar_hbox.pack_start (*hbox, false, false);
        }
 
        if (!ARDOUR::Profile->get_trx()) {
                hbox->pack_start (snap_box, false, false);
                hbox->pack_start (*nudge_box, false, false);
        }
-       hbox->pack_start (panic_box, false, false);
 
        hbox->show_all ();
 
@@ -3541,11 +3665,7 @@ Editor::cycle_edit_mode ()
 {
        switch (Config->get_edit_mode()) {
        case Slide:
-               if (Profile->get_sae()) {
-                       Config->set_edit_mode (Lock);
-               } else {
-                       Config->set_edit_mode (Ripple);
-               }
+               Config->set_edit_mode (Ripple);
                break;
        case Splice:
        case Ripple:
@@ -3753,7 +3873,7 @@ bool
 Editor::edit_controls_button_release (GdkEventButton* ev)
 {
        if (Keyboard::is_context_menu_event (ev)) {
-               ARDOUR_UI::instance()->add_route (current_toplevel());
+               ARDOUR_UI::instance()->add_route ();
        } else if (ev->button == 1) {
                selection->clear_tracks ();
        }
@@ -3813,92 +3933,6 @@ Editor::cycle_zoom_focus ()
        }
 }
 
-void
-Editor::pane_allocation_handler (Allocation &alloc, Paned* which)
-{
-       /* recover or initialize pane positions. do this here rather than earlier because
-          we don't want the positions to change the child allocations, which they seem to do.
-        */
-
-       int pos;
-       XMLProperty* prop;
-       char buf[32];
-       XMLNode* node = ARDOUR_UI::instance()->editor_settings();
-
-       enum Pane {
-               Horizontal = 0x1,
-               Vertical = 0x2
-       };
-
-       static Pane done;
-
-       XMLNode* geometry = find_named_node (*node, "geometry");
-
-       if (which == static_cast<Paned*> (&edit_pane)) {
-
-               if (done & Horizontal) {
-                       return;
-               }
-
-               if (geometry && (prop = geometry->property ("notebook-shrunk"))) {
-                       _notebook_shrunk = string_is_affirmative (prop->value ());
-               }
-
-               if (!geometry || (prop = geometry->property ("edit-horizontal-pane-pos")) == 0) {
-                       /* initial allocation is 90% to canvas, 10% to notebook */
-                       pos = (int) floor (alloc.get_width() * 0.90f);
-                       snprintf (buf, sizeof(buf), "%d", pos);
-               } else {
-                       pos = atoi (prop->value());
-               }
-
-               if (GTK_WIDGET(edit_pane.gobj())->allocation.width > pos) {
-                       edit_pane.set_position (pos);
-               }
-
-               done = (Pane) (done | Horizontal);
-
-       } else if (which == static_cast<Paned*> (&editor_summary_pane)) {
-
-               if (done & Vertical) {
-                       return;
-               }
-
-               if (!geometry || (prop = geometry->property ("edit-vertical-pane-pos")) == 0) {
-                       /* initial allocation is 90% to canvas, 10% to summary */
-                       pos = (int) floor (alloc.get_height() * 0.90f);
-                       snprintf (buf, sizeof(buf), "%d", pos);
-               } else {
-
-                       pos = atoi (prop->value());
-               }
-
-               if (GTK_WIDGET(editor_summary_pane.gobj())->allocation.height > pos) {
-                       editor_summary_pane.set_position (pos);
-               }
-
-               done = (Pane) (done | Vertical);
-       }
-}
-
-void
-Editor::detach_tearoff (Box* /*b*/, Window* /*w*/)
-{
-       if ((_tools_tearoff->torn_off() || !_tools_tearoff->visible()) &&
-           (_mouse_mode_tearoff->torn_off() || !_mouse_mode_tearoff->visible()) &&
-           (_zoom_tearoff && (_zoom_tearoff->torn_off() || !_zoom_tearoff->visible()))) {
-               top_hbox.remove (toolbar_frame);
-       }
-}
-
-void
-Editor::reattach_tearoff (Box* /*b*/, Window* /*w*/, int32_t /*n*/)
-{
-       if (toolbar_frame.get_parent() == 0) {
-               top_hbox.pack_end (toolbar_frame);
-       }
-}
-
 void
 Editor::set_show_measures (bool yn)
 {
@@ -3910,11 +3944,9 @@ Editor::set_show_measures (bool yn)
                                tempo_lines->show();
                        }
 
-                       ARDOUR::TempoMap::BBTPointList::const_iterator begin;
-                       ARDOUR::TempoMap::BBTPointList::const_iterator end;
-
-                       compute_current_bbt_points (leftmost_frame, leftmost_frame + current_page_samples(), begin, end);
-                       draw_measures (begin, end);
+                       std::vector<TempoMap::BBTPoint> grid;
+                       compute_current_bbt_points (grid, leftmost_frame, leftmost_frame + current_page_samples());
+                       draw_measures (grid);
                }
 
                instant_save ();
@@ -4020,6 +4052,47 @@ Editor::get_grid_beat_divisions(framepos_t position)
        return 0;
 }
 
+/** returns the current musical grid divisiions using the supplied modifier mask from a GtkEvent.
+    if the grid is non-musical, returns 0.
+    if the grid is snapped to bars, returns -1.
+    @param event_state the current keyboard modifier mask.
+*/
+unsigned
+Editor::get_grid_music_divisions (uint32_t event_state)
+{
+       if (snap_mode() == Editing::SnapOff && !ArdourKeyboard::indicates_snap (event_state)) {
+               return 0;
+       }
+
+       if (snap_mode() != Editing::SnapOff && ArdourKeyboard::indicates_snap (event_state)) {
+               return 0;
+       }
+
+       switch (_snap_type) {
+       case SnapToBeatDiv128: return 128;
+       case SnapToBeatDiv64:  return 64;
+       case SnapToBeatDiv32:  return 32;
+       case SnapToBeatDiv28:  return 28;
+       case SnapToBeatDiv24:  return 24;
+       case SnapToBeatDiv20:  return 20;
+       case SnapToBeatDiv16:  return 16;
+       case SnapToBeatDiv14:  return 14;
+       case SnapToBeatDiv12:  return 12;
+       case SnapToBeatDiv10:  return 10;
+       case SnapToBeatDiv8:   return 8;
+       case SnapToBeatDiv7:   return 7;
+       case SnapToBeatDiv6:   return 6;
+       case SnapToBeatDiv5:   return 5;
+       case SnapToBeatDiv4:   return 4;
+       case SnapToBeatDiv3:   return 3;
+       case SnapToBeatDiv2:   return 2;
+       case SnapToBeat:       return 1;
+       case SnapToBar :       return -1;
+       default:               return 0;
+       }
+       return 0;
+}
+
 Evoral::Beats
 Editor::get_grid_type_as_beats (bool& success, framepos_t position)
 {
@@ -4035,7 +4108,7 @@ Editor::get_grid_type_as_beats (bool& success, framepos_t position)
                return Evoral::Beats(1.0);
        case SnapToBar:
                if (_session) {
-                       return Evoral::Beats(_session->tempo_map().meter_at (position).divisions_per_bar());
+                       return Evoral::Beats(_session->tempo_map().meter_at_frame (position).divisions_per_bar());
                }
                break;
        default:
@@ -4178,25 +4251,6 @@ Editor::session_state_saved (string)
        _snapshots->redisplay ();
 }
 
-void
-Editor::update_tearoff_visibility()
-{
-       bool visible = UIConfiguration::instance().get_keep_tearoffs();
-       _mouse_mode_tearoff->set_visible (visible);
-       _tools_tearoff->set_visible (visible);
-       if (_zoom_tearoff) {
-               _zoom_tearoff->set_visible (visible);
-       }
-}
-
-void
-Editor::reattach_all_tearoffs ()
-{
-       if (_mouse_mode_tearoff) _mouse_mode_tearoff->put_it_back ();
-       if (_tools_tearoff) _tools_tearoff->put_it_back ();
-       if (_zoom_tearoff) _zoom_tearoff->put_it_back ();
-}
-
 void
 Editor::maximise_editing_space ()
 {
@@ -4204,9 +4258,12 @@ Editor::maximise_editing_space ()
                return;
        }
 
-       current_toplevel()->fullscreen ();
+       Gtk::Window* toplevel = current_toplevel();
 
-       _maximised = true;
+       if (toplevel) {
+               toplevel->fullscreen ();
+               _maximised = true;
+       }
 }
 
 void
@@ -4216,9 +4273,12 @@ Editor::restore_editing_space ()
                return;
        }
 
-       current_toplevel()->unfullscreen();
+       Gtk::Window* toplevel = current_toplevel();
 
-       _maximised = false;
+       if (toplevel) {
+               toplevel->unfullscreen();
+               _maximised = false;
+       }
 }
 
 /**
@@ -4233,7 +4293,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, ARDOUR::Properties::select.property_id);
+       mapover_tracks (sigc::bind (sigc::mem_fun (*this, &Editor::mapped_use_new_playlist), playlists), v, ARDOUR::Properties::group_select.property_id);
        commit_reversible_command ();
 }
 
@@ -4249,7 +4309,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, ARDOUR::Properties::select.property_id);
+       mapover_tracks (sigc::bind (sigc::mem_fun (*this, &Editor::mapped_use_copy_playlist), playlists), v, ARDOUR::Properties::group_select.property_id);
        commit_reversible_command ();
 }
 
@@ -4264,7 +4324,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, ARDOUR::Properties::select.property_id);
+       mapover_tracks (sigc::mem_fun (*this, &Editor::mapped_clear_playlist), v, ARDOUR::Properties::group_select.property_id);
        commit_reversible_command ();
 }
 
@@ -4354,7 +4414,7 @@ Editor::current_visual_state (bool with_tracks)
        vs->zoom_focus = zoom_focus;
 
        if (with_tracks) {
-               *vs->gui_state = *ARDOUR_UI::instance()->gui_object_state;
+               vs->gui_state->set_state (ARDOUR_UI::instance()->gui_object_state->get_state());
        }
 
        return vs;
@@ -4419,7 +4479,7 @@ Editor::use_visual_state (VisualState& vs)
        reposition_and_zoom (vs.leftmost_frame, vs.samples_per_pixel);
 
        if (vs.gui_state) {
-               *ARDOUR_UI::instance()->gui_object_state = *vs.gui_state;
+               ARDOUR_UI::instance()->gui_object_state->set_state (vs.gui_state->get_state());
 
                for (TrackViewList::iterator i = track_views.begin(); i != track_views.end(); ++i) {
                        (*i)->clear_property_cache();
@@ -4553,14 +4613,10 @@ Editor::visual_changer (const VisualChange& vc)
 
                compute_fixed_ruler_scale ();
 
-               ARDOUR::TempoMap::BBTPointList::const_iterator current_bbt_points_begin;
-               ARDOUR::TempoMap::BBTPointList::const_iterator current_bbt_points_end;
-
-               compute_current_bbt_points (vc.time_origin, pending_visual_change.time_origin + current_page_samples(),
-                                           current_bbt_points_begin, current_bbt_points_end);
-               compute_bbt_ruler_scale (vc.time_origin, pending_visual_change.time_origin + current_page_samples(),
-                                        current_bbt_points_begin, current_bbt_points_end);
-               update_tempo_based_rulers (current_bbt_points_begin, current_bbt_points_end);
+               std::vector<TempoMap::BBTPoint> grid;
+               compute_current_bbt_points (grid, vc.time_origin, pending_visual_change.time_origin + current_page_samples());
+               compute_bbt_ruler_scale (grid, vc.time_origin, pending_visual_change.time_origin + current_page_samples());
+               update_tempo_based_rulers (grid);
 
                update_video_timeline();
        }
@@ -4878,7 +4934,7 @@ Editor::get_regions_from_selection_and_mouse (framepos_t pos)
  */
 
 RegionSelection
-Editor::get_regions_from_selection_and_entered ()
+Editor::get_regions_from_selection_and_entered () const
 {
        RegionSelection regions = selection->regions;
 
@@ -4990,12 +5046,11 @@ Editor::first_idle ()
        if (track_views.size() > 1) {
                Timers::TimerSuspender t;
                dialog = new MessageDialog (
-                       *current_toplevel(),
                        string_compose (_("Please wait while %1 loads visual data."), PROGRAM_NAME),
                        true
                        );
                dialog->present ();
-               ARDOUR_UI::instance()->flush_pending ();
+               ARDOUR_UI::instance()->flush_pending (60);
        }
 
        for (TrackViewList::iterator t = track_views.begin(); t != track_views.end(); ++t) {
@@ -5131,16 +5186,13 @@ Editor::region_view_removed ()
        _summary->set_background_dirty ();
 }
 
-RouteTimeAxisView*
-Editor::axis_view_from_route (boost::shared_ptr<Route> r) const
+TimeAxisView*
+Editor::axis_view_from_stripable (boost::shared_ptr<Stripable> s) const
 {
-       TrackViewList::const_iterator j = track_views.begin ();
-       while (j != track_views.end()) {
-               RouteTimeAxisView* rtv = dynamic_cast<RouteTimeAxisView*> (*j);
-               if (rtv && rtv->route() == r) {
-                       return rtv;
+       for (TrackViewList::const_iterator j = track_views.begin (); j != track_views.end(); ++j) {
+               if ((*j)->stripable() == s) {
+                       return *j;
                }
-               ++j;
        }
 
        return 0;
@@ -5153,7 +5205,7 @@ Editor::axis_views_from_routes (boost::shared_ptr<RouteList> r) const
        TrackViewList t;
 
        for (RouteList::const_iterator i = r->begin(); i != r->end(); ++i) {
-               TimeAxisView* tv = axis_view_from_route (*i);
+               TimeAxisView* tv = axis_view_from_stripable (*i);
                if (tv) {
                        t.push_back (tv);
                }
@@ -5180,56 +5232,94 @@ Editor::resume_route_redisplay ()
 }
 
 void
-Editor::add_routes (RouteList& routes)
+Editor::add_vcas (VCAList& vlist)
 {
-       ENSURE_GUI_THREAD (*this, &Editor::handle_new_route, routes)
+       StripableList sl;
 
-       RouteTimeAxisView *rtv;
-       list<RouteTimeAxisView*> new_views;
+       for (VCAList::iterator v = vlist.begin(); v != vlist.end(); ++v) {
+               sl.push_back (boost::dynamic_pointer_cast<Stripable> (*v));
+       }
+
+       add_stripables (sl);
+}
+
+void
+Editor::add_routes (RouteList& rlist)
+{
+       StripableList sl;
+
+       for (RouteList::iterator r = rlist.begin(); r != rlist.end(); ++r) {
+               sl.push_back (*r);
+       }
+
+       add_stripables (sl);
+}
+
+void
+Editor::add_stripables (StripableList& sl)
+{
+       list<TimeAxisView*> new_views;
+       boost::shared_ptr<VCA> v;
+       boost::shared_ptr<Route> r;
        TrackViewList new_selection;
        bool from_scratch = (track_views.size() == 0);
 
-       for (RouteList::iterator x = routes.begin(); x != routes.end(); ++x) {
-               boost::shared_ptr<Route> route = (*x);
+       sl.sort (StripablePresentationInfoSorter());
 
-               if (route->is_auditioner() || route->is_monitor()) {
-                       continue;
-               }
+       for (StripableList::iterator s = sl.begin(); s != sl.end(); ++s) {
 
-               DataType dt = route->input()->default_type();
+               if ((v = boost::dynamic_pointer_cast<VCA> (*s)) != 0) {
 
-               if (dt == ARDOUR::DataType::AUDIO) {
-                       rtv = new AudioTimeAxisView (*this, _session, *_track_canvas);
-                       rtv->set_route (route);
-               } else if (dt == ARDOUR::DataType::MIDI) {
-                       rtv = new MidiTimeAxisView (*this, _session, *_track_canvas);
-                       rtv->set_route (route);
-               } else {
-                       throw unknown_type();
-               }
+                       VCATimeAxisView* vtv = new VCATimeAxisView (*this, _session, *_track_canvas);
+                       vtv->set_vca (v);
+                       new_views.push_back (vtv);
+
+               } else if ((r = boost::dynamic_pointer_cast<Route> (*s)) != 0) {
+
+                       if (r->is_auditioner() || r->is_monitor()) {
+                               continue;
+                       }
+
+                       RouteTimeAxisView* rtv;
+                       DataType dt = r->input()->default_type();
 
-               new_views.push_back (rtv);
-               track_views.push_back (rtv);
-               new_selection.push_back (rtv);
+                       if (dt == ARDOUR::DataType::AUDIO) {
+                               rtv = new AudioTimeAxisView (*this, _session, *_track_canvas);
+                               rtv->set_route (r);
+                       } else if (dt == ARDOUR::DataType::MIDI) {
+                               rtv = new MidiTimeAxisView (*this, _session, *_track_canvas);
+                               rtv->set_route (r);
+                       } else {
+                               throw unknown_type();
+                       }
 
-               rtv->effective_gain_display ();
+                       new_views.push_back (rtv);
+                       track_views.push_back (rtv);
+                       new_selection.push_back (rtv);
 
-               rtv->view()->RegionViewAdded.connect (sigc::mem_fun (*this, &Editor::region_view_added));
-               rtv->view()->RegionViewRemoved.connect (sigc::mem_fun (*this, &Editor::region_view_removed));
+                       rtv->effective_gain_display ();
+
+                       rtv->view()->RegionViewAdded.connect (sigc::mem_fun (*this, &Editor::region_view_added));
+                       rtv->view()->RegionViewRemoved.connect (sigc::mem_fun (*this, &Editor::region_view_removed));
+               }
        }
 
        if (new_views.size() > 0) {
-               _routes->routes_added (new_views);
-               _summary->routes_added (new_views);
+               _routes->time_axis_views_added (new_views);
+               //_summary->routes_added (new_selection); /* XXX requires RouteTimeAxisViewList */
        }
 
-       if (!from_scratch) {
+       /* note: !new_selection.empty() means that we got some routes rather
+        * than just VCAs
+        */
+
+       if (!from_scratch && !new_selection.empty()) {
                selection->tracks.clear();
                selection->add (new_selection);
                begin_selection_op_history();
        }
 
-       if (show_editor_mixer_when_tracks_arrive) {
+       if (show_editor_mixer_when_tracks_arrive && !new_selection.empty()) {
                show_editor_mixer (true);
        }
 
@@ -5694,18 +5784,41 @@ Editor::session_going_away ()
        stop_step_editing ();
 
        if (own_window()) {
-       
+
                /* get rid of any existing editor mixer strip */
-               
+
                WindowTitle title(Glib::get_application_name());
                title += _("Editor");
-               
+
                own_window()->set_title (title.get_string());
        }
 
        SessionHandlePtr::session_going_away ();
 }
 
+void
+Editor::trigger_script (int i)
+{
+       LuaInstance::instance()-> call_action (i);
+}
+
+void
+Editor::set_script_action_name (int i, const std::string& n)
+{
+       string const a = string_compose (X_("script-action-%1"), i + 1);
+       Glib::RefPtr<Action> act = ActionManager::get_action(X_("Editor"), a.c_str());
+       assert (act);
+       if (n.empty ()) {
+               act->set_label (string_compose (_("Unset #%1"), i + 1));
+               act->set_tooltip (_("no action bound"));
+               act->set_sensitive (false);
+       } else {
+               act->set_label (n);
+               act->set_tooltip (n);
+               act->set_sensitive (true);
+       }
+       KeyEditor::UpdateBindings ();
+}
 
 void
 Editor::show_editor_list (bool yn)
@@ -5821,16 +5934,16 @@ Editor::notebook_tab_clicked (GdkEventButton* ev, Gtk::Widget* page)
 
                if (_notebook_shrunk) {
                        if (pre_notebook_shrink_pane_width) {
-                               edit_pane.set_position (*pre_notebook_shrink_pane_width);
+                               edit_pane.set_divider (0, *pre_notebook_shrink_pane_width);
                        }
                        _notebook_shrunk = false;
                } else {
-                       pre_notebook_shrink_pane_width = edit_pane.get_position();
+                       pre_notebook_shrink_pane_width = edit_pane.get_divider();
 
                        /* this expands the LHS of the edit pane to cover the notebook
                           PAGE but leaves the tabs visible.
                         */
-                       edit_pane.set_position (edit_pane.get_position() + page->get_width());
+                       edit_pane.set_divider (0, edit_pane.get_divider() + page->get_width());
                        _notebook_shrunk = true;
                }
        }
@@ -5927,6 +6040,9 @@ Editor::ui_parameter_changed (string parameter)
                }
                _cursors->set_cursor_set (UIConfiguration::instance().get_icon_set());
                _cursor_stack.push_back(_cursors->grabber);
+               edit_pane.set_drag_cursor (*_cursors->expand_left_right);
+               editor_summary_pane.set_drag_cursor (*_cursors->expand_up_down);
+
        } else if (parameter == "draggable-playhead") {
                if (_verbose_cursor) {
                        playhead_cursor->set_sensitive (UIConfiguration::instance().get_draggable_playhead());
@@ -5935,11 +6051,11 @@ Editor::ui_parameter_changed (string parameter)
 }
 
 Gtk::Window*
-Editor::use_own_window ()
+Editor::use_own_window (bool and_fill_it)
 {
        bool new_window = !own_window();
-       
-       Gtk::Window* win = Tabbable::use_own_window ();
+
+       Gtk::Window* win = Tabbable::use_own_window (and_fill_it);
 
        if (win && new_window) {
                win->set_name ("EditorWindow");
@@ -5947,40 +6063,42 @@ Editor::use_own_window ()
                ARDOUR_UI::instance()->setup_toplevel_window (*win, _("Editor"), this);
 
                // win->signal_realize().connect (*this, &Editor::on_realize);
+               win->signal_event().connect (sigc::bind (sigc::ptr_fun (&Keyboard::catch_user_event_for_pre_dialog_focus), win));
                win->signal_event().connect (sigc::mem_fun (*this, &Editor::generic_event_handler));
-               
+               win->set_data ("ardour-bindings", bindings);
+
                update_title ();
        }
 
        DisplaySuspender ds;
        contents().show_all ();
-       
+
        /* XXX: this is a bit unfortunate; it would probably
           be nicer if we could just call show () above rather
           than needing the show_all ()
        */
-       
+
        /* re-hide stuff if necessary */
        editor_list_button_toggled ();
        parameter_changed ("show-summary");
        parameter_changed ("show-group-tabs");
        parameter_changed ("show-zoom-tools");
-       
+
        /* now reset all audio_time_axis heights, because widgets might need
           to be re-hidden
        */
-       
+
        TimeAxisView *tv;
-       
+
        for (TrackViewList::iterator i = track_views.begin(); i != track_views.end(); ++i) {
                tv = (static_cast<TimeAxisView*>(*i));
                tv->reset_height ();
        }
-       
+
        if (current_mixer_strip) {
                current_mixer_strip->hide_things ();
                current_mixer_strip->parameter_changed ("mixer-element-visibility");
        }
-       
+
        return win;
 }