first part of MIDI cut/copy/paste ; fix for input/output_streams of an IOProcessor...
[ardour.git] / gtk2_ardour / editor_ops.cc
index a9cc7ae7d6d6b0de1c488bbcde9d02f4419cfaf2..0a7543b214a4ae6ce233a47bd7fbd5a06c253a07 100644 (file)
@@ -55,6 +55,7 @@
 #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"
 
@@ -108,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
@@ -200,7 +198,7 @@ Editor::split_regions_at (nframes64_t where, RegionSelection& regions)
 }
 
 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;
@@ -859,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;
@@ -928,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; 
@@ -972,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; 
@@ -1415,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;
@@ -1443,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;
@@ -1482,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);
 }
@@ -1511,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);
 }
@@ -1569,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) {
@@ -1712,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) {
@@ -1738,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 */
                
@@ -1747,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()) {
@@ -1756,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;
@@ -1803,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;
 
@@ -1813,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));
 
@@ -2318,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 */
@@ -2594,7 +2579,7 @@ Editor::rename_region()
                strip_whitespace_edges (str);
                if (!str.empty()) {
                        rs.front()->region()->set_name (str);
-                       redisplay_regions ();
+                       _regions->redisplay ();
                }
        }
 }
@@ -2708,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;
@@ -2759,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));
        }
@@ -2801,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) {
@@ -2910,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 ()
 {
@@ -3118,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;
 
@@ -3704,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);
@@ -3882,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;
                }
        }
        
@@ -3902,75 +3900,92 @@ Editor::cut_copy (CutCopyOp op)
                }
 
                break_drag ();
+               delete _drag;
+               _drag = 0;
 
                return;
        }
 
-       RegionSelection rs; 
-
-       /* we only want to cut regions if some are selected */
-
-       if (!selection->regions.empty()) {
-               get_regions_for_action (rs);
-       }
+       if (internal_editing()) {
 
-       switch (current_mouse_mode()) {
-       case MouseObject: 
-               if (!rs.empty() || !selection->points.empty()) {
+               switch (current_mouse_mode()) {
+               case MouseObject:
+               case MouseRange:
+                       cut_copy_midi (op);
+                       break;
+               default:
+                       break;
+               }
+       } else {
+               
+               RegionSelection rs; 
+               
+               /* we only want to cut regions if some are selected */
+               
+               if (!selection->regions.empty()) {
+                       get_regions_for_action (rs);
+               }
+               
+               switch (current_mouse_mode()) {
+               case MouseObject: 
+                       if (!rs.empty() || !selection->points.empty()) {
 
-                       begin_reversible_command (opname + _(" objects"));
+                               begin_reversible_command (opname + _(" objects"));
 
-                       if (!rs.empty()) {
-                               cut_copy_regions (op, rs);
+                               if (!rs.empty()) {
+                                       cut_copy_regions (op, rs);
                                
-                               if (op == Cut) {
-                                       selection->clear_regions ();
+                                       if (op == Cut) {
+                                               selection->clear_regions ();
+                                       }
                                }
-                       }
 
-                       if (!selection->points.empty()) {
-                               cut_copy_points (op);
+                               if (!selection->points.empty()) {
+                                       cut_copy_points (op);
 
-                               if (op == Cut) {
-                                       selection->clear_points ();
+                                       if (op == Cut) {
+                                               selection->clear_points ();
+                                       }
                                }
-                       }
 
-                       commit_reversible_command ();   
-                       break; // terminate case statement here
-               } 
-               if (!selection->time.empty()) {
-                       /* don't cause suprises */
-                       break;
-               }
-               // fall thru if there was nothing selected
+                               commit_reversible_command ();   
+                               break; // terminate case statement here
+                       
+                       if (!selection->time.empty()) {
+                               /* don't cause suprises */
+                               break;
+                       }
+                       // fall thru if there was nothing selected
                
-       case MouseRange:
-               if (selection->time.empty()) {
-                       nframes64_t start, end;
-                       if (!get_edit_op_range (start, end)) {
-                               return;
+               case MouseRange:
+                       if (selection->time.empty()) {
+                               nframes64_t start, end;
+                               if (!get_edit_op_range (start, end)) {
+                                       return;
+                               }
+                               selection->set ((TimeAxisView*) 0, start, end);
                        }
-                       selection->set ((TimeAxisView*) 0, start, end);
-               }
                        
-               begin_reversible_command (opname + _(" range"));
-               cut_copy_ranges (op);
-               commit_reversible_command ();
+                       begin_reversible_command (opname + _(" range"));
+                       cut_copy_ranges (op);
+                       commit_reversible_command ();
                
-               if (op == Cut) {
-                       selection->clear_time ();
-               }
+                       if (op == Cut) {
+                               selection->clear_time ();
+                       }
 
-               break;
+                       break;
                
-       default:
-               break;
+               default:
+                       break;
+               }
        }
-
-
+               
        if (op == Cut || op == Clear) {
                break_drag ();
+               delete _drag;
+               _drag = 0;
        }
 }
 
@@ -3990,6 +4005,20 @@ Editor::cut_copy_points (CutCopyOp op)
        }
 }
 
+/** Cut, copy or clear selected automation points.
+ * @param op Operation (Cut, Copy or Clear)
+ */
+void
+Editor::cut_copy_midi (CutCopyOp op)
+{
+       cerr << "CCM: there are " << selection->midi.size() << " MRV's to work on\n";
+
+       for (MidiSelection::iterator i = selection->midi.begin(); i != selection->midi.end(); ++i) {
+               MidiRegionView* mrv = *i;
+               mrv->cut_copy_clear (op);
+       }
+}
+
 struct PlaylistState {
     boost::shared_ptr<Playlist> playlist;
     XMLNode*  before;
@@ -4305,7 +4334,6 @@ Editor::paste_internal (nframes64_t position, float times)
 
        /* get everything in the correct order */
 
-
        if (!selection->tracks.empty()) {
                sort_track_selection ();
                ts = selection->tracks;
@@ -4477,14 +4505,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);
 }
 
@@ -4569,17 +4597,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"));
 
@@ -4591,12 +4637,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 ();
 }
 
 
@@ -4867,9 +4915,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;
@@ -4894,80 +4940,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
@@ -4997,20 +5097,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;
@@ -5386,7 +5482,7 @@ Editor::split ()
 {
        RegionSelection rs; 
        
-       get_regions_for_action (rs);
+       get_regions_for_action (rs, true);
 
        nframes64_t where = get_preferred_edit_position();
 
@@ -6040,10 +6136,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);
        }
@@ -6153,23 +6245,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 ()
 {
@@ -6177,61 +6252,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) {
@@ -6240,20 +6308,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;
 
@@ -6316,21 +6388,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;
@@ -6339,11 +6421,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;
        }
@@ -6353,7 +6435,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) {
@@ -6364,7 +6446,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;
                }