Improve sensitization of buttons in the advanced options
[ardour.git] / gtk2_ardour / editor.cc
index 6dfe47230fbd0b3a705e43d5582ffd9542e9e6be..5d92ccb08e63dd3259becad06297e1eefa461cd8 100644 (file)
 #include "gtkmm2ext/cell_renderer_pixbuf_toggle.h"
 
 #include "ardour/audio_track.h"
-#include "ardour/audioplaylist.h"
+#include "ardour/audioengine.h"
 #include "ardour/audioregion.h"
 #include "ardour/location.h"
-#include "ardour/midi_region.h"
-#include "ardour/plugin_manager.h"
 #include "ardour/profile.h"
 #include "ardour/route_group.h"
-#include "ardour/session_directory.h"
-#include "ardour/session_route.h"
-#include "ardour/session_state_utils.h"
+#include "ardour/session_playlists.h"
 #include "ardour/tempo.h"
 #include "ardour/utils.h"
-#include "ardour/session_playlists.h"
-#include "ardour/audioengine.h"
 
 #include "control_protocol/control_protocol.h"
 
@@ -220,18 +214,22 @@ show_me_the_size (Requisition* r, const char* what)
        cerr << "size of " << what << " = " << r->width << " x " << r->height << endl;
 }
 
-#ifdef GTKOSX
 static void
 pane_size_watcher (Paned* pane)
 {
        /* if the handle of a pane vanishes into (at least) the tabs of a notebook,
-          it is no longer accessible. so stop that. this doesn't happen on X11,
-          just the quartz backend.
+          it is:
+
+             X: hard to access
+             Quartz: impossible to access
+             
+          so stop that by preventing it from ever getting too narrow. 35
+          pixels is basically a rough guess at the tab width.
 
           ugh.
        */
 
-       int max_width_of_lhs = GTK_WIDGET(pane->gobj())->allocation.width - 25;
+       int max_width_of_lhs = GTK_WIDGET(pane->gobj())->allocation.width - 35;
 
        gint pos = pane->get_position ();
 
@@ -239,7 +237,6 @@ pane_size_watcher (Paned* pane)
                pane->set_position (max_width_of_lhs);
        }
 }
-#endif
 
 Editor::Editor ()
        : _join_object_range_state (JOIN_OBJECT_RANGE_NONE)
@@ -287,6 +284,8 @@ Editor::Editor ()
 
        , _region_selection_change_updates_region_list (true)
        , _following_mixer_selection (false)
+       , _control_point_toggled_on_press (false)
+       , _stepping_axis_view (0)
 {
        constructed = false;
 
@@ -606,13 +605,13 @@ Editor::Editor ()
 
        editor_summary_pane.signal_size_allocate().connect (sigc::bind (sigc::mem_fun (*this, &Editor::pane_allocation_handler), static_cast<Paned*> (&editor_summary_pane)));
 
-       /* XXX: editor_summary_pane might need similar special OS X treatment to the edit_pane */
+       /* XXX: editor_summary_pane might need similar to the edit_pane */
 
        edit_pane.signal_size_allocate().connect (sigc::bind (sigc::mem_fun(*this, &Editor::pane_allocation_handler), static_cast<Paned*> (&edit_pane)));
-#ifdef GTKOSX
+
        Glib::PropertyProxy<int> proxy = edit_pane.property_position();
        proxy.signal_changed().connect (bind (sigc::ptr_fun (pane_size_watcher), static_cast<Paned*> (&edit_pane)));
-#endif
+
        top_hbox.pack_start (toolbar_frame);
 
        HBox *hbox = manage (new HBox);
@@ -701,6 +700,8 @@ Editor::Editor ()
        signal_configure_event().connect (sigc::mem_fun (*ARDOUR_UI::instance(), &ARDOUR_UI::configure_handler));
        signal_delete_event().connect (sigc::mem_fun (*ARDOUR_UI::instance(), &ARDOUR_UI::exit_on_main_window_close));
 
+       Gtkmm2ext::Keyboard::the_keyboard().ShiftReleased.connect (sigc::mem_fun (*this, &Editor::shift_key_released));
+       
        /* allow external control surfaces/protocols to do various things */
 
        ControlProtocol::ZoomToSession.connect (*this, invalidator (*this), boost::bind (&Editor::temporal_zoom_session, this), gui_context());
@@ -786,7 +787,6 @@ Editor::button_settings () const
        XMLNode* node = find_named_node (*settings, X_("Buttons"));
 
        if (!node) {
-                cerr << "new empty Button node\n";
                node = new XMLNode (X_("Buttons"));
        }
 
@@ -920,12 +920,9 @@ Editor::zoom_adjustment_changed ()
        }
 
        double fpu = zoom_range_clock->current_duration() / _canvas_width;
-
-       if (fpu < 1.0) {
-               fpu = 1.0;
-               zoom_range_clock->set ((framepos_t) floor (fpu * _canvas_width));
-       } else if (fpu > _session->current_end_frame() / _canvas_width) {
-               fpu = _session->current_end_frame() / _canvas_width;
+       bool clamped = clamp_frames_per_unit (fpu);
+       
+       if (clamped) {
                zoom_range_clock->set ((framepos_t) floor (fpu * _canvas_width));
        }
 
@@ -1274,7 +1271,7 @@ Editor::set_session (Session *t)
        _session->StepEditStatusChange.connect (_session_connections, invalidator (*this), boost::bind (&Editor::step_edit_status_change, this, _1), gui_context());
        _session->TransportStateChange.connect (_session_connections, invalidator (*this), boost::bind (&Editor::map_transport_state, this), gui_context());
        _session->PositionChanged.connect (_session_connections, invalidator (*this), boost::bind (&Editor::map_position_change, this, _1), gui_context());
-       _session->RouteAdded.connect (_session_connections, invalidator (*this), boost::bind (&Editor::handle_new_route, this, _1), gui_context());
+       _session->RouteAdded.connect (_session_connections, invalidator (*this), boost::bind (&Editor::add_routes, this, _1), gui_context());
        _session->DirtyChanged.connect (_session_connections, invalidator (*this), boost::bind (&Editor::update_title, this), gui_context());
        _session->tempo_map().PropertyChanged.connect (_session_connections, invalidator (*this), boost::bind (&Editor::tempo_map_changed, this, _1), gui_context());
        _session->Located.connect (_session_connections, invalidator (*this), boost::bind (&Editor::located, this), gui_context());
@@ -1341,64 +1338,98 @@ Editor::action_pre_activated (Glib::RefPtr<Action> const & a)
        }
 }
 
-/** Pop up a context menu for when the user clicks on a crossfade */
 void
-Editor::popup_xfade_context_menu (int button, int32_t time, ArdourCanvas::Item* item, ItemType item_type)
+Editor::fill_xfade_menu (Menu_Helpers::MenuList& items, bool start)
 {
        using namespace Menu_Helpers;
 
-       MenuList& items (xfade_context_menu.items());
+       void (Editor::*emf)(FadeShape);
+       std::map<ARDOUR::FadeShape,Gtk::Image*>* images;
+
+       if (start) {
+               images = &_xfade_in_images;
+               emf = &Editor::set_fade_in_shape;
+       } else {
+               images = &_xfade_out_images;
+               emf = &Editor::set_fade_out_shape;
+       }
+
+       items.push_back (
+               ImageMenuElem (
+                       _("Linear (for highly correlated material)"),
+                       *(*images)[FadeLinear],
+                       sigc::bind (sigc::mem_fun (*this, emf), FadeLinear)
+                       )
+               );
+       
+       dynamic_cast<ImageMenuItem*>(&items.back())->set_always_show_image ();
+       
+       items.push_back (
+               ImageMenuElem (
+                       _("ConstantPower"),
+                       *(*images)[FadeConstantPower],
+                       sigc::bind (sigc::mem_fun (*this, emf), FadeConstantPower)
+                       ));
        
+       dynamic_cast<ImageMenuItem*>(&items.back())->set_always_show_image ();
+       
+       items.push_back (
+               ImageMenuElem (
+                       _("Symmetric"),
+                       *(*images)[FadeSymmetric],
+                       sigc::bind (sigc::mem_fun (*this, emf), FadeSymmetric)
+                       )
+               );
+       
+       dynamic_cast<ImageMenuItem*>(&items.back())->set_always_show_image ();
+       
+       items.push_back (
+               ImageMenuElem (
+                       _("Slow"),
+                       *(*images)[FadeSlow],
+                       sigc::bind (sigc::mem_fun (*this, emf), FadeSlow)
+                       ));
+       
+       dynamic_cast<ImageMenuItem*>(&items.back())->set_always_show_image ();
+       
+       items.push_back (
+               ImageMenuElem (
+                       _("Fast"),
+                       *(*images)[FadeFast],
+                       sigc::bind (sigc::mem_fun (*this, emf), FadeFast)
+                       ));
+       
+       dynamic_cast<ImageMenuItem*>(&items.back())->set_always_show_image ();
+}
+
+/** Pop up a context menu for when the user clicks on a start crossfade */
+void
+Editor::popup_xfade_in_context_menu (int button, int32_t time, ArdourCanvas::Item* /*item*/, ItemType /*item_type*/)
+{
+       using namespace Menu_Helpers;
+
+       MenuList& items (xfade_in_context_menu.items());
+
        if (items.empty()) {
-               items.push_back (
-                       ImageMenuElem (
-                               _("Linear (for highly correlated material)"),
-                               *_xfade_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 (
-                               _("ConstantPower (-6dB)"),
-                               *_xfade_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 (
-                               _("Linear-dB"),
-                               *_xfade_images[FadeSlow],
-                               sigc::bind (sigc::mem_fun (*this, &Editor::set_fade_in_shape), FadeSlow)
-                               )
-                       );
-               
-               dynamic_cast<ImageMenuItem*>(&items.back())->set_always_show_image ();
-
-               items.push_back (
-                       ImageMenuElem (
-                               _("Smooth"),
-                               *_xfade_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"),
-                               *_xfade_images[FadeLogA],
-                               sigc::bind (sigc::mem_fun (*this, &Editor::set_fade_in_shape), FadeLogA)
-                               ));
-               
-               dynamic_cast<ImageMenuItem*>(&items.back())->set_always_show_image ();
+               fill_xfade_menu (items, true);
+       }
+
+       xfade_in_context_menu.popup (button, time);
+}
+
+/** Pop up a context menu for when the user clicks on an end crossfade */
+void
+Editor::popup_xfade_out_context_menu (int button, int32_t time, ArdourCanvas::Item* /*item*/, ItemType /*item_type*/)
+{
+       using namespace Menu_Helpers;
+
+       MenuList& items (xfade_out_context_menu.items());
+
+       if (items.empty()) {
+               fill_xfade_menu (items, false);
        }
 
-       xfade_context_menu.popup (button, time);
+       xfade_out_context_menu.popup (button, time);
 }
 
 
@@ -1415,7 +1446,6 @@ Editor::popup_fade_context_menu (int button, int32_t time, ArdourCanvas::Item* i
        }
 
        MenuList& items (fade_context_menu.items());
-
        items.clear ();
 
        switch (item_type) {
@@ -1426,16 +1456,16 @@ Editor::popup_fade_context_menu (int button, int32_t time, ArdourCanvas::Item* i
                } else {
                        items.push_back (MenuElem (_("Activate"), sigc::bind (sigc::mem_fun (*this, &Editor::set_fade_in_active), true)));
                }
-
+               
                items.push_back (SeparatorElem());
-
+               
                if (Profile->get_sae()) {
-
+                       
                        items.push_back (MenuElem (_("Linear"), sigc::bind (sigc::mem_fun (*this, &Editor::set_fade_in_shape), FadeLinear)));
                        items.push_back (MenuElem (_("Slowest"), sigc::bind (sigc::mem_fun (*this, &Editor::set_fade_in_shape), FadeFast)));
-
+                       
                } else {
-
+                       
                        items.push_back (
                                ImageMenuElem (
                                        _("Linear"),
@@ -1443,41 +1473,39 @@ Editor::popup_fade_context_menu (int button, int32_t time, ArdourCanvas::Item* i
                                        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)
+                                       _("Slow"),
+                                       *_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 ();
-
+                               
                        items.push_back (
                                ImageMenuElem (
-                                       _("Slow"),
-                                       *_fade_in_images[FadeLogB],
-                                       sigc::bind (sigc::mem_fun (*this, &Editor::set_fade_in_shape), FadeLogB)
+                                       _("Fast"),
+                                       *_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 (
-                                       _("Fast"),
-                                       *_fade_in_images[FadeLogA],
-                                       sigc::bind (sigc::mem_fun (*this, &Editor::set_fade_in_shape), FadeLogA)
+                                       _("Symmetric"),
+                                       *_fade_in_images[FadeSymmetric],
+                                       sigc::bind (sigc::mem_fun (*this, &Editor::set_fade_in_shape), FadeSymmetric)
                                        ));
-
-                       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)
+                                       _("Constant Power"),
+                                       *_fade_in_images[FadeConstantPower],
+                                       sigc::bind (sigc::mem_fun (*this, &Editor::set_fade_in_shape), FadeConstantPower)
                                        ));
 
                        dynamic_cast<ImageMenuItem*>(&items.back())->set_always_show_image ();
@@ -1512,8 +1540,8 @@ Editor::popup_fade_context_menu (int button, int32_t time, ArdourCanvas::Item* i
 
                        items.push_back (
                                ImageMenuElem (
-                                       _("Slowest"),
-                                       *_fade_out_images[FadeFast],
+                                       _("Slow"),
+                                       *_fade_out_images[FadeSlow],
                                        sigc::bind (sigc::mem_fun (*this, &Editor::set_fade_out_shape), FadeSlow)
                                        ));
 
@@ -1521,27 +1549,25 @@ Editor::popup_fade_context_menu (int button, int32_t time, ArdourCanvas::Item* i
 
                        items.push_back (
                                ImageMenuElem (
-                                       _("Slow"),
-                                       *_fade_out_images[FadeLogB],
-                                       sigc::bind (sigc::mem_fun (*this, &Editor::set_fade_out_shape), FadeLogA)
+                                       _("Fast"),
+                                       *_fade_out_images[FadeFast],
+                                       sigc::bind (sigc::mem_fun (*this, &Editor::set_fade_out_shape), FadeFast)
                                        ));
 
                        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)
+                                       _("Symmetric"),
+                                       *_fade_out_images[FadeSymmetric],
+                                       sigc::bind (sigc::mem_fun (*this, &Editor::set_fade_out_shape), FadeSymmetric)
                                        ));
 
-                       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)
+                                       _("Constant Power"),
+                                       *_fade_out_images[FadeConstantPower],
+                                       sigc::bind (sigc::mem_fun (*this, &Editor::set_fade_out_shape), FadeConstantPower)
                                        ));
 
                        dynamic_cast<ImageMenuItem*>(&items.back())->set_always_show_image ();
@@ -1883,7 +1909,7 @@ Editor::add_selection_context_items (Menu_Helpers::MenuList& edit_items)
        edit_items.push_back (SeparatorElem());
        edit_items.push_back (MenuElem (_("Crop Region to Range"), sigc::mem_fun(*this, &Editor::crop_region_to_selection)));
        edit_items.push_back (MenuElem (_("Fill Range with Region"), sigc::mem_fun(*this, &Editor::region_fill_selection)));
-       edit_items.push_back (MenuElem (_("Duplicate Range"), sigc::bind (sigc::mem_fun(*this, &Editor::duplicate_dialog), false)));
+       edit_items.push_back (MenuElem (_("Duplicate Range"), sigc::bind (sigc::mem_fun(*this, &Editor::duplicate_range), false)));
 
        edit_items.push_back (SeparatorElem());
        edit_items.push_back (MenuElem (_("Consolidate Range"), sigc::bind (sigc::mem_fun(*this, &Editor::bounce_range_selection), true, false)));
@@ -1967,10 +1993,10 @@ Editor::add_dstream_context_items (Menu_Helpers::MenuList& edit_items)
        nudge_menu->set_name ("ArdourContextMenu");
 
        edit_items.push_back (SeparatorElem());
-       nudge_items.push_back (MenuElem (_("Nudge Entire Track Forward"), (sigc::bind (sigc::mem_fun(*this, &Editor::nudge_track), false, true))));
-       nudge_items.push_back (MenuElem (_("Nudge Track After Edit Point Forward"), (sigc::bind (sigc::mem_fun(*this, &Editor::nudge_track), true, true))));
-       nudge_items.push_back (MenuElem (_("Nudge Entire Track Backward"), (sigc::bind (sigc::mem_fun(*this, &Editor::nudge_track), false, false))));
-       nudge_items.push_back (MenuElem (_("Nudge Track After Edit Point Backward"), (sigc::bind (sigc::mem_fun(*this, &Editor::nudge_track), true, false))));
+       nudge_items.push_back (MenuElem (_("Nudge Entire Track Later"), (sigc::bind (sigc::mem_fun(*this, &Editor::nudge_track), false, true))));
+       nudge_items.push_back (MenuElem (_("Nudge Track After Edit Point Later"), (sigc::bind (sigc::mem_fun(*this, &Editor::nudge_track), true, true))));
+       nudge_items.push_back (MenuElem (_("Nudge Entire Track Earlier"), (sigc::bind (sigc::mem_fun(*this, &Editor::nudge_track), false, false))));
+       nudge_items.push_back (MenuElem (_("Nudge Track After Edit Point Earlier"), (sigc::bind (sigc::mem_fun(*this, &Editor::nudge_track), true, false))));
 
        edit_items.push_back (MenuElem (_("Nudge"), *nudge_menu));
 }
@@ -2023,10 +2049,10 @@ Editor::add_bus_context_items (Menu_Helpers::MenuList& edit_items)
        nudge_menu->set_name ("ArdourContextMenu");
 
        edit_items.push_back (SeparatorElem());
-       nudge_items.push_back (MenuElem (_("Nudge Entire Track Forward"), (sigc::bind (sigc::mem_fun(*this, &Editor::nudge_track), false, true))));
-       nudge_items.push_back (MenuElem (_("Nudge Track After Edit Point Forward"), (sigc::bind (sigc::mem_fun(*this, &Editor::nudge_track), true, true))));
-       nudge_items.push_back (MenuElem (_("Nudge Entire Track Backward"), (sigc::bind (sigc::mem_fun(*this, &Editor::nudge_track), false, false))));
-       nudge_items.push_back (MenuElem (_("Nudge Track After Edit Point Backward"), (sigc::bind (sigc::mem_fun(*this, &Editor::nudge_track), true, false))));
+       nudge_items.push_back (MenuElem (_("Nudge Entire Track Later"), (sigc::bind (sigc::mem_fun(*this, &Editor::nudge_track), false, true))));
+       nudge_items.push_back (MenuElem (_("Nudge Track After Edit Point Later"), (sigc::bind (sigc::mem_fun(*this, &Editor::nudge_track), true, true))));
+       nudge_items.push_back (MenuElem (_("Nudge Entire Track Earlier"), (sigc::bind (sigc::mem_fun(*this, &Editor::nudge_track), false, false))));
+       nudge_items.push_back (MenuElem (_("Nudge Track After Edit Point Earlier"), (sigc::bind (sigc::mem_fun(*this, &Editor::nudge_track), true, false))));
 
        edit_items.push_back (MenuElem (_("Nudge"), *nudge_menu));
 }
@@ -2717,7 +2743,10 @@ Editor::snap_to_internal (framepos_t& start, int32_t direction, bool for_mark)
 
                _session->locations()->marks_either_side (start, before, after);
 
-               if (before == max_framepos) {
+               if (before == max_framepos && after == max_framepos) {
+                       /* No marks to snap to, so just don't snap */
+                       return;
+               } else if (before == max_framepos) {
                        start = after;
                } else if (after == max_framepos) {
                        start = before;
@@ -3031,8 +3060,8 @@ Editor::setup_tooltips ()
        ARDOUR_UI::instance()->set_tip (smart_mode_joiner, _("Smart Mode (Select/Move Objects + Ranges)"));
        ARDOUR_UI::instance()->set_tip (internal_edit_button, _("Edit Region Contents (e.g. notes)"));
        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 (nudge_forward_button, _("Nudge Region/Selection Later"));
+       ARDOUR_UI::instance()->set_tip (nudge_backward_button, _("Nudge Region/Selection Earlier"));
        ARDOUR_UI::instance()->set_tip (zoom_in_button, _("Zoom In"));
        ARDOUR_UI::instance()->set_tip (zoom_out_button, _("Zoom Out"));
        ARDOUR_UI::instance()->set_tip (zoom_out_full_button, _("Zoom to Session"));
@@ -3083,8 +3112,8 @@ Editor::convert_drop_to_paths (
                const char* q;
 
                p = (const char *) malloc (txt.length() + 1);
-               txt.copy ((char *) p, txt.length(), 0);
-               ((char*)p)[txt.length()] = '\0';
+               txt.copy (const_cast<char *> (p), txt.length(), 0);
+               const_cast<char*>(p)[txt.length()] = '\0';
 
                while (p)
                {
@@ -3225,7 +3254,7 @@ Editor::history_changed ()
 }
 
 void
-Editor::duplicate_dialog (bool with_dialog)
+Editor::duplicate_range (bool with_dialog)
 {
        float times = 1.0f;
 
@@ -3905,6 +3934,15 @@ Editor::session_state_saved (string)
        _snapshots->redisplay ();
 }
 
+void
+Editor::update_tearoff_visibility()
+{
+       bool visible = Config->get_keep_tearoffs();
+       _mouse_mode_tearoff->set_visible (visible);
+       _tools_tearoff->set_visible (visible);
+       _zoom_tearoff->set_visible (visible);
+}
+
 void
 Editor::maximise_editing_space ()
 {
@@ -3914,15 +3952,6 @@ Editor::maximise_editing_space ()
 
        fullscreen ();
 
-       if (!Config->get_keep_tearoffs()) {
-               /* these calls will leave each tearoff visible *if* it is torn off, 
-                  but invisible otherwise.
-               */
-               _mouse_mode_tearoff->set_visible (false);
-               _tools_tearoff->set_visible (false);
-               _zoom_tearoff->set_visible (false);
-       }
-
        _maximised = true;
 }
 
@@ -3935,12 +3964,6 @@ Editor::restore_editing_space ()
 
        unfullscreen();
 
-       if (!Config->get_keep_tearoffs()) {
-               _mouse_mode_tearoff->set_visible (true);
-               _tools_tearoff->set_visible (true);
-               _zoom_tearoff->set_visible (true);
-       }
-
        _maximised = false;
 }
 
@@ -4028,19 +4051,31 @@ Editor::on_key_release_event (GdkEventKey* ev)
 void
 Editor::reset_x_origin (framepos_t frame)
 {
-       queue_visual_change (frame);
+       pending_visual_change.add (VisualChange::TimeOrigin);
+       pending_visual_change.time_origin = frame;
+       ensure_visual_change_idle_handler ();
 }
 
 void
 Editor::reset_y_origin (double y)
 {
-       queue_visual_change_y (y);
+       pending_visual_change.add (VisualChange::YOrigin);
+       pending_visual_change.y_origin = y;
+       ensure_visual_change_idle_handler ();
 }
 
 void
 Editor::reset_zoom (double fpu)
 {
-       queue_visual_change (fpu);
+       clamp_frames_per_unit (fpu);
+
+       if (fpu == frames_per_unit) {
+               return;
+       }
+
+       pending_visual_change.add (VisualChange::ZoomLevel);
+       pending_visual_change.frames_per_unit = fpu;
+       ensure_visual_change_idle_handler ();
 }
 
 void
@@ -4145,41 +4180,20 @@ Editor::use_visual_state (VisualState& vs)
        _routes->resume_redisplay ();
 }
 
+/** This is the core function that controls the zoom level of the canvas. It is called
+ *  whenever one or more calls are made to reset_zoom().  It executes in an idle handler.
+ *  @param fpu New frames per unit; should already have been clamped so that it is sensible.
+ */
 void
 Editor::set_frames_per_unit (double fpu)
 {
-       /* this is the core function that controls the zoom level of the canvas. it is called
-          whenever one or more calls are made to reset_zoom(). it executes in an idle handler.
-       */
-
-       if (fpu == frames_per_unit) {
-               return;
-       }
-
-       if (fpu < 2.0) {
-               fpu = 2.0;
-       }
-
-
-       /* don't allow zooms that fit more than the maximum number
-          of frames into an 800 pixel wide space.
-       */
-
-       if (max_framepos / fpu < 800.0) {
-               return;
-       }
-
-       if (tempo_lines)
+       if (tempo_lines) {
                tempo_lines->tempo_map_changed();
+       }
 
        frames_per_unit = fpu;
-       post_zoom ();
-}
 
-void
-Editor::post_zoom ()
-{
-       // convert fpu to frame count
+       /* convert fpu to frame count */
 
        framepos_t frames = (framepos_t) floor (frames_per_unit * _canvas_width);
 
@@ -4213,32 +4227,6 @@ Editor::post_zoom ()
        instant_save ();
 }
 
-void
-Editor::queue_visual_change (framepos_t where)
-{
-       pending_visual_change.add (VisualChange::TimeOrigin);
-       pending_visual_change.time_origin = where;
-       ensure_visual_change_idle_handler ();
-}
-
-void
-Editor::queue_visual_change (double fpu)
-{
-       pending_visual_change.add (VisualChange::ZoomLevel);
-       pending_visual_change.frames_per_unit = fpu;
-
-       ensure_visual_change_idle_handler ();
-}
-
-void
-Editor::queue_visual_change_y (double y)
-{
-       pending_visual_change.add (VisualChange::YOrigin);
-       pending_visual_change.y_origin = y;
-
-       ensure_visual_change_idle_handler ();
-}
-
 void
 Editor::ensure_visual_change_idle_handler ()
 {
@@ -4256,6 +4244,18 @@ Editor::_idle_visual_changer (void* arg)
 int
 Editor::idle_visual_changer ()
 {
+       /* set_horizontal_position() below (and maybe other calls) call
+          gtk_main_iteration(), so it's possible that a signal will be handled
+          half-way through this method.  If this signal wants an
+          idle_visual_changer we must schedule another one after this one, so
+          mark the idle_handler_id as -1 here to allow that.  Also make a note
+          that we are doing the visual change, so that changes in response to
+          super-rapid-screen-update can be dropped if we are still processing
+          the last one.
+       */
+       pending_visual_change.idle_handler_id = -1;
+       pending_visual_change.being_handled = true;
+       
        VisualChange::Type p = pending_visual_change.pending;
        pending_visual_change.pending = (VisualChange::Type) 0;
 
@@ -4284,7 +4284,7 @@ Editor::idle_visual_changer ()
 
        _summary->set_overlays_dirty ();
 
-       pending_visual_change.idle_handler_id = -1;
+       pending_visual_change.being_handled = false;
        return 0; /* this is always a one-shot call */
 }
 
@@ -4394,7 +4394,7 @@ Editor::set_punch_range (framepos_t start, framepos_t end, string cmd)
        Location* tpl;
 
        if ((tpl = transport_punch_location()) == 0) {
-               Location* loc = new Location (*_session, start, end, _("Loop"),  Location::IsAutoPunch);
+               Location* loc = new Location (*_session, start, end, _("Punch"),  Location::IsAutoPunch);
                XMLNode &before = _session->locations()->get_state();
                _session->locations()->add (loc, true);
                _session->set_auto_loop_location (loc);
@@ -4504,16 +4504,15 @@ Editor::get_regions_from_selection ()
  *  the edit point is `mouse' and the mouse is over an unselected
  *  region.  In this case, start with just that region.
  *
- *  Then, make an initial track list of the tracks that these
- *  regions are on, and if the edit point is not `mouse', add the
- *  selected tracks.
+ *  Then, add equivalent regions in active edit groups to the region list.
  *
- *  Look at this track list and add any other tracks that are on the
- *  same active edit-enabled route group as one of the initial tracks.
+ *  Then, search the list of selected tracks to find any selected tracks which
+ *  do not contain regions already in the region list. If there are no selected
+ *  tracks and 'No Selection = All Tracks' is active, search all tracks rather
+ *  than just the selected.
  *
- *  Finally take the initial region list and add any regions that are
- *  under the edit point on one of the tracks on the track list to get
- *  the returned region list.
+ *  Add any regions that are under the edit point on these tracks to get the
+ *  returned region list.
  *
  *  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
@@ -4541,22 +4540,40 @@ Editor::get_regions_from_selection_and_edit_point ()
                tracks = selection->tracks;
        }
 
-       /* Add any other tracks that have regions that are in the same
+       /* Add any other regions that are in the same
           edit-activated route group as one of our regions.
         */
-       for (RegionSelection::iterator i = regions.begin (); i != regions.end(); ++i) {
-
-               RouteGroup* g = (*i)->get_time_axis_view().route_group ();
+       regions = get_equivalent_regions (regions, ARDOUR::Properties::edit.property_id);
+       framepos_t const where = get_preferred_edit_position ();
 
-               if (g && g->is_active() && g->is_edit()) {
-                       tracks.add (axis_views_from_routes (g->route_list()));
-               }
+       if (_route_groups->all_group_active_button().get_active() && tracks.empty()) {
+               /* tracks is empty (no track selected), and 'No Selection = All Tracks'
+                * is enabled, so consider all tracks
+                */
+               tracks = track_views; 
        }
 
        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 (regions, where, tracks);
+               /* now search the selected tracks for tracks which don't
+                  already contain regions to be acted upon, and get regions at
+                  the edit point on those tracks too.
+                */
+               TrackViewList tracks_without_relevant_regions;
+
+               for (TrackViewList::iterator t = tracks.begin (); t != tracks.end (); ++t) {
+                       if (!regions.involves (**t)) {
+                               /* there are no equivalent regions on this track */
+                               tracks_without_relevant_regions.push_back (*t);
+                       }
+               }
+
+               if (!tracks_without_relevant_regions.empty()) {
+                       /* there are some selected tracks with neither selected
+                        * regions or their equivalents: act upon all regions in
+                        * those tracks
+                        */
+                       get_regions_at (regions, where, tracks_without_relevant_regions);
+               }
        }
 
        return regions;
@@ -4771,9 +4788,8 @@ Editor::axis_views_from_routes (boost::shared_ptr<RouteList> r) const
        return t;
 }
 
-
 void
-Editor::handle_new_route (RouteList& routes)
+Editor::add_routes (RouteList& routes)
 {
        ENSURE_GUI_THREAD (*this, &Editor::handle_new_route, routes)
 
@@ -5189,7 +5205,14 @@ Editor::super_rapid_screen_update ()
 
                if (!_stationary_playhead) {
 
-                       if (!_dragging_playhead && _follow_playhead && _session->requested_return_frame() < 0) {
+                       if (!_dragging_playhead && _follow_playhead && _session->requested_return_frame() < 0 && !pending_visual_change.being_handled) {
+                               /* We only do this if we aren't already
+                                  handling a visual change (ie if
+                                  pending_visual_change.being_handled is
+                                  false) so that these requests don't stack
+                                  up there are too many of them to handle in
+                                  time.
+                               */
                                reset_x_origin_to_follow_playhead ();
                        }
 
@@ -5345,22 +5368,28 @@ void
 Editor::setup_fade_images ()
 {
        _fade_in_images[FadeLinear] = new Gtk::Image (get_icon_path (X_("fadein-linear")));
-       _fade_in_images[FadeFast] = new Gtk::Image (get_icon_path (X_("fadein-short-cut")));
-       _fade_in_images[FadeLogB] = new Gtk::Image (get_icon_path (X_("fadein-slow-cut")));
-       _fade_in_images[FadeLogA] = new Gtk::Image (get_icon_path (X_("fadein-fast-cut")));
-       _fade_in_images[FadeSlow] = new Gtk::Image (get_icon_path (X_("fadein-long-cut")));
+       _fade_in_images[FadeSymmetric] = new Gtk::Image (get_icon_path (X_("fadein-short-cut")));
+       _fade_in_images[FadeFast] = new Gtk::Image (get_icon_path (X_("fadein-slow-cut")));
+       _fade_in_images[FadeSlow] = new Gtk::Image (get_icon_path (X_("fadein-fast-cut")));
+       _fade_in_images[FadeConstantPower] = new Gtk::Image (get_icon_path (X_("fadein-long-cut")));
 
        _fade_out_images[FadeLinear] = new Gtk::Image (get_icon_path (X_("fadeout-linear")));
-       _fade_out_images[FadeFast] = new Gtk::Image (get_icon_path (X_("fadeout-short-cut")));
-       _fade_out_images[FadeLogB] = new Gtk::Image (get_icon_path (X_("fadeout-slow-cut")));
-       _fade_out_images[FadeLogA] = new Gtk::Image (get_icon_path (X_("fadeout-fast-cut")));
-       _fade_out_images[FadeSlow] = new Gtk::Image (get_icon_path (X_("fadeout-long-cut")));
+       _fade_out_images[FadeSymmetric] = new Gtk::Image (get_icon_path (X_("fadeout-short-cut")));
+       _fade_out_images[FadeFast] = new Gtk::Image (get_icon_path (X_("fadeout-slow-cut")));
+       _fade_out_images[FadeSlow] = new Gtk::Image (get_icon_path (X_("fadeout-fast-cut")));
+       _fade_out_images[FadeConstantPower] = new Gtk::Image (get_icon_path (X_("fadeout-long-cut")));
+       
+       _xfade_in_images[FadeLinear] = new Gtk::Image (get_icon_path (X_("fadeout-linear")));
+       _xfade_in_images[FadeSymmetric] = new Gtk::Image (get_icon_path (X_("fadeout-short-cut")));
+       _xfade_in_images[FadeFast] = new Gtk::Image (get_icon_path (X_("fadeout-slow-cut")));
+       _xfade_in_images[FadeSlow] = new Gtk::Image (get_icon_path (X_("fadeout-fast-cut")));
+       _xfade_in_images[FadeConstantPower] = new Gtk::Image (get_icon_path (X_("fadeout-long-cut")));
 
-       _xfade_images[FadeLinear] = new Gtk::Image (get_icon_path (X_("fadeout-linear")));
-       _xfade_images[FadeFast] = new Gtk::Image (get_icon_path (X_("fadeout-short-cut")));
-       _xfade_images[FadeLogB] = new Gtk::Image (get_icon_path (X_("fadeout-slow-cut")));
-       _xfade_images[FadeLogA] = new Gtk::Image (get_icon_path (X_("fadeout-fast-cut")));
-       _xfade_images[FadeSlow] = new Gtk::Image (get_icon_path (X_("fadeout-long-cut")));
+       _xfade_out_images[FadeLinear] = new Gtk::Image (get_icon_path (X_("fadeout-linear")));
+       _xfade_out_images[FadeSymmetric] = new Gtk::Image (get_icon_path (X_("fadeout-short-cut")));
+       _xfade_out_images[FadeFast] = new Gtk::Image (get_icon_path (X_("fadeout-slow-cut")));
+       _xfade_out_images[FadeSlow] = new Gtk::Image (get_icon_path (X_("fadeout-fast-cut")));
+       _xfade_out_images[FadeConstantPower] = new Gtk::Image (get_icon_path (X_("fadeout-long-cut")));
 
 }
 
@@ -5433,3 +5462,8 @@ Editor::popup_control_point_context_menu (ArdourCanvas::Item* item, GdkEvent* ev
        _control_point_context_menu.popup (event->button.button, event->button.time);
 }
 
+void
+Editor::shift_key_released ()
+{
+       _stepping_axis_view = 0;
+}