use syntactic sugar
[ardour.git] / gtk2_ardour / editor.cc
index 37e55aded040b106ae37dea2e54f4464f659dd23..3f78e49ca775802169b9a9398056c12850737952 100644 (file)
 #include <gtkmm/menuitem.h>
 
 #include "gtkmm2ext/bindings.h"
-#include "gtkmm2ext/eventboxext.h"
-#include "gtkmm2ext/grouped_buttons.h"
 #include "gtkmm2ext/gtk_ui.h"
-#include <gtkmm2ext/keyboard.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/vca.h"
 
 #include "canvas/debug.h"
+#include "canvas/note.h"
 #include "canvas/text.h"
 
+#include "widgets/ardour_spacer.h"
+#include "widgets/eventboxext.h"
+#include "widgets/tooltips.h"
+
 #include "control_protocol/control_protocol.h"
 
 #include "actions.h"
 #include "analysis_window.h"
-#include "ardour_spacer.h"
 #include "audio_clock.h"
 #include "audio_region_view.h"
 #include "audio_streamview.h"
 #include "selection.h"
 #include "simple_progress_dialog.h"
 #include "sfdb_ui.h"
-#include "tempo_lines.h"
+#include "grid_lines.h"
 #include "time_axis_view.h"
 #include "time_info_box.h"
 #include "timers.h"
-#include "tooltips.h"
 #include "ui_config.h"
 #include "utils.h"
 #include "vca_time_axis.h"
 
 using namespace std;
 using namespace ARDOUR;
+using namespace ArdourWidgets;
 using namespace ARDOUR_UI_UTILS;
 using namespace PBD;
 using namespace Gtk;
@@ -160,44 +161,28 @@ using Gtkmm2ext::Keyboard;
 
 double Editor::timebar_height = 15.0;
 
-static const gchar *_snap_type_strings[] = {
-       N_("CD Frames"),
-       N_("TC Frames"),
-       N_("TC Seconds"),
-       N_("TC Minutes"),
-       N_("Seconds"),
-       N_("Minutes"),
-       N_("Beats/128"),
-       N_("Beats/64"),
-       N_("Beats/32"),
-       N_("Beats/28"),
-       N_("Beats/24"),
-       N_("Beats/20"),
-       N_("Beats/16"),
-       N_("Beats/14"),
-       N_("Beats/12"),
-       N_("Beats/10"),
-       N_("Beats/8"),
-       N_("Beats/7"),
-       N_("Beats/6"),
-       N_("Beats/5"),
-       N_("Beats/4"),
-       N_("Beats/3"),
-       N_("Beats/2"),
-       N_("Beats"),
-       N_("Bars"),
-       N_("Marks"),
-       N_("Region starts"),
-       N_("Region ends"),
-       N_("Region syncs"),
-       N_("Region bounds"),
-       0
-};
-
-static const gchar *_snap_mode_strings[] = {
+static const gchar *_grid_type_strings[] = {
        N_("No Grid"),
-       N_("Grid"),
-       N_("Magnetic"),
+       N_("Bar"),
+       N_("1/4 Note"),
+       N_("1/8 Note"),
+       N_("1/16 Note"),
+       N_("1/32 Note"),
+       N_("1/64 Note"),
+       N_("1/128 Note"),
+       N_("1/3 (8th triplet)"), // or "1/12" ?
+       N_("1/6 (16th triplet)"),
+       N_("1/12 (32nd triplet)"),
+       N_("1/24 (64th triplet)"),
+       N_("1/5 (8th quintuplet)"),
+       N_("1/10 (16th quintuplet)"),
+       N_("1/20 (32nd quintuplet)"),
+       N_("1/7 (8th septuplet)"),
+       N_("1/14 (16th septuplet)"),
+       N_("1/28 (32nd septuplet)"),
+       N_("Timecode"),
+       N_("MinSec"),
+       N_("CD Frames"),
        0
 };
 
@@ -239,7 +224,12 @@ static const gchar *_rb_opt_strings[] = {
 };
 #endif
 
-#define COMBO_TRIANGLE_WIDTH 25 // ArdourButton _diameter (11) + 2 * arrow-padding (2*2) + 2 * text-padding (2*5)
+/* Robin says: this should be odd to accomodate cairo drawing offset (width/2 rounds up to pixel boundary) */
+#ifdef __APPLE__
+#define COMBO_TRIANGLE_WIDTH 19 // ArdourButton _diameter (11) + 2 * arrow-padding (2*2) + 2 * text-padding (2*5)
+#else
+#define COMBO_TRIANGLE_WIDTH 11 // as-measured for win/linux.
+#endif
 
 Editor::Editor ()
        : PublicEditor (global_hpacker)
@@ -248,13 +238,13 @@ Editor::Editor ()
        , _playlist_selector (0)
        , _time_info_box (0)
        , no_save_visual (false)
-       , leftmost_frame (0)
+       , _leftmost_sample (0)
        , samples_per_pixel (2048)
        , zoom_focus (ZoomFocusPlayhead)
        , mouse_mode (MouseObject)
-       , pre_internal_snap_type (SnapToBeat)
+       , pre_internal_grid_type (GridTypeBeat)
        , pre_internal_snap_mode (SnapOff)
-       , internal_snap_type (SnapToBeat)
+       , internal_grid_type (GridTypeBeat)
        , internal_snap_mode (SnapOff)
        , _join_object_range_state (JOIN_OBJECT_RANGE_NONE)
        , _notebook_shrunk (false)
@@ -302,9 +292,11 @@ Editor::Editor ()
        , minsec_mark_interval (0)
        , minsec_mark_modulo (0)
        , minsec_nmarks (0)
+       , timecode_ruler_scale (timecode_show_many_hours)
        , timecode_mark_modulo (0)
        , timecode_nmarks (0)
        , _samples_ruler_interval (0)
+       , bbt_ruler_scale (bbt_show_many)
        , bbt_bars (0)
        , bbt_nmarks (0)
        , bbt_bar_helper_on (0)
@@ -333,7 +325,9 @@ Editor::Editor ()
        , cd_mark_label (_("CD Markers"))
        , videotl_label (_("Video Timeline"))
        , videotl_group (0)
+       , snapped_cursor (0)
        , playhead_cursor (0)
+       , _region_boundary_cache_dirty (true)
        , edit_packer (4, 4, true)
        , vertical_adjustment (0.0, 0.0, 10.0, 400.0)
        , horizontal_adjustment (0.0, 0.0, 1e16)
@@ -351,7 +345,7 @@ Editor::Editor ()
        , cut_buffer_start (0)
        , cut_buffer_length (0)
        , button_bindings (0)
-       , last_paste_pos (0)
+       , last_paste_pos (-1)
        , paste_count (0)
        , sfbrowser (0)
        , current_interthread_info (0)
@@ -363,20 +357,18 @@ Editor::Editor ()
        , scrub_reverse_distance (0)
        , have_pending_keyboard_selection (false)
        , pending_keyboard_selection_start (0)
-       , _snap_type (SnapToBeat)
+       , _grid_type (GridTypeBeat)
        , _snap_mode (SnapOff)
-       , snap_threshold (5.0)
        , ignore_gui_changes (false)
        , _drags (new DragManager (this))
        , lock_dialog (0)
          /* , last_event_time { 0, 0 } */ /* this initialization style requires C++11 */
        , _dragging_playhead (false)
        , _dragging_edit_point (false)
-       , _show_measures (true)
        , _follow_playhead (true)
        , _stationary_playhead (false)
        , _maximised (false)
-       , tempo_lines (0)
+       , grid_lines (0)
        , global_rect_group (0)
        , time_line_group (0)
        , tempo_marker_menu (0)
@@ -398,6 +390,7 @@ Editor::Editor ()
        , _ignore_region_action (false)
        , _last_region_menu_was_main (false)
        , _track_selection_change_without_scroll (false)
+       , _editor_track_selection_change_without_scroll (false)
        , cd_marker_bar_drag_rect (0)
        , range_bar_drag_rect (0)
        , transport_bar_drag_rect (0)
@@ -455,7 +448,6 @@ Editor::Editor ()
        , _stepping_axis_view (0)
        , quantize_dialog (0)
        , _main_menu_disabler (0)
-       , myactions (X_("editor"))
 {
        /* we are a singleton */
 
@@ -469,8 +461,7 @@ Editor::Editor ()
        selection_op_history.clear();
        before.clear();
 
-       snap_type_strings =  I18N (_snap_type_strings);
-       snap_mode_strings =  I18N (_snap_mode_strings);
+       grid_type_strings =  I18N (_grid_type_strings);
        zoom_focus_strings = I18N (_zoom_focus_strings);
        edit_mode_strings = I18N (_edit_mode_strings);
        edit_point_strings = I18N (_edit_point_strings);
@@ -482,8 +473,7 @@ Editor::Editor ()
        build_edit_mode_menu();
        build_zoom_focus_menu();
        build_track_count_menu();
-       build_snap_mode_menu();
-       build_snap_type_menu();
+       build_grid_type_menu();
        build_edit_point_menu();
 
        location_marker_color = UIConfiguration::instance().color ("location marker");
@@ -583,6 +573,7 @@ Editor::Editor ()
        _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));
 
@@ -620,7 +611,7 @@ Editor::Editor ()
        pad_line_1->set_outline_color (0xFF0000FF);
        pad_line_1->show();
 
-       // CAIROCANVAS
+       /* CAIROCANVAS */
        time_pad->show();
 
        edit_packer.set_col_spacings (0);
@@ -633,12 +624,19 @@ Editor::Editor ()
        time_bars_event_box.set_events (Gdk::BUTTON_PRESS_MASK|Gdk::BUTTON_RELEASE_MASK);
        time_bars_event_box.signal_button_release_event().connect (sigc::mem_fun(*this, &Editor::ruler_label_button_release));
 
+       ArdourWidgets::ArdourDropShadow *axis_view_shadow = manage (new (ArdourWidgets::ArdourDropShadow));
+       axis_view_shadow->set_size_request (4, -1);
+       axis_view_shadow->set_name("EditorWindow");
+       axis_view_shadow->show();
+
+       edit_packer.attach (*axis_view_shadow,     0, 1, 0, 2,    FILL,        FILL|EXPAND, 0, 0);
+
        /* labels for the time bars */
-       edit_packer.attach (time_bars_event_box,     0, 1, 0, 1,    FILL,        SHRINK, 0, 0);
+       edit_packer.attach (time_bars_event_box,     1, 2, 0, 1,    FILL,        SHRINK, 0, 0);
        /* track controls */
-       edit_packer.attach (controls_layout,         0, 1, 1, 2,    FILL,        FILL|EXPAND, 0, 0);
+       edit_packer.attach (controls_layout,         1, 2, 1, 2,    FILL,        FILL|EXPAND, 0, 0);
        /* canvas */
-       edit_packer.attach (*_track_canvas_viewport,  1, 2, 0, 2,    FILL|EXPAND, FILL|EXPAND, 0, 0);
+       edit_packer.attach (*_track_canvas_viewport,  2, 3, 0, 2,    FILL|EXPAND, FILL|EXPAND, 0, 0);
 
        bottom_hbox.set_border_width (2);
        bottom_hbox.set_spacing (3);
@@ -684,33 +682,21 @@ Editor::Editor ()
        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)));
-       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_arrow_left = manage (new Button);
+       summary_arrow_left->add (*manage (new Arrow (ARROW_LEFT, SHADOW_NONE)));
+       summary_arrow_left->signal_pressed().connect (sigc::hide_return (sigc::bind (sigc::mem_fun (*this, &Editor::scroll_press), LEFT)));
+       summary_arrow_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));
+       Button* summary_arrow_right = manage (new Button);
+       summary_arrow_right->add (*manage (new Arrow (ARROW_RIGHT, SHADOW_NONE)));
+       summary_arrow_right->signal_pressed().connect (sigc::hide_return (sigc::bind (sigc::mem_fun (*this, &Editor::scroll_press), RIGHT)));
+       summary_arrow_right->signal_released().connect (sigc::mem_fun (*this, &Editor::scroll_release));
 
        VBox* summary_arrows_left = manage (new VBox);
-       summary_arrows_left->pack_start (*summary_arrows_left_left);
-       summary_arrows_left->pack_start (*summary_arrows_left_right);
-
-       Button* summary_arrows_right_up = manage (new Button);
-       summary_arrows_right_up->add (*manage (new Arrow (ARROW_UP, SHADOW_NONE)));
-       summary_arrows_right_up->signal_pressed().connect (sigc::hide_return (sigc::bind (sigc::mem_fun (*this, &Editor::scroll_press), UP)));
-       summary_arrows_right_up->signal_released().connect (sigc::mem_fun (*this, &Editor::scroll_release));
-
-       Button* summary_arrows_right_down = manage (new Button);
-       summary_arrows_right_down->add (*manage (new Arrow (ARROW_DOWN, SHADOW_NONE)));
-       summary_arrows_right_down->signal_pressed().connect (sigc::hide_return (sigc::bind (sigc::mem_fun (*this, &Editor::scroll_press), DOWN)));
-       summary_arrows_right_down->signal_released().connect (sigc::mem_fun (*this, &Editor::scroll_release));
+       summary_arrows_left->pack_start (*summary_arrow_left);
 
        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);
+       summary_arrows_right->pack_start (*summary_arrow_right);
 
        Frame* summary_frame = manage (new Frame);
        summary_frame->set_shadow_type (Gtk::SHADOW_ETCHED_IN);
@@ -751,23 +737,30 @@ Editor::Editor ()
        }
        editor_summary_pane.set_divider (0, fract);
 
-       global_vpacker.set_spacing (2);
+       global_vpacker.set_spacing (0);
        global_vpacker.set_border_width (0);
 
-       //the next three EventBoxes provide the ability for their child widgets to have a background color.  That is all.
+       /* the next three EventBoxes provide the ability for their child widgets to have a background color.  That is all. */
 
-       Gtk::EventBox* ebox = manage (new Gtk::EventBox);  //a themeable box
+       Gtk::EventBox* ebox = manage (new Gtk::EventBox); // a themeable box
        ebox->set_name("EditorWindow");
-       ebox->add (toolbar_hbox);
+       ebox->add (ebox_hpacker);
 
-       Gtk::EventBox* epane_box = manage (new Gtkmm2ext::EventBoxExt);  //a themeable box
+       Gtk::EventBox* epane_box = manage (new EventBoxExt); // a themeable box
        epane_box->set_name("EditorWindow");
        epane_box->add (edit_pane);
 
-       Gtk::EventBox* epane_box2 = manage (new Gtkmm2ext::EventBoxExt);  //a themeable box
+       Gtk::EventBox* epane_box2 = manage (new EventBoxExt); // a themeable box
        epane_box2->set_name("EditorWindow");
        epane_box2->add (global_vpacker);
 
+       ArdourWidgets::ArdourDropShadow *toolbar_shadow = manage (new (ArdourWidgets::ArdourDropShadow));
+       toolbar_shadow->set_size_request (-1, 4);
+       toolbar_shadow->set_mode(ArdourWidgets::ArdourDropShadow::DropShadowBoth);
+       toolbar_shadow->set_name("EditorWindow");
+       toolbar_shadow->show();
+
+       global_vpacker.pack_start (*toolbar_shadow, false, false);
        global_vpacker.pack_start (*ebox, false, false);
        global_vpacker.pack_start (*epane_box, true, true);
        global_hpacker.pack_start (*epane_box2, true, true);
@@ -776,11 +769,13 @@ Editor::Editor ()
         */
 
        global_hpacker.show ();
+       ebox_hpacker.show();
+       ebox->show();
 
        /* register actions now so that set_state() can find them and set toggles/checks etc */
 
-       register_actions ();
        load_bindings ();
+       register_actions ();
 
        setup_toolbar ();
 
@@ -845,16 +840,16 @@ Editor::Editor ()
 
        _show_marker_lines = false;
 
-        /* Button bindings */
+       /* Button 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_operation (**i);
-                }
-        }
+       if (node) {
+               for (XMLNodeList::const_iterator i = node->children().begin(); i != node->children().end(); ++i) {
+                       button_bindings->load_operation (**i);
+               }
+       }
 
        constructed = true;
 
@@ -864,7 +859,7 @@ Editor::Editor ()
 
        setup_fade_images ();
 
-       instant_save ();
+       set_grid_to (GridTypeNone);
 }
 
 Editor::~Editor()
@@ -917,7 +912,7 @@ Editor::button_settings () const
 bool
 Editor::get_smart_mode () const
 {
-       return ((current_mouse_mode() == Editing::MouseObject) && smart_mode_action->get_active());
+       return ((current_mouse_mode() == MouseObject) && smart_mode_action->get_active());
 }
 
 void
@@ -986,15 +981,11 @@ Editor::set_entered_track (TimeAxisView* tav)
 void
 Editor::instant_save ()
 {
-       if (!constructed || !ARDOUR_UI::instance()->session_loaded || no_save_instant) {
+       if (!constructed || !_session || no_save_instant) {
                return;
        }
 
-       if (_session) {
-               _session->add_instant_xml(get_state());
-       } else {
-               Config->add_instant_xml(get_state());
-       }
+       _session->add_instant_xml(get_state());
 }
 
 void
@@ -1083,25 +1074,25 @@ Editor::control_scroll (float fraction)
        /*
                _control_scroll_target is an optional<T>
 
-               it acts like a pointer to an framepos_t, with
+               it acts like a pointer to an samplepos_t, with
                a operator conversion to boolean to check
                that it has a value could possibly use
-               playhead_cursor->current_frame to store the
+               playhead_cursor->current_sample to store the
                value and a boolean in the class to know
                when it's out of date
        */
 
        if (!_control_scroll_target) {
-               _control_scroll_target = _session->transport_frame();
+               _control_scroll_target = _session->transport_sample();
                _dragging_playhead = true;
        }
 
-       if ((fraction < 0.0f) && (*_control_scroll_target <= (framepos_t) fabs(step))) {
+       if ((fraction < 0.0f) && (*_control_scroll_target <= (samplepos_t) fabs(step))) {
                *_control_scroll_target = 0;
-       } else if ((fraction > 0.0f) && (max_framepos - *_control_scroll_target < step)) {
-               *_control_scroll_target = max_framepos - (current_page_samples()*2); // allow room for slop in where the PH is on the screen
+       } else if ((fraction > 0.0f) && (max_samplepos - *_control_scroll_target < step)) {
+               *_control_scroll_target = max_samplepos - (current_page_samples()*2); // allow room for slop in where the PH is on the screen
        } else {
-               *_control_scroll_target += (framepos_t) trunc (step);
+               *_control_scroll_target += (samplepos_t) trunc (step);
        }
 
        /* move visuals, we'll catch up with it later */
@@ -1132,17 +1123,17 @@ Editor::control_scroll (float fraction)
 }
 
 bool
-Editor::deferred_control_scroll (framepos_t /*target*/)
+Editor::deferred_control_scroll (samplepos_t /*target*/)
 {
        _session->request_locate (*_control_scroll_target, _session->transport_rolling());
-       // reset for next stream
+       /* reset for next stream */
        _control_scroll_target = boost::none;
        _dragging_playhead = false;
        return false;
 }
 
 void
-Editor::access_action (std::string action_group, std::string action_item)
+Editor::access_action (const std::string& action_group, const std::string& action_item)
 {
        if (!_session) {
                return;
@@ -1151,13 +1142,19 @@ Editor::access_action (std::string action_group, std::string action_item)
        ENSURE_GUI_THREAD (*this, &Editor::access_action, action_group, action_item)
 
        RefPtr<Action> act;
-       act = ActionManager::get_action( action_group.c_str(), action_item.c_str() );
+       act = ActionManager::get_action (action_group.c_str(), action_item.c_str());
 
        if (act) {
                act->activate();
        }
 }
 
+void
+Editor::set_toggleaction (const std::string& action_group, const std::string& action_item, bool s)
+{
+       ActionManager::set_toggleaction_state (action_group.c_str(), action_item.c_str(), s);
+}
+
 void
 Editor::on_realize ()
 {
@@ -1238,46 +1235,46 @@ Editor::lock_timeout_callback ()
 }
 
 void
-Editor::map_position_change (framepos_t frame)
+Editor::map_position_change (samplepos_t sample)
 {
-       ENSURE_GUI_THREAD (*this, &Editor::map_position_change, frame)
+       ENSURE_GUI_THREAD (*this, &Editor::map_position_change, sample)
 
        if (_session == 0) {
                return;
        }
 
        if (_follow_playhead) {
-               center_screen (frame);
+               center_screen (sample);
        }
 
-       playhead_cursor->set_position (frame);
+       playhead_cursor->set_position (sample);
 }
 
 void
-Editor::center_screen (framepos_t frame)
+Editor::center_screen (samplepos_t sample)
 {
-       framecnt_t const page = _visible_canvas_width * samples_per_pixel;
+       samplecnt_t const page = _visible_canvas_width * samples_per_pixel;
 
        /* if we're off the page, then scroll.
         */
 
-       if (frame < leftmost_frame || frame >= leftmost_frame + page) {
-               center_screen_internal (frame, page);
+       if (sample < _leftmost_sample || sample >= _leftmost_sample + page) {
+               center_screen_internal (sample, page);
        }
 }
 
 void
-Editor::center_screen_internal (framepos_t frame, float page)
+Editor::center_screen_internal (samplepos_t sample, float page)
 {
        page /= 2;
 
-       if (frame > page) {
-               frame -= (framepos_t) page;
+       if (sample > page) {
+               sample -= (samplepos_t) page;
        } else {
-               frame = 0;
+               sample = 0;
        }
 
-       reset_x_origin (frame);
+       reset_x_origin (sample);
 }
 
 
@@ -1323,6 +1320,11 @@ Editor::set_session (Session *t)
                return;
        }
 
+       /* initialize _leftmost_sample to the extents of the session
+        * this prevents a bogus setting of leftmost = "0" if the summary view asks for the leftmost sample
+        * before the visible state has been loaded from instant.xml */
+       _leftmost_sample = session_gui_extents().first;
+
        _playlist_selector->set_session (_session);
        nudge_clock->set_session (_session);
        _summary->set_session (_session);
@@ -1357,15 +1359,15 @@ Editor::set_session (Session *t)
 
        loc = _session->locations()->auto_punch_location();
        if (loc != 0) {
-               // force name
+               /* 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.
-       */
+        * 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);
 
@@ -1377,7 +1379,7 @@ Editor::set_session (Session *t)
 
        /* catch up with the playhead */
 
-       _session->request_locate (playhead_cursor->current_frame ());
+       _session->request_locate (playhead_cursor->current_sample ());
        _pending_initial_locate = true;
 
        update_title ();
@@ -1405,8 +1407,13 @@ Editor::set_session (Session *t)
        _session->locations()->changed.connect (_session_connections, invalidator (*this), boost::bind (&Editor::refresh_location_display, this), gui_context());
        _session->history().Changed.connect (_session_connections, invalidator (*this), boost::bind (&Editor::history_changed, this), gui_context());
 
+       playhead_cursor->track_canvas_item().reparent ((ArdourCanvas::Item*) get_cursor_scroll_group());
        playhead_cursor->show ();
 
+       snapped_cursor->track_canvas_item().reparent ((ArdourCanvas::Item*) get_cursor_scroll_group());
+       snapped_cursor->set_color (UIConfiguration::instance().color ("edit point"));
+       snapped_cursor->show ();
+
        boost::function<void (string)> pc (boost::bind (&Editor::parameter_changed, this, _1));
        Config->map_parameters (pc);
        _session->config.map_parameters (pc);
@@ -1423,18 +1430,6 @@ Editor::set_session (Session *t)
                sigc::mem_fun (*this, &Editor::super_rapid_screen_update)
                );
 
-       switch (_snap_type) {
-       case SnapToRegionStart:
-       case SnapToRegionEnd:
-       case SnapToRegionSync:
-       case SnapToRegionBoundary:
-               build_region_boundary_cache ();
-               break;
-
-       default:
-               break;
-       }
-
        /* register for undo history */
        _session->register_with_memento_command_factory(id(), this);
        _session->register_with_memento_command_factory(_selection_memento->id(), _selection_memento);
@@ -1740,7 +1735,7 @@ Editor::loudness_analyze_region_selection ()
        Selection& s (PublicEditor::instance ().get_selection ());
        RegionSelection ars = s.regions;
        ARDOUR::AnalysisGraph ag (_session);
-       framecnt_t total_work = 0;
+       samplecnt_t total_work = 0;
 
        for (RegionSelection::iterator j = ars.begin (); j != ars.end (); ++j) {
                AudioRegionView* arv = dynamic_cast<AudioRegionView*> (*j);
@@ -1756,7 +1751,7 @@ Editor::loudness_analyze_region_selection ()
 
        SimpleProgressDialog spd (_("Region Loudness Analysis"), sigc::mem_fun (ag, &AnalysisGraph::cancel));
        ScopedConnection c;
-       ag.set_total_frames (total_work);
+       ag.set_total_samples (total_work);
        ag.Progress.connect_same_thread (c, boost::bind (&SimpleProgressDialog::update_progress, &spd, _1, _2));
        spd.show();
 
@@ -1787,7 +1782,7 @@ Editor::loudness_analyze_range_selection ()
        Selection& s (PublicEditor::instance ().get_selection ());
        TimeSelection ts = s.time;
        ARDOUR::AnalysisGraph ag (_session);
-       framecnt_t total_work = 0;
+       samplecnt_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 ());
@@ -1805,7 +1800,7 @@ Editor::loudness_analyze_range_selection ()
 
        SimpleProgressDialog spd (_("Range Loudness Analysis"), sigc::mem_fun (ag, &AnalysisGraph::cancel));
        ScopedConnection c;
-       ag.set_total_frames (total_work);
+       ag.set_total_samples (total_work);
        ag.Progress.connect_same_thread (c, boost::bind (&SimpleProgressDialog::update_progress, &spd, _1, _2));
        spd.show();
 
@@ -1888,20 +1883,10 @@ Editor::add_region_context_items (Menu_Helpers::MenuList& edit_items, boost::sha
 
        RegionSelection rs = get_regions_from_selection_and_entered ();
 
-       string::size_type pos = 0;
        string menu_item_name = (rs.size() == 1) ? rs.front()->region()->name() : _("Selected Regions");
 
-       /* we have to hack up the region name because "_" has a special
-          meaning for menu titles.
-       */
-
-       while ((pos = menu_item_name.find ("_", pos)) != string::npos) {
-               menu_item_name.replace (pos, 1, "__");
-               pos += 2;
-       }
-
        if (_popup_region_menu_item == 0) {
-               _popup_region_menu_item = new MenuItem (menu_item_name);
+               _popup_region_menu_item = new MenuItem (menu_item_name, false);
                _popup_region_menu_item->set_submenu (*dynamic_cast<Menu*> (ActionManager::get_widget (X_("/PopupRegionMenu"))));
                _popup_region_menu_item->show ();
        } else {
@@ -1916,7 +1901,7 @@ Editor::add_region_context_items (Menu_Helpers::MenuList& edit_items, boost::sha
                act->set_sensitive (true);
        }
 
-       const framepos_t position = get_preferred_edit_position (EDIT_IGNORE_NONE, true);
+       const samplepos_t position = get_preferred_edit_position (EDIT_IGNORE_NONE, true);
 
        edit_items.push_back (*_popup_region_menu_item);
        if (Config->get_layer_model() == Manual && track->playlist()->count_regions_at (position) > 1 && (layering_order_editor == 0 || !layering_order_editor->is_visible ())) {
@@ -2142,43 +2127,73 @@ Editor::add_bus_context_items (Menu_Helpers::MenuList& edit_items)
        edit_items.push_back (MenuElem (_("Nudge"), *nudge_menu));
 }
 
-SnapType
-Editor::snap_type() const
+GridType
+Editor::grid_type() const
 {
-       return _snap_type;
+       return _grid_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:
+Editor::grid_musical() const
+{
+       switch (_grid_type) {
+       case GridTypeBeatDiv32:
+       case GridTypeBeatDiv28:
+       case GridTypeBeatDiv24:
+       case GridTypeBeatDiv20:
+       case GridTypeBeatDiv16:
+       case GridTypeBeatDiv14:
+       case GridTypeBeatDiv12:
+       case GridTypeBeatDiv10:
+       case GridTypeBeatDiv8:
+       case GridTypeBeatDiv7:
+       case GridTypeBeatDiv6:
+       case GridTypeBeatDiv5:
+       case GridTypeBeatDiv4:
+       case GridTypeBeatDiv3:
+       case GridTypeBeatDiv2:
+       case GridTypeBeat:
+       case GridTypeBar:
                return true;
-       default:
-               break;
+       case GridTypeNone:
+       case GridTypeTimecode:
+       case GridTypeMinSec:
+       case GridTypeCDFrame:
+               return false;
        }
-
        return false;
 }
 
+bool
+Editor::grid_nonmusical() const
+{
+       switch (_grid_type) {
+       case GridTypeTimecode:
+       case GridTypeMinSec:
+       case GridTypeCDFrame:
+               return true;
+       case GridTypeBeatDiv32:
+       case GridTypeBeatDiv28:
+       case GridTypeBeatDiv24:
+       case GridTypeBeatDiv20:
+       case GridTypeBeatDiv16:
+       case GridTypeBeatDiv14:
+       case GridTypeBeatDiv12:
+       case GridTypeBeatDiv10:
+       case GridTypeBeatDiv8:
+       case GridTypeBeatDiv7:
+       case GridTypeBeatDiv6:
+       case GridTypeBeatDiv5:
+       case GridTypeBeatDiv4:
+       case GridTypeBeatDiv3:
+       case GridTypeBeatDiv2:
+       case GridTypeBeat:
+       case GridTypeBar:
+       case GridTypeNone:
+               return false;
+       }
+       return false;
+}
 SnapMode
 Editor::snap_mode() const
 {
@@ -2186,67 +2201,95 @@ Editor::snap_mode() const
 }
 
 void
-Editor::set_snap_to (SnapType st)
+Editor::show_rulers_for_grid ()
 {
-       unsigned int snap_ind = (unsigned int)st;
+       /* show appropriate rulers for this grid setting. */
+       if (grid_musical()) {
+               ruler_tempo_action->set_active(true);
+               ruler_meter_action->set_active(true);
+               ruler_bbt_action->set_active(true);
 
-       if (internal_editing()) {
-               internal_snap_type = st;
+               if (UIConfiguration::instance().get_rulers_follow_grid()) {
+                       ruler_timecode_action->set_active(false);
+                       ruler_minsec_action->set_active(false);
+                       ruler_samples_action->set_active(false);
+               }
+       } else if (_grid_type == GridTypeTimecode) {
+               ruler_timecode_action->set_active(true);
+
+               if (UIConfiguration::instance().get_rulers_follow_grid()) {
+                       ruler_tempo_action->set_active(false);
+                       ruler_meter_action->set_active(false);
+                       ruler_bbt_action->set_active(false);
+                       ruler_minsec_action->set_active(false);
+                       ruler_samples_action->set_active(false);
+               }
+       } else if (_grid_type == GridTypeMinSec) {
+               ruler_minsec_action->set_active(true);
+
+               if (UIConfiguration::instance().get_rulers_follow_grid()) {
+                       ruler_tempo_action->set_active(false);
+                       ruler_meter_action->set_active(false);
+                       ruler_bbt_action->set_active(false);
+                       ruler_timecode_action->set_active(false);
+                       ruler_samples_action->set_active(false);
+               }
+       } else if (_grid_type == GridTypeCDFrame) {
+               ruler_cd_marker_action->set_active(true);
+               ruler_minsec_action->set_active(true);
+
+               if (UIConfiguration::instance().get_rulers_follow_grid()) {
+                       ruler_tempo_action->set_active(false);
+                       ruler_meter_action->set_active(false);
+                       ruler_bbt_action->set_active(false);
+                       ruler_timecode_action->set_active(false);
+                       ruler_samples_action->set_active(false);
+               }
+       }
+}
+
+void
+Editor::set_grid_to (GridType gt)
+{
+       if (_grid_type == gt) { // already set
+               return;
+       }
+
+       unsigned int grid_ind = (unsigned int)gt;
+
+       if (internal_editing() && UIConfiguration::instance().get_grid_follows_internal()) {
+               internal_grid_type = gt;
        } else {
-               pre_internal_snap_type = st;
+               pre_internal_grid_type = gt;
        }
 
-       _snap_type = st;
+       _grid_type = gt;
 
-       if (snap_ind > snap_type_strings.size() - 1) {
-               snap_ind = 0;
-               _snap_type = (SnapType)snap_ind;
+       if (grid_ind > grid_type_strings.size() - 1) {
+               grid_ind = 0;
+               _grid_type = (GridType)grid_ind;
        }
 
-       string str = snap_type_strings[snap_ind];
+       string str = grid_type_strings[grid_ind];
 
-       if (str != snap_type_selector.get_text()) {
-               snap_type_selector.set_text (str);
+       if (str != grid_type_selector.get_text()) {
+               grid_type_selector.set_text (str);
+       }
+
+       if (UIConfiguration::instance().get_show_grids_ruler()) {
+               show_rulers_for_grid ();
        }
 
        instant_save ();
 
-       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: {
-               compute_bbt_ruler_scale (leftmost_frame, leftmost_frame + current_page_samples());
+       if (grid_musical()) {
+               compute_bbt_ruler_scale (_leftmost_sample, _leftmost_sample + current_page_samples());
                update_tempo_based_rulers ();
-               break;
        }
 
-       case SnapToRegionStart:
-       case SnapToRegionEnd:
-       case SnapToRegionSync:
-       case SnapToRegionBoundary:
-               build_region_boundary_cache ();
-               break;
-
-       default:
-               /* relax */
-               break;
-       }
+       mark_region_boundary_cache_dirty ();
 
-       redisplay_tempo (false);
+       redisplay_grid (false);
 
        SnapChanged (); /* EMIT SIGNAL */
 }
@@ -2254,8 +2297,6 @@ Editor::set_snap_to (SnapType st)
 void
 Editor::set_snap_mode (SnapMode mode)
 {
-       string str = snap_mode_strings[(int)mode];
-
        if (internal_editing()) {
                internal_snap_mode = mode;
        } else {
@@ -2264,8 +2305,10 @@ Editor::set_snap_mode (SnapMode mode)
 
        _snap_mode = mode;
 
-       if (str != snap_mode_selector.get_text ()) {
-               snap_mode_selector.set_text (str);
+       if (_snap_mode == SnapOff) {
+               snap_mode_button.set_active_state (Gtkmm2ext::Off);
+       } else {
+               snap_mode_button.set_active_state (Gtkmm2ext::ExplicitActive);
        }
 
        instant_save ();
@@ -2311,10 +2354,10 @@ Editor::set_edit_point_preference (EditPoint ep, bool force)
                Glib::RefPtr<RadioAction>::cast_dynamic(act)->set_active (true);
        }
 
-       framepos_t foo;
+       samplepos_t foo;
        bool in_track_canvas;
 
-       if (!mouse_frame (foo, in_track_canvas)) {
+       if (!mouse_sample (foo, in_track_canvas)) {
                in_track_canvas = false;
        }
 
@@ -2333,7 +2376,7 @@ Editor::set_state (const XMLNode& node, int version)
 
        Tabbable::set_state (node, version);
 
-       framepos_t ph_pos;
+       samplepos_t ph_pos;
        if (_session && node.get_property ("playhead", ph_pos)) {
                if (ph_pos >= 0) {
                        playhead_cursor->set_position (ph_pos);
@@ -2363,11 +2406,11 @@ Editor::set_state (const XMLNode& node, int version)
                set_visible_track_count (cnt);
        }
 
-       SnapType snap_type;
-       if (!node.get_property ("snap-to", snap_type)) {
-               snap_type = _snap_type;
+       GridType grid_type;
+       if (!node.get_property ("grid-type", grid_type)) {
+               grid_type = _grid_type;
        }
-       set_snap_to (snap_type);
+       set_grid_to (grid_type);
 
        SnapMode sm;
        if (node.get_property ("snap-mode", sm)) {
@@ -2381,9 +2424,9 @@ Editor::set_state (const XMLNode& node, int version)
                set_snap_mode (_snap_mode);
        }
 
-       node.get_property ("internal-snap-to", internal_snap_type);
+       node.get_property ("internal-grid-type", internal_grid_type);
        node.get_property ("internal-snap-mode", internal_snap_mode);
-       node.get_property ("pre-internal-snap-to", pre_internal_snap_type);
+       node.get_property ("pre-internal-grid-type", pre_internal_grid_type);
        node.get_property ("pre-internal-snap-mode", pre_internal_snap_mode);
 
        std::string mm_str;
@@ -2394,7 +2437,7 @@ Editor::set_state (const XMLNode& node, int version)
                set_mouse_mode (MouseObject, true);
        }
 
-       framepos_t lf_pos;
+       samplepos_t lf_pos;
        if (node.get_property ("left-frame", lf_pos)) {
                if (lf_pos < 0) {
                        lf_pos = 0;
@@ -2424,8 +2467,6 @@ Editor::set_state (const XMLNode& node, int version)
                set_edit_point_preference (_edit_point);
        }
 
-       node.get_property ("show-measures", _show_measures);
-
        if (node.get_property ("follow-playhead", yn)) {
                set_follow_playhead (yn);
        }
@@ -2496,12 +2537,12 @@ Editor::set_state (const XMLNode& node, int version)
                }
        }
 
-       framepos_t nudge_clock_value;
+       samplepos_t nudge_clock_value;
        if (node.get_property ("nudge-clock-value", nudge_clock_value)) {
                nudge_clock->set (nudge_clock_value);
        } else {
                nudge_clock->set_mode (AudioClock::Timecode);
-               nudge_clock->set (_session->frame_rate() * 5, true);
+               nudge_clock->set (_session->sample_rate() * 5, true);
        }
 
        {
@@ -2511,15 +2552,6 @@ Editor::set_state (const XMLNode& node, int version)
                 */
                RefPtr<Action> act;
 
-               act = ActionManager::get_action (X_("Editor"), X_("ToggleMeasureVisibility"));
-               if (act) {
-                       yn = _show_measures;
-                       RefPtr<ToggleAction> tact = RefPtr<ToggleAction>::cast_dynamic(act);
-                       /* do it twice to force the change */
-                       tact->set_active (!yn);
-                       tact->set_active (yn);
-               }
-
                act = ActionManager::get_action (X_("Editor"), X_("toggle-follow-playhead"));
                yn = _follow_playhead;
                if (act) {
@@ -2539,7 +2571,7 @@ Editor::set_state (const XMLNode& node, int version)
                }
        }
 
-       return LuaInstance::instance()->set_state(node);
+       return 0;
 }
 
 XMLNode&
@@ -2560,20 +2592,19 @@ Editor::get_state ()
        node->set_property ("zoom-focus", zoom_focus);
 
        node->set_property ("zoom", samples_per_pixel);
-       node->set_property ("snap-to", _snap_type);
+       node->set_property ("grid-type", _grid_type);
        node->set_property ("snap-mode", _snap_mode);
-       node->set_property ("internal-snap-to", internal_snap_type);
+       node->set_property ("internal-grid-type", internal_grid_type);
        node->set_property ("internal-snap-mode", internal_snap_mode);
-       node->set_property ("pre-internal-snap-to", pre_internal_snap_type);
+       node->set_property ("pre-internal-grid-type", pre_internal_grid_type);
        node->set_property ("pre-internal-snap-mode", pre_internal_snap_mode);
        node->set_property ("edit-point", _edit_point);
        node->set_property ("visible-track-count", _visible_track_count);
 
-       node->set_property ("playhead", playhead_cursor->current_frame ());
-       node->set_property ("left-frame", leftmost_frame);
+       node->set_property ("playhead", playhead_cursor->current_sample ());
+       node->set_property ("left-frame", _leftmost_sample);
        node->set_property ("y-origin", vertical_adjustment.get_value ());
 
-       node->set_property ("show-measures", _show_measures);
        node->set_property ("maximised", _maximised);
        node->set_property ("follow-playhead", _follow_playhead);
        node->set_property ("stationary-playhead", _stationary_playhead);
@@ -2608,8 +2639,6 @@ Editor::get_state ()
 
        node->set_property ("nudge-clock-value", nudge_clock->current_duration());
 
-       node->add_child_nocopy (LuaInstance::instance()->get_action_state());
-       node->add_child_nocopy (LuaInstance::instance()->get_hook_state());
        node->add_child_nocopy (_locations->get_state ());
 
        return *node;
@@ -2631,7 +2660,7 @@ Editor::trackview_by_y_position (double y, bool trackview_relative_offset) const
        }
 
        if (y < 0) {
-               return std::make_pair ( (TimeAxisView *) 0, 0);
+               return std::make_pair ((TimeAxisView *) 0, 0);
        }
 
        for (TrackViewList::const_iterator iter = track_views.begin(); iter != track_views.end(); ++iter) {
@@ -2643,16 +2672,25 @@ Editor::trackview_by_y_position (double y, bool trackview_relative_offset) const
                }
        }
 
-       return std::make_pair ( (TimeAxisView *) 0, 0);
+       return std::make_pair ((TimeAxisView *) 0, 0);
 }
 
+void
+Editor::set_snapped_cursor_position (samplepos_t pos)
+{
+       if (_edit_point == EditAtMouse) {
+               snapped_cursor->set_position(pos);
+       }
+}
+
+
 /** Snap a position to the grid, if appropriate, taking into account current
  *  grid settings and also the state of any snap modifier keys that may be pressed.
  *  @param start Position to snap.
  *  @param event Event to get current key modifier information from, or 0.
  */
 void
-Editor::snap_to_with_modifier (MusicFrame& start, GdkEvent const * event, RoundMode direction, bool for_mark)
+Editor::snap_to_with_modifier (MusicSample& start, GdkEvent const * event, RoundMode direction, SnapPref pref)
 {
        if (!_session || !event) {
                return;
@@ -2660,53 +2698,68 @@ Editor::snap_to_with_modifier (MusicFrame& start, GdkEvent const * event, RoundM
 
        if (ArdourKeyboard::indicates_snap (event->button.state)) {
                if (_snap_mode == SnapOff) {
-                       snap_to_internal (start, direction, for_mark);
+                       snap_to_internal (start, direction, pref);
                } else {
-                       start.set (start.frame, 0);
+                       start.set (start.sample, 0);
                }
        } else {
                if (_snap_mode != SnapOff) {
-                       snap_to_internal (start, direction, for_mark);
+                       snap_to_internal (start, direction, pref);
                } else if (ArdourKeyboard::indicates_snap_delta (event->button.state)) {
                        /* SnapOff, but we pressed the snap_delta modifier */
-                       snap_to_internal (start, direction, for_mark);
+                       snap_to_internal (start, direction, pref);
                } else {
-                       start.set (start.frame, 0);
+                       start.set (start.sample, 0);
                }
        }
 }
 
 void
-Editor::snap_to (MusicFrame& start, RoundMode direction, bool for_mark, bool ensure_snap)
+Editor::snap_to (MusicSample& start, RoundMode direction, SnapPref pref, bool ensure_snap)
 {
        if (!_session || (_snap_mode == SnapOff && !ensure_snap)) {
-               start.set (start.frame, 0);
+               start.set (start.sample, 0);
                return;
        }
 
-       snap_to_internal (start, direction, for_mark, ensure_snap);
+       snap_to_internal (start, direction, pref, ensure_snap);
 }
 
 void
-Editor::timecode_snap_to_internal (MusicFrame& pos, RoundMode direction, bool /*for_mark*/)
+check_best_snap (samplepos_t presnap, samplepos_t &test, samplepos_t &dist, samplepos_t &best)
 {
-       framepos_t start = pos.frame;
-       const framepos_t one_timecode_second = (framepos_t)(rint(_session->timecode_frames_per_second()) * _session->samples_per_timecode_frame());
-       framepos_t one_timecode_minute = (framepos_t)(rint(_session->timecode_frames_per_second()) * _session->samples_per_timecode_frame() * 60);
+       samplepos_t diff = abs (test - presnap);
+       if (diff < dist) {
+               dist = diff;
+               best = test;
+       }
 
-       switch (_snap_type) {
-       case SnapToTimecodeFrame:
+       test = max_samplepos; // reset this so it doesn't get accidentally reused
+}
+
+MusicSample
+Editor::snap_to_timecode (MusicSample presnap, RoundMode direction, SnapPref gpref)
+{
+       samplepos_t start = presnap.sample;
+       const samplepos_t one_timecode_second = (samplepos_t)(rint(_session->timecode_frames_per_second()) * _session->samples_per_timecode_frame());
+       samplepos_t one_timecode_minute = (samplepos_t)(rint(_session->timecode_frames_per_second()) * _session->samples_per_timecode_frame() * 60);
+
+       TimecodeRulerScale scale = (gpref != SnapToGrid_Unscaled) ? timecode_ruler_scale : timecode_show_samples;
+
+       switch (scale) {
+       case timecode_show_bits:
+       case timecode_show_samples:
                if ((direction == RoundUpMaybe || direction == RoundDownMaybe) &&
                    fmod((double)start, (double)_session->samples_per_timecode_frame()) == 0) {
                        /* start is already on a whole timecode frame, do nothing */
                } else if (((direction == 0) && (fmod((double)start, (double)_session->samples_per_timecode_frame()) > (_session->samples_per_timecode_frame() / 2))) || (direction > 0)) {
-                       start = (framepos_t) (ceil ((double) start / _session->samples_per_timecode_frame()) * _session->samples_per_timecode_frame());
+                       start = (samplepos_t) (ceil ((double) start / _session->samples_per_timecode_frame()) * _session->samples_per_timecode_frame());
                } else {
-                       start = (framepos_t) (floor ((double) start / _session->samples_per_timecode_frame()) *  _session->samples_per_timecode_frame());
+                       start = (samplepos_t) (floor ((double) start / _session->samples_per_timecode_frame()) *  _session->samples_per_timecode_frame());
                }
                break;
 
-       case SnapToTimecodeSeconds:
+       case timecode_show_seconds:
                if (_session->config.get_timecode_offset_negative()) {
                        start += _session->config.get_timecode_offset ();
                } else {
@@ -2716,9 +2769,9 @@ Editor::timecode_snap_to_internal (MusicFrame& pos, RoundMode direction, bool /*
                    (start % one_timecode_second == 0)) {
                        /* start is already on a whole second, do nothing */
                } else if (((direction == 0) && (start % one_timecode_second > one_timecode_second / 2)) || direction > 0) {
-                       start = (framepos_t) ceil ((double) start / one_timecode_second) * one_timecode_second;
+                       start = (samplepos_t) ceil ((double) start / one_timecode_second) * one_timecode_second;
                } else {
-                       start = (framepos_t) floor ((double) start / one_timecode_second) * one_timecode_second;
+                       start = (samplepos_t) floor ((double) start / one_timecode_second) * one_timecode_second;
                }
 
                if (_session->config.get_timecode_offset_negative()) {
@@ -2728,7 +2781,9 @@ Editor::timecode_snap_to_internal (MusicFrame& pos, RoundMode direction, bool /*
                }
                break;
 
-       case SnapToTimecodeMinutes:
+       case timecode_show_minutes:
+       case timecode_show_hours:
+       case timecode_show_many_hours:
                if (_session->config.get_timecode_offset_negative()) {
                        start += _session->config.get_timecode_offset ();
                } else {
@@ -2738,9 +2793,9 @@ Editor::timecode_snap_to_internal (MusicFrame& pos, RoundMode direction, bool /*
                    (start % one_timecode_minute == 0)) {
                        /* start is already on a whole minute, do nothing */
                } else if (((direction == 0) && (start % one_timecode_minute > one_timecode_minute / 2)) || direction > 0) {
-                       start = (framepos_t) ceil ((double) start / one_timecode_minute) * one_timecode_minute;
+                       start = (samplepos_t) ceil ((double) start / one_timecode_minute) * one_timecode_minute;
                } else {
-                       start = (framepos_t) floor ((double) start / one_timecode_minute) * one_timecode_minute;
+                       start = (samplepos_t) floor ((double) start / one_timecode_minute) * one_timecode_minute;
                }
                if (_session->config.get_timecode_offset_negative()) {
                        start -= _session->config.get_timecode_offset ();
@@ -2750,221 +2805,274 @@ Editor::timecode_snap_to_internal (MusicFrame& pos, RoundMode direction, bool /*
                break;
        default:
                fatal << "Editor::smpte_snap_to_internal() called with non-timecode snap type!" << endmsg;
-               abort(); /*NOTREACHED*/
        }
 
-       pos.set (start, 0);
+       MusicSample ret(start,0);
+       return ret;
 }
 
-void
-Editor::snap_to_internal (MusicFrame& start, RoundMode direction, bool for_mark, bool ensure_snap)
+MusicSample
+Editor::snap_to_minsec (MusicSample presnap, RoundMode direction, SnapPref gpref)
 {
-       const framepos_t one_second = _session->frame_rate();
-       const framepos_t one_minute = _session->frame_rate() * 60;
-       framepos_t presnap = start.frame;
-       framepos_t before;
-       framepos_t after;
-
-       switch (_snap_type) {
-       case SnapToTimecodeFrame:
-       case SnapToTimecodeSeconds:
-       case SnapToTimecodeMinutes:
-               return timecode_snap_to_internal (start, direction, for_mark);
+       MusicSample ret(presnap);
+       
+       const samplepos_t one_second = _session->sample_rate();
+       const samplepos_t one_minute = one_second * 60;
+       const samplepos_t one_hour = one_minute * 60;
 
-       case SnapToCDFrame:
-               if ((direction == RoundUpMaybe || direction == RoundDownMaybe) &&
-                   start.frame % (one_second/75) == 0) {
-                       /* start is already on a whole CD frame, do nothing */
-               } else if (((direction == 0) && (start.frame % (one_second/75) > (one_second/75) / 2)) || (direction > 0)) {
-                       start.frame = (framepos_t) ceil ((double) start.frame / (one_second / 75)) * (one_second / 75);
-               } else {
-                       start.frame = (framepos_t) floor ((double) start.frame / (one_second / 75)) * (one_second / 75);
-               }
+       MinsecRulerScale scale = (gpref != SnapToGrid_Unscaled) ? minsec_ruler_scale : minsec_show_seconds;
 
-               start.set (start.frame, 0);
+       switch (scale) {
+               case minsec_show_msecs:
+               case minsec_show_seconds: {
+                       if ((direction == RoundUpMaybe || direction == RoundDownMaybe) &&
+                               presnap.sample % one_second == 0) {
+                               /* start is already on a whole second, do nothing */
+                       } else if (((direction == 0) && (presnap.sample % one_second > one_second / 2)) || (direction > 0)) {
+                               ret.sample = (samplepos_t) ceil ((double) presnap.sample / one_second) * one_second;
+                       } else {
+                               ret.sample = (samplepos_t) floor ((double) presnap.sample / one_second) * one_second;
+                       }
+               } break;
+
+               case minsec_show_minutes: {
+                       if ((direction == RoundUpMaybe || direction == RoundDownMaybe) &&
+                               presnap.sample % one_minute == 0) {
+                               /* start is already on a whole minute, do nothing */
+                       } else if (((direction == 0) && (presnap.sample % one_minute > one_minute / 2)) || (direction > 0)) {
+                               ret.sample = (samplepos_t) ceil ((double) presnap.sample / one_minute) * one_minute;
+                       } else {
+                               ret.sample = (samplepos_t) floor ((double) presnap.sample / one_minute) * one_minute;
+                       }
+               } break;
+
+               default: {
+                       if ((direction == RoundUpMaybe || direction == RoundDownMaybe) &&
+                               presnap.sample % one_hour == 0) {
+                               /* start is already on a whole hour, do nothing */
+                       } else if (((direction == 0) && (presnap.sample % one_hour > one_hour / 2)) || (direction > 0)) {
+                               ret.sample = (samplepos_t) ceil ((double) presnap.sample / one_hour) * one_hour;
+                       } else {
+                               ret.sample = (samplepos_t) floor ((double) presnap.sample / one_hour) * one_hour;
+                       }
+               } break;
+       }
+       
+       return ret;
+}
 
-               break;
+MusicSample
+Editor::snap_to_cd_frames (MusicSample presnap, RoundMode direction, SnapPref gpref)
+{
+       if ((gpref != SnapToGrid_Unscaled) && (minsec_ruler_scale != minsec_show_msecs)) {
+               return snap_to_minsec (presnap, direction, gpref);
+       }       
+       
+       const samplepos_t one_second = _session->sample_rate();
 
-       case SnapToSeconds:
-               if ((direction == RoundUpMaybe || direction == RoundDownMaybe) &&
-                   start.frame % one_second == 0) {
-                       /* start is already on a whole second, do nothing */
-               } else if (((direction == 0) && (start.frame % one_second > one_second / 2)) || (direction > 0)) {
-                       start.frame = (framepos_t) ceil ((double) start.frame / one_second) * one_second;
-               } else {
-                       start.frame = (framepos_t) floor ((double) start.frame / one_second) * one_second;
-               }
+       MusicSample ret(presnap);
+       
+       if ((direction == RoundUpMaybe || direction == RoundDownMaybe) &&
+               presnap.sample % (one_second/75) == 0) {
+               /* start is already on a whole CD sample, do nothing */
+       } else if (((direction == 0) && (presnap.sample % (one_second/75) > (one_second/75) / 2)) || (direction > 0)) {
+               ret.sample = (samplepos_t) ceil ((double) presnap.sample / (one_second / 75)) * (one_second / 75);
+       } else {
+               ret.sample = (samplepos_t) floor ((double) presnap.sample / (one_second / 75)) * (one_second / 75);
+       }
 
-               start.set (start.frame, 0);
+       return ret;
+}
 
-               break;
+MusicSample
+Editor::snap_to_bbt (MusicSample presnap, RoundMode direction, SnapPref gpref)
+{
+       MusicSample ret(presnap);
+       
+       if (gpref != SnapToGrid_Unscaled) { // use the visual grid lines which are limited by the zoom scale that the user selected
 
-       case SnapToMinutes:
-               if ((direction == RoundUpMaybe || direction == RoundDownMaybe) &&
-                   start.frame % one_minute == 0) {
-                       /* start is already on a whole minute, do nothing */
-               } else if (((direction == 0) && (start.frame % one_minute > one_minute / 2)) || (direction > 0)) {
-                       start.frame = (framepos_t) ceil ((double) start.frame / one_minute) * one_minute;
-               } else {
-                       start.frame = (framepos_t) floor ((double) start.frame / one_minute) * one_minute;
+               int divisor = 2;
+               switch (_grid_type) {
+               case GridTypeBeatDiv3:
+               case GridTypeBeatDiv6:
+               case GridTypeBeatDiv12:
+               case GridTypeBeatDiv24:
+                       divisor = 3;
+                       break;
+               case GridTypeBeatDiv5:
+               case GridTypeBeatDiv10:
+               case GridTypeBeatDiv20:
+                       divisor = 5;
+                       break;
+               case GridTypeBeatDiv7:
+               case GridTypeBeatDiv14:
+               case GridTypeBeatDiv28:
+                       divisor = 7;
+                       break;
+               default:
+                       divisor = 2;
+               };
+
+               BBTRulerScale scale = bbt_ruler_scale;
+               switch (scale) {
+                       case bbt_show_many:
+                       case bbt_show_64:
+                       case bbt_show_16:
+                       case bbt_show_4:
+                       case bbt_show_1:
+                               ret = _session->tempo_map().round_to_bar (presnap.sample, direction);
+                               break;
+                       case bbt_show_quarters:
+                               ret = _session->tempo_map().round_to_beat (presnap.sample, direction);
+                               break;
+                       case bbt_show_eighths:
+                               ret = _session->tempo_map().round_to_quarter_note_subdivision (presnap.sample, 1 * divisor, direction);
+                               break;
+                       case bbt_show_sixteenths:
+                               ret = _session->tempo_map().round_to_quarter_note_subdivision (presnap.sample, 2 * divisor, direction);
+                               break;
+                       case bbt_show_thirtyseconds:
+                               ret = _session->tempo_map().round_to_quarter_note_subdivision (presnap.sample, 4 * divisor, direction);
+                               break;
                }
+       } else {
+               ret = _session->tempo_map().round_to_quarter_note_subdivision (presnap.sample, get_grid_beat_divisions(_grid_type), direction);
+       }
+       
+       return ret;
+}
 
-               start.set (start.frame, 0);
-
-               break;
-
-       case SnapToBar:
-               start = _session->tempo_map().round_to_bar (start.frame, direction);
-               break;
-
-       case SnapToBeat:
-               start = _session->tempo_map().round_to_beat (start.frame, direction);
-               break;
-
-       case SnapToBeatDiv128:
-               start = _session->tempo_map().round_to_quarter_note_subdivision (start.frame, 128, direction);
-               break;
-       case SnapToBeatDiv64:
-               start = _session->tempo_map().round_to_quarter_note_subdivision (start.frame, 64, direction);
-               break;
-       case SnapToBeatDiv32:
-               start = _session->tempo_map().round_to_quarter_note_subdivision (start.frame, 32, direction);
-               break;
-       case SnapToBeatDiv28:
-               start = _session->tempo_map().round_to_quarter_note_subdivision (start.frame, 28, direction);
-               break;
-       case SnapToBeatDiv24:
-               start = _session->tempo_map().round_to_quarter_note_subdivision (start.frame, 24, direction);
-               break;
-       case SnapToBeatDiv20:
-               start = _session->tempo_map().round_to_quarter_note_subdivision (start.frame, 20, direction);
-               break;
-       case SnapToBeatDiv16:
-               start = _session->tempo_map().round_to_quarter_note_subdivision (start.frame, 16, direction);
-               break;
-       case SnapToBeatDiv14:
-               start = _session->tempo_map().round_to_quarter_note_subdivision (start.frame, 14, direction);
-               break;
-       case SnapToBeatDiv12:
-               start = _session->tempo_map().round_to_quarter_note_subdivision (start.frame, 12, direction);
-               break;
-       case SnapToBeatDiv10:
-               start = _session->tempo_map().round_to_quarter_note_subdivision (start.frame, 10, direction);
-               break;
-       case SnapToBeatDiv8:
-               start = _session->tempo_map().round_to_quarter_note_subdivision (start.frame, 8, direction);
-               break;
-       case SnapToBeatDiv7:
-               start = _session->tempo_map().round_to_quarter_note_subdivision (start.frame, 7, direction);
-               break;
-       case SnapToBeatDiv6:
-               start = _session->tempo_map().round_to_quarter_note_subdivision (start.frame, 6, direction);
-               break;
-       case SnapToBeatDiv5:
-               start = _session->tempo_map().round_to_quarter_note_subdivision (start.frame, 5, direction);
-               break;
-       case SnapToBeatDiv4:
-               start = _session->tempo_map().round_to_quarter_note_subdivision (start.frame, 4, direction);
-               break;
-       case SnapToBeatDiv3:
-               start = _session->tempo_map().round_to_quarter_note_subdivision (start.frame, 3, direction);
-               break;
-       case SnapToBeatDiv2:
-               start = _session->tempo_map().round_to_quarter_note_subdivision (start.frame, 2, direction);
-               break;
-
-       case SnapToMark:
-               if (for_mark) {
-                       return;
-               }
+ARDOUR::MusicSample
+Editor::snap_to_grid (MusicSample presnap, RoundMode direction, SnapPref gpref)
+{
+       MusicSample ret(presnap);
+       
+       if (grid_musical()) {
+               ret = snap_to_bbt (presnap, direction, gpref);
+       }
+       
+       switch (_grid_type) {
+               case GridTypeTimecode:
+                       ret = snap_to_timecode(presnap, direction, gpref);
+                       break;
+               case GridTypeMinSec:
+                       ret = snap_to_minsec(presnap, direction, gpref);
+                       break;
+               case GridTypeCDFrame:
+                       ret = snap_to_cd_frames(presnap, direction, gpref);
+                       break;
+               default:
+                       {}
+       };
 
-               _session->locations()->marks_either_side (start.frame, before, after);
+       return ret;
+}
 
-               if (before == max_framepos && after == max_framepos) {
-                       /* No marks to snap to, so just don't snap */
-                       return;
-               } else if (before == max_framepos) {
-                       start.frame = after;
-               } else if (after == max_framepos) {
-                       start.frame = before;
-               } else if (before != max_framepos && after != max_framepos) {
-                       if ((direction == RoundUpMaybe || direction == RoundUpAlways))
-                               start.frame = after;
-                       else if ((direction == RoundDownMaybe || direction == RoundDownAlways))
-                               start.frame = before;
-                       else if (direction ==  0 ) {
-                               if ((start.frame - before) < (after - start.frame)) {
-                                       start.frame = before;
-                               } else {
-                                       start.frame = after;
-                               }
+samplepos_t
+Editor::snap_to_marker (samplepos_t presnap, RoundMode direction)
+{
+       samplepos_t before;
+       samplepos_t after;
+       samplepos_t test;
+
+       _session->locations()->marks_either_side (presnap, before, after);
+
+       if (before == max_samplepos && after == max_samplepos) {
+               /* No marks to snap to, so just don't snap */
+               return presnap;
+       } else if (before == max_samplepos) {
+               test = after;
+       } else if (after == max_samplepos) {
+               test = before;
+       } else  {
+               if ((direction == RoundUpMaybe || direction == RoundUpAlways)) {
+                       test = after;
+               } else if ((direction == RoundDownMaybe || direction == RoundDownAlways)) {
+                       test = before;
+               } else if (direction ==  0) {
+                       if ((presnap - before) < (after - presnap)) {
+                               test = before;
+                       } else {
+                               test = after;
                        }
                }
+       }
 
-               start.set (start.frame, 0);
+       return test;
+}
 
-               break;
+void
+Editor::snap_to_internal (MusicSample& start, RoundMode direction, SnapPref pref, bool ensure_snap)
+{
+       const samplepos_t presnap = start.sample;
 
-       case SnapToRegionStart:
-       case SnapToRegionEnd:
-       case SnapToRegionSync:
-       case SnapToRegionBoundary:
-               if (!region_boundary_cache.empty()) {
+       samplepos_t test = max_samplepos; // for each snap, we'll use this value
+       samplepos_t dist = max_samplepos; // this records the distance of the best snap result we've found so far
+       samplepos_t best = max_samplepos; // this records the best snap-result we've found so far
 
-                       vector<framepos_t>::iterator prev = region_boundary_cache.end ();
-                       vector<framepos_t>::iterator next = region_boundary_cache.end ();
+       /* check snap-to-marker */
+       if ((pref == SnapToAny_Visual) && UIConfiguration::instance().get_snap_to_marks()) {
+               test = snap_to_marker (presnap, direction);
+               check_best_snap(presnap, test, dist, best);
+       }
 
-                       if (direction > 0) {
-                               next = std::upper_bound (region_boundary_cache.begin(), region_boundary_cache.end(), start.frame);
-                       } else {
-                               next = std::lower_bound (region_boundary_cache.begin(), region_boundary_cache.end(), start.frame);
-                       }
+       /* check snap-to-region-{start/end/sync} */
+       if (
+               (pref == SnapToAny_Visual) &&
+               (UIConfiguration::instance().get_snap_to_region_start() || UIConfiguration::instance().get_snap_to_region_end() || UIConfiguration::instance().get_snap_to_region_sync())
+               ) {
+               if (!region_boundary_cache.empty()) {
 
+                       vector<samplepos_t>::iterator prev = region_boundary_cache.begin();
+                       vector<samplepos_t>::iterator next = std::upper_bound (region_boundary_cache.begin(), region_boundary_cache.end(), presnap);
                        if (next != region_boundary_cache.begin ()) {
                                prev = next;
                                prev--;
                        }
 
-                       framepos_t const p = (prev == region_boundary_cache.end()) ? region_boundary_cache.front () : *prev;
-                       framepos_t const n = (next == region_boundary_cache.end()) ? region_boundary_cache.back () : *next;
-
-                       if (start.frame > (p + n) / 2) {
-                               start.frame = n;
-                       } else {
-                               start.frame = p;
+                       if ((direction == RoundUpMaybe || direction == RoundUpAlways))
+                               test = *next;
+                       else if ((direction == RoundDownMaybe || direction == RoundDownAlways))
+                               test = *prev;
+                       else if (direction ==  0) {
+                               if ((presnap - *prev) < (*next - presnap)) {
+                                       test = *prev;
+                               } else {
+                                       test = *next;
+                               }
                        }
+
                }
 
-               start.set (start.frame, 0);
+               check_best_snap(presnap, test, dist, best);
+       }
 
-               break;
+       /* check Grid */
+       if (UIConfiguration::instance().get_snap_to_grid() && (_grid_type != GridTypeNone)) {
+               MusicSample pre(presnap, 0);
+               MusicSample post = snap_to_grid (pre, direction, pref);
+               check_best_snap(presnap, post.sample, dist, best);
        }
 
-       switch (_snap_mode) {
-       case SnapNormal:
+       /* now check "magnetic" state: is the grid within reasonable on-screen distance to trigger a snap?
+        * this also helps to avoid snapping to somewhere the user can't see.  (i.e.: I clicked on a region and it disappeared!!)
+        * ToDo: Perhaps this should only occur if EditPointMouse?
+        */
+       int snap_threshold_s = pixel_to_sample(UIConfiguration::instance().get_snap_threshold());
+       if (ensure_snap) {
+               start.set (best, 0);
                return;
-
-       case SnapMagnetic:
-
-               if (ensure_snap) {
-                       return;
+       } else if (presnap > best) {
+               if (presnap > (best+ snap_threshold_s)) {
+                       best = presnap;
                }
-
-               if (presnap > start.frame) {
-                       if (presnap > (start.frame + pixel_to_sample(snap_threshold))) {
-                               start.set (presnap, 0);
-                       }
-
-               } else if (presnap < start.frame) {
-                       if (presnap < (start.frame - pixel_to_sample(snap_threshold))) {
-                               start.set (presnap, 0);
-                       }
+       } else if (presnap < best) {
+               if (presnap < (best - snap_threshold_s)) {
+                        best = presnap;
                }
-
-       default:
-               /* handled at entry */
-               return;
        }
+
+       start.set (best, 0);
 }
 
 
@@ -3002,8 +3110,8 @@ Editor::setup_toolbar ()
                mouse_mode_size_group->add_widget (visible_tracks_selector);
        }
 
-       mouse_mode_size_group->add_widget (snap_type_selector);
-       mouse_mode_size_group->add_widget (snap_mode_selector);
+       mouse_mode_size_group->add_widget (grid_type_selector);
+       mouse_mode_size_group->add_widget (snap_mode_button);
 
        mouse_mode_size_group->add_widget (edit_point_selector);
        mouse_mode_size_group->add_widget (edit_mode_selector);
@@ -3023,11 +3131,11 @@ Editor::setup_toolbar ()
 
        if (!ARDOUR::Profile->get_mixbus()) {
                mouse_mode_hbox->pack_start (mouse_cut_button, false, false);
+               mouse_mode_hbox->pack_start (mouse_audition_button, false, false);
        }
 
        if (!ARDOUR::Profile->get_trx()) {
                mouse_mode_hbox->pack_start (mouse_timefx_button, false, false);
-               mouse_mode_hbox->pack_start (mouse_audition_button, false, false);
                mouse_mode_hbox->pack_start (mouse_draw_button, false, false);
                mouse_mode_hbox->pack_start (mouse_content_button, false, false);
        }
@@ -3043,6 +3151,9 @@ Editor::setup_toolbar ()
 
        if (!ARDOUR::Profile->get_trx()) {
                mode_box->pack_start (edit_mode_selector, false, false);
+               mode_box->pack_start (*(manage (new ArdourVSpacer ())), false, false, 3);
+               mode_box->pack_start (edit_point_selector, false, false);
+               mode_box->pack_start (*(manage (new ArdourVSpacer ())), false, false, 3);
        }
 
        mode_box->pack_start (*mouse_mode_box, false, false);
@@ -3121,21 +3232,14 @@ Editor::setup_toolbar ()
        snap_box.set_spacing (2);
        snap_box.set_border_width (2);
 
-       snap_type_selector.set_name ("mouse mode button");
+       grid_type_selector.set_name ("mouse mode button");
 
-       snap_mode_selector.set_name ("mouse mode button");
+       snap_mode_button.set_name ("mouse mode button");
 
        edit_point_selector.set_name ("mouse mode button");
 
-       snap_box.pack_start (snap_mode_selector, false, false);
-       snap_box.pack_start (snap_type_selector, false, false);
-
-       /* Edit Point*/
-       HBox *ep_box = manage (new HBox);
-       ep_box->set_spacing (2);
-       ep_box->set_border_width (2);
-
-       ep_box->pack_start (edit_point_selector, false, false);
+       snap_box.pack_start (snap_mode_button, false, false);
+       snap_box.pack_start (grid_type_selector, false, false);
 
        /* Nudge */
 
@@ -3156,17 +3260,25 @@ Editor::setup_toolbar ()
        toolbar_hbox.set_spacing (2);
        toolbar_hbox.set_border_width (2);
 
-       toolbar_hbox.pack_start (*mode_box, false, false);
+       ArdourWidgets::ArdourDropShadow *tool_shadow = manage (new (ArdourWidgets::ArdourDropShadow));
+       tool_shadow->set_size_request (4, -1);
+       tool_shadow->show();
 
-       if (!ARDOUR::Profile->get_trx()) {
+       ebox_hpacker.pack_start (*tool_shadow, false, false);
+       ebox_hpacker.pack_start(ebox_vpacker, true, true);
 
-               toolbar_hbox.pack_start (*(manage (new ArdourVSpacer ())), false, false, 3);
+       Gtk::EventBox* spacer = manage (new Gtk::EventBox); // extra space under the mouse toolbar, for aesthetics
+       spacer->set_name("EditorWindow");
+       spacer->set_size_request(-1,4);
+       spacer->show();
 
-               toolbar_hbox.pack_start (_zoom_box, false, false);
+       ebox_vpacker.pack_start(toolbar_hbox, false, false);
+       ebox_vpacker.pack_start(*spacer, false, false);
+       ebox_vpacker.show();
 
-               toolbar_hbox.pack_start (*(manage (new ArdourVSpacer ())), false, false, 3);
+       toolbar_hbox.pack_start (*mode_box, false, false);
 
-               toolbar_hbox.pack_start (_track_box, false, false);
+       if (!ARDOUR::Profile->get_trx()) {
 
                toolbar_hbox.pack_start (*(manage (new ArdourVSpacer ())), false, false, 3);
 
@@ -3174,11 +3286,14 @@ Editor::setup_toolbar ()
 
                toolbar_hbox.pack_start (*(manage (new ArdourVSpacer ())), false, false, 3);
 
-               toolbar_hbox.pack_start (*ep_box, false, false);
+               toolbar_hbox.pack_start (*nudge_box, false, false);
 
-               toolbar_hbox.pack_start (*(manage (new ArdourVSpacer ())), false, false, 3);
+               toolbar_hbox.pack_end (_zoom_box, false, false, 2);
+
+               toolbar_hbox.pack_end (*(manage (new ArdourVSpacer ())), false, false, 3);
+
+               toolbar_hbox.pack_end (_track_box, false, false);
 
-               toolbar_hbox.pack_start (*nudge_box, false, false);
        }
 
        toolbar_hbox.show_all ();
@@ -3189,10 +3304,10 @@ Editor::build_edit_point_menu ()
 {
        using namespace Menu_Helpers;
 
-       edit_point_selector.AddMenuElem (MenuElem ( edit_point_strings[(int)EditAtPlayhead], sigc::bind (sigc::mem_fun(*this, &Editor::edit_point_selection_done), (EditPoint) EditAtPlayhead)));
+       edit_point_selector.AddMenuElem (MenuElem (edit_point_strings[(int)EditAtPlayhead], sigc::bind (sigc::mem_fun(*this, &Editor::edit_point_selection_done), (EditPoint) EditAtPlayhead)));
        if(!Profile->get_mixbus())
-               edit_point_selector.AddMenuElem (MenuElem ( edit_point_strings[(int)EditAtSelectedMarker], sigc::bind (sigc::mem_fun(*this, &Editor::edit_point_selection_done), (EditPoint) EditAtSelectedMarker)));
-       edit_point_selector.AddMenuElem (MenuElem ( edit_point_strings[(int)EditAtMouse], sigc::bind (sigc::mem_fun(*this, &Editor::edit_point_selection_done), (EditPoint) EditAtMouse)));
+               edit_point_selector.AddMenuElem (MenuElem (edit_point_strings[(int)EditAtSelectedMarker], sigc::bind (sigc::mem_fun(*this, &Editor::edit_point_selection_done), (EditPoint) EditAtSelectedMarker)));
+       edit_point_selector.AddMenuElem (MenuElem (edit_point_strings[(int)EditAtMouse], sigc::bind (sigc::mem_fun(*this, &Editor::edit_point_selection_done), (EditPoint) EditAtMouse)));
 
        set_size_request_to_display_given_text (edit_point_selector, edit_point_strings, COMBO_TRIANGLE_WIDTH, 2);
 }
@@ -3202,64 +3317,65 @@ Editor::build_edit_mode_menu ()
 {
        using namespace Menu_Helpers;
 
-       edit_mode_selector.AddMenuElem (MenuElem ( edit_mode_strings[(int)Slide], sigc::bind (sigc::mem_fun(*this, &Editor::edit_mode_selection_done), (EditMode) Slide)));
-//     edit_mode_selector.AddMenuElem (MenuElem ( edit_mode_strings[(int)Splice], sigc::bind (sigc::mem_fun(*this, &Editor::edit_mode_selection_done), (EditMode) Splice)));
-       edit_mode_selector.AddMenuElem (MenuElem ( edit_mode_strings[(int)Ripple], sigc::bind (sigc::mem_fun(*this, &Editor::edit_mode_selection_done), (EditMode) Ripple)));
-       edit_mode_selector.AddMenuElem (MenuElem ( edit_mode_strings[(int)Lock], sigc::bind (sigc::mem_fun(*this, &Editor::edit_mode_selection_done), (EditMode)  Lock)));
+       edit_mode_selector.AddMenuElem (MenuElem (edit_mode_strings[(int)Slide], sigc::bind (sigc::mem_fun(*this, &Editor::edit_mode_selection_done), (EditMode) Slide)));
+       edit_mode_selector.AddMenuElem (MenuElem (edit_mode_strings[(int)Ripple], sigc::bind (sigc::mem_fun(*this, &Editor::edit_mode_selection_done), (EditMode) Ripple)));
+       edit_mode_selector.AddMenuElem (MenuElem (edit_mode_strings[(int)Lock], sigc::bind (sigc::mem_fun(*this, &Editor::edit_mode_selection_done), (EditMode)  Lock)));
+       /* Note: Splice was removed */
 
        set_size_request_to_display_given_text (edit_mode_selector, edit_mode_strings, COMBO_TRIANGLE_WIDTH, 2);
 }
 
 void
-Editor::build_snap_mode_menu ()
+Editor::build_grid_type_menu ()
 {
        using namespace Menu_Helpers;
 
-       snap_mode_selector.AddMenuElem (MenuElem ( snap_mode_strings[(int)SnapOff], sigc::bind (sigc::mem_fun(*this, &Editor::snap_mode_selection_done), (SnapMode) SnapOff)));
-       snap_mode_selector.AddMenuElem (MenuElem ( snap_mode_strings[(int)SnapNormal], sigc::bind (sigc::mem_fun(*this, &Editor::snap_mode_selection_done), (SnapMode) SnapNormal)));
-       snap_mode_selector.AddMenuElem (MenuElem ( snap_mode_strings[(int)SnapMagnetic], sigc::bind (sigc::mem_fun(*this, &Editor::snap_mode_selection_done), (SnapMode) SnapMagnetic)));
-
-       set_size_request_to_display_given_text (snap_mode_selector, snap_mode_strings, COMBO_TRIANGLE_WIDTH, 2);
-}
+       /* main grid: bars, quarter-notes, etc */
+       grid_type_selector.AddMenuElem (MenuElem (grid_type_strings[(int)GridTypeNone],      sigc::bind (sigc::mem_fun(*this, &Editor::grid_type_selection_done), (GridType) GridTypeNone)));
+       grid_type_selector.AddMenuElem (MenuElem (grid_type_strings[(int)GridTypeBar],       sigc::bind (sigc::mem_fun(*this, &Editor::grid_type_selection_done), (GridType) GridTypeBar)));
+       grid_type_selector.AddMenuElem (MenuElem (grid_type_strings[(int)GridTypeBeat],      sigc::bind (sigc::mem_fun(*this, &Editor::grid_type_selection_done), (GridType) GridTypeBeat)));
+       grid_type_selector.AddMenuElem (MenuElem (grid_type_strings[(int)GridTypeBeatDiv2],  sigc::bind (sigc::mem_fun(*this, &Editor::grid_type_selection_done), (GridType) GridTypeBeatDiv2)));
+       grid_type_selector.AddMenuElem (MenuElem (grid_type_strings[(int)GridTypeBeatDiv4],  sigc::bind (sigc::mem_fun(*this, &Editor::grid_type_selection_done), (GridType) GridTypeBeatDiv4)));
+       grid_type_selector.AddMenuElem (MenuElem (grid_type_strings[(int)GridTypeBeatDiv8],  sigc::bind (sigc::mem_fun(*this, &Editor::grid_type_selection_done), (GridType) GridTypeBeatDiv8)));
+       grid_type_selector.AddMenuElem (MenuElem (grid_type_strings[(int)GridTypeBeatDiv16], sigc::bind (sigc::mem_fun(*this, &Editor::grid_type_selection_done), (GridType) GridTypeBeatDiv16)));
+       grid_type_selector.AddMenuElem (MenuElem (grid_type_strings[(int)GridTypeBeatDiv32], sigc::bind (sigc::mem_fun(*this, &Editor::grid_type_selection_done), (GridType) GridTypeBeatDiv32)));
+
+       /* triplet grid */
+       grid_type_selector.AddMenuElem(SeparatorElem());
+       Gtk::Menu *_triplet_menu = manage (new Menu);
+       MenuList& triplet_items (_triplet_menu->items());
+       {
+               triplet_items.push_back (MenuElem (grid_type_strings[(int)GridTypeBeatDiv3],  sigc::bind (sigc::mem_fun(*this, &Editor::grid_type_selection_done), (GridType) GridTypeBeatDiv3)));
+               triplet_items.push_back (MenuElem (grid_type_strings[(int)GridTypeBeatDiv6],  sigc::bind (sigc::mem_fun(*this, &Editor::grid_type_selection_done), (GridType) GridTypeBeatDiv6)));
+               triplet_items.push_back (MenuElem (grid_type_strings[(int)GridTypeBeatDiv12], sigc::bind (sigc::mem_fun(*this, &Editor::grid_type_selection_done), (GridType) GridTypeBeatDiv12)));
+               triplet_items.push_back (MenuElem (grid_type_strings[(int)GridTypeBeatDiv24], sigc::bind (sigc::mem_fun(*this, &Editor::grid_type_selection_done), (GridType) GridTypeBeatDiv24)));
+       }
+       grid_type_selector.AddMenuElem (Menu_Helpers::MenuElem (_("Triplets"), *_triplet_menu));
 
-void
-Editor::build_snap_type_menu ()
-{
-       using namespace Menu_Helpers;
+       /* quintuplet grid */
+       Gtk::Menu *_quintuplet_menu = manage (new Menu);
+       MenuList& quintuplet_items (_quintuplet_menu->items());
+       {
+               quintuplet_items.push_back (MenuElem (grid_type_strings[(int)GridTypeBeatDiv5],  sigc::bind (sigc::mem_fun(*this, &Editor::grid_type_selection_done), (GridType) GridTypeBeatDiv5)));
+               quintuplet_items.push_back (MenuElem (grid_type_strings[(int)GridTypeBeatDiv10], sigc::bind (sigc::mem_fun(*this, &Editor::grid_type_selection_done), (GridType) GridTypeBeatDiv10)));
+               quintuplet_items.push_back (MenuElem (grid_type_strings[(int)GridTypeBeatDiv20], sigc::bind (sigc::mem_fun(*this, &Editor::grid_type_selection_done), (GridType) GridTypeBeatDiv20)));
+       }
+       grid_type_selector.AddMenuElem (Menu_Helpers::MenuElem (_("Quintuplets"), *_quintuplet_menu));
 
-       snap_type_selector.AddMenuElem (MenuElem ( snap_type_strings[(int)SnapToCDFrame], sigc::bind (sigc::mem_fun(*this, &Editor::snap_type_selection_done), (SnapType) SnapToCDFrame)));
-       snap_type_selector.AddMenuElem (MenuElem ( snap_type_strings[(int)SnapToTimecodeFrame], sigc::bind (sigc::mem_fun(*this, &Editor::snap_type_selection_done), (SnapType) SnapToTimecodeFrame)));
-       snap_type_selector.AddMenuElem (MenuElem ( snap_type_strings[(int)SnapToTimecodeSeconds], sigc::bind (sigc::mem_fun(*this, &Editor::snap_type_selection_done), (SnapType) SnapToTimecodeSeconds)));
-       snap_type_selector.AddMenuElem (MenuElem ( snap_type_strings[(int)SnapToTimecodeMinutes], sigc::bind (sigc::mem_fun(*this, &Editor::snap_type_selection_done), (SnapType) SnapToTimecodeMinutes)));
-       snap_type_selector.AddMenuElem (MenuElem ( snap_type_strings[(int)SnapToSeconds], sigc::bind (sigc::mem_fun(*this, &Editor::snap_type_selection_done), (SnapType) SnapToSeconds)));
-       snap_type_selector.AddMenuElem (MenuElem ( snap_type_strings[(int)SnapToMinutes], sigc::bind (sigc::mem_fun(*this, &Editor::snap_type_selection_done), (SnapType) SnapToMinutes)));
-       snap_type_selector.AddMenuElem (MenuElem ( snap_type_strings[(int)SnapToBeatDiv128], sigc::bind (sigc::mem_fun(*this, &Editor::snap_type_selection_done), (SnapType) SnapToBeatDiv128)));
-       snap_type_selector.AddMenuElem (MenuElem ( snap_type_strings[(int)SnapToBeatDiv64], sigc::bind (sigc::mem_fun(*this, &Editor::snap_type_selection_done), (SnapType) SnapToBeatDiv64)));
-       snap_type_selector.AddMenuElem (MenuElem ( snap_type_strings[(int)SnapToBeatDiv32], sigc::bind (sigc::mem_fun(*this, &Editor::snap_type_selection_done), (SnapType) SnapToBeatDiv32)));
-       snap_type_selector.AddMenuElem (MenuElem ( snap_type_strings[(int)SnapToBeatDiv28], sigc::bind (sigc::mem_fun(*this, &Editor::snap_type_selection_done), (SnapType) SnapToBeatDiv28)));
-       snap_type_selector.AddMenuElem (MenuElem ( snap_type_strings[(int)SnapToBeatDiv24], sigc::bind (sigc::mem_fun(*this, &Editor::snap_type_selection_done), (SnapType) SnapToBeatDiv24)));
-       snap_type_selector.AddMenuElem (MenuElem ( snap_type_strings[(int)SnapToBeatDiv20], sigc::bind (sigc::mem_fun(*this, &Editor::snap_type_selection_done), (SnapType) SnapToBeatDiv20)));
-       snap_type_selector.AddMenuElem (MenuElem ( snap_type_strings[(int)SnapToBeatDiv16], sigc::bind (sigc::mem_fun(*this, &Editor::snap_type_selection_done), (SnapType) SnapToBeatDiv16)));
-       snap_type_selector.AddMenuElem (MenuElem ( snap_type_strings[(int)SnapToBeatDiv14], sigc::bind (sigc::mem_fun(*this, &Editor::snap_type_selection_done), (SnapType) SnapToBeatDiv14)));
-       snap_type_selector.AddMenuElem (MenuElem ( snap_type_strings[(int)SnapToBeatDiv12], sigc::bind (sigc::mem_fun(*this, &Editor::snap_type_selection_done), (SnapType) SnapToBeatDiv12)));
-       snap_type_selector.AddMenuElem (MenuElem ( snap_type_strings[(int)SnapToBeatDiv10], sigc::bind (sigc::mem_fun(*this, &Editor::snap_type_selection_done), (SnapType) SnapToBeatDiv10)));
-       snap_type_selector.AddMenuElem (MenuElem ( snap_type_strings[(int)SnapToBeatDiv8], sigc::bind (sigc::mem_fun(*this, &Editor::snap_type_selection_done), (SnapType) SnapToBeatDiv8)));
-       snap_type_selector.AddMenuElem (MenuElem ( snap_type_strings[(int)SnapToBeatDiv7], sigc::bind (sigc::mem_fun(*this, &Editor::snap_type_selection_done), (SnapType) SnapToBeatDiv7)));
-       snap_type_selector.AddMenuElem (MenuElem ( snap_type_strings[(int)SnapToBeatDiv6], sigc::bind (sigc::mem_fun(*this, &Editor::snap_type_selection_done), (SnapType) SnapToBeatDiv6)));
-       snap_type_selector.AddMenuElem (MenuElem ( snap_type_strings[(int)SnapToBeatDiv5], sigc::bind (sigc::mem_fun(*this, &Editor::snap_type_selection_done), (SnapType) SnapToBeatDiv5)));
-       snap_type_selector.AddMenuElem (MenuElem ( snap_type_strings[(int)SnapToBeatDiv4], sigc::bind (sigc::mem_fun(*this, &Editor::snap_type_selection_done), (SnapType) SnapToBeatDiv4)));
-       snap_type_selector.AddMenuElem (MenuElem ( snap_type_strings[(int)SnapToBeatDiv3], sigc::bind (sigc::mem_fun(*this, &Editor::snap_type_selection_done), (SnapType) SnapToBeatDiv3)));
-       snap_type_selector.AddMenuElem (MenuElem ( snap_type_strings[(int)SnapToBeatDiv2], sigc::bind (sigc::mem_fun(*this, &Editor::snap_type_selection_done), (SnapType) SnapToBeatDiv2)));
-       snap_type_selector.AddMenuElem (MenuElem ( snap_type_strings[(int)SnapToBeat], sigc::bind (sigc::mem_fun(*this, &Editor::snap_type_selection_done), (SnapType) SnapToBeat)));
-       snap_type_selector.AddMenuElem (MenuElem ( snap_type_strings[(int)SnapToBar], sigc::bind (sigc::mem_fun(*this, &Editor::snap_type_selection_done), (SnapType) SnapToBar)));
-       snap_type_selector.AddMenuElem (MenuElem ( snap_type_strings[(int)SnapToMark], sigc::bind (sigc::mem_fun(*this, &Editor::snap_type_selection_done), (SnapType) SnapToMark)));
-       snap_type_selector.AddMenuElem (MenuElem ( snap_type_strings[(int)SnapToRegionStart], sigc::bind (sigc::mem_fun(*this, &Editor::snap_type_selection_done), (SnapType) SnapToRegionStart)));
-       snap_type_selector.AddMenuElem (MenuElem ( snap_type_strings[(int)SnapToRegionEnd], sigc::bind (sigc::mem_fun(*this, &Editor::snap_type_selection_done), (SnapType) SnapToRegionEnd)));
-       snap_type_selector.AddMenuElem (MenuElem ( snap_type_strings[(int)SnapToRegionSync], sigc::bind (sigc::mem_fun(*this, &Editor::snap_type_selection_done), (SnapType) SnapToRegionSync)));
-       snap_type_selector.AddMenuElem (MenuElem ( snap_type_strings[(int)SnapToRegionBoundary], sigc::bind (sigc::mem_fun(*this, &Editor::snap_type_selection_done), (SnapType) SnapToRegionBoundary)));
-
-       set_size_request_to_display_given_text (snap_type_selector, snap_type_strings, COMBO_TRIANGLE_WIDTH, 2);
+       /* septuplet grid */
+       Gtk::Menu *_septuplet_menu = manage (new Menu);
+       MenuList& septuplet_items (_septuplet_menu->items());
+       {
+               septuplet_items.push_back (MenuElem (grid_type_strings[(int)GridTypeBeatDiv7],  sigc::bind (sigc::mem_fun(*this, &Editor::grid_type_selection_done), (GridType) GridTypeBeatDiv7)));
+               septuplet_items.push_back (MenuElem (grid_type_strings[(int)GridTypeBeatDiv14], sigc::bind (sigc::mem_fun(*this, &Editor::grid_type_selection_done), (GridType) GridTypeBeatDiv14)));
+               septuplet_items.push_back (MenuElem (grid_type_strings[(int)GridTypeBeatDiv28], sigc::bind (sigc::mem_fun(*this, &Editor::grid_type_selection_done), (GridType) GridTypeBeatDiv28)));
+       }
+       grid_type_selector.AddMenuElem (Menu_Helpers::MenuElem (_("Septuplets"), *_septuplet_menu));
 
+       grid_type_selector.AddMenuElem(SeparatorElem());
+       grid_type_selector.AddMenuElem (MenuElem (grid_type_strings[(int)GridTypeTimecode], sigc::bind (sigc::mem_fun(*this, &Editor::grid_type_selection_done), (GridType) GridTypeTimecode)));
+       grid_type_selector.AddMenuElem (MenuElem (grid_type_strings[(int)GridTypeMinSec], sigc::bind (sigc::mem_fun(*this, &Editor::grid_type_selection_done), (GridType) GridTypeMinSec)));
+       grid_type_selector.AddMenuElem (MenuElem (grid_type_strings[(int)GridTypeCDFrame], sigc::bind (sigc::mem_fun(*this, &Editor::grid_type_selection_done), (GridType) GridTypeCDFrame)));
 }
 
 void
@@ -3284,8 +3400,8 @@ Editor::setup_tooltips ()
        set_tooltip (tav_expand_button, _("Expand Tracks"));
        set_tooltip (tav_shrink_button, _("Shrink Tracks"));
        set_tooltip (visible_tracks_selector, _("Number of visible tracks"));
-       set_tooltip (snap_type_selector, _("Snap/Grid Units"));
-       set_tooltip (snap_mode_selector, _("Snap/Grid Mode"));
+       set_tooltip (grid_type_selector, _("Grid Mode"));
+       set_tooltip (snap_mode_button, _("Snap Mode\n\nRight-click to visit Snap preferences."));
        set_tooltip (edit_point_selector, _("Edit Point"));
        set_tooltip (edit_mode_selector, _("Edit Mode"));
        set_tooltip (nudge_clock, _("Nudge Clock\n(controls distance used to nudge regions and selections)"));
@@ -3437,16 +3553,15 @@ Editor::commit_reversible_selection_op ()
                if (selection_op_cmd_depth == 1) {
 
                        if (selection_op_history_it > 0 && selection_op_history_it < selection_op_history.size()) {
-                               /**
-                                   The user has undone some selection ops and then made a new one,
-                                   making anything earlier in the list invalid.
-                               */
+                               /* The user has undone some selection ops and then made a new one,
+                                * making anything earlier in the list invalid.
+                                */
 
                                list<XMLNode *>::iterator it = selection_op_history.begin();
                                list<XMLNode *>::iterator e_it = it;
                                advance (e_it, selection_op_history_it);
 
-                               for ( ; it != e_it; ++it) {
+                               for (; it != e_it; ++it) {
                                        delete *it;
                                }
                                selection_op_history.erase (selection_op_history.begin(), e_it);
@@ -3591,7 +3706,7 @@ Editor::duplicate_range (bool with_dialog)
 
        RegionSelection rs = get_regions_from_selection_and_entered ();
 
-       if ( selection->time.length() == 0 && rs.empty()) {
+       if (selection->time.length() == 0 && rs.empty()) {
                return;
        }
 
@@ -3636,7 +3751,7 @@ Editor::duplicate_range (bool with_dialog)
                times = adjustment.get_value();
        }
 
-       if ((current_mouse_mode() == Editing::MouseRange)) {
+       if ((current_mouse_mode() == MouseRange)) {
                if (selection->time.length()) {
                        duplicate_selection (times);
                }
@@ -3674,15 +3789,15 @@ Editor::cycle_edit_mode ()
 }
 
 void
-Editor::edit_mode_selection_done ( EditMode m )
+Editor::edit_mode_selection_done (EditMode m)
 {
-       Config->set_edit_mode ( m );
+       Config->set_edit_mode (m);
 }
 
 void
-Editor::snap_type_selection_done (SnapType snaptype)
+Editor::grid_type_selection_done (GridType gridtype)
 {
-       RefPtr<RadioAction> ract = snap_type_action (snaptype);
+       RefPtr<RadioAction> ract = grid_type_action (gridtype);
        if (ract) {
                ract->set_active ();
        }
@@ -3724,7 +3839,7 @@ Editor::cycle_edit_point (bool with_marker)
 void
 Editor::edit_point_selection_done (EditPoint ep)
 {
-       set_edit_point_preference ( ep );
+       set_edit_point_preference (ep);
 }
 
 void
@@ -3732,18 +3847,18 @@ Editor::build_zoom_focus_menu ()
 {
        using namespace Menu_Helpers;
 
-       zoom_focus_selector.AddMenuElem (MenuElem ( zoom_focus_strings[(int)ZoomFocusLeft], sigc::bind (sigc::mem_fun(*this, &Editor::zoom_focus_selection_done), (ZoomFocus) ZoomFocusLeft)));
-       zoom_focus_selector.AddMenuElem (MenuElem ( zoom_focus_strings[(int)ZoomFocusRight], sigc::bind (sigc::mem_fun(*this, &Editor::zoom_focus_selection_done), (ZoomFocus) ZoomFocusRight)));
-       zoom_focus_selector.AddMenuElem (MenuElem ( zoom_focus_strings[(int)ZoomFocusCenter], sigc::bind (sigc::mem_fun(*this, &Editor::zoom_focus_selection_done), (ZoomFocus) ZoomFocusCenter)));
-       zoom_focus_selector.AddMenuElem (MenuElem ( zoom_focus_strings[(int)ZoomFocusPlayhead], sigc::bind (sigc::mem_fun(*this, &Editor::zoom_focus_selection_done), (ZoomFocus) ZoomFocusPlayhead)));
-       zoom_focus_selector.AddMenuElem (MenuElem ( zoom_focus_strings[(int)ZoomFocusMouse], sigc::bind (sigc::mem_fun(*this, &Editor::zoom_focus_selection_done), (ZoomFocus) ZoomFocusMouse)));
-       zoom_focus_selector.AddMenuElem (MenuElem ( zoom_focus_strings[(int)ZoomFocusEdit], sigc::bind (sigc::mem_fun(*this, &Editor::zoom_focus_selection_done), (ZoomFocus) ZoomFocusEdit)));
+       zoom_focus_selector.AddMenuElem (MenuElem (zoom_focus_strings[(int)ZoomFocusLeft], sigc::bind (sigc::mem_fun(*this, &Editor::zoom_focus_selection_done), (ZoomFocus) ZoomFocusLeft)));
+       zoom_focus_selector.AddMenuElem (MenuElem (zoom_focus_strings[(int)ZoomFocusRight], sigc::bind (sigc::mem_fun(*this, &Editor::zoom_focus_selection_done), (ZoomFocus) ZoomFocusRight)));
+       zoom_focus_selector.AddMenuElem (MenuElem (zoom_focus_strings[(int)ZoomFocusCenter], sigc::bind (sigc::mem_fun(*this, &Editor::zoom_focus_selection_done), (ZoomFocus) ZoomFocusCenter)));
+       zoom_focus_selector.AddMenuElem (MenuElem (zoom_focus_strings[(int)ZoomFocusPlayhead], sigc::bind (sigc::mem_fun(*this, &Editor::zoom_focus_selection_done), (ZoomFocus) ZoomFocusPlayhead)));
+       zoom_focus_selector.AddMenuElem (MenuElem (zoom_focus_strings[(int)ZoomFocusMouse], sigc::bind (sigc::mem_fun(*this, &Editor::zoom_focus_selection_done), (ZoomFocus) ZoomFocusMouse)));
+       zoom_focus_selector.AddMenuElem (MenuElem (zoom_focus_strings[(int)ZoomFocusEdit], sigc::bind (sigc::mem_fun(*this, &Editor::zoom_focus_selection_done), (ZoomFocus) ZoomFocusEdit)));
 
        set_size_request_to_display_given_text (zoom_focus_selector, zoom_focus_strings, COMBO_TRIANGLE_WIDTH, 2);
 }
 
 void
-Editor::zoom_focus_selection_done ( ZoomFocus f )
+Editor::zoom_focus_selection_done (ZoomFocus f)
 {
        RefPtr<RadioAction> ract = zoom_focus_action (f);
        if (ract) {
@@ -3792,6 +3907,7 @@ Editor::build_track_count_menu ()
                zoom_preset_selector.AddMenuElem (MenuElem (_("Zoom to 8 hours"), sigc::bind (sigc::mem_fun(*this, &Editor::set_zoom_preset), 8 * 60 * 60 * 1000)));
                zoom_preset_selector.AddMenuElem (MenuElem (_("Zoom to 24 hours"), sigc::bind (sigc::mem_fun(*this, &Editor::set_zoom_preset), 24 * 60 * 60 * 1000)));
                zoom_preset_selector.AddMenuElem (MenuElem (_("Zoom to Session"), sigc::mem_fun(*this, &Editor::temporal_zoom_session)));
+               zoom_preset_selector.AddMenuElem (MenuElem (_("Zoom to Extents"), sigc::mem_fun(*this, &Editor::temporal_zoom_extents)));
                zoom_preset_selector.AddMenuElem (MenuElem (_("Zoom to Range/Region Selection"), sigc::bind (sigc::mem_fun(*this, &Editor::temporal_zoom_selection), Horizontal)));
        }
 }
@@ -3799,13 +3915,13 @@ Editor::build_track_count_menu ()
 void
 Editor::set_zoom_preset (int64_t ms)
 {
-       if ( ms <= 0 ) {
+       if (ms <= 0) {
                temporal_zoom_session();
                return;
        }
 
-       ARDOUR::framecnt_t const sample_rate = ARDOUR::AudioEngine::instance()->sample_rate();
-       temporal_zoom( (sample_rate * ms / 1000) / _visible_canvas_width );
+       ARDOUR::samplecnt_t const sample_rate = ARDOUR::AudioEngine::instance()->sample_rate();
+       temporal_zoom ((sample_rate * ms / 1000) / _visible_canvas_width);
 }
 
 void
@@ -3834,11 +3950,21 @@ Editor::set_visible_track_count (int32_t n)
                str = s.str();
        } else if (_visible_track_count == 0) {
                uint32_t n = 0;
-               for (TrackViewList::iterator i = track_views.begin(); i != track_views.end(); ++i) {
+               for (TrackViewList::const_iterator i = track_views.begin(); i != track_views.end(); ++i) {
                        if ((*i)->marked_for_display()) {
                                ++n;
+                               TimeAxisView::Children cl ((*i)->get_child_list ());
+                               for (TimeAxisView::Children::const_iterator j = cl.begin(); j != cl.end(); ++j) {
+                                       if ((*j)->marked_for_display()) {
+                                               ++n;
+                                       }
+                               }
                        }
                }
+               if (n == 0) {
+                       visible_tracks_selector.set_text (X_("*"));
+                       return;
+               }
                h = trackviews_height() / n;
                str = _("All");
        } else {
@@ -3862,7 +3988,7 @@ void
 Editor::override_visible_track_count ()
 {
        _visible_track_count = -1;
-       visible_tracks_selector.set_text ( _("*") );
+       visible_tracks_selector.set_text (_("*"));
 }
 
 bool
@@ -3930,22 +4056,18 @@ Editor::cycle_zoom_focus ()
 }
 
 void
-Editor::set_show_measures (bool yn)
+Editor::update_grid ()
 {
-       if (_show_measures != yn) {
-               hide_measures ();
-
-               if ((_show_measures = yn) == true) {
-                       if (tempo_lines) {
-                               tempo_lines->show();
-                       }
-
-                       std::vector<TempoMap::BBTPoint> grid;
-                       compute_current_bbt_points (grid, leftmost_frame, leftmost_frame + current_page_samples());
-                       draw_measures (grid);
+       if (grid_musical()) {
+               std::vector<TempoMap::BBTPoint> grid;
+               if (bbt_ruler_scale != bbt_show_many) {
+                       compute_current_bbt_points (grid, _leftmost_sample, _leftmost_sample + current_page_samples());
                }
-
-               instant_save ();
+               maybe_draw_grid_lines ();
+       } else if (grid_nonmusical()) {
+               maybe_draw_grid_lines ();
+       } else {
+               hide_grid_lines ();
        }
 }
 
@@ -3989,8 +4111,7 @@ Editor::set_stationary_playhead (bool yn)
 {
        if (_stationary_playhead != yn) {
                if ((_stationary_playhead = yn) == true) {
-                       /* catch up */
-                       // FIXME need a 3.0 equivalent of this 2.X call
+                       /* catch up -- FIXME need a 3.0 equivalent of this 2.X call */
                        // update_current_screen ();
                }
                instant_save ();
@@ -4003,8 +4124,8 @@ Editor::playlist_selector () const
        return *_playlist_selector;
 }
 
-framecnt_t
-Editor::get_paste_offset (framepos_t pos, unsigned paste_count, framecnt_t duration)
+samplecnt_t
+Editor::get_paste_offset (samplepos_t pos, unsigned paste_count, samplecnt_t duration)
 {
        if (paste_count == 0) {
                /* don't bother calculating an offset that will be zero anyway */
@@ -4012,38 +4133,43 @@ Editor::get_paste_offset (framepos_t pos, unsigned paste_count, framecnt_t durat
        }
 
        /* calculate basic unsnapped multi-paste offset */
-       framecnt_t offset = paste_count * duration;
+       samplecnt_t offset = paste_count * duration;
 
        /* snap offset so pos + offset is aligned to the grid */
-       MusicFrame offset_pos (pos + offset, 0);
+       MusicSample offset_pos (pos + offset, 0);
        snap_to(offset_pos, RoundUpMaybe);
-       offset = offset_pos.frame - pos;
+       offset = offset_pos.sample - pos;
 
        return offset;
 }
 
 unsigned
-Editor::get_grid_beat_divisions(framepos_t position)
-{
-       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;
-       default:               return 0;
+Editor::get_grid_beat_divisions(samplepos_t position)
+{
+       switch (_grid_type) {
+       case GridTypeBeatDiv32:  return 32;
+       case GridTypeBeatDiv28:  return 28;
+       case GridTypeBeatDiv24:  return 24;
+       case GridTypeBeatDiv20:  return 20;
+       case GridTypeBeatDiv16:  return 16;
+       case GridTypeBeatDiv14:  return 14;
+       case GridTypeBeatDiv12:  return 12;
+       case GridTypeBeatDiv10:  return 10;
+       case GridTypeBeatDiv8:   return 8;
+       case GridTypeBeatDiv7:   return 7;
+       case GridTypeBeatDiv6:   return 6;
+       case GridTypeBeatDiv5:   return 5;
+       case GridTypeBeatDiv4:   return 4;
+       case GridTypeBeatDiv3:   return 3;
+       case GridTypeBeatDiv2:   return 2;
+       case GridTypeBeat:       return 1;
+       case GridTypeBar:        return 1;
+
+       case GridTypeNone:       return 0;
+       case GridTypeTimecode:   return 0;
+       case GridTypeMinSec:     return 0;
+       case GridTypeCDFrame:    return 0;
+       default:                 return 0;
        }
        return 0;
 }
@@ -4056,56 +4182,58 @@ Editor::get_grid_beat_divisions(framepos_t position)
 int32_t
 Editor::get_grid_music_divisions (uint32_t event_state)
 {
-       if (snap_mode() == Editing::SnapOff && !ArdourKeyboard::indicates_snap (event_state)) {
+       if (snap_mode() == SnapOff && !ArdourKeyboard::indicates_snap (event_state)) {
                return 0;
        }
 
-       if (snap_mode() != Editing::SnapOff && ArdourKeyboard::indicates_snap (event_state)) {
+       if (snap_mode() != 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;
+       switch (_grid_type) {
+       case GridTypeBeatDiv32:  return 32;
+       case GridTypeBeatDiv28:  return 28;
+       case GridTypeBeatDiv24:  return 24;
+       case GridTypeBeatDiv20:  return 20;
+       case GridTypeBeatDiv16:  return 16;
+       case GridTypeBeatDiv14:  return 14;
+       case GridTypeBeatDiv12:  return 12;
+       case GridTypeBeatDiv10:  return 10;
+       case GridTypeBeatDiv8:   return 8;
+       case GridTypeBeatDiv7:   return 7;
+       case GridTypeBeatDiv6:   return 6;
+       case GridTypeBeatDiv5:   return 5;
+       case GridTypeBeatDiv4:   return 4;
+       case GridTypeBeatDiv3:   return 3;
+       case GridTypeBeatDiv2:   return 2;
+       case GridTypeBeat:       return 1;
+       case GridTypeBar :       return -1;
+
+       case GridTypeNone:       return 0;
+       case GridTypeTimecode:   return 0;
+       case GridTypeMinSec:     return 0;
+       case GridTypeCDFrame:    return 0;
        }
        return 0;
 }
 
-Evoral::Beats
-Editor::get_grid_type_as_beats (bool& success, framepos_t position)
+Temporal::Beats
+Editor::get_grid_type_as_beats (bool& success, samplepos_t position)
 {
        success = true;
 
        const unsigned divisions = get_grid_beat_divisions(position);
        if (divisions) {
-               return Evoral::Beats(1.0 / (double)get_grid_beat_divisions(position));
+               return Temporal::Beats(1.0 / (double)get_grid_beat_divisions(position));
        }
 
-       switch (_snap_type) {
-       case SnapToBeat:
-               return Evoral::Beats(4.0 / _session->tempo_map().meter_at_frame (position).note_divisor());
-       case SnapToBar:
+       switch (_grid_type) {
+       case GridTypeBeat:
+               return Temporal::Beats(4.0 / _session->tempo_map().meter_at_sample (position).note_divisor());
+       case GridTypeBar:
                if (_session) {
-                       const Meter& m = _session->tempo_map().meter_at_frame (position);
-                       return Evoral::Beats((4.0 * m.divisions_per_bar()) / m.note_divisor());
+                       const Meter& m = _session->tempo_map().meter_at_sample (position);
+                       return Temporal::Beats((4.0 * m.divisions_per_bar()) / m.note_divisor());
                }
                break;
        default:
@@ -4113,13 +4241,13 @@ Editor::get_grid_type_as_beats (bool& success, framepos_t position)
                break;
        }
 
-       return Evoral::Beats();
+       return Temporal::Beats();
 }
 
-framecnt_t
-Editor::get_nudge_distance (framepos_t pos, framecnt_t& next)
+samplecnt_t
+Editor::get_nudge_distance (samplepos_t pos, samplecnt_t& next)
 {
-       framecnt_t ret;
+       samplecnt_t ret;
 
        ret = nudge_clock->current_duration (pos);
        next = ret + 1; /* XXXX fix me */
@@ -4147,7 +4275,7 @@ Editor::playlist_deletion_dialog (boost::shared_ptr<Playlist> pl)
        dialog.add_button (_("Keep Remaining"), RESPONSE_NO); // ditto
        dialog.add_button (_("Cancel"), RESPONSE_CANCEL);
 
-       // by default gtk uses the left most button
+       /* by default gtk uses the left most button */
        keep->grab_focus ();
 
        switch (dialog.run ()) {
@@ -4179,7 +4307,7 @@ Editor::playlist_deletion_dialog (boost::shared_ptr<Playlist> pl)
 }
 
 bool
-Editor::audio_region_selection_covers (framepos_t where)
+Editor::audio_region_selection_covers (samplepos_t where)
 {
        for (RegionSelection::iterator a = selection->regions.begin(); a != selection->regions.end(); ++a) {
                if ((*a)->region()->covers (where)) {
@@ -4350,13 +4478,13 @@ Editor::get_y_origin () const
 }
 
 /** Queue up a change to the viewport x origin.
- *  @param frame New x origin.
+ *  @param sample New x origin.
  */
 void
-Editor::reset_x_origin (framepos_t frame)
+Editor::reset_x_origin (samplepos_t sample)
 {
        pending_visual_change.add (VisualChange::TimeOrigin);
-       pending_visual_change.time_origin = frame;
+       pending_visual_change.time_origin = sample;
        ensure_visual_change_idle_handler ();
 }
 
@@ -4369,7 +4497,7 @@ Editor::reset_y_origin (double y)
 }
 
 void
-Editor::reset_zoom (framecnt_t spp)
+Editor::reset_zoom (samplecnt_t spp)
 {
        if (spp == samples_per_pixel) {
                return;
@@ -4381,9 +4509,9 @@ Editor::reset_zoom (framecnt_t spp)
 }
 
 void
-Editor::reposition_and_zoom (framepos_t frame, double fpu)
+Editor::reposition_and_zoom (samplepos_t sample, double fpu)
 {
-       reset_x_origin (frame);
+       reset_x_origin (sample);
        reset_zoom (fpu);
 
        if (!no_save_visual) {
@@ -4407,7 +4535,7 @@ Editor::current_visual_state (bool with_tracks)
        VisualState* vs = new VisualState (with_tracks);
        vs->y_position = vertical_adjustment.get_value();
        vs->samples_per_pixel = samples_per_pixel;
-       vs->leftmost_frame = leftmost_frame;
+       vs->_leftmost_sample = _leftmost_sample;
        vs->zoom_focus = zoom_focus;
 
        if (with_tracks) {
@@ -4445,8 +4573,7 @@ Editor::redo_visual_state ()
        VisualState* vs = redo_visual_stack.back();
        redo_visual_stack.pop_back();
 
-       // can 'vs' really be 0? Is there a place that puts NULL pointers onto the stack?
-       // why do we check here?
+       /* XXX: can 'vs' really be 0? Is there a place that puts NULL pointers onto the stack? */
        undo_visual_stack.push_back (current_visual_state (vs ? (vs->gui_state != 0) : false));
 
        if (vs) {
@@ -4473,7 +4600,7 @@ Editor::use_visual_state (VisualState& vs)
        vertical_adjustment.set_value (vs.y_position);
 
        set_zoom_focus (vs.zoom_focus);
-       reposition_and_zoom (vs.leftmost_frame, vs.samples_per_pixel);
+       reposition_and_zoom (vs._leftmost_sample, vs.samples_per_pixel);
 
        if (vs.gui_state) {
                ARDOUR_UI::instance()->gui_object_state->set_state (vs.gui_state->get_state());
@@ -4492,14 +4619,14 @@ Editor::use_visual_state (VisualState& vs)
  *  @param spp new number of samples per pixel
  */
 void
-Editor::set_samples_per_pixel (framecnt_t spp)
+Editor::set_samples_per_pixel (samplecnt_t spp)
 {
        if (spp < 1) {
                return;
        }
 
-       const framecnt_t three_days = 3 * 24 * 60 * 60 * (_session ? _session->frame_rate() : 48000);
-       const framecnt_t lots_of_pixels = 4000;
+       const samplecnt_t three_days = 3 * 24 * 60 * 60 * (_session ? _session->sample_rate() : 48000);
+       const samplecnt_t lots_of_pixels = 4000;
 
        /* if the zoom level is greater than what you'd get trying to display 3
         * days of audio on a really big screen, then it's too big.
@@ -4510,14 +4637,14 @@ Editor::set_samples_per_pixel (framecnt_t spp)
        }
 
        samples_per_pixel = spp;
+}
 
-       if (tempo_lines) {
-               tempo_lines->tempo_map_changed(_session->tempo_map().music_origin());
-       }
-
+void
+Editor::on_samples_per_pixel_changed ()
+{
        bool const showing_time_selection = selection->time.length() > 0;
 
-       if (showing_time_selection && selection->time.start () != selection->time.end_frame ()) {
+       if (showing_time_selection && selection->time.start () != selection->time.end_sample ()) {
                for (TrackViewList::iterator i = selection->tracks.begin(); i != selection->tracks.end(); ++i) {
                        (*i)->reshow_selection (selection->time);
                }
@@ -4533,7 +4660,7 @@ Editor::set_samples_per_pixel (framecnt_t spp)
        }
 
        if (playhead_cursor) {
-               playhead_cursor->set_position (playhead_cursor->current_frame ());
+               playhead_cursor->set_position (playhead_cursor->current_sample ());
        }
 
        refresh_location_display();
@@ -4544,22 +4671,16 @@ Editor::set_samples_per_pixel (framecnt_t spp)
        instant_save ();
 }
 
-framepos_t
+samplepos_t
 Editor::playhead_cursor_sample () const
 {
-       return playhead_cursor->current_frame();
+       return playhead_cursor->current_sample();
 }
 
 void
 Editor::queue_visual_videotimeline_update ()
 {
-       /* TODO:
-        * pending_visual_change.add (VisualChange::VideoTimeline);
-        * or maybe even more specific: which videotimeline-image
-        * currently it calls update_video_timeline() to update
-        * _all outdated_ images on the video-timeline.
-        * see 'exposeimg()' in video_image_frame.cc
-        */
+       pending_visual_change.add (VisualChange::VideoTimeline);
        ensure_visual_change_idle_handler ();
 }
 
@@ -4567,7 +4688,7 @@ void
 Editor::ensure_visual_change_idle_handler ()
 {
        if (pending_visual_change.idle_handler_id < 0) {
-               // see comment in add_to_idle_resize above.
+               /* see comment in add_to_idle_resize above. */
                pending_visual_change.idle_handler_id = g_idle_add_full (G_PRIORITY_HIGH_IDLE + 10, _idle_visual_changer, this, NULL);
                pending_visual_change.being_handled = false;
        }
@@ -4592,6 +4713,8 @@ Editor::pre_render ()
 int
 Editor::idle_visual_changer ()
 {
+       pending_visual_change.idle_handler_id = -1;
+
        if (pending_visual_change.pending == 0) {
                return 0;
        }
@@ -4606,8 +4729,6 @@ Editor::idle_visual_changer ()
           the last one.
        */
 
-       pending_visual_change.idle_handler_id = -1;
-
        if (visual_change_queued) {
                return 0;
        }
@@ -4630,34 +4751,53 @@ Editor::idle_visual_changer ()
 void
 Editor::visual_changer (const VisualChange& vc)
 {
-       double const last_time_origin = horizontal_position ();
-
+       /**
+        * Changed first so the correct horizontal canvas position is calculated in
+        * Editor::set_horizontal_position
+        */
        if (vc.pending & VisualChange::ZoomLevel) {
                set_samples_per_pixel (vc.samples_per_pixel);
-
-               compute_fixed_ruler_scale ();
-
-               compute_bbt_ruler_scale (vc.time_origin, pending_visual_change.time_origin + current_page_samples());
-               update_tempo_based_rulers ();
-
-               update_video_timeline();
        }
 
        if (vc.pending & VisualChange::TimeOrigin) {
-               set_horizontal_position (sample_to_pixel_unrounded (vc.time_origin));
+               double new_time_origin = sample_to_pixel_unrounded (vc.time_origin);
+               set_horizontal_position (new_time_origin);
        }
 
        if (vc.pending & VisualChange::YOrigin) {
                vertical_adjustment.set_value (vc.y_origin);
        }
 
-       if (last_time_origin == horizontal_position ()) {
-               /* changed signal not emitted */
-               update_fixed_rulers ();
-               redisplay_tempo (true);
+       /**
+        * Now the canvas is in the final state before render the canvas items that
+        * support the Item::prepare_for_render interface can calculate the correct
+        * item to visible canvas intersection.
+        */
+       if (vc.pending & VisualChange::ZoomLevel) {
+               on_samples_per_pixel_changed ();
+
+               compute_fixed_ruler_scale ();
+
+               compute_bbt_ruler_scale (vc.time_origin, pending_visual_change.time_origin + current_page_samples());
+               update_tempo_based_rulers ();
        }
 
        if (!(vc.pending & VisualChange::ZoomLevel)) {
+               /* If the canvas is not being zoomed then the canvas items will not change
+                * and cause Item::prepare_for_render to be called so do it here manually.
+                * Not ideal, but I can't think of a better solution atm.
+                */
+               _track_canvas->prepare_for_render();
+       }
+
+       /* If we are only scrolling vertically there is no need to update these */
+       if (vc.pending != VisualChange::YOrigin) {
+               update_fixed_rulers ();
+               redisplay_grid (true);
+
+               /* video frames & position need to be updated for zoom, horiz-scroll
+                * and (explicitly) VisualChange::VideoTimeline.
+                */
                update_video_timeline();
        }
 
@@ -4677,11 +4817,11 @@ Editor::sort_track_selection (TrackViewList& sel)
        sel.sort (cmp);
 }
 
-framepos_t
+samplepos_t
 Editor::get_preferred_edit_position (EditIgnoreOption ignore, bool from_context_menu, bool from_outside_canvas)
 {
        bool ignored;
-       framepos_t where = 0;
+       samplepos_t where = 0;
        EditPoint ep = _edit_point;
 
        if (Profile->get_mixbus()) {
@@ -4697,28 +4837,29 @@ Editor::get_preferred_edit_position (EditIgnoreOption ignore, bool from_context_
        }
 
        if (entered_marker) {
-                DEBUG_TRACE (DEBUG::CutNPaste, string_compose ("GPEP: use entered marker @ %1\n", entered_marker->position()));
+               DEBUG_TRACE (DEBUG::CutNPaste, string_compose ("GPEP: use entered marker @ %1\n", entered_marker->position()));
                return entered_marker->position();
        }
 
-       if ( (ignore==EDIT_IGNORE_PHEAD) && ep == EditAtPlayhead) {
+       if ((ignore == EDIT_IGNORE_PHEAD) && ep == EditAtPlayhead) {
                ep = EditAtSelectedMarker;
        }
 
-       if ( (ignore==EDIT_IGNORE_MOUSE) && ep == EditAtMouse) {
+       if ((ignore == EDIT_IGNORE_MOUSE) && ep == EditAtMouse) {
                ep = EditAtPlayhead;
        }
 
-       MusicFrame snap_mf (0, 0);
+       MusicSample snap_mf (0, 0);
 
        switch (ep) {
        case EditAtPlayhead:
-               if (_dragging_playhead && _control_scroll_target) {
-                       where = *_control_scroll_target;
+               if (_dragging_playhead) {
+                       /* NOTE: since the user is dragging with the mouse, this operation will implicitly be Snapped */
+                       where = playhead_cursor->current_sample();
                } else {
-                       where = _session->audible_frame();
+                       where = _session->audible_sample();
                }
-                DEBUG_TRACE (DEBUG::CutNPaste, string_compose ("GPEP: use playhead @ %1\n", where));
+               DEBUG_TRACE (DEBUG::CutNPaste, string_compose ("GPEP: use playhead @ %1\n", where));
                break;
 
        case EditAtSelectedMarker:
@@ -4731,7 +4872,7 @@ Editor::get_preferred_edit_position (EditIgnoreOption ignore, bool from_context_
                                } else {
                                        where = loc->end();
                                }
-                                DEBUG_TRACE (DEBUG::CutNPaste, string_compose ("GPEP: use selected marker @ %1\n", where));
+                               DEBUG_TRACE (DEBUG::CutNPaste, string_compose ("GPEP: use selected marker @ %1\n", where));
                                break;
                        }
                }
@@ -4739,14 +4880,14 @@ Editor::get_preferred_edit_position (EditIgnoreOption ignore, bool from_context_
 
        default:
        case EditAtMouse:
-               if (!mouse_frame (where, ignored)) {
+               if (!mouse_sample (where, ignored)) {
                        /* XXX not right but what can we do ? */
                        return 0;
                }
-               snap_mf.frame = where;
+               snap_mf.sample = where;
                snap_to (snap_mf);
-               where = snap_mf.frame;
-                DEBUG_TRACE (DEBUG::CutNPaste, string_compose ("GPEP: use mouse @ %1\n", where));
+               where = snap_mf.sample;
+               DEBUG_TRACE (DEBUG::CutNPaste, string_compose ("GPEP: use mouse @ %1\n", where));
                break;
        }
 
@@ -4754,7 +4895,7 @@ Editor::get_preferred_edit_position (EditIgnoreOption ignore, bool from_context_
 }
 
 void
-Editor::set_loop_range (framepos_t start, framepos_t end, string cmd)
+Editor::set_loop_range (samplepos_t start, samplepos_t end, string cmd)
 {
        if (!_session) return;
 
@@ -4781,7 +4922,7 @@ Editor::set_loop_range (framepos_t start, framepos_t end, string cmd)
 }
 
 void
-Editor::set_punch_range (framepos_t start, framepos_t end, string cmd)
+Editor::set_punch_range (samplepos_t start, samplepos_t end, string cmd)
 {
        if (!_session) return;
 
@@ -4813,7 +4954,7 @@ Editor::set_punch_range (framepos_t start, framepos_t end, string cmd)
  *  @param ts Tracks to look on; if this is empty, all tracks are examined.
  */
 void
-Editor::get_regions_at (RegionSelection& rs, framepos_t where, const TrackViewList& ts) const
+Editor::get_regions_at (RegionSelection& rs, samplepos_t where, const TrackViewList& ts) const
 {
        const TrackViewList* tracks;
 
@@ -4833,8 +4974,7 @@ Editor::get_regions_at (RegionSelection& rs, framepos_t where, const TrackViewLi
 
                        if ((tr = rtv->track()) && ((pl = tr->playlist()))) {
 
-                               boost::shared_ptr<RegionList> regions = pl->regions_at (
-                                               (framepos_t) floor ( (double) where * tr->speed()));
+                               boost::shared_ptr<RegionList> regions = pl->regions_at (where);
 
                                for (RegionList::iterator i = regions->begin(); i != regions->end(); ++i) {
                                        RegionView* rv = rtv->view()->find_view (*i);
@@ -4848,7 +4988,7 @@ Editor::get_regions_at (RegionSelection& rs, framepos_t where, const TrackViewLi
 }
 
 void
-Editor::get_regions_after (RegionSelection& rs, framepos_t where, const TrackViewList& ts) const
+Editor::get_regions_after (RegionSelection& rs, samplepos_t where, const TrackViewList& ts) const
 {
        const TrackViewList* tracks;
 
@@ -4866,8 +5006,7 @@ Editor::get_regions_after (RegionSelection& rs, framepos_t where, const TrackVie
 
                        if ((tr = rtv->track()) && ((pl = tr->playlist()))) {
 
-                               boost::shared_ptr<RegionList> regions = pl->regions_touched (
-                                       (framepos_t) floor ( (double)where * tr->speed()), max_framepos);
+                               boost::shared_ptr<RegionList> regions = pl->regions_touched (where, max_samplepos);
 
                                for (RegionList::iterator i = regions->begin(); i != regions->end(); ++i) {
 
@@ -4899,20 +5038,20 @@ Editor::get_regions_from_selection_and_edit_point (EditIgnoreOption ignore, bool
 {
        RegionSelection regions;
 
-       if (_edit_point == EditAtMouse && entered_regionview && selection->tracks.empty() && selection->regions.empty() ) {
+       if (_edit_point == EditAtMouse && entered_regionview && selection->tracks.empty() && selection->regions.empty()) {
                regions.add (entered_regionview);
        } else {
                regions = selection->regions;
        }
 
-       if ( regions.empty() ) {
+       if (regions.empty()) {
                TrackViewList tracks = selection->tracks;
 
                if (!tracks.empty()) {
                        /* no region selected or entered, but some selected tracks:
                         * act on all regions on the selected tracks at the edit point
                         */
-                       framepos_t const where = get_preferred_edit_position (ignore, from_context_menu, from_outside_canvas);
+                       samplepos_t const where = get_preferred_edit_position (ignore, from_context_menu, from_outside_canvas);
                        get_regions_at(regions, where, tracks);
                }
        }
@@ -4932,17 +5071,17 @@ Editor::get_regions_from_selection_and_edit_point (EditIgnoreOption ignore, bool
  *  Note that we have forced the rule that selected regions and selected tracks are mutually exclusive
  */
 RegionSelection
-Editor::get_regions_from_selection_and_mouse (framepos_t pos)
+Editor::get_regions_from_selection_and_mouse (samplepos_t pos)
 {
        RegionSelection regions;
 
-       if (entered_regionview && selection->tracks.empty() && selection->regions.empty() ) {
+       if (entered_regionview && selection->tracks.empty() && selection->regions.empty()) {
                regions.add (entered_regionview);
        } else {
                regions = selection->regions;
        }
 
-       if ( regions.empty() ) {
+       if (regions.empty()) {
                TrackViewList tracks = selection->tracks;
 
                if (!tracks.empty()) {
@@ -5003,7 +5142,7 @@ Editor::get_regionviews_by_id (PBD::ID const id, RegionSelection & regions) cons
 }
 
 void
-Editor::get_per_region_note_selection (list<pair<PBD::ID, set<boost::shared_ptr<Evoral::Note<Evoral::Beats> > > > > &selection) const
+Editor::get_per_region_note_selection (list<pair<PBD::ID, set<boost::shared_ptr<Evoral::Note<Temporal::Beats> > > > > &selection) const
 {
 
        for (TrackViewList::const_iterator i = track_views.begin(); i != track_views.end(); ++i) {
@@ -5128,7 +5267,7 @@ Editor::first_idle ()
 
        selection->set (rs);
 
-       // first idle adds route children (automation tracks), so we need to redisplay here
+       /* first idle adds route children (automation tracks), so we need to redisplay here */
        _routes->redisplay ();
 
        delete dialog;
@@ -5215,7 +5354,7 @@ Editor::located ()
        ENSURE_GUI_THREAD (*this, &Editor::located);
 
        if (_session) {
-               playhead_cursor->set_position (_session->audible_frame ());
+               playhead_cursor->set_position (_session->audible_sample ());
                if (_follow_playhead && !_pending_initial_locate) {
                        reset_x_origin_to_follow_playhead ();
                }
@@ -5242,12 +5381,16 @@ Editor::region_view_added (RegionView * rv)
        }
 
        _summary->set_background_dirty ();
+
+       mark_region_boundary_cache_dirty ();
 }
 
 void
 Editor::region_view_removed ()
 {
        _summary->set_background_dirty ();
+
+       mark_region_boundary_cache_dirty ();
 }
 
 AxisView*
@@ -5490,8 +5633,12 @@ Editor::hide_track_in_display (TimeAxisView* tv, bool apply_to_selection)
        if (!tv) {
                return;
        }
+
+       DisplaySuspender ds;
+       PresentationInfo::ChangeSuspender cs;
+
        if (apply_to_selection) {
-               for (TrackSelection::iterator i = selection->tracks.begin(); i != selection->tracks.end(); ) {
+               for (TrackSelection::iterator i = selection->tracks.begin(); i != selection->tracks.end();) {
 
                        TrackSelection::iterator j = i;
                        ++j;
@@ -5504,7 +5651,7 @@ Editor::hide_track_in_display (TimeAxisView* tv, bool apply_to_selection)
                RouteTimeAxisView* rtv = dynamic_cast<RouteTimeAxisView*> (tv);
 
                if (rtv && current_mixer_strip && (rtv->route() == current_mixer_strip->route())) {
-                       // this will hide the mixer strip
+                       /* this will hide the mixer strip */
                        set_selected_mixer_strip (*tv);
                }
 
@@ -5543,15 +5690,15 @@ 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 (const PBD::ID& id) const
+/** Find a StripableTimeAxisView by the ID of its stripable */
+StripableTimeAxisView*
+Editor::get_stripable_time_axis_by_id (const PBD::ID& id) const
 {
-       RouteTimeAxisView* v;
+       StripableTimeAxisView* v;
 
        for (TrackViewList::const_iterator i = track_views.begin(); i != track_views.end(); ++i) {
-               if((v = dynamic_cast<RouteTimeAxisView*>(*i)) != 0) {
-                       if(v->route()->id() == id) {
+               if((v = dynamic_cast<StripableTimeAxisView*>(*i)) != 0) {
+                       if(v->stripable()->id() == id) {
                                return v;
                        }
                }
@@ -5705,39 +5852,39 @@ Editor::scroll_release ()
 void
 Editor::reset_x_origin_to_follow_playhead ()
 {
-       framepos_t const frame = playhead_cursor->current_frame ();
+       samplepos_t const sample = playhead_cursor->current_sample ();
 
-       if (frame < leftmost_frame || frame > leftmost_frame + current_page_samples()) {
+       if (sample < _leftmost_sample || sample > _leftmost_sample + current_page_samples()) {
 
                if (_session->transport_speed() < 0) {
 
-                       if (frame > (current_page_samples() / 2)) {
-                               center_screen (frame-(current_page_samples()/2));
+                       if (sample > (current_page_samples() / 2)) {
+                               center_screen (sample-(current_page_samples()/2));
                        } else {
                                center_screen (current_page_samples()/2);
                        }
 
                } else {
 
-                       framepos_t l = 0;
+                       samplepos_t l = 0;
 
-                       if (frame < leftmost_frame) {
+                       if (sample < _leftmost_sample) {
                                /* moving left */
                                if (_session->transport_rolling()) {
                                        /* rolling; end up with the playhead at the right of the page */
-                                       l = frame - current_page_samples ();
+                                       l = sample - current_page_samples ();
                                } else {
                                        /* not rolling: end up with the playhead 1/4 of the way along the page */
-                                       l = frame - current_page_samples() / 4;
+                                       l = sample - current_page_samples() / 4;
                                }
                        } else {
                                /* moving right */
                                if (_session->transport_rolling()) {
                                        /* rolling: end up with the playhead on the left of the page */
-                                       l = frame;
+                                       l = sample;
                                } else {
                                        /* not rolling: end up with the playhead 3/4 of the way along the page */
-                                       l = frame - 3 * current_page_samples() / 4;
+                                       l = sample - 3 * current_page_samples() / 4;
                                }
                        }
 
@@ -5774,47 +5921,39 @@ Editor::super_rapid_screen_update ()
                current_mixer_strip->fast_update ();
        }
 
-       /* PLAYHEAD AND VIEWPORT */
+       bool latent_locate = false;
+       samplepos_t sample = _session->audible_sample (&latent_locate);
+       const int64_t now = g_get_monotonic_time ();
+       double err = 0;
 
-       /* There are a few reasons why we might not update the playhead / viewport stuff:
-        *
-        * 1.  we don't update things when there's a pending locate request, otherwise
-        *     when the editor requests a locate there is a chance that this method
-        *     will move the playhead before the locate request is processed, causing
-        *     a visual glitch.
-        * 2.  if we're not rolling, there's nothing to do here (locates are handled elsewhere).
-        * 3.  if we're still at the same frame that we were last time, there's nothing to do.
-        */
-       if (_pending_locate_request || !_session->transport_rolling ()) {
+       if (_session->exporting ()) {
+               /* freewheel/export may be faster or slower than transport_speed() / SR.
+                * Also exporting multiple ranges locates/jumps without a _pending_locate_request.
+                */
                _last_update_time = 0;
-               return;
        }
 
-       if (_dragging_playhead) {
+       if (!_session->transport_rolling () || _session->is_auditioning ()) {
+               /* Do not interpolate the playhead position; just set it */
                _last_update_time = 0;
-               return;
        }
 
-       bool latent_locate = false;
-       framepos_t frame = _session->audible_frame (&latent_locate);
-       const int64_t now = g_get_monotonic_time ();
-       double err = 0;
-
        if (_last_update_time > 0) {
-               const double ds =  (now - _last_update_time) * _session->transport_speed() * _session->nominal_frame_rate () * 1e-6;
-               framepos_t guess = playhead_cursor->current_frame () + rint (ds);
-               err = frame - guess;
+               /* interpolate and smoothen playhead position */
+               const double ds =  (now - _last_update_time) * _session->transport_speed() * _session->nominal_sample_rate () * 1e-6;
+               samplepos_t guess = playhead_cursor->current_sample () + rint (ds);
+               err = sample - guess;
 
                guess += err * .12 + _err_screen_engine; // time-constant based on 25fps (super_rapid_screen_update)
                _err_screen_engine += .0144 * (err - _err_screen_engine); // tc^2
 
 #if 0 // DEBUG
                printf ("eng: %ld  gui:%ld (%+6.1f)  diff: %6.1f (err: %7.2f)\n",
-                               frame, guess, ds,
+                               sample, guess, ds,
                                err, _err_screen_engine);
 #endif
 
-               frame = guess;
+               sample = guess;
        } else {
                _err_screen_engine = 0;
        }
@@ -5822,17 +5961,60 @@ Editor::super_rapid_screen_update ()
        if (err > 8192 || latent_locate) {
                // in case of x-runs or freewheeling
                _last_update_time = 0;
+               sample = _session->audible_sample ();
        } else {
                _last_update_time = now;
        }
 
-       if (playhead_cursor->current_frame () == frame) {
+       /* snapped cursor stuff (the snapped_cursor shows where an operation is going to occur) */
+       bool ignored;
+       MusicSample where (sample, 0);
+       if (!UIConfiguration::instance().get_show_snapped_cursor()) {
+               snapped_cursor->hide ();
+       } else if (_edit_point == EditAtPlayhead && !_dragging_playhead) {
+               /* EditAtPlayhead does not snap */
+       } else if (_edit_point == EditAtSelectedMarker) {
+               /* NOTE: I don't think EditAtSelectedMarker should snap. They are what they are.
+                * however, the current editing code -does- snap so I'll draw it that way for now.
+                */
+               if (!selection->markers.empty()) {
+                       MusicSample ms (selection->markers.front()->position(), 0);
+                       snap_to (ms); // should use snap_to_with_modifier?
+                       snapped_cursor->set_position (ms.sample);
+                       snapped_cursor->show ();
+               }
+       } else if (mouse_sample (where.sample, ignored)) { // cursor is in the editing canvas. show it.
+               snapped_cursor->show ();
+       } else { // mouse is out of the editing canvas. hide the snapped_cursor
+               snapped_cursor->hide ();
+       }
+
+       /* There are a few reasons why we might not update the playhead / viewport stuff:
+        *
+        * 1.  we don't update things when there's a pending locate request, otherwise
+        *     when the editor requests a locate there is a chance that this method
+        *     will move the playhead before the locate request is processed, causing
+        *     a visual glitch.
+        * 2.  if we're not rolling, there's nothing to do here (locates are handled elsewhere).
+        * 3.  if we're still at the same frame that we were last time, there's nothing to do.
+        */
+       if (_pending_locate_request) {
+               _last_update_time = 0;
+               return;
+       }
+
+       if (_dragging_playhead) {
+               _last_update_time = 0;
+               return;
+       }
+
+       if (playhead_cursor->current_sample () == sample) {
                return;
        }
 
-       playhead_cursor->set_position (frame);
+       playhead_cursor->set_position (sample);
 
-       if (_session->requested_return_frame() >= 0) {
+       if (_session->requested_return_sample() >= 0) {
                _last_update_time = 0;
                return;
        }
@@ -5851,13 +6033,13 @@ Editor::super_rapid_screen_update ()
        if (!_stationary_playhead) {
                reset_x_origin_to_follow_playhead ();
        } else {
-               framepos_t const frame = playhead_cursor->current_frame ();
-               double target = ((double)frame - (double)current_page_samples() / 2.0);
+               samplepos_t const sample = playhead_cursor->current_sample ();
+               double target = ((double)sample - (double)current_page_samples() / 2.0);
                if (target <= 0.0) {
                        target = 0.0;
                }
-               // compare to EditorCursor::set_position()
-               double const old_pos = sample_to_pixel_unrounded (leftmost_frame);
+               /* compare to EditorCursor::set_position() */
+               double const old_pos = sample_to_pixel_unrounded (_leftmost_sample);
                double const new_pos = sample_to_pixel_unrounded (target);
                if (rint (new_pos) != rint (old_pos)) {
                        reset_x_origin (pixel_to_sample (new_pos));
@@ -5920,11 +6102,11 @@ Editor::session_going_away ()
 
        /* clear tempo/meter rulers */
        remove_metric_marks ();
-       hide_measures ();
        clear_marker_display ();
 
-       delete tempo_lines;
-       tempo_lines = 0;
+       hide_grid_lines ();
+       delete grid_lines;
+       grid_lines = 0;
 
        stop_step_editing ();
 
@@ -5960,7 +6142,7 @@ Editor::show_editor_list (bool yn)
 void
 Editor::change_region_layering_order (bool from_context_menu)
 {
-       const framepos_t position = get_preferred_edit_position (EDIT_IGNORE_NONE, from_context_menu);
+       const samplepos_t position = get_preferred_edit_position (EDIT_IGNORE_NONE, from_context_menu);
 
        if (!clicked_routeview) {
                if (layering_order_editor) {
@@ -6162,6 +6344,11 @@ Editor::ui_parameter_changed (string parameter)
                if (_verbose_cursor) {
                        playhead_cursor->set_sensitive (UIConfiguration::instance().get_draggable_playhead());
                }
+       } else if (parameter == "use-note-bars-for-velocity") {
+               ArdourCanvas::Note::set_show_velocity_bars (UIConfiguration::instance().get_use_note_bars_for_velocity());
+               _track_canvas->request_redraw (_track_canvas->visible_area());
+       } else if (parameter == "use-note-color-for-velocity") {
+               /* handled individually by each MidiRegionView */
        }
 }