HIG-ify the locations UI a bit. Should fix #3526.
[ardour.git] / gtk2_ardour / editor.cc
index aa0a76edcda0759ad9fb2586bf9e2ddfc7cf8c28..950208b19e6bb31d40daaa5965268e16e7199874 100644 (file)
@@ -220,7 +220,8 @@ Gdk::Cursor* Editor::fade_out_cursor = 0;
 Gdk::Cursor* Editor::grabber_cursor = 0;
 Gdk::Cursor* Editor::grabber_note_cursor = 0;
 Gdk::Cursor* Editor::grabber_edit_point_cursor = 0;
-Gdk::Cursor* Editor::zoom_cursor = 0;
+Gdk::Cursor* Editor::zoom_in_cursor = 0;
+Gdk::Cursor* Editor::zoom_out_cursor = 0;
 Gdk::Cursor* Editor::time_fx_cursor = 0;
 Gdk::Cursor* Editor::fader_cursor = 0;
 Gdk::Cursor* Editor::speaker_cursor = 0;
@@ -303,6 +304,9 @@ Editor::Editor ()
        , meters_running(false)
        , _pending_locate_request (false)
        , _pending_initial_locate (false)
+       , _last_cut_copy_source_track (0)
+
+       , _region_selection_change_updates_region_list (true)
 {
        constructed = false;
 
@@ -321,6 +325,7 @@ Editor::Editor ()
        clicked_crossfadeview = 0;
        clicked_control_point = 0;
        last_update_frame = 0;
+        pre_press_cursor = 0;
        _drags = new DragManager (this);
        current_mixer_strip = 0;
        current_bbt_points = 0;
@@ -352,7 +357,6 @@ Editor::Editor ()
        show_gain_after_trim = false;
        verbose_cursor_on = true;
        last_item_entered = 0;
-       last_item_entered_n = 0;
 
        have_pending_keyboard_selection = false;
        _follow_playhead = true;
@@ -731,10 +735,14 @@ Editor::Editor ()
 
        TimeAxisView::CatchDeletion.connect (*this, invalidator (*this), ui_bind (&Editor::timeaxisview_deleted, this, _1), gui_context());
 
-       _last_normalization_value = 0;
+       _ignore_region_action = false;
+       _last_region_menu_was_main = false;
+       _popup_region_menu_item = 0;
 
        constructed = true;
        instant_save ();
+
+       setup_fade_images ();
 }
 
 Editor::~Editor()
@@ -782,6 +790,10 @@ Editor::catch_vanishing_regionview (RegionView *rv)
        if (entered_regionview == rv) {
                set_entered_regionview (0);
        }
+
+       if (!_all_region_actions_sensitized) {
+               sensitize_all_region_actions (true);
+       }
 }
 
 void
@@ -798,6 +810,13 @@ Editor::set_entered_regionview (RegionView* rv)
        if ((entered_regionview = rv) != 0) {
                entered_regionview->entered (internal_editing ());
        }
+
+       if (!_all_region_actions_sensitized && _last_region_menu_was_main) {
+               /* This RegionView entry might have changed what region actions
+                  are allowed, so sensitize them all in case a key is pressed.
+               */
+               sensitize_all_region_actions (true);
+       }
 }
 
 void
@@ -836,8 +855,6 @@ Editor::show_window ()
                        tv = (static_cast<TimeAxisView*>(*i));
                        tv->reset_height ();
                }
-
-               reset_zoom (frames_per_unit);
        }
 
        present ();
@@ -868,10 +885,10 @@ Editor::zoom_adjustment_changed ()
 
        if (fpu < 1.0) {
                fpu = 1.0;
-               zoom_range_clock.set ((nframes64_t) floor (fpu * _canvas_width));
+               zoom_range_clock.set ((framepos_t) floor (fpu * _canvas_width));
        } else if (fpu > _session->current_end_frame() / _canvas_width) {
                fpu = _session->current_end_frame() / _canvas_width;
-               zoom_range_clock.set ((nframes64_t) floor (fpu * _canvas_width));
+               zoom_range_clock.set ((framepos_t) floor (fpu * _canvas_width));
        }
 
        temporal_zoom (fpu);
@@ -891,7 +908,7 @@ Editor::control_scroll (float fraction)
        /*
                _control_scroll_target is an optional<T>
 
-               it acts like a pointer to an nframes64_t, with
+               it acts like a pointer to an framepos_t, with
                a operator conversion to boolean to check
                that it has a value could possibly use
                playhead_cursor->current_frame to store the
@@ -904,12 +921,12 @@ Editor::control_scroll (float fraction)
                _dragging_playhead = true;
        }
 
-       if ((fraction < 0.0f) && (*_control_scroll_target < (nframes64_t) fabs(step))) {
+       if ((fraction < 0.0f) && (*_control_scroll_target < (framepos_t) fabs(step))) {
                *_control_scroll_target = 0;
-       } else if ((fraction > 0.0f) && (max_frames - *_control_scroll_target < step)) {
-               *_control_scroll_target = max_frames - (current_page_frames()*2); // allow room for slop in where the PH is on the screen
+       } else if ((fraction > 0.0f) && (max_framepos - *_control_scroll_target < step)) {
+               *_control_scroll_target = max_framepos - (current_page_frames()*2); // allow room for slop in where the PH is on the screen
        } else {
-               *_control_scroll_target += (nframes64_t) floor (step);
+               *_control_scroll_target += (framepos_t) floor (step);
        }
 
        /* move visuals, we'll catch up with it later */
@@ -940,7 +957,7 @@ Editor::control_scroll (float fraction)
 }
 
 bool
-Editor::deferred_control_scroll (nframes64_t /*target*/)
+Editor::deferred_control_scroll (framepos_t /*target*/)
 {
        _session->request_locate (*_control_scroll_target, _session->transport_rolling());
        // reset for next stream
@@ -974,7 +991,7 @@ Editor::on_realize ()
 }
 
 void
-Editor::map_position_change (nframes64_t frame)
+Editor::map_position_change (framepos_t frame)
 {
        ENSURE_GUI_THREAD (*this, &Editor::map_position_change, frame)
 
@@ -990,7 +1007,7 @@ Editor::map_position_change (nframes64_t frame)
 }
 
 void
-Editor::center_screen (nframes64_t frame)
+Editor::center_screen (framepos_t frame)
 {
        double page = _canvas_width * frames_per_unit;
 
@@ -1003,12 +1020,12 @@ Editor::center_screen (nframes64_t frame)
 }
 
 void
-Editor::center_screen_internal (nframes64_t frame, float page)
+Editor::center_screen_internal (framepos_t frame, float page)
 {
        page /= 2;
 
        if (frame > page) {
-               frame -= (nframes64_t) page;
+               frame -= (framepos_t) page;
        } else {
                frame = 0;
        }
@@ -1077,9 +1094,6 @@ Editor::set_session (Session *t)
 
        compute_fixed_ruler_scale ();
 
-       /* there are never any selected regions at startup */
-       sensitize_the_right_region_actions (false);
-
        XMLNode* node = ARDOUR_UI::instance()->editor_settings();
        set_state (*node, Stateful::loading_state_version);
 
@@ -1185,24 +1199,43 @@ Editor::set_session (Session *t)
        /* register for undo history */
        _session->register_with_memento_command_factory(_id, this);
 
+       ActionManager::ui_manager->signal_pre_activate().connect (sigc::mem_fun (*this, &Editor::action_pre_activated));
+
        start_updating_meters ();
 }
 
+void
+Editor::action_pre_activated (Glib::RefPtr<Action> const & a)
+{
+       if (a->get_name() == "RegionMenu") {
+               /* When the main menu's region menu is opened, we setup the actions so that they look right
+                  in the menu.  I can't find a way of getting a signal when this menu is subsequently closed,
+                  so we resensitize all region actions when the entered regionview or the region selection
+                  changes.  HOWEVER we can't always resensitize on entered_regionview change because that
+                  happens after the region context menu is opened.  So we set a flag here, too.
+
+                  What a carry on :(
+               */
+               sensitize_the_right_region_actions ();
+               _last_region_menu_was_main = true;
+       }
+}
+
 void
 Editor::build_cursors ()
 {
        using namespace Gdk;
 
-       Gdk::Color mbg ("#000000" ); /* Black */
-       Gdk::Color mfg ("#0000ff" ); /* Blue. */
-
        {
-               RefPtr<Bitmap> source, mask;
-               source = Bitmap::create (mag_bits, mag_width, mag_height);
-               mask = Bitmap::create (magmask_bits, mag_width, mag_height);
-               zoom_cursor = new Gdk::Cursor (source, mask, mfg, mbg, mag_x_hot, mag_y_hot);
+               Glib::RefPtr<Gdk::Pixbuf> zoom_in_cursor_pixbuf (::get_icon ("zoom_in_cursor"));
+               zoom_in_cursor = new Gdk::Cursor (Gdk::Display::get_default(), zoom_in_cursor_pixbuf, 5, 5);
        }
 
+       {
+               Glib::RefPtr<Gdk::Pixbuf> zoom_out_cursor_pixbuf (::get_icon ("zoom_out_cursor"));
+               zoom_out_cursor = new Gdk::Cursor (Gdk::Display::get_default(), zoom_out_cursor_pixbuf, 5, 5);
+       }
+       
        Gdk::Color fbg ("#ffffff" );
        Gdk::Color ffg  ("#000000" );
 
@@ -1237,8 +1270,10 @@ Editor::build_cursors ()
                transparent_cursor = new Gdk::Cursor (bits, bits, c, c, 0, 0);
        }
 
-
-       grabber_cursor = new Gdk::Cursor (HAND2);
+       {
+               Glib::RefPtr<Gdk::Pixbuf> grabber_pixbuf (::get_icon ("grabber"));
+               grabber_cursor = new Gdk::Cursor (Gdk::Display::get_default(), grabber_pixbuf, 5, 0);
+       }
 
        {
                Glib::RefPtr<Gdk::Pixbuf> grabber_note_pixbuf (::get_icon ("grabber_note"));
@@ -1312,14 +1347,57 @@ Editor::popup_fade_context_menu (int button, int32_t time, ArdourCanvas::Item* i
                items.push_back (SeparatorElem());
 
                if (Profile->get_sae()) {
-                       items.push_back (MenuElem (_("Linear"), sigc::bind (sigc::mem_fun (*this, &Editor::set_fade_in_shape), AudioRegion::Linear)));
-                       items.push_back (MenuElem (_("Slowest"), sigc::bind (sigc::mem_fun (*this, &Editor::set_fade_in_shape), AudioRegion::Fast)));
+                       
+                       items.push_back (MenuElem (_("Linear"), sigc::bind (sigc::mem_fun (*this, &Editor::set_fade_in_shape), FadeLinear)));
+                       items.push_back (MenuElem (_("Slowest"), sigc::bind (sigc::mem_fun (*this, &Editor::set_fade_in_shape), FadeFast)));
+                       
                } else {
-                       items.push_back (MenuElem (_("Linear"), sigc::bind (sigc::mem_fun (*this, &Editor::set_fade_in_shape), AudioRegion::Linear)));
-                       items.push_back (MenuElem (_("Slowest"), sigc::bind (sigc::mem_fun (*this, &Editor::set_fade_in_shape), AudioRegion::Fast)));
-                       items.push_back (MenuElem (_("Slow"), sigc::bind (sigc::mem_fun (*this, &Editor::set_fade_in_shape), AudioRegion::LogB)));
-                       items.push_back (MenuElem (_("Fast"), sigc::bind (sigc::mem_fun (*this, &Editor::set_fade_in_shape), AudioRegion::LogA)));
-                       items.push_back (MenuElem (_("Fastest"), sigc::bind (sigc::mem_fun (*this, &Editor::set_fade_in_shape), AudioRegion::Slow)));
+
+                       items.push_back (
+                               ImageMenuElem (
+                                       _("Linear"),
+                                       *_fade_in_images[FadeLinear],
+                                       sigc::bind (sigc::mem_fun (*this, &Editor::set_fade_in_shape), FadeLinear)
+                                       )
+                               );
+
+                       dynamic_cast<ImageMenuItem*>(&items.back())->set_always_show_image ();
+
+                       items.push_back (
+                               ImageMenuElem (
+                                       _("Slowest"),
+                                       *_fade_in_images[FadeFast],
+                                       sigc::bind (sigc::mem_fun (*this, &Editor::set_fade_in_shape), FadeFast)
+                                       ));
+                       
+                       dynamic_cast<ImageMenuItem*>(&items.back())->set_always_show_image ();
+                       
+                       items.push_back (
+                               ImageMenuElem (
+                                       _("Slow"),
+                                       *_fade_in_images[FadeLogB],
+                                       sigc::bind (sigc::mem_fun (*this, &Editor::set_fade_in_shape), FadeLogB)
+                                       ));
+                       
+                       dynamic_cast<ImageMenuItem*>(&items.back())->set_always_show_image ();
+                       
+                       items.push_back (
+                               ImageMenuElem (
+                                       _("Fast"),
+                                       *_fade_in_images[FadeLogA],
+                                       sigc::bind (sigc::mem_fun (*this, &Editor::set_fade_in_shape), FadeLogA)
+                                       ));
+                       
+                       dynamic_cast<ImageMenuItem*>(&items.back())->set_always_show_image ();
+                       
+                       items.push_back (
+                               ImageMenuElem (
+                                       _("Fastest"),
+                                       *_fade_in_images[FadeSlow],
+                                       sigc::bind (sigc::mem_fun (*this, &Editor::set_fade_in_shape), FadeSlow)
+                                       ));
+                       
+                       dynamic_cast<ImageMenuItem*>(&items.back())->set_always_show_image ();
                }
 
                break;
@@ -1335,14 +1413,55 @@ Editor::popup_fade_context_menu (int button, int32_t time, ArdourCanvas::Item* i
                items.push_back (SeparatorElem());
 
                if (Profile->get_sae()) {
-                       items.push_back (MenuElem (_("Linear"), sigc::bind (sigc::mem_fun (*this, &Editor::set_fade_out_shape), AudioRegion::Linear)));
-                       items.push_back (MenuElem (_("Slowest"), sigc::bind (sigc::mem_fun (*this, &Editor::set_fade_out_shape), AudioRegion::Slow)));
+                       items.push_back (MenuElem (_("Linear"), sigc::bind (sigc::mem_fun (*this, &Editor::set_fade_out_shape), FadeLinear)));
+                       items.push_back (MenuElem (_("Slowest"), sigc::bind (sigc::mem_fun (*this, &Editor::set_fade_out_shape), FadeSlow)));
                } else {
-                       items.push_back (MenuElem (_("Linear"), sigc::bind (sigc::mem_fun (*this, &Editor::set_fade_out_shape), AudioRegion::Linear)));
-                       items.push_back (MenuElem (_("Slowest"), sigc::bind (sigc::mem_fun (*this, &Editor::set_fade_out_shape), AudioRegion::Slow)));
-                       items.push_back (MenuElem (_("Slow"), sigc::bind (sigc::mem_fun (*this, &Editor::set_fade_out_shape), AudioRegion::LogA)));
-                       items.push_back (MenuElem (_("Fast"), sigc::bind (sigc::mem_fun (*this, &Editor::set_fade_out_shape), AudioRegion::LogB)));
-                       items.push_back (MenuElem (_("Fastest"), sigc::bind (sigc::mem_fun (*this, &Editor::set_fade_out_shape), AudioRegion::Fast)));
+
+                       items.push_back (
+                               ImageMenuElem (
+                                       _("Linear"),
+                                       *_fade_out_images[FadeLinear],
+                                       sigc::bind (sigc::mem_fun (*this, &Editor::set_fade_out_shape), FadeLinear)
+                                       )
+                               );
+
+                       dynamic_cast<ImageMenuItem*>(&items.back())->set_always_show_image ();
+
+                       items.push_back (
+                               ImageMenuElem (
+                                       _("Slowest"),
+                                       *_fade_out_images[FadeFast],
+                                       sigc::bind (sigc::mem_fun (*this, &Editor::set_fade_out_shape), FadeSlow)
+                                       ));
+                       
+                       dynamic_cast<ImageMenuItem*>(&items.back())->set_always_show_image ();
+                       
+                       items.push_back (
+                               ImageMenuElem (
+                                       _("Slow"),
+                                       *_fade_out_images[FadeLogB],
+                                       sigc::bind (sigc::mem_fun (*this, &Editor::set_fade_out_shape), FadeLogA)
+                                       ));
+                       
+                       dynamic_cast<ImageMenuItem*>(&items.back())->set_always_show_image ();
+                       
+                       items.push_back (
+                               ImageMenuElem (
+                                       _("Fast"),
+                                       *_fade_out_images[FadeLogA],
+                                       sigc::bind (sigc::mem_fun (*this, &Editor::set_fade_out_shape), FadeLogB)
+                                       ));
+                       
+                       dynamic_cast<ImageMenuItem*>(&items.back())->set_always_show_image ();
+                       
+                       items.push_back (
+                               ImageMenuElem (
+                                       _("Fastest"),
+                                       *_fade_out_images[FadeSlow],
+                                       sigc::bind (sigc::mem_fun (*this, &Editor::set_fade_out_shape), FadeFast)
+                                       ));
+                       
+                       dynamic_cast<ImageMenuItem*>(&items.back())->set_always_show_image ();
                }
 
                break;
@@ -1358,10 +1477,10 @@ Editor::popup_fade_context_menu (int button, int32_t time, ArdourCanvas::Item* i
 }
 
 void
-Editor::popup_track_context_menu (int button, int32_t time, ItemType item_type, bool with_selection, nframes64_t frame)
+Editor::popup_track_context_menu (int button, int32_t time, ItemType item_type, bool with_selection, framepos_t frame)
 {
        using namespace Menu_Helpers;
-       Menu* (Editor::*build_menu_function)(nframes64_t);
+       Menu* (Editor::*build_menu_function)(framepos_t);
        Menu *menu;
 
        switch (item_type) {
@@ -1474,11 +1593,18 @@ Editor::popup_track_context_menu (int button, int32_t time, ItemType item_type,
                clicked_routeview->build_underlay_menu(menu);
        }
 
+       /* When the region menu is opened, we setup the actions so that they look right
+          in the menu.
+       */
+       sensitize_the_right_region_actions ();
+       _last_region_menu_was_main = false;
+
+       menu->signal_hide().connect (sigc::bind (sigc::mem_fun (*this, &Editor::sensitize_all_region_actions), true));
        menu->popup (button, time);
 }
 
 Menu*
-Editor::build_track_context_menu (nframes64_t)
+Editor::build_track_context_menu (framepos_t)
 {
        using namespace Menu_Helpers;
 
@@ -1490,7 +1616,7 @@ Editor::build_track_context_menu (nframes64_t)
 }
 
 Menu*
-Editor::build_track_bus_context_menu (nframes64_t)
+Editor::build_track_bus_context_menu (framepos_t)
 {
        using namespace Menu_Helpers;
 
@@ -1502,7 +1628,7 @@ Editor::build_track_bus_context_menu (nframes64_t)
 }
 
 Menu*
-Editor::build_track_region_context_menu (nframes64_t frame)
+Editor::build_track_region_context_menu (framepos_t frame)
 {
        using namespace Menu_Helpers;
        MenuList& edit_items  = track_region_context_menu.items();
@@ -1526,24 +1652,7 @@ Editor::build_track_region_context_menu (nframes64_t frame)
                if ((tr = rtv->track()) && ((pl = tr->playlist())) && !internal_editing()) {
                         framepos_t framepos = (framepos_t) floor ((double)frame * tr->speed());
                         uint32_t regions_at = pl->count_regions_at (framepos);
-                        list<boost::shared_ptr<Region> > regions_for_menu;
-
-                       if (selection->regions.size() > 1) {
-                               // there's already a multiple selection: just add a
-                               // single region context menu that will act on all
-                               // selected regions
-
-                               for (RegionSelection::iterator i = selection->regions.begin(); i != selection->regions.end(); ++i) {
-                                       regions_for_menu.push_back ((*i)->region ());
-                               }
-                        } else {
-                                boost::shared_ptr<Region> top_region = pl->top_region_at (framepos);
-                                if (top_region) {
-                                        regions_for_menu.push_back (top_region);
-                                }
-                        }
-
-                        add_region_context_items (rtv->view(), regions_for_menu, edit_items, framepos, regions_at > 1);
+                        add_region_context_items (edit_items, regions_at > 1);
                }
        }
 
@@ -1553,7 +1662,7 @@ Editor::build_track_region_context_menu (nframes64_t frame)
 }
 
 Menu*
-Editor::build_track_crossfade_context_menu (nframes64_t frame)
+Editor::build_track_crossfade_context_menu (framepos_t frame)
 {
        using namespace Menu_Helpers;
        MenuList& edit_items  = track_crossfade_context_menu.items();
@@ -1580,24 +1689,7 @@ Editor::build_track_crossfade_context_menu (nframes64_t frame)
 
                        framepos_t framepos = (framepos_t) floor ((double)frame * tr->speed());
                        uint32_t regions_at = pl->count_regions_at (framepos);
-                        list<boost::shared_ptr<Region> > regions_for_menu;
-
-                       if (selection->regions.size() > 1) {
-                               // there's already a multiple selection: just add a
-                               // single region context menu that will act on all
-                               // selected regions
-
-                               for (RegionSelection::iterator i = selection->regions.begin(); i != selection->regions.end(); ++i) {
-                                       regions_for_menu.push_back ((*i)->region ());
-                               }
-                        } else {
-                                boost::shared_ptr<Region> top_region = pl->top_region_at (framepos);
-                                if (top_region) {
-                                        regions_for_menu.push_back (top_region);
-                                }
-                        }
-                        
-                        add_region_context_items (atv->audio_view(), regions_for_menu, edit_items, framepos, regions_at > 1);
+                        add_region_context_items (edit_items, regions_at > 1);
                }
        }
 
@@ -1607,7 +1699,7 @@ Editor::build_track_crossfade_context_menu (nframes64_t frame)
 }
 
 void
-Editor::analyze_region_selection()
+Editor::analyze_region_selection ()
 {
        if (analysis_window == 0) {
                analysis_window = new AnalysisWindow();
@@ -1643,7 +1735,7 @@ Editor::analyze_range_selection()
 }
 
 Menu*
-Editor::build_track_selection_context_menu (nframes64_t)
+Editor::build_track_selection_context_menu (framepos_t)
 {
        using namespace Menu_Helpers;
        MenuList& edit_items  = track_selection_context_menu.items();
@@ -1717,288 +1809,10 @@ Editor::xfade_edit_right_region ()
 }
 
 void
-Editor::add_region_context_items (StreamView* sv, list<boost::shared_ptr<Region> > regions, Menu_Helpers::MenuList& edit_items,
-                                  framepos_t position, bool multiple_regions_at_position)
+Editor::add_region_context_items (Menu_Helpers::MenuList& edit_items, bool multiple_regions_at_position)
 {
        using namespace Menu_Helpers;
-       Gtk::MenuItem* foo_item;
-       Menu     *region_menu = manage (new Menu);
-       MenuList& items       = region_menu->items();
-       region_menu->set_name ("ArdourContextMenu");
-
-       /* Look through the regions that we are handling and make notes about what we have got */
        
-       bool have_audio = false;
-       bool have_midi = false;
-       bool have_locked = false;
-       bool have_unlocked = false;
-       bool have_position_lock_style_audio = false;
-       bool have_position_lock_style_music = false;
-       bool have_muted = false;
-       bool have_unmuted = false;
-       bool have_opaque = false;
-       bool have_non_opaque = false;
-       bool have_not_at_natural_position = false;
-       bool have_envelope_visible = false;
-       bool have_envelope_invisible = false;
-       bool have_envelope_active = false;
-       bool have_envelope_inactive = false;
-       bool have_non_unity_scale_amplitude = false;
-
-       for (list<boost::shared_ptr<Region> >::const_iterator i = regions.begin(); i != regions.end(); ++i) {
-               boost::shared_ptr<AudioRegion> ar = boost::dynamic_pointer_cast<AudioRegion> (*i);
-               if (ar) {
-                       have_audio = true;
-               }
-               if (boost::dynamic_pointer_cast<MidiRegion> (*i)) {
-                       have_midi = true;
-               }
-
-               if ((*i)->locked()) {
-                       have_locked = true;
-               } else {
-                       have_unlocked = true;
-               }
-
-               if ((*i)->position_lock_style() == MusicTime) {
-                       have_position_lock_style_music = true;
-               } else {
-                       have_position_lock_style_audio = true;
-               }
-
-               if ((*i)->muted()) {
-                       have_muted = true;
-               } else {
-                       have_unmuted = true;
-               }
-
-               if ((*i)->opaque()) {
-                       have_opaque = true;
-               } else {
-                       have_non_opaque = true;
-               }
-
-               if (!(*i)->at_natural_position()) {
-                       have_not_at_natural_position = true;
-               }
-
-               if (ar) {
-                       RegionView* rv = sv->find_view (ar);
-                       AudioRegionView* arv = dynamic_cast<AudioRegionView*> (rv);
-                       
-                       if (arv->envelope_visible()) {
-                               have_envelope_visible = true;
-                       } else {
-                               have_envelope_invisible = true;
-                       }
-
-                       if (ar->envelope_active()) {
-                               have_envelope_active = true;
-                       } else {
-                               have_envelope_inactive = true;
-                       }
-
-                       if (ar->scale_amplitude() != 1) {
-                               have_non_unity_scale_amplitude = true;
-                       }
-               }
-       }
-
-       if (regions.size() == 1) {
-
-               /* when this particular menu pops up, make the relevant region
-                  become selected.
-               */
-
-               region_menu->signal_map_event().connect (
-                       sigc::bind (sigc::mem_fun(*this, &Editor::set_selected_regionview_from_map_event), sv, boost::weak_ptr<Region> (regions.front()))
-                       );
-
-               items.push_back (MenuElem (_("Rename..."), sigc::mem_fun(*this, &Editor::rename_region)));
-               
-               if (have_midi) {
-                       items.push_back (MenuElem (_("List Editor..."), sigc::mem_fun(*this, &Editor::show_midi_list_editor)));
-               }
-               
-               items.push_back (MenuElem (_("Region Properties..."), sigc::mem_fun(*this, &Editor::edit_region)));
-       }
-
-       items.push_back (MenuElem (_("Raise to Top Layer"), sigc::mem_fun(*this, &Editor::raise_region_to_top)));
-       items.push_back (MenuElem (_("Lower to Bottom Layer"), sigc::mem_fun  (*this, &Editor::lower_region_to_bottom)));
-       items.push_back (SeparatorElem());
-       items.push_back (MenuElem (_("Define Sync Point"), sigc::mem_fun(*this, &Editor::set_region_sync_from_edit_point)));
-       if (_edit_point == EditAtMouse) {
-               items.back ().set_sensitive (false);
-       }
-       items.push_back (MenuElem (_("Remove Sync Point"), sigc::mem_fun(*this, &Editor::remove_region_sync)));
-       items.push_back (SeparatorElem());
-
-       items.push_back (MenuElem (_("Audition"), sigc::mem_fun(*this, &Editor::play_selected_region)));
-       items.push_back (MenuElem (_("Export..."), sigc::mem_fun(*this, &Editor::export_region)));
-       items.push_back (MenuElem (_("Bounce"), sigc::mem_fun(*this, &Editor::bounce_region_selection)));
-
-       if (have_audio) {
-               items.push_back (MenuElem (_("Spectral Analysis..."), sigc::mem_fun(*this, &Editor::analyze_region_selection)));
-       }
-
-       items.push_back (SeparatorElem());
-
-       items.push_back (CheckMenuElem (_("Lock")));
-       CheckMenuItem* region_lock_item = static_cast<CheckMenuItem*>(&items.back());
-       if (have_locked && !have_unlocked) {
-               region_lock_item->set_active();
-       } else if (have_locked && have_unlocked) {
-               region_lock_item->set_inconsistent ();
-       }
-       region_lock_item->signal_activate().connect (sigc::mem_fun(*this, &Editor::toggle_region_lock));
-
-       items.push_back (CheckMenuElem (_("Glue to Bars and Beats")));
-       CheckMenuItem* bbt_glue_item = static_cast<CheckMenuItem*>(&items.back());
-
-       if (have_position_lock_style_music && !have_position_lock_style_audio) {
-               bbt_glue_item->set_active ();
-       } else if (have_position_lock_style_music && have_position_lock_style_audio) {
-               bbt_glue_item->set_inconsistent ();
-       }
-
-       bbt_glue_item->signal_activate().connect (sigc::mem_fun (*this, &Editor::toggle_region_lock_style));
-
-       items.push_back (CheckMenuElem (_("Mute")));
-       CheckMenuItem* region_mute_item = static_cast<CheckMenuItem*>(&items.back());
-
-       if (have_muted && !have_unmuted) {
-               region_mute_item->set_active();
-       } else if (have_muted && have_unmuted) {
-               region_mute_item->set_inconsistent ();
-       }
-        
-       region_mute_item->signal_activate().connect (sigc::mem_fun(*this, &Editor::toggle_region_mute));
-       
-        items.push_back (MenuElem (_("Transpose..."), mem_fun(*this, &Editor::pitch_shift_regions)));
-
-       if (!Profile->get_sae()) {
-               items.push_back (CheckMenuElem (_("Opaque")));
-               CheckMenuItem* region_opaque_item = static_cast<CheckMenuItem*>(&items.back());
-               if (have_opaque && !have_non_opaque) {
-                       region_opaque_item->set_active();
-               } else if (have_opaque && have_non_opaque) {
-                       region_opaque_item->set_inconsistent ();
-               }
-               region_opaque_item->signal_activate().connect (sigc::mem_fun(*this, &Editor::toggle_region_opaque));
-       }
-
-       items.push_back (CheckMenuElem (_("Original Position"), sigc::mem_fun(*this, &Editor::naturalize)));
-       if (!have_not_at_natural_position) {
-               items.back().set_sensitive (false);
-       }
-
-       items.push_back (SeparatorElem());
-
-       if (have_audio) {
-
-               if (!Profile->get_sae()) {
-                       items.push_back (MenuElem (_("Reset Envelope"), sigc::mem_fun(*this, &Editor::reset_region_gain_envelopes)));
-
-                       items.push_back (CheckMenuElem (_("Envelope Visible")));
-                       CheckMenuItem* region_envelope_visible_item = static_cast<CheckMenuItem*> (&items.back());
-                       if (have_envelope_visible && !have_envelope_invisible) {
-                               region_envelope_visible_item->set_active ();
-                       } else if (have_envelope_visible && have_envelope_invisible) {
-                               region_envelope_visible_item->set_inconsistent ();
-                       }
-                       region_envelope_visible_item->signal_activate().connect (sigc::mem_fun(*this, &Editor::toggle_gain_envelope_visibility));
-
-                       items.push_back (CheckMenuElem (_("Envelope Active")));
-                       CheckMenuItem* region_envelope_active_item = static_cast<CheckMenuItem*> (&items.back());
-
-                       if (have_envelope_active && !have_envelope_inactive) {
-                               region_envelope_active_item->set_active ();
-                       } else if (have_envelope_active && have_envelope_inactive) {
-                               region_envelope_active_item->set_inconsistent ();
-                       }
-
-                       region_envelope_active_item->signal_activate().connect (sigc::mem_fun(*this, &Editor::toggle_gain_envelope_active));
-                       items.push_back (SeparatorElem());
-               }
-
-               items.push_back (MenuElem (_("Normalize..."), sigc::mem_fun(*this, &Editor::normalize_region)));
-               if (have_non_unity_scale_amplitude) {
-                       items.push_back (MenuElem (_("Reset Gain"), sigc::mem_fun(*this, &Editor::reset_region_scale_amplitude)));
-               }
-
-       } else if (have_midi) {
-               items.push_back (MenuElem (_("Quantize"), sigc::mem_fun(*this, &Editor::quantize_region)));
-               items.push_back (MenuElem (_("Fork"), sigc::mem_fun(*this, &Editor::fork_region)));
-               items.push_back (SeparatorElem());
-       }
-
-       items.push_back (MenuElem (_("Strip Silence..."), sigc::mem_fun (*this, &Editor::strip_region_silence)));
-       items.push_back (MenuElem (_("Reverse"), sigc::mem_fun(*this, &Editor::reverse_region)));
-       items.push_back (SeparatorElem());
-
-       /* range related stuff */
-
-       items.push_back (MenuElem (_("Add Single Range"), sigc::mem_fun (*this, &Editor::add_location_from_audio_region)));
-       items.push_back (MenuElem (_("Add Range Markers"), sigc::mem_fun (*this, &Editor::add_locations_from_audio_region)));
-       if (selection->regions.size() < 2) {
-               items.back().set_sensitive (false);
-       }
-
-       items.push_back (MenuElem (_("Set Range Selection"), sigc::mem_fun (*this, &Editor::set_selection_from_region)));
-       items.push_back (SeparatorElem());
-
-       /* Nudge region */
-
-       Menu *nudge_menu = manage (new Menu());
-       MenuList& nudge_items = nudge_menu->items();
-       nudge_menu->set_name ("ArdourContextMenu");
-
-       nudge_items.push_back (MenuElem (_("Nudge Forward"), (sigc::bind (sigc::mem_fun(*this, &Editor::nudge_forward), false, false))));
-       nudge_items.push_back (MenuElem (_("Nudge Backward"), (sigc::bind (sigc::mem_fun(*this, &Editor::nudge_backward), false, false))));
-       nudge_items.push_back (MenuElem (_("Nudge Forward by Capture Offset"), (sigc::mem_fun(*this, &Editor::nudge_forward_capture_offset))));
-       nudge_items.push_back (MenuElem (_("Nudge Backward by Capture Offset"), (sigc::mem_fun(*this, &Editor::nudge_backward_capture_offset))));
-
-       items.push_back (MenuElem (_("Nudge"), *nudge_menu));
-       items.push_back (SeparatorElem());
-
-       Menu *trim_menu = manage (new Menu);
-       MenuList& trim_items = trim_menu->items();
-       trim_menu->set_name ("ArdourContextMenu");
-
-       trim_items.push_back (MenuElem (_("Start to Edit Point"), sigc::mem_fun(*this, &Editor::trim_region_from_edit_point)));
-       foo_item = &trim_items.back();
-       if (_edit_point == EditAtMouse) {
-               foo_item->set_sensitive (false);
-       }
-       trim_items.push_back (MenuElem (_("Edit Point to End"), sigc::mem_fun(*this, &Editor::trim_region_to_edit_point)));
-       foo_item = &trim_items.back();
-       if (_edit_point == EditAtMouse) {
-               foo_item->set_sensitive (false);
-       }
-       trim_items.push_back (MenuElem (_("Trim to Loop"), sigc::mem_fun(*this, &Editor::trim_region_to_loop)));
-       trim_items.push_back (MenuElem (_("Trim to Punch"), sigc::mem_fun(*this, &Editor::trim_region_to_punch)));
-
-       items.push_back (MenuElem (_("Trim"), *trim_menu));
-       items.push_back (SeparatorElem());
-
-       items.push_back (MenuElem (_("Split"), (sigc::mem_fun(*this, &Editor::split))));
-       region_edit_menu_split_item = &items.back();
-
-       if (_edit_point == EditAtMouse) {
-               region_edit_menu_split_item->set_sensitive (false);
-       }
-
-       if (have_audio) {
-               items.push_back (MenuElem (_("Make Mono Regions"), (sigc::mem_fun(*this, &Editor::split_multichannel_region))));
-               region_edit_menu_split_multichannel_item = &items.back();
-       }
-
-       items.push_back (MenuElem (_("Duplicate"), (sigc::bind (sigc::mem_fun(*this, &Editor::duplicate_dialog), false))));
-       items.push_back (MenuElem (_("Multi-Duplicate..."), (sigc::bind (sigc::mem_fun(*this, &Editor::duplicate_dialog), true))));
-       items.push_back (MenuElem (_("Fill Track"), (sigc::mem_fun(*this, &Editor::region_fill_track))));
-       items.push_back (SeparatorElem());
-       items.push_back (MenuElem (_("Remove"), sigc::mem_fun(*this, &Editor::remove_selected_regions)));
-
        /* OK, stick the region submenu at the top of the list, and then add
           the standard items.
        */
@@ -2007,17 +1821,27 @@ Editor::add_region_context_items (StreamView* sv, list<boost::shared_ptr<Region>
           meaning for menu titles.
        */
 
+       RegionSelection rs = get_regions_from_selection_and_entered ();
+       
        string::size_type pos = 0;
-       string menu_item_name = (regions.size() == 1) ? regions.front()->name() : _("Selected Regions");
+       string menu_item_name = (rs.size() == 1) ? rs.front()->region()->name() : _("Selected Regions");
 
        while ((pos = menu_item_name.find ("_", pos)) != string::npos) {
                menu_item_name.replace (pos, 1, "__");
                pos += 2;
        }
 
-       edit_items.push_back (MenuElem (menu_item_name, *region_menu));
+       if (_popup_region_menu_item == 0) {
+               _popup_region_menu_item = new MenuItem (menu_item_name);
+               _popup_region_menu_item->set_submenu (*dynamic_cast<Menu*> (ActionManager::get_widget (X_("/PopupRegionMenu"))));
+               _popup_region_menu_item->show ();
+       } else {
+               _popup_region_menu_item->set_label (menu_item_name);
+       }
+
+       edit_items.push_back (*_popup_region_menu_item);
        if (multiple_regions_at_position && (layering_order_editor == 0 || !layering_order_editor->is_visible ())) {
-               edit_items.push_back (MenuElem (_("Choose top region"), (bind (mem_fun(*this, &Editor::change_region_layering_order), position))));
+               edit_items.push_back (action_menu_item ("choose-top-region"));
        }
        edit_items.push_back (SeparatorElem());
 }
@@ -2085,7 +1909,7 @@ Editor::add_dstream_context_items (Menu_Helpers::MenuList& edit_items)
        play_items.push_back (MenuElem (_("Play From Start"), sigc::mem_fun(*this, &Editor::play_from_start)));
        play_items.push_back (MenuElem (_("Play Region"), sigc::mem_fun(*this, &Editor::play_selected_region)));
        play_items.push_back (SeparatorElem());
-       play_items.push_back (MenuElem (_("Loop Region"), sigc::mem_fun(*this, &Editor::loop_selected_region)));
+       play_items.push_back (MenuElem (_("Loop Region"), sigc::bind (sigc::mem_fun (*this, &Editor::set_loop_from_region), true)));
 
        edit_items.push_back (MenuElem (_("Play"), *play_menu));
 
@@ -2125,8 +1949,8 @@ Editor::add_dstream_context_items (Menu_Helpers::MenuList& edit_items)
 
        cutnpaste_items.push_back (SeparatorElem());
 
-       cutnpaste_items.push_back (MenuElem (_("Align"), sigc::bind (sigc::mem_fun(*this, &Editor::align), ARDOUR::SyncPoint)));
-       cutnpaste_items.push_back (MenuElem (_("Align Relative"), sigc::bind (sigc::mem_fun(*this, &Editor::align_relative), ARDOUR::SyncPoint)));
+       cutnpaste_items.push_back (MenuElem (_("Align"), sigc::bind (sigc::mem_fun (*this, &Editor::align_regions), ARDOUR::SyncPoint)));
+       cutnpaste_items.push_back (MenuElem (_("Align Relative"), sigc::bind (sigc::mem_fun (*this, &Editor::align_regions_relative), ARDOUR::SyncPoint)));
 
        edit_items.push_back (MenuElem (_("Edit"), *cutnpaste_menu));
 
@@ -2323,7 +2147,7 @@ Editor::set_edit_point_preference (EditPoint ep, bool force)
                Glib::RefPtr<RadioAction>::cast_dynamic(act)->set_active (true);
        }
 
-       nframes64_t foo;
+       framepos_t foo;
        bool in_track_canvas;
 
        if (!mouse_frame (foo, in_track_canvas)) {
@@ -2403,7 +2227,7 @@ Editor::set_state (const XMLNode& node, int /*version*/)
        move (x, y);
 
        if (_session && (prop = node.property ("playhead"))) {
-               nframes64_t pos;
+               framepos_t pos;
                 sscanf (prop->value().c_str(), "%" PRIi64, &pos);
                playhead_cursor->set_position (pos);
        } else {
@@ -2420,6 +2244,8 @@ Editor::set_state (const XMLNode& node, int /*version*/)
 
        if ((prop = node.property ("zoom"))) {
                reset_zoom (PBD::atof (prop->value()));
+       } else {
+               reset_zoom (frames_per_unit);
        }
 
        if ((prop = node.property ("snap-to"))) {
@@ -2438,7 +2264,7 @@ Editor::set_state (const XMLNode& node, int /*version*/)
        }
 
        if ((prop = node.property ("left-frame")) != 0){
-               nframes64_t pos;
+               framepos_t pos;
                if (sscanf (prop->value().c_str(), "%" PRId64, &pos) == 1) {
                        reset_x_origin (pos);
                }
@@ -2468,7 +2294,7 @@ Editor::set_state (const XMLNode& node, int /*version*/)
 
        if ((prop = node.property ("show-measures"))) {
                bool yn = string_is_affirmative (prop->value());
-               _show_measures = !yn;
+               _show_measures = yn;
                RefPtr<Action> act = ActionManager::get_action (X_("Editor"), X_("ToggleMeasureVisibility"));
                if (act) {
                        RefPtr<ToggleAction> tact = RefPtr<ToggleAction>::cast_dynamic(act);
@@ -2551,6 +2377,7 @@ Editor::set_state (const XMLNode& node, int /*version*/)
        XMLNodeList children = node.children ();
        for (XMLNodeList::const_iterator i = children.begin(); i != children.end(); ++i) {
                selection->set_state (**i, Stateful::current_state_version);
+               _regions->set_state (**i);
        }
 
        return 0;
@@ -2640,6 +2467,7 @@ Editor::get_state ()
        node->add_property (X_("editor-list-page"), buf);
 
        node->add_child_nocopy (selection->get_state ());
+       node->add_child_nocopy (_regions->get_state ());
        
        return *node;
 }
@@ -2671,7 +2499,7 @@ Editor::trackview_by_y_position (double y)
  *  @param event Event to get current key modifier information from, or 0.
  */
 void
-Editor::snap_to_with_modifier (nframes64_t& start, GdkEvent const * event, int32_t direction, bool for_mark)
+Editor::snap_to_with_modifier (framepos_t& start, GdkEvent const * event, int32_t direction, bool for_mark)
 {
        if (!_session || !event) {
                return;
@@ -2689,7 +2517,7 @@ Editor::snap_to_with_modifier (nframes64_t& start, GdkEvent const * event, int32
 }
 
 void
-Editor::snap_to (nframes64_t& start, int32_t direction, bool for_mark)
+Editor::snap_to (framepos_t& start, int32_t direction, bool for_mark)
 {
        if (!_session || _snap_mode == SnapOff) {
                return;
@@ -2699,17 +2527,17 @@ Editor::snap_to (nframes64_t& start, int32_t direction, bool for_mark)
 }
 
 void
-Editor::timecode_snap_to_internal (nframes64_t& start, int32_t direction, bool /*for_mark*/)
+Editor::timecode_snap_to_internal (framepos_t& start, int32_t direction, bool /*for_mark*/)
 {
-       const nframes64_t one_timecode_second = (nframes64_t)(rint(_session->timecode_frames_per_second()) * _session->frames_per_timecode_frame());
-       nframes64_t one_timecode_minute = (nframes64_t)(rint(_session->timecode_frames_per_second()) * _session->frames_per_timecode_frame() * 60);
+       const framepos_t one_timecode_second = (framepos_t)(rint(_session->timecode_frames_per_second()) * _session->frames_per_timecode_frame());
+       framepos_t one_timecode_minute = (framepos_t)(rint(_session->timecode_frames_per_second()) * _session->frames_per_timecode_frame() * 60);
 
        switch (_snap_type) {
        case SnapToTimecodeFrame:
                if (((direction == 0) && (fmod((double)start, (double)_session->frames_per_timecode_frame()) > (_session->frames_per_timecode_frame() / 2))) || (direction > 0)) {
-                       start = (nframes64_t) (ceil ((double) start / _session->frames_per_timecode_frame()) * _session->frames_per_timecode_frame());
+                       start = (framepos_t) (ceil ((double) start / _session->frames_per_timecode_frame()) * _session->frames_per_timecode_frame());
                } else {
-                       start = (nframes64_t) (floor ((double) start / _session->frames_per_timecode_frame()) *  _session->frames_per_timecode_frame());
+                       start = (framepos_t) (floor ((double) start / _session->frames_per_timecode_frame()) *  _session->frames_per_timecode_frame());
                }
                break;
 
@@ -2721,9 +2549,9 @@ Editor::timecode_snap_to_internal (nframes64_t& start, int32_t direction, bool /
                        start -= _session->timecode_offset ();
                }
                if (((direction == 0) && (start % one_timecode_second > one_timecode_second / 2)) || direction > 0) {
-                       start = (nframes64_t) ceil ((double) start / one_timecode_second) * one_timecode_second;
+                       start = (framepos_t) ceil ((double) start / one_timecode_second) * one_timecode_second;
                } else {
-                       start = (nframes64_t) floor ((double) start / one_timecode_second) * one_timecode_second;
+                       start = (framepos_t) floor ((double) start / one_timecode_second) * one_timecode_second;
                }
 
                if (_session->timecode_offset_negative())
@@ -2742,9 +2570,9 @@ Editor::timecode_snap_to_internal (nframes64_t& start, int32_t direction, bool /
                        start -= _session->timecode_offset ();
                }
                if (((direction == 0) && (start % one_timecode_minute > one_timecode_minute / 2)) || direction > 0) {
-                       start = (nframes64_t) ceil ((double) start / one_timecode_minute) * one_timecode_minute;
+                       start = (framepos_t) ceil ((double) start / one_timecode_minute) * one_timecode_minute;
                } else {
-                       start = (nframes64_t) floor ((double) start / one_timecode_minute) * one_timecode_minute;
+                       start = (framepos_t) floor ((double) start / one_timecode_minute) * one_timecode_minute;
                }
                if (_session->timecode_offset_negative())
                {
@@ -2760,13 +2588,13 @@ Editor::timecode_snap_to_internal (nframes64_t& start, int32_t direction, bool /
 }
 
 void
-Editor::snap_to_internal (nframes64_t& start, int32_t direction, bool for_mark)
+Editor::snap_to_internal (framepos_t& start, int32_t direction, bool for_mark)
 {
-       const nframes64_t one_second = _session->frame_rate();
-       const nframes64_t one_minute = _session->frame_rate() * 60;
-       nframes64_t presnap = start;
-       nframes64_t before;
-       nframes64_t after;
+       const framepos_t one_second = _session->frame_rate();
+       const framepos_t one_minute = _session->frame_rate() * 60;
+       framepos_t presnap = start;
+       framepos_t before;
+       framepos_t after;
 
        switch (_snap_type) {
        case SnapToTimecodeFrame:
@@ -2776,25 +2604,25 @@ Editor::snap_to_internal (nframes64_t& start, int32_t direction, bool for_mark)
 
        case SnapToCDFrame:
                if (((direction == 0) && (start % (one_second/75) > (one_second/75) / 2)) || (direction > 0)) {
-                       start = (nframes64_t) ceil ((double) start / (one_second / 75)) * (one_second / 75);
+                       start = (framepos_t) ceil ((double) start / (one_second / 75)) * (one_second / 75);
                } else {
-                       start = (nframes64_t) floor ((double) start / (one_second / 75)) * (one_second / 75);
+                       start = (framepos_t) floor ((double) start / (one_second / 75)) * (one_second / 75);
                }
                break;
 
        case SnapToSeconds:
                if (((direction == 0) && (start % one_second > one_second / 2)) || (direction > 0)) {
-                       start = (nframes64_t) ceil ((double) start / one_second) * one_second;
+                       start = (framepos_t) ceil ((double) start / one_second) * one_second;
                } else {
-                       start = (nframes64_t) floor ((double) start / one_second) * one_second;
+                       start = (framepos_t) floor ((double) start / one_second) * one_second;
                }
                break;
 
        case SnapToMinutes:
                if (((direction == 0) && (start % one_minute > one_minute / 2)) || (direction > 0)) {
-                       start = (nframes64_t) ceil ((double) start / one_minute) * one_minute;
+                       start = (framepos_t) ceil ((double) start / one_minute) * one_minute;
                } else {
-                       start = (nframes64_t) floor ((double) start / one_minute) * one_minute;
+                       start = (framepos_t) floor ((double) start / one_minute) * one_minute;
                }
                break;
 
@@ -2859,11 +2687,11 @@ Editor::snap_to_internal (nframes64_t& start, int32_t direction, bool for_mark)
 
                _session->locations()->marks_either_side (start, before, after);
 
-               if (before == max_frames) {
+               if (before == max_framepos) {
                        start = after;
-               } else if (after == max_frames) {
+               } else if (after == max_framepos) {
                        start = before;
-               } else if (before != max_frames && after != max_frames) {
+               } else if (before != max_framepos && after != max_framepos) {
                        /* have before and after */
                        if ((start - before) < (after - start)) {
                                start = before;
@@ -2880,8 +2708,8 @@ Editor::snap_to_internal (nframes64_t& start, int32_t direction, bool for_mark)
        case SnapToRegionBoundary:
                if (!region_boundary_cache.empty()) {
 
-                       vector<nframes64_t>::iterator prev = region_boundary_cache.end ();
-                       vector<nframes64_t>::iterator next = region_boundary_cache.end ();
+                       vector<framepos_t>::iterator prev = region_boundary_cache.end ();
+                       vector<framepos_t>::iterator next = region_boundary_cache.end ();
 
                        if (direction > 0) {
                                next = std::upper_bound (region_boundary_cache.begin(), region_boundary_cache.end(), start);
@@ -2894,8 +2722,8 @@ Editor::snap_to_internal (nframes64_t& start, int32_t direction, bool for_mark)
                                prev--;
                        }
 
-                       nframes64_t const p = (prev == region_boundary_cache.end()) ? region_boundary_cache.front () : *prev;
-                       nframes64_t const n = (next == region_boundary_cache.end()) ? region_boundary_cache.back () : *next;
+                       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 > (p + n) / 2) {
                                start = n;
@@ -3162,13 +2990,13 @@ void
 Editor::setup_tooltips ()
 {
        ARDOUR_UI::instance()->set_tip (mouse_move_button, _("Select/Move Objects"));
-       ARDOUR_UI::instance()->set_tip (mouse_gain_button, _("Draw Gain Automation"));
+       ARDOUR_UI::instance()->set_tip (mouse_gain_button, _("Draw Region Gain"));
        ARDOUR_UI::instance()->set_tip (mouse_zoom_button, _("Select Zoom Range"));
        ARDOUR_UI::instance()->set_tip (mouse_timefx_button, _("Stretch/Shrink Regions and MIDI Notes"));
        ARDOUR_UI::instance()->set_tip (mouse_audition_button, _("Listen to Specific Regions"));
        ARDOUR_UI::instance()->set_tip (join_object_range_button, _("Select/Move Objects or Ranges"));
        ARDOUR_UI::instance()->set_tip (internal_edit_button, _("Edit Region Contents (e.g. notes)"));
-       ARDOUR_UI::instance()->set_tip (*_group_tabs, _("Groups: context-click for possible operations"));
+       ARDOUR_UI::instance()->set_tip (*_group_tabs, _("Groups: click to (de)activate\nContext-click for other operations"));
        ARDOUR_UI::instance()->set_tip (nudge_forward_button, _("Nudge Region/Selection Forwards"));
        ARDOUR_UI::instance()->set_tip (nudge_backward_button, _("Nudge Region/Selection Backwards"));
        ARDOUR_UI::instance()->set_tip (zoom_in_button, _("Zoom In"));
@@ -3217,7 +3045,7 @@ Editor::setup_midi_toolbar ()
 
 int
 Editor::convert_drop_to_paths (
-               vector<ustring>&                paths,
+               vector<string>&                paths,
                const RefPtr<Gdk::DragContext>& /*context*/,
                gint                            /*x*/,
                gint                            /*y*/,
@@ -3228,8 +3056,8 @@ Editor::convert_drop_to_paths (
        if (_session == 0) {
                return -1;
        }
-
-       vector<ustring> uris = data.get_uris();
+        
+       vector<string> uris = data.get_uris();
 
        if (uris.empty()) {
 
@@ -3247,7 +3075,7 @@ Editor::convert_drop_to_paths (
                   THERE MAY BE NO NULL TERMINATING CHAR!!!
                */
 
-               ustring txt = data.get_text();
+               string txt = data.get_text();
                const char* p;
                const char* q;
 
@@ -3275,7 +3103,7 @@ Editor::convert_drop_to_paths (
 
                                        if (q > p)
                                        {
-                                               uris.push_back (ustring (p, q - p + 1));
+                                               uris.push_back (string (p, q - p + 1));
                                        }
                                }
                        }
@@ -3291,18 +3119,18 @@ Editor::convert_drop_to_paths (
                }
        }
 
-       for (vector<ustring>::iterator i = uris.begin(); i != uris.end(); ++i) {
+       for (vector<string>::iterator i = uris.begin(); i != uris.end(); ++i) {
 
                if ((*i).substr (0,7) == "file://") {
 
-                       ustring p = *i;
+                       string p = *i;
                        PBD::url_decode (p);
 
                        // scan forward past three slashes
 
-                       ustring::size_type slashcnt = 0;
-                       ustring::size_type n = 0;
-                       ustring::iterator x = p.begin();
+                       string::size_type slashcnt = 0;
+                       string::size_type n = 0;
+                       string::iterator x = p.begin();
 
                        while (slashcnt < 3 && x != p.end()) {
                                if ((*x) == '/') {
@@ -3431,14 +3259,10 @@ Editor::duplicate_dialog (bool with_dialog)
                }
        }
 
-       RegionSelection rs;
-       get_regions_for_action (rs);
+       RegionSelection rs = get_regions_from_selection_and_entered ();
 
-       if (mouse_mode != MouseRange) {
-
-               if (rs.empty()) {
-                       return;
-               }
+       if (mouse_mode != MouseRange && rs.empty()) {
+               return;
        }
 
        if (with_dialog) {
@@ -3529,7 +3353,7 @@ Editor::clamp_verbose_cursor_y (double y)
 }
 
 void
-Editor::show_verbose_canvas_cursor_with (const string & txt)
+Editor::show_verbose_canvas_cursor_with (const string & txt, int32_t xoffset, int32_t yoffset)
 {
        verbose_canvas_cursor->property_text() = txt.c_str();
 
@@ -3539,6 +3363,9 @@ Editor::show_verbose_canvas_cursor_with (const string & txt)
        track_canvas->get_pointer (x, y);
        track_canvas->window_to_world (x, y, wx, wy);
 
+        wx += xoffset;
+        wy += yoffset;
+
        /* don't get too close to the edge */
        verbose_canvas_cursor->property_x() = clamp_verbose_cursor_x (wx);
        verbose_canvas_cursor->property_y() = clamp_verbose_cursor_y (wy);
@@ -4006,7 +3833,7 @@ Editor::playlist_selector () const
 }
 
 Evoral::MusicalTime
-Editor::get_grid_type_as_beats (bool& success, nframes64_t position)
+Editor::get_grid_type_as_beats (bool& success, framepos_t position)
 {
        success = true;
 
@@ -4085,10 +3912,10 @@ Editor::get_grid_type_as_beats (bool& success, nframes64_t position)
        return 0.0;
 }
 
-nframes64_t
-Editor::get_nudge_distance (nframes64_t pos, nframes64_t& next)
+framecnt_t
+Editor::get_nudge_distance (framepos_t pos, framecnt_t& next)
 {
-       nframes64_t ret;
+       framecnt_t ret;
 
        ret = nudge_clock.current_duration (pos);
        next = ret + 1; /* XXXX fix me */
@@ -4133,7 +3960,7 @@ Editor::playlist_deletion_dialog (boost::shared_ptr<Playlist> pl)
 }
 
 bool
-Editor::audio_region_selection_covers (nframes64_t where)
+Editor::audio_region_selection_covers (framepos_t where)
 {
        for (RegionSelection::iterator a = selection->regions.begin(); a != selection->regions.end(); ++a) {
                if ((*a)->region()->covers (where)) {
@@ -4208,10 +4035,8 @@ Editor::control_layout_scroll (GdkEventScroll* ev)
 }
 
 void
-Editor::session_state_saved (string snap_name)
+Editor::session_state_saved (string)
 {
-       ENSURE_GUI_THREAD (*this, &Editor::session_state_saved, snap_name);
-       
        update_title ();        
        _snapshots->redisplay ();
 }
@@ -4366,7 +4191,7 @@ Editor::on_key_release_event (GdkEventKey* ev)
  *  @param frame New x origin.
  */
 void
-Editor::reset_x_origin (nframes64_t frame)
+Editor::reset_x_origin (framepos_t frame)
 {
        queue_visual_change (frame);
 }
@@ -4384,7 +4209,7 @@ Editor::reset_zoom (double fpu)
 }
 
 void
-Editor::reposition_and_zoom (nframes64_t frame, double fpu)
+Editor::reposition_and_zoom (framepos_t frame, double fpu)
 {
        reset_x_origin (frame);
        reset_zoom (fpu);
@@ -4502,7 +4327,7 @@ Editor::set_frames_per_unit (double fpu)
           of frames into an 800 pixel wide space.
        */
 
-       if (max_frames / fpu < 800.0) {
+       if (max_framepos / fpu < 800.0) {
                return;
        }
 
@@ -4518,7 +4343,7 @@ Editor::post_zoom ()
 {
        // convert fpu to frame count
 
-       nframes64_t frames = (nframes64_t) floor (frames_per_unit * _canvas_width);
+       framepos_t frames = (framepos_t) floor (frames_per_unit * _canvas_width);
 
        if (frames_per_unit != zoom_range_clock.current_duration()) {
                zoom_range_clock.set (frames);
@@ -4545,7 +4370,7 @@ Editor::post_zoom ()
 }
 
 void
-Editor::queue_visual_change (nframes64_t where)
+Editor::queue_visual_change (framepos_t where)
 {
        pending_visual_change.add (VisualChange::TimeOrigin);
        pending_visual_change.time_origin = where;
@@ -4590,7 +4415,19 @@ Editor::idle_visual_changer ()
        VisualChange::Type p = pending_visual_change.pending;
        pending_visual_change.pending = (VisualChange::Type) 0;
 
-       double last_time_origin = horizontal_position ();
+       double const last_time_origin = horizontal_position ();
+
+       if (p & VisualChange::TimeOrigin) {
+               /* This is a bit of a hack, but set_frames_per_unit
+                  below will (if called) end up with the
+                  CrossfadeViews looking at Editor::leftmost_frame,
+                  and if we're changing origin and zoom in the same
+                  operation it will be the wrong value unless we
+                  update it here.
+               */
+
+               leftmost_frame = pending_visual_change.time_origin;
+       }
 
        if (p & VisualChange::ZoomLevel) {
                set_frames_per_unit (pending_visual_change.frames_per_unit);
@@ -4637,11 +4474,11 @@ Editor::sort_track_selection (TrackViewList* sel)
        }
 }
 
-nframes64_t
+framepos_t
 Editor::get_preferred_edit_position (bool ignore_playhead)
 {
        bool ignored;
-       nframes64_t where = 0;
+       framepos_t where = 0;
        EditPoint ep = _edit_point;
 
        if (entered_marker) {
@@ -4686,7 +4523,7 @@ Editor::get_preferred_edit_position (bool ignore_playhead)
 }
 
 void
-Editor::set_loop_range (nframes64_t start, nframes64_t end, string cmd)
+Editor::set_loop_range (framepos_t start, framepos_t end, string cmd)
 {
        if (!_session) return;
 
@@ -4713,7 +4550,7 @@ Editor::set_loop_range (nframes64_t start, nframes64_t end, string cmd)
 }
 
 void
-Editor::set_punch_range (nframes64_t start, nframes64_t end, string cmd)
+Editor::set_punch_range (framepos_t start, framepos_t end, string cmd)
 {
        if (!_session) return;
 
@@ -4746,7 +4583,7 @@ Editor::set_punch_range (nframes64_t start, nframes64_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, nframes64_t where, const TrackViewList& ts) const
+Editor::get_regions_at (RegionSelection& rs, framepos_t where, const TrackViewList& ts) const
 {
        const TrackViewList* tracks;
 
@@ -4765,7 +4602,7 @@ Editor::get_regions_at (RegionSelection& rs, nframes64_t where, const TrackViewL
                        if ((tr = rtv->track()) && ((pl = tr->playlist()))) {
 
                                Playlist::RegionList* regions = pl->regions_at (
-                                               (nframes64_t) floor ( (double)where * tr->speed()));
+                                               (framepos_t) floor ( (double)where * tr->speed()));
 
                                for (Playlist::RegionList::iterator i = regions->begin(); i != regions->end(); ++i) {
                                        RegionView* rv = rtv->view()->find_view (*i);
@@ -4781,7 +4618,7 @@ Editor::get_regions_at (RegionSelection& rs, nframes64_t where, const TrackViewL
 }
 
 void
-Editor::get_regions_after (RegionSelection& rs, nframes64_t where, const TrackViewList& ts) const
+Editor::get_regions_after (RegionSelection& rs, framepos_t where, const TrackViewList& ts) const
 {
        const TrackViewList* tracks;
 
@@ -4800,7 +4637,7 @@ Editor::get_regions_after (RegionSelection& rs, nframes64_t where, const TrackVi
                        if ((tr = rtv->track()) && ((pl = tr->playlist()))) {
 
                                Playlist::RegionList* regions = pl->regions_touched (
-                                               (nframes64_t) floor ( (double)where * tr->speed()), max_frames);
+                                        (framepos_t) floor ( (double)where * tr->speed()), max_framepos);
 
                                for (Playlist::RegionList::iterator i = regions->begin(); i != regions->end(); ++i) {
 
@@ -4817,47 +4654,74 @@ Editor::get_regions_after (RegionSelection& rs, nframes64_t where, const TrackVi
        }
 }
 
-/** Find all regions which are either:
- *      - selected or
- *      - the entered_regionview (if allow_entered == true) or
- *      - under the preferred edit position AND on a selected track, or on a track
- *        which is in the same active edit-enable route group as a selected region (if allow_edit_position == true)
+/** Get regions using the following conditions:
+ *    1.  If the edit point is `mouse':
+ *          if the mouse is over a selected region, or no region, return all selected regions.
+ *          if the mouse is over an unselected region, return just that region.
+ *    2.  For all other edit points:
+ *          return the selected regions AND those that are both under the edit position
+ *          AND on a selected track, or on a track which is in the same active edit-enabled route group
+ *          as a selected region.
+ *
+ *  The rationale here is that the mouse edit point is special in that its position describes
+ *  both a time and a track; the other edit modes only describe a time.
+ *
  *  @param rs Returned region list.
- *  @param allow_entered true to include the entered_regionview in the list.
  */
-void
-Editor::get_regions_for_action (RegionSelection& rs, bool allow_entered, bool allow_edit_position)
-{
-       /* Start with selected regions */
-       rs = selection->regions;
 
-       /* Add the entered_regionview, if requested */
-       if (allow_entered && entered_regionview) {
-               rs.add (entered_regionview);
+RegionSelection
+Editor::get_regions_from_selection_and_edit_point ()
+{
+       if (_edit_point == EditAtMouse) {
+               if (entered_regionview == 0 || selection->regions.contains (entered_regionview)) {
+                       return selection->regions;
+               } else {
+                       RegionSelection rs;
+                       rs.add (entered_regionview);
+                       return rs;
+               }
        }
 
-       if (allow_edit_position) {
+       /* We're using the edit point, but its not EditAtMouse */
 
-               TrackViewList tracks = selection->tracks;
+       /* Start with selected regions */
+       RegionSelection rs = selection->regions;
 
-               /* tracks is currently the set of selected tracks; add any other tracks that
-                * have regions that are in the same edit-activated route group as one of
-                * our regions */
-               for (RegionSelection::iterator i = rs.begin (); i != rs.end(); ++i) {
+       TrackViewList tracks = selection->tracks;
 
-                       RouteGroup* g = (*i)->get_time_axis_view().route_group ();
-                       if (g && g->is_active() && g->is_edit()) {
-                               tracks.add (axis_views_from_routes (g->route_list()));
-                       }
-                       
+       /* Tracks is currently the set of selected tracks; add any other tracks that
+          have regions that are in the same edit-activated route group as one of
+          our regions.
+        */
+       for (RegionSelection::iterator i = rs.begin (); i != rs.end(); ++i) {
+               
+               RouteGroup* g = (*i)->get_time_axis_view().route_group ();
+               if (g && g->is_active() && g->is_edit()) {
+                       tracks.add (axis_views_from_routes (g->route_list()));
                }
+               
+       }
+       
+       if (!tracks.empty()) {
+               /* now find regions that are at the edit position on those tracks */
+               framepos_t const where = get_preferred_edit_position ();
+               get_regions_at (rs, where, tracks);
+       }
 
-               if (!tracks.empty()) {
-                       /* now find regions that are at the edit position on those tracks */
-                       nframes64_t const where = get_preferred_edit_position ();
-                       get_regions_at (rs, where, tracks);
-               }
+       return rs;
+}
+
+
+RegionSelection
+Editor::get_regions_from_selection_and_entered ()
+{
+       RegionSelection rs = selection->regions;
+       
+       if (rs.empty() && entered_regionview) {
+               rs.add (entered_regionview);
        }
+
+       return rs;
 }
 
 void
@@ -5364,7 +5228,7 @@ Editor::horizontal_scroll_right_release ()
 void
 Editor::reset_x_origin_to_follow_playhead ()
 {
-       nframes64_t const frame = playhead_cursor->current_frame;
+       framepos_t const frame = playhead_cursor->current_frame;
 
        if (frame < leftmost_frame || frame > leftmost_frame + current_page_frames()) {
 
@@ -5380,7 +5244,7 @@ Editor::reset_x_origin_to_follow_playhead ()
                                                
                        if (frame < leftmost_frame) {
                                /* moving left */
-                               nframes64_t l = 0;
+                               framepos_t l = 0;
                                if (_session->transport_rolling()) {
                                        /* rolling; end up with the playhead at the right of the page */
                                        l = frame - current_page_frames ();
@@ -5434,7 +5298,7 @@ Editor::super_rapid_screen_update ()
 
        /* PLAYHEAD AND VIEWPORT */
 
-       nframes64_t const frame = _session->audible_frame();
+       framepos_t const frame = _session->audible_frame();
 
        /* There are a few reasons why we might not update the playhead / viewport stuff:
         *
@@ -5570,19 +5434,17 @@ Editor::show_editor_list (bool yn)
 }
 
 void
-Editor::change_region_layering_order (framepos_t position)
+Editor::change_region_layering_order ()
 {
-       if (!clicked_regionview) {
+       framepos_t const position = get_preferred_edit_position ();
+       
+        if (!clicked_routeview) {
                 if (layering_order_editor) {
                         layering_order_editor->hide ();
                 }
                return;
        }
 
-        if (!clicked_routeview) {
-               return;
-       }
-
         boost::shared_ptr<Track> track = boost::dynamic_pointer_cast<Track> (clicked_routeview->route());
 
         if (!track) {
@@ -5604,9 +5466,33 @@ Editor::change_region_layering_order (framepos_t position)
 }
 
 void
-Editor::update_region_layering_order_editor (framepos_t frame)
+Editor::update_region_layering_order_editor ()
 {
        if (layering_order_editor && layering_order_editor->is_visible ()) {
-               change_region_layering_order (frame);
+               change_region_layering_order ();
        }
 }
+
+void
+Editor::setup_fade_images ()
+{
+       _fade_in_images[FadeLinear] = new Gtk::Image (get_icon_path (X_("crossfade-in-linear")));
+       _fade_in_images[FadeFast] = new Gtk::Image (get_icon_path (X_("crossfade-in-short-cut")));
+       _fade_in_images[FadeLogB] = new Gtk::Image (get_icon_path (X_("crossfade-in-slow-cut")));
+       _fade_in_images[FadeLogA] = new Gtk::Image (get_icon_path (X_("crossfade-in-fast-cut")));
+       _fade_in_images[FadeSlow] = new Gtk::Image (get_icon_path (X_("crossfade-in-long-cut")));
+
+       _fade_out_images[FadeLinear] = new Gtk::Image (get_icon_path (X_("crossfade-out-linear")));
+       _fade_out_images[FadeFast] = new Gtk::Image (get_icon_path (X_("crossfade-out-short-cut")));
+       _fade_out_images[FadeLogB] = new Gtk::Image (get_icon_path (X_("crossfade-out-slow-cut")));
+       _fade_out_images[FadeLogA] = new Gtk::Image (get_icon_path (X_("crossfade-out-fast-cut")));
+       _fade_out_images[FadeSlow] = new Gtk::Image (get_icon_path (X_("crossfade-out-long-cut")));
+}
+
+
+/** @return Gtk::manage()d menu item for a given action from `editor_actions' */
+Gtk::MenuItem&
+Editor::action_menu_item (std::string const & name)
+{
+       return *manage (editor_actions->get_action(name)->create_menu_item ());
+}