Make editor route list rec button respect group settings.
[ardour.git] / gtk2_ardour / editor_ops.cc
index 5826033072b8586c203d8c7171f4fd82f26b7300..55382bbacaed10f2821ad28e01364dbb72248ad4 100644 (file)
@@ -54,6 +54,8 @@
 #include "ardour/transient_detector.h"
 #include "ardour/dB.h"
 #include "ardour/quantize.h"
+#include "ardour/strip_silence.h"
+#include "ardour/route_group.h"
 
 #include "ardour_ui.h"
 #include "editor.h"
 #include "gui_thread.h"
 #include "keyboard.h"
 #include "utils.h"
+#include "editor_drag.h"
+#include "strip_silence_dialog.h"
+#include "editor_routes.h"
+#include "editor_regions.h"
 
 #include "i18n.h"
 
@@ -106,17 +112,11 @@ Editor::redo (uint32_t n)
 
 void
 Editor::split_region ()
-{
-       split_region_at (get_preferred_edit_position());
-}
-
-void
-Editor::split_region_at (nframes64_t where)
 {
        RegionSelection rs;
 
-       get_regions_for_action (rs);
-       split_regions_at (where, selection->regions);
+       get_regions_for_action (rs, true);
+       split_regions_at (get_preferred_edit_position (), selection->regions);
 }
 
 void
@@ -197,76 +197,8 @@ Editor::split_regions_at (nframes64_t where, RegionSelection& regions)
        _new_regionviews_show_envelope = false;
 }
 
-
-/** Remove `clicked_regionview' */
-void
-Editor::remove_clicked_region ()
-{
-       if (clicked_routeview == 0 || clicked_regionview == 0) {
-               return;
-       }
-
-       boost::shared_ptr<Playlist> playlist = clicked_routeview->playlist();
-       
-       begin_reversible_command (_("remove region"));
-        XMLNode &before = playlist->get_state();
-       playlist->remove_region (clicked_regionview->region());
-        XMLNode &after = playlist->get_state();
-       session->add_command(new MementoCommand<Playlist>(*playlist, &before, &after));
-       commit_reversible_command ();
-}
-
-
-/** Remove the selected regions */
-void
-Editor::remove_selected_regions ()
-{
-       RegionSelection rs; 
-       get_regions_for_action (rs);
-       
-       if (!session) {
-               return;
-       }
-
-       if (rs.empty()) {
-               return;
-       }
-
-       begin_reversible_command (_("remove region"));
-
-       list<boost::shared_ptr<Region> > regions_to_remove;
-
-       for (RegionSelection::iterator i = rs.begin(); i != rs.end(); ++i) {
-               // we can't just remove the region(s) in this loop because
-               // this removes them from the RegionSelection, and they thus
-               // disappear from underneath the iterator, and the ++i above
-               // SEGVs in a puzzling fashion.
-
-               // so, first iterate over the regions to be removed from rs and
-               // add them to the regions_to_remove list, and then
-               // iterate over the list to actually remove them.
-               
-               regions_to_remove.push_back ((*i)->region());
-       }
-       
-       for (list<boost::shared_ptr<Region> >::iterator rl = regions_to_remove.begin(); rl != regions_to_remove.end(); ++rl) {
-               boost::shared_ptr<Playlist> playlist = (*rl)->playlist();
-               if (!playlist) {
-                       // is this check necessary?
-                       continue;
-               }
-
-               XMLNode &before = playlist->get_state();
-               playlist->remove_region (*rl);
-               XMLNode &after = playlist->get_state();
-               session->add_command(new MementoCommand<Playlist>(*playlist, &before, &after));
-       }
-
-       commit_reversible_command ();
-}
-
 boost::shared_ptr<Region>
-Editor::select_region_for_operation (int dir, TimeAxisView **tv)
+Editor::select_region_for_operation (int /*dir*/, TimeAxisView **tv)
 {
        RegionView* rv;
        boost::shared_ptr<Region> region;
@@ -925,7 +857,7 @@ Editor::cursor_to_previous_region_boundary (bool with_selection)
 }
 
 void
-Editor::cursor_to_region_point (Cursor* cursor, RegionPoint point, int32_t dir)
+Editor::cursor_to_region_point (EditorCursor* cursor, RegionPoint point, int32_t dir)
 {
        boost::shared_ptr<Region> r;
        nframes64_t pos = cursor->current_frame;
@@ -994,19 +926,19 @@ Editor::cursor_to_region_point (Cursor* cursor, RegionPoint point, int32_t dir)
 }
 
 void
-Editor::cursor_to_next_region_point (Cursor* cursor, RegionPoint point)
+Editor::cursor_to_next_region_point (EditorCursor* cursor, RegionPoint point)
 {
        cursor_to_region_point (cursor, point, 1);
 }
 
 void
-Editor::cursor_to_previous_region_point (Cursor* cursor, RegionPoint point)
+Editor::cursor_to_previous_region_point (EditorCursor* cursor, RegionPoint point)
 {
        cursor_to_region_point (cursor, point, -1);
 }
 
 void
-Editor::cursor_to_selection_start (Cursor *cursor)
+Editor::cursor_to_selection_start (EditorCursor *cursor)
 {
        nframes64_t pos = 0;
        RegionSelection rs; 
@@ -1038,7 +970,7 @@ Editor::cursor_to_selection_start (Cursor *cursor)
 }
 
 void
-Editor::cursor_to_selection_end (Cursor *cursor)
+Editor::cursor_to_selection_end (EditorCursor *cursor)
 {
        nframes64_t pos = 0;
        RegionSelection rs; 
@@ -1481,7 +1413,7 @@ void
 Editor::scroll_backward (float pages)
 {
        nframes64_t frame;
-       nframes64_t one_page = (nframes64_t) rint (canvas_width * frames_per_unit);
+       nframes64_t one_page = (nframes64_t) rint (_canvas_width * frames_per_unit);
        bool was_floating;
        float prefix;
        nframes64_t cnt;
@@ -1509,7 +1441,7 @@ void
 Editor::scroll_forward (float pages)
 {
        nframes64_t frame;
-       nframes64_t one_page = (nframes64_t) rint (canvas_width * frames_per_unit);
+       nframes64_t one_page = (nframes64_t) rint (_canvas_width * frames_per_unit);
        bool was_floating;
        float prefix;
        nframes64_t cnt;
@@ -1548,8 +1480,8 @@ Editor::scroll_tracks_down ()
 
        double vert_value = vertical_adjustment.get_value() + (cnt *
                vertical_adjustment.get_page_size());
-       if (vert_value > vertical_adjustment.get_upper() - canvas_height) {
-               vert_value = vertical_adjustment.get_upper() - canvas_height;
+       if (vert_value > vertical_adjustment.get_upper() - _canvas_height) {
+               vert_value = vertical_adjustment.get_upper() - _canvas_height;
        }
        vertical_adjustment.set_value (vert_value);
 }
@@ -1577,8 +1509,8 @@ Editor::scroll_tracks_down_line ()
         Gtk::Adjustment* adj = edit_vscrollbar.get_adjustment();
        double vert_value = adj->get_value() + 60;
 
-       if (vert_value>adj->get_upper() - canvas_height) {
-               vert_value = adj->get_upper() - canvas_height;
+       if (vert_value>adj->get_upper() - _canvas_height) {
+               vert_value = adj->get_upper() - _canvas_height;
        }
        adj->set_value (vert_value);
 }
@@ -1635,7 +1567,7 @@ Editor::temporal_zoom (gdouble fpu)
 
        nfpu = fpu;
        
-       new_page_size = (nframes64_t) floor (canvas_width * nfpu);
+       new_page_size = (nframes64_t) floor (_canvas_width * nfpu);
        half_page_size = new_page_size / 2;
 
        switch (zoom_focus) {
@@ -1778,7 +1710,7 @@ Editor::temporal_zoom_region (bool both_axes)
        }
 
        nframes64_t range = end - start;
-       double new_fpu = (double)range / (double)canvas_width;
+       double new_fpu = (double)range / (double)_canvas_width;
        nframes64_t extra_samples = (nframes64_t) floor (one_centimeter_in_pixels * new_fpu);
 
        if (start > extra_samples) {
@@ -1804,7 +1736,7 @@ Editor::temporal_zoom_region (bool both_axes)
        temporal_zoom_by_frame (start, end, "zoom to region");
 
        if (both_axes) {
-               uint32_t per_track_height = (uint32_t) floor ((canvas_height - canvas_timebars_vsize - 10.0) / tracks.size());
+               uint32_t per_track_height = (uint32_t) floor ((_canvas_height - canvas_timebars_vsize - 10.0) / tracks.size());
                
                /* set visible track heights appropriately */
                
@@ -1813,8 +1745,8 @@ Editor::temporal_zoom_region (bool both_axes)
                }
                
                /* hide irrelevant tracks */
-               
-               no_route_list_redisplay = true;
+
+               _routes->suspend_redisplay ();
 
                for (TrackViewList::iterator i = track_views.begin(); i != track_views.end(); ++i) {
                        if (find (tracks.begin(), tracks.end(), (*i)) == tracks.end()) {
@@ -1822,8 +1754,7 @@ Editor::temporal_zoom_region (bool both_axes)
                        }
                }
 
-               no_route_list_redisplay = false;
-               redisplay_route_list ();
+               _routes->resume_redisplay ();
 
                vertical_adjustment.set_value (0.0);
                no_save_visual = false;
@@ -1869,7 +1800,7 @@ Editor::temporal_zoom_session ()
 }
 
 void
-Editor::temporal_zoom_by_frame (nframes64_t start, nframes64_t end, const string & op)
+Editor::temporal_zoom_by_frame (nframes64_t start, nframes64_t end, const string & /*op*/)
 {
        if (!session) return;
 
@@ -1879,9 +1810,9 @@ Editor::temporal_zoom_by_frame (nframes64_t start, nframes64_t end, const string
 
        nframes64_t range = end - start;
 
-       double new_fpu = (double)range / (double)canvas_width;
+       double new_fpu = (double)range / (double)_canvas_width;
        
-       nframes64_t new_page = (nframes64_t) floor (canvas_width * new_fpu);
+       nframes64_t new_page = (nframes64_t) floor (_canvas_width * new_fpu);
        nframes64_t middle = (nframes64_t) floor( (double)start + ((double)range / 2.0f ));
        nframes64_t new_leftmost = (nframes64_t) floor( (double)middle - ((double)new_page/2.0f));
 
@@ -2384,29 +2315,17 @@ Editor::insert_region_list_selection (float times)
        if ((playlist = tv->playlist()) == 0) {
                return;
        }
-       
-       Glib::RefPtr<TreeSelection> selected = region_list_display.get_selection();
-       
-       if (selected->count_selected_rows() != 1) {
+
+       boost::shared_ptr<Region> region = _regions->get_single_selection ();
+       if (region == 0) {
                return;
        }
-       
-       TreeView::Selection::ListHandle_Path rows = selected->get_selected_rows ();
-
-       /* only one row selected, so rows.begin() is it */
-
-       TreeIter iter;
-
-       if ((iter = region_list_model->get_iter (*rows.begin()))) {
-
-               boost::shared_ptr<Region> region = (*iter)[region_list_columns.region];
                
-               begin_reversible_command (_("insert region"));
-               XMLNode &before = playlist->get_state();
-               playlist->add_region ((RegionFactory::create (region)), get_preferred_edit_position(), times);
-               session->add_command(new MementoCommand<Playlist>(*playlist, &before, &playlist->get_state()));
-               commit_reversible_command ();
-       } 
+       begin_reversible_command (_("insert region"));
+       XMLNode &before = playlist->get_state();
+       playlist->add_region ((RegionFactory::create (region)), get_preferred_edit_position(), times);
+       session->add_command(new MementoCommand<Playlist>(*playlist, &before, &playlist->get_state()));
+       commit_reversible_command ();
 }
 
 /* BUILT-IN EFFECTS */
@@ -2660,7 +2579,7 @@ Editor::rename_region()
                strip_whitespace_edges (str);
                if (!str.empty()) {
                        rs.front()->region()->set_name (str);
-                       redisplay_regions ();
+                       _regions->redisplay ();
                }
        }
 }
@@ -2774,9 +2693,11 @@ Editor::region_from_selection ()
        nframes64_t start = selection->time[clicked_selection].start;
        nframes64_t end = selection->time[clicked_selection].end;
 
+       TrackSelection tracks = get_tracks_for_range_action ();
+
        nframes64_t selection_cnt = end - start + 1;
        
-       for (TrackSelection::iterator i = selection->tracks.begin(); i != selection->tracks.end(); ++i) {
+       for (TrackSelection::iterator i = tracks.begin(); i != tracks.end(); ++i) {
                boost::shared_ptr<Region> current;
                boost::shared_ptr<Playlist> pl;
                nframes64_t internal_start;
@@ -2825,7 +2746,7 @@ Editor::create_region_from_selection (vector<boost::shared_ptr<Region> >& new_re
 
                internal_start = start - current->position();
                session->region_name (new_name, current->name(), true);
-               
+
                new_regions.push_back (RegionFactory::create (current,
                                        internal_start, end - start + 1, new_name));
        }
@@ -2867,41 +2788,52 @@ add_if_covered (RegionView* rv, const AudioRange* ar, RegionSelection* rs)
        }
 }
 
-void
-Editor::separate_regions_between (const TimeSelection& ts)
+/** Return either:
+ *    - selected tracks, or if there are none...
+ *    - tracks containing selected regions, or if there are none...
+ *    - all tracks
+ * @return tracks.
+ */
+TrackSelection
+Editor::get_tracks_for_range_action () const
 {
-       bool in_command = false;
-       boost::shared_ptr<Playlist> playlist;
-       RegionSelection new_selection;
-       TrackSelection tmptracks;
-
+       TrackSelection t;
+       
        if (selection->tracks.empty()) {
                
                /* use tracks with selected regions */
 
-               RegionSelection rs; 
-
-               get_regions_for_action (rs);
+               RegionSelection rs = selection->regions;
 
                for (RegionSelection::iterator i = rs.begin(); i != rs.end(); ++i) {
                        TimeAxisView* tv = &(*i)->get_time_axis_view();
 
-                       if (find (tmptracks.begin(), tmptracks.end(), tv) == tmptracks.end()) {
-                               tmptracks.push_back (tv);
+                       if (!t.contains (tv)) {
+                               t.push_back (tv);
                        }
                }
 
-               if (tmptracks.empty()) {
-                       /* no regions selected: do nothing */
-                       return;
+               if (t.empty()) {
+                       /* no regions and no tracks: use all tracks */
+                       t = track_views;
                }
 
        } else {
 
-               tmptracks = selection->tracks;
-
+               t = selection->tracks;
        }
 
+       return t;
+}
+
+void
+Editor::separate_regions_between (const TimeSelection& ts)
+{
+       bool in_command = false;
+       boost::shared_ptr<Playlist> playlist;
+       RegionSelection new_selection;
+
+       TrackSelection tmptracks = get_tracks_for_range_action ();
        sort_track_selection (&tmptracks);
 
        for (TrackSelection::iterator i = tmptracks.begin(); i != tmptracks.end(); ++i) {
@@ -2976,6 +2908,10 @@ Editor::separate_regions_between (const TimeSelection& ts)
        }
 }
 
+/** Take tracks from get_tracks_for_range_action and cut any regions
+ *  on those tracks so that the tracks are empty over the time
+ *  selection.
+ */
 void
 Editor::separate_region_from_selection ()
 {
@@ -3184,16 +3120,11 @@ Editor::region_fill_selection ()
                return;
        }
 
-
-       Glib::RefPtr<TreeSelection> selected = region_list_display.get_selection();
-
-       if (selected->count_selected_rows() != 1) {
+       boost::shared_ptr<Region> region = _regions->get_single_selection ();
+       if (region == 0) {
                return;
        }
 
-       TreeModel::iterator i = region_list_display.get_selection()->get_selected();
-       boost::shared_ptr<Region> region = (*i)[region_list_columns.region];
-
        nframes64_t start = selection->time[clicked_selection].start;
        nframes64_t end = selection->time[clicked_selection].end;
 
@@ -3770,7 +3701,7 @@ Editor::freeze_thread ()
 }
 
 gint
-Editor::freeze_progress_timeout (void *arg)
+Editor::freeze_progress_timeout (void */*arg*/)
 {
        interthread_progress_bar.set_fraction (current_interthread_info->progress);
        return !(current_interthread_info->done || current_interthread_info->cancel);
@@ -3948,9 +3879,10 @@ Editor::cut_copy (CutCopyOp op)
        */
 
        if (op == Cut || op == Clear) {
-               if (drag_info.item) {
-                       drag_info.item->ungrab (0);
-                       drag_info.item = 0;
+               if (_drag) {
+                       _drag->item()->ungrab (0);
+                       delete _drag;
+                       _drag = 0;
                }
        }
        
@@ -3968,6 +3900,8 @@ Editor::cut_copy (CutCopyOp op)
                }
 
                break_drag ();
+               delete _drag;
+               _drag = 0;
 
                return;
        }
@@ -4037,6 +3971,8 @@ Editor::cut_copy (CutCopyOp op)
 
        if (op == Cut || op == Clear) {
                break_drag ();
+               delete _drag;
+               _drag = 0;
        }
 }
 
@@ -4074,13 +4010,106 @@ struct PlaylistMapping {
     PlaylistMapping (TimeAxisView* tvp) : tv (tvp) {}
 };
 
+/** Remove `clicked_regionview' */
+void
+Editor::remove_clicked_region ()
+{
+       if (clicked_routeview == 0 || clicked_regionview == 0) {
+               return;
+       }
+
+       boost::shared_ptr<Playlist> playlist = clicked_routeview->playlist();
+       
+       begin_reversible_command (_("remove region"));
+        XMLNode &before = playlist->get_state();
+       playlist->remove_region (clicked_regionview->region());
+        XMLNode &after = playlist->get_state();
+       session->add_command(new MementoCommand<Playlist>(*playlist, &before, &after));
+       commit_reversible_command ();
+}
+
+
+/** Remove the selected regions */
+void
+Editor::remove_selected_regions ()
+{
+       RegionSelection rs; 
+       get_regions_for_action (rs);
+       
+       if (!session) {
+               return;
+       }
+
+       if (rs.empty()) {
+               return;
+       }
+
+       begin_reversible_command (_("remove region"));
+
+       list<boost::shared_ptr<Region> > regions_to_remove;
+
+       for (RegionSelection::iterator i = rs.begin(); i != rs.end(); ++i) {
+               // we can't just remove the region(s) in this loop because
+               // this removes them from the RegionSelection, and they thus
+               // disappear from underneath the iterator, and the ++i above
+               // SEGVs in a puzzling fashion.
+
+               // so, first iterate over the regions to be removed from rs and
+               // add them to the regions_to_remove list, and then
+               // iterate over the list to actually remove them.
+               
+               regions_to_remove.push_back ((*i)->region());
+       }
+
+       vector<PlaylistState> playlists;
+       
+       for (list<boost::shared_ptr<Region> >::iterator rl = regions_to_remove.begin(); rl != regions_to_remove.end(); ++rl) {
+
+               boost::shared_ptr<Playlist> playlist = (*rl)->playlist();
+
+               if (!playlist) {
+                       // is this check necessary?
+                       continue;
+               }
+
+               vector<PlaylistState>::iterator i;
+
+               //only take state if this is a new playlist.
+               for (i = playlists.begin(); i != playlists.end(); ++i) {
+                       if ((*i).playlist == playlist) {
+                               break;
+                       }
+               }
+
+               if (i == playlists.end()) {
+
+                       PlaylistState before;
+                       before.playlist = playlist;
+                       before.before = &playlist->get_state();
+
+                       playlist->freeze ();
+                       playlists.push_back(before);
+               }
+
+               playlist->remove_region (*rl);          
+       }
+
+       vector<PlaylistState>::iterator pl;
+
+       for (pl = playlists.begin(); pl != playlists.end(); ++pl) {
+               (*pl).playlist->thaw ();
+               session->add_command(new MementoCommand<Playlist>(*(*pl).playlist, (*pl).before, &(*pl).playlist->get_state()));
+       }
+
+       commit_reversible_command ();
+}
 
 /** Cut, copy or clear selected regions.
  * @param op Operation (Cut, Copy or Clear)
  */
 void
 Editor::cut_copy_regions (CutCopyOp op, RegionSelection& rs)
-{      
+{
        /* we can't use a std::map here because the ordering is important, and we can't trivially sort
           a map when we want ordered access to both elements. i think.
        */
@@ -4104,15 +4133,21 @@ Editor::cut_copy_regions (CutCopyOp op, RegionSelection& rs)
                        boost::shared_ptr<Playlist> pl = (*x)->region()->playlist();
 
                        if (pl) {
+                               set<PlaylistState, lt_playlist>::iterator fl;
 
-                               PlaylistState before;
-                               before.playlist = pl;
-                               before.before = &pl->get_state();
-                               
-                               insert_result = freezelist.insert (before);
-                               
-                               if (insert_result.second) {
+                               //only take state if this is a new playlist.
+                               for (fl = freezelist.begin(); fl != freezelist.end(); ++fl) {
+                                       if ((*fl).playlist == pl) {
+                                               break;
+                                       }
+                               }
+               
+                               if (fl == freezelist.end()) {
+                                       PlaylistState before;
+                                       before.playlist = pl;
+                                       before.before = &pl->get_state();
                                        pl->freeze ();
+                                       insert_result = freezelist.insert (before);
                                }
                        }
                }
@@ -4444,14 +4479,14 @@ Editor::reset_point_selection ()
 void
 Editor::center_playhead ()
 {
-       float page = canvas_width * frames_per_unit;
+       float page = _canvas_width * frames_per_unit;
        center_screen_internal (playhead_cursor->current_frame, page);
 }
 
 void
 Editor::center_edit_point ()
 {
-       float page = canvas_width * frames_per_unit;
+       float page = _canvas_width * frames_per_unit;
        center_screen_internal (get_preferred_edit_position(), page);
 }
 
@@ -4536,17 +4571,35 @@ Editor::remove_last_capture ()
 void
 Editor::normalize_region ()
 {
-       RegionSelection rs; 
-
-       get_regions_for_action (rs);
-       
        if (!session) {
                return;
        }
 
+       RegionSelection rs; 
+       get_regions_for_action (rs);
+
        if (rs.empty()) {
                return;
        }
+       
+       Dialog dialog (rs.size() > 1 ? _("Normalize regions") : _("Normalize region"));
+       HBox hbox;
+       hbox.pack_start (*manage (new Label (_("Normalize to:"))));
+       SpinButton spin (0.2, 2);
+       spin.set_range (-112, 0);
+       spin.set_increments (0.1, 1);
+       spin.set_value (0);
+       hbox.pack_start (spin);
+       spin.set_value (_last_normalization_value);
+       hbox.pack_start (*manage (new Label (_("dbFS"))));
+       hbox.show_all ();
+       dialog.get_vbox()->pack_start (hbox);
+       dialog.add_button (Stock::CANCEL, RESPONSE_CANCEL);
+       dialog.add_button (_("Normalize"), RESPONSE_ACCEPT);
+
+       if (dialog.run () == RESPONSE_CANCEL) {
+               return;
+       }
 
        begin_reversible_command (_("normalize"));
 
@@ -4558,12 +4611,14 @@ Editor::normalize_region ()
                if (!arv)
                        continue;
                XMLNode &before = arv->region()->get_state();
-               arv->audio_region()->normalize_to (0.0f);
+               arv->audio_region()->normalize_to (spin.get_value());
                session->add_command (new MementoCommand<Region>(*(arv->region().get()), &before, &arv->region()->get_state()));
        }
 
        commit_reversible_command ();
        track_canvas->get_window()->set_cursor (*current_canvas_cursor);
+
+       _last_normalization_value = spin.get_value ();
 }
 
 
@@ -4660,6 +4715,38 @@ Editor::reverse_region ()
        apply_filter (rev, _("reverse regions"));
 }
 
+void
+Editor::strip_region_silence ()
+{
+       if (!session) {
+               return;
+       }
+
+       RegionSelection rs;
+       get_regions_for_action (rs);
+
+       if (rs.empty()) {
+               return;
+       }
+
+       std::list<boost::shared_ptr<AudioRegion> > ar;
+       
+       for (RegionSelection::iterator i = rs.begin(); i != rs.end(); ++i) {
+               AudioRegionView* const arv = dynamic_cast<AudioRegionView*> (*i);
+               if (arv) {
+                       ar.push_back (arv->audio_region ());
+               }
+       }
+       
+       StripSilenceDialog d (ar);
+       int const r = d.run ();
+
+       if (r == Gtk::RESPONSE_OK) {
+               StripSilence s (*session, d.threshold (), d.minimum_length (), d.fade_length ());
+               apply_filter (s, _("strip silence"));
+       }
+}
+
 
 void
 Editor::quantize_region ()
@@ -4707,7 +4794,28 @@ Editor::apply_filter (Filter& filter, string command)
                        if (arv->audio_region()->apply (filter) == 0) {
 
                                XMLNode &before = playlist->get_state();
-                               playlist->replace_region (arv->region(), filter.results.front(), arv->region()->position());
+
+                               if (filter.results.empty ()) {
+
+                                       /* no regions returned; remove the old one */
+                                       playlist->remove_region (arv->region ());
+                                       
+                               } else {
+
+                                       std::vector<boost::shared_ptr<Region> >::iterator res = filter.results.begin ();
+
+                                       /* first region replaces the old one */
+                                       playlist->replace_region (arv->region(), *res, (*res)->position());
+                                       ++res;
+
+                                       /* add the rest */
+                                       while (res != filter.results.end()) {
+                                               playlist->add_region (*res, (*res)->position());
+                                               ++res;
+                                       }
+                                       
+                               }
+
                                XMLNode &after = playlist->get_state();
                                session->add_command(new MementoCommand<Playlist>(*playlist, &before, &after));
                        } else {
@@ -4781,9 +4889,7 @@ Editor::brush (nframes64_t pos)
 void
 Editor::reset_region_gain_envelopes ()
 {
-       RegionSelection rs; 
-
-       get_regions_for_action (rs);
+       RegionSelection rs = get_equivalent_regions (selection->regions, RouteGroup::Edit);
 
        if (!session || rs.empty()) {
                return;
@@ -4808,80 +4914,134 @@ Editor::reset_region_gain_envelopes ()
 void
 Editor::toggle_gain_envelope_visibility ()
 {
-       RegionSelection rs
+       RegionSelection rs = get_equivalent_regions (selection->regions, RouteGroup::Edit);
 
-       get_regions_for_action (rs);
+       if (!session || rs.empty()) {
+               return;
+       }
+
+       session->begin_reversible_command (_("region gain envelope visible"));
 
        for (RegionSelection::iterator i = rs.begin(); i != rs.end(); ++i) {
                AudioRegionView* const arv = dynamic_cast<AudioRegionView*>(*i);
                if (arv) {
+                       XMLNode &before = arv->region()->get_state ();
                        arv->set_envelope_visible (!arv->envelope_visible());
+                       XMLNode &after = arv->region()->get_state ();
+                       session->add_command (new MementoCommand<Region> (*(arv->region().get()), &before, &after));
                }
        }
+
+       session->commit_reversible_command ();
 }
 
 void
 Editor::toggle_gain_envelope_active ()
 {
-       RegionSelection rs
+       RegionSelection rs = get_equivalent_regions (selection->regions, RouteGroup::Edit);
 
-       get_regions_for_action (rs);
+       if (!session || rs.empty()) {
+               return;
+       }
+       
+       session->begin_reversible_command (_("region gain envelope active"));
 
        for (RegionSelection::iterator i = rs.begin(); i != rs.end(); ++i) {
                AudioRegionView* const arv = dynamic_cast<AudioRegionView*>(*i);
                if (arv) {
+                       XMLNode &before = arv->region()->get_state ();
                        arv->audio_region()->set_envelope_active (!arv->audio_region()->envelope_active());
+                       XMLNode &after = arv->region()->get_state ();
+                       session->add_command (new MementoCommand<Region> (*(arv->region().get()), &before, &after));
                }
        }
+
+       session->commit_reversible_command ();
 }
 
 void
 Editor::toggle_region_lock ()
 {
-       RegionSelection rs
+       RegionSelection rs = get_equivalent_regions (selection->regions, RouteGroup::Edit);
 
-       get_regions_for_action (rs);
+       if (!session || rs.empty()) {
+               return;
+       }
+       
+       session->begin_reversible_command (_("region lock"));
 
        for (RegionSelection::iterator i = rs.begin(); i != rs.end(); ++i) {
+               XMLNode &before = (*i)->region()->get_state ();
                (*i)->region()->set_locked (!(*i)->region()->locked());
+               XMLNode &after = (*i)->region()->get_state ();
+               session->add_command (new MementoCommand<Region> (*((*i)->region().get()), &before, &after));
        }
+
+       session->commit_reversible_command ();
 }
 
 void
 Editor::set_region_lock_style (Region::PositionLockStyle ps)
 {
-       RegionSelection rs
+       RegionSelection rs = get_equivalent_regions (selection->regions, RouteGroup::Edit);
 
-       get_regions_for_action (rs);
+       if (!session || rs.empty()) {
+               return;
+       }
+       
+       session->begin_reversible_command (_("region lock style"));
 
        for (RegionSelection::iterator i = rs.begin(); i != rs.end(); ++i) {
+               XMLNode &before = (*i)->region()->get_state ();
                (*i)->region()->set_position_lock_style (ps);
+               XMLNode &after = (*i)->region()->get_state ();
+               session->add_command (new MementoCommand<Region> (*((*i)->region().get()), &before, &after));
        }
+
+       session->commit_reversible_command ();
 }
 
 
 void
 Editor::toggle_region_mute ()
 {
-       RegionSelection rs
+       RegionSelection rs = get_equivalent_regions (selection->regions, RouteGroup::Edit);
 
-       get_regions_for_action (rs);
+       if (!session || rs.empty()) {
+               return;
+       }
+       
+       session->begin_reversible_command (_("region mute"));
 
        for (RegionSelection::iterator i = rs.begin(); i != rs.end(); ++i) {
+               XMLNode &before = (*i)->region()->get_state ();
                (*i)->region()->set_muted (!(*i)->region()->muted());
+               XMLNode &after = (*i)->region()->get_state ();
+               session->add_command (new MementoCommand<Region> (*((*i)->region().get()), &before, &after));
        }
+
+       session->commit_reversible_command ();
 }
 
 void
 Editor::toggle_region_opaque ()
 {
-       RegionSelection rs
+       RegionSelection rs = get_equivalent_regions (selection->regions, RouteGroup::Edit);
 
-       get_regions_for_action (rs);
+       if (!session || rs.empty()) {
+               return;
+       }
+       
+       session->begin_reversible_command (_("region opacity"));
 
        for (RegionSelection::iterator i = rs.begin(); i != rs.end(); ++i) {
+               XMLNode &before = (*i)->region()->get_state ();
                (*i)->region()->set_opaque (!(*i)->region()->opaque());
+               XMLNode &after = (*i)->region()->get_state ();
+               session->add_command (new MementoCommand<Region> (*((*i)->region().get()), &before, &after));
        }
+
+       session->commit_reversible_command ();
 }
 
 void
@@ -4911,20 +5071,16 @@ Editor::set_fade_length (bool in)
 {
        RegionSelection rs; 
 
-       get_regions_for_action (rs);
-
-       /* we need a region to measure the offset from the start */
+       get_regions_for_action (rs, true);
 
-       RegionView* rv;
-
-       if (!rs.empty()) {
-               rv = rs.front();
-       } else if (entered_regionview) {
-               rv = entered_regionview;
-       } else {
+       if (rs.empty()) {
                return;
        }
 
+       /* we need a region to measure the offset from the start */
+
+       RegionView* rv = rs.front ();
+
        nframes64_t pos = get_preferred_edit_position();
        nframes64_t len;
        char* cmd;
@@ -5215,7 +5371,7 @@ Editor::toggle_selected_region_fades (int dir)
 void
 Editor::update_region_fade_visibility ()
 {
-       bool _fade_visibility = Config->get_show_region_fades ();
+       bool _fade_visibility = session->config.get_show_region_fades ();
 
        for (TrackViewList::iterator i = track_views.begin(); i != track_views.end(); ++i) {
                AudioTimeAxisView* v = dynamic_cast<AudioTimeAxisView*>(*i);
@@ -5233,7 +5389,7 @@ Editor::update_region_fade_visibility ()
 void
 Editor::update_xfade_visibility ()
 {
-       _xfade_visibility = Config->get_xfades_visible ();
+       _xfade_visibility = session->config.get_xfades_visible ();
        
        for (TrackViewList::iterator i = track_views.begin(); i != track_views.end(); ++i) {
                AudioTimeAxisView* v = dynamic_cast<AudioTimeAxisView*>(*i);
@@ -5300,7 +5456,7 @@ Editor::split ()
 {
        RegionSelection rs; 
        
-       get_regions_for_action (rs);
+       get_regions_for_action (rs, true);
 
        nframes64_t where = get_preferred_edit_position();
 
@@ -5954,10 +6110,6 @@ Editor::set_track_height (uint32_t h)
 {
        TrackSelection& ts (selection->tracks);
 
-       if (ts.empty()) {
-               return;
-       }
-
        for (TrackSelection::iterator x = ts.begin(); x != ts.end(); ++x) {
                (*x)->set_height (h);
        }
@@ -6067,23 +6219,6 @@ Editor::remove_tracks ()
        }
 }
 
-void
-Editor::set_waveform_scale (WaveformScale ws)
-{
-       TrackSelection& ts (selection->tracks);
-
-       if (ts.empty()) {
-               return;
-       }
-
-       for (TrackSelection::iterator x = ts.begin(); x != ts.end(); ++x) {
-               AudioTimeAxisView* atv = dynamic_cast<AudioTimeAxisView*> (*x);
-               if (atv) {
-                       atv->set_waveform_scale (ws);
-               }
-       }
-}      
-
 void
 Editor::do_insert_time ()
 {
@@ -6091,61 +6226,54 @@ Editor::do_insert_time ()
                return;
        }
 
-       nframes64_t pos = get_preferred_edit_position ();
        ArdourDialog d (*this, _("Insert Time"));
-       VButtonBox button_box;
-       VBox option_box;
-       RadioButtonGroup group;
-       RadioButton leave_button (group, _("Stay in position"));
-       RadioButton move_button (group, _("Move"));
-       RadioButton split_button (group, _("Split & Later Section Moves"));
-       Label intersect_option_label (_("Intersected regions should:"));
-       CheckButton glue_button (_("Move Glued Regions"));
-       CheckButton marker_button (_("Move Markers"));
-       AudioClock clock ("insertTimeClock", true, X_("InsertTimeClock"), true, true, true);
-       HBox clock_box;
-
-       clock.set (0);
-       clock.set_session (session);
-       clock.set_bbt_reference (pos);
 
-       clock_box.pack_start (clock, false, true);
+       nframes64_t const pos = get_preferred_edit_position ();
+       
+       d.get_vbox()->set_border_width (12);
+       d.get_vbox()->set_spacing (4);
 
-       option_box.set_spacing (6);
-       option_box.pack_start (intersect_option_label, false, false);
-       option_box.pack_start (button_box, false, false);
-       option_box.pack_start (glue_button, false, false);
-       option_box.pack_start (marker_button, false, false);
+       Table table (2, 2);
+       table.set_spacings (4);
 
-       button_box.pack_start (leave_button, false, false);
-       button_box.pack_start (move_button, false, false);
-       button_box.pack_start (split_button, false, false);
-                                     
-       d.get_vbox()->set_border_width (12);
-       d.get_vbox()->pack_start (clock_box, false, false);
-       d.get_vbox()->pack_start (option_box, false, false);
+       Label time_label (_("Time to insert:"));
+       time_label.set_alignment (1, 0.5);
+       table.attach (time_label, 0, 1, 0, 1, FILL | EXPAND);
+       AudioClock clock ("insertTimeClock", true, X_("InsertTimeClock"), true, true, true);
+       clock.set (0);
+       clock.set_session (session);
+       clock.set_bbt_reference (pos);  
+       table.attach (clock, 1, 2, 0, 1);
+
+       Label intersected_label (_("Intersected regions should:"));
+       intersected_label.set_alignment (1, 0.5);
+       table.attach (intersected_label, 0, 1, 1, 2, FILL | EXPAND);
+       ComboBoxText intersected_combo;
+       intersected_combo.append_text (_("stay in position"));
+       intersected_combo.append_text (_("move"));
+       intersected_combo.append_text (_("be split"));
+       intersected_combo.set_active (0);
+       table.attach (intersected_combo, 1, 2, 1, 2);
+
+       d.get_vbox()->pack_start (table);
+
+       CheckButton move_glued (_("Move glued regions"));
+       d.get_vbox()->pack_start (move_glued);
+       CheckButton move_markers (_("Move markers"));
+       d.get_vbox()->pack_start (move_markers);
+       CheckButton move_tempos (_("Move tempo and meter changes"));
+       d.get_vbox()->pack_start (move_tempos);
        
-       leave_button.show ();
-       move_button.show ();
-       split_button.show ();
-       intersect_option_label.show ();
-       option_box.show ();
-       button_box.show ();
-       glue_button.show ();
-       clock.show_all();
-       clock_box.show ();
-       marker_button.show ();
-
        d.add_button (Gtk::Stock::CANCEL, Gtk::RESPONSE_CANCEL);
-       d.add_button (Gtk::Stock::OK, Gtk::RESPONSE_OK);
-       d.show ();
+       d.add_button (_("Insert time"), Gtk::RESPONSE_OK);
+       d.show_all ();
 
        int response = d.run ();
 
        if (response != RESPONSE_OK) {
                return;
        }
-       
+
        nframes64_t distance = clock.current_duration (pos);
 
        if (distance == 0) {
@@ -6154,20 +6282,24 @@ Editor::do_insert_time ()
 
        InsertTimeOption opt;
 
-       if (leave_button.get_active()) {
+       switch (intersected_combo.get_active_row_number ()) {
+       case 0:
                opt = LeaveIntersected;
-       } else if (move_button.get_active()) {
+               break;
+       case 1:
                opt = MoveIntersected;
-       } else {
+               break;
+       case 2:
                opt = SplitIntersected;
+               break;
        }
 
-       insert_time (pos, distance, opt, glue_button.get_active(), marker_button.get_active());
+       insert_time (pos, distance, opt, move_glued.get_active(), move_markers.get_active(), move_tempos.get_active());
 }
 
 void
 Editor::insert_time (nframes64_t pos, nframes64_t frames, InsertTimeOption opt, 
-                    bool ignore_music_glue, bool markers_too)
+                    bool ignore_music_glue, bool markers_too, bool tempo_too)
 {
        bool commit = false;
 
@@ -6230,21 +6362,31 @@ Editor::insert_time (nframes64_t pos, nframes64_t frames, InsertTimeOption opt,
                }
        }
 
+       if (tempo_too) {
+               session->tempo_map().insert_time (pos, frames);
+       }
+
        if (commit) {
                commit_reversible_command ();
        }
 }
 
 void
-Editor::fit_tracks ()
+Editor::fit_selected_tracks ()
 {
-       if (selection->tracks.empty()) {
+       fit_tracks (selection->tracks);
+}
+
+void
+Editor::fit_tracks (TrackSelection & tracks)
+{
+       if (tracks.empty()) {
                return;
        }
 
        uint32_t child_heights = 0;
 
-       for (TrackSelection::iterator t = selection->tracks.begin(); t != selection->tracks.end(); ++t) {
+       for (TrackSelection::iterator t = tracks.begin(); t != tracks.end(); ++t) {
 
                if (!(*t)->marked_for_display()) {
                        continue;
@@ -6253,11 +6395,11 @@ Editor::fit_tracks ()
                child_heights += (*t)->effective_height() - (*t)->current_height();
        }
 
-       uint32_t h = (uint32_t) floor ((canvas_height - child_heights - canvas_timebars_vsize)/selection->tracks.size());
+       uint32_t h = (uint32_t) floor ((_canvas_height - child_heights - canvas_timebars_vsize) / tracks.size());
        double first_y_pos = DBL_MAX;
 
        if (h < TimeAxisView::hSmall) {
-               MessageDialog msg (*this, _("There are too many selected tracks to fit in the current window"));
+               MessageDialog msg (*this, _("There are too many tracks to fit in the current window"));
                /* too small to be displayed */
                return;
        }
@@ -6267,7 +6409,7 @@ Editor::fit_tracks ()
        /* operate on all tracks, hide unselected ones that are in the middle of selected ones */
        
        bool prev_was_selected = false;
-       bool is_selected = selection->selected (track_views.front());
+       bool is_selected = tracks.contains (track_views.front());
        bool next_is_selected;
 
        for (TrackViewList::iterator t = track_views.begin(); t != track_views.end(); ++t) {
@@ -6278,7 +6420,7 @@ Editor::fit_tracks ()
                ++next;
                
                if (next != track_views.end()) {
-                       next_is_selected = selection->selected (*next);
+                       next_is_selected = tracks.contains (*next);
                } else {
                        next_is_selected = false;
                }