Fix placement of automation child tracks.
[ardour.git] / gtk2_ardour / editor_mouse.cc
index 048277fea7ab09bae5d7ff35cb7d649a7279a4e6..d0931e4579bdbf416eb5350fcbf09e1d76541ed4 100644 (file)
@@ -1,3 +1,4 @@
+
 /*
     Copyright (C) 2000-2001 Paul Davis 
 
@@ -82,29 +83,25 @@ Editor::mouse_frame (nframes64_t& where, bool& in_track_canvas) const
        int x, y;
        double wx, wy;
        Gdk::ModifierType mask;
-       Glib::RefPtr<Gdk::Window> canvas_window = const_cast<Editor*>(this)->track_canvas.get_window();
+       Glib::RefPtr<Gdk::Window> canvas_window = const_cast<Editor*>(this)->track_canvas->get_window();
        Glib::RefPtr<const Gdk::Window> pointer_window;
+
+       if (!canvas_window) {
+               return false;
+       }
        
        pointer_window = canvas_window->get_pointer (x, y, mask);
 
-       if (pointer_window == track_canvas.get_bin_window()) {
-
-               track_canvas.window_to_world (x, y, wx, wy);
+       if (pointer_window == track_canvas->get_bin_window()) {
+               wx = x;
+               wy = y;
                in_track_canvas = true;
 
        } else {
                in_track_canvas = false;
-
-               if (pointer_window == time_canvas.get_bin_window()) {
-                       time_canvas.window_to_world (x, y, wx, wy);
-               } else {
                        return false;
-               }
        }
 
-       wx += horizontal_adjustment.get_value();
-       wy += vertical_adjustment.get_value();
-
        GdkEvent event;
        event.type = GDK_BUTTON_RELEASE;
        event.button.x = wx;
@@ -134,18 +131,24 @@ Editor::event_frame (GdkEvent* event, double* pcx, double* pcy) const
        case GDK_BUTTON_PRESS:
        case GDK_2BUTTON_PRESS:
        case GDK_3BUTTON_PRESS:
-               track_canvas.w2c(event->button.x, event->button.y, *pcx, *pcy);
+
+               *pcx = event->button.x;
+               *pcy = event->button.y;
+               _trackview_group->w2i(*pcx, *pcy);
                break;
        case GDK_MOTION_NOTIFY:
-               track_canvas.w2c(event->motion.x, event->motion.y, *pcx, *pcy);
+       
+               *pcx = event->motion.x;
+               *pcy = event->motion.y;
+               _trackview_group->w2i(*pcx, *pcy);
                break;
        case GDK_ENTER_NOTIFY:
        case GDK_LEAVE_NOTIFY:
-               track_canvas.w2c(event->crossing.x, event->crossing.y, *pcx, *pcy);
+               track_canvas->w2c(event->crossing.x, event->crossing.y, *pcx, *pcy);
                break;
        case GDK_KEY_PRESS:
        case GDK_KEY_RELEASE:
-               // track_canvas.w2c(event->key.x, event->key.y, *pcx, *pcy);
+               // track_canvas->w2c(event->key.x, event->key.y, *pcx, *pcy);
                break;
        default:
                warning << string_compose (_("Editor::event_frame() used on unhandled event type %1"), event->type) << endmsg;
@@ -215,6 +218,57 @@ Editor::mouse_mode_toggled (MouseMode m)
        }
 }      
 
+Gdk::Cursor*
+Editor::which_grabber_cursor ()
+{
+       switch (_edit_point) {
+       case EditAtMouse:
+               return grabber_edit_point_cursor;
+               break;
+       default:
+               break;
+       }
+       return grabber_cursor;
+}
+
+void
+Editor::set_canvas_cursor ()
+{
+       switch (mouse_mode) {
+       case MouseRange:
+               current_canvas_cursor = selector_cursor;
+               break;
+
+       case MouseObject:
+               current_canvas_cursor = which_grabber_cursor();
+               break;
+
+       case MouseGain:
+               current_canvas_cursor = cross_hair_cursor;
+               break;
+
+       case MouseZoom:
+               current_canvas_cursor = zoom_cursor;
+               break;
+
+       case MouseTimeFX:
+               current_canvas_cursor = time_fx_cursor; // just use playhead
+               break;
+
+       case MouseAudition:
+               current_canvas_cursor = speaker_cursor;
+               break;
+       
+       case MouseNote:
+               set_midi_edit_cursor (current_midi_edit_mode());
+               break;
+       }
+
+       if (is_drawable()) {
+               track_canvas->get_window()->set_cursor(*current_canvas_cursor);
+       }
+}
+
 void
 Editor::set_mouse_mode (MouseMode m, bool force)
 {
@@ -266,36 +320,26 @@ Editor::set_mouse_mode (MouseMode m, bool force)
        switch (mouse_mode) {
        case MouseRange:
                mouse_select_button.set_active (true);
-               current_canvas_cursor = selector_cursor;
                break;
 
        case MouseObject:
                mouse_move_button.set_active (true);
-               if (Profile->get_sae()) {
-                       current_canvas_cursor = timebar_cursor;
-               } else {
-                       current_canvas_cursor = grabber_cursor;
-               }
                break;
 
        case MouseGain:
                mouse_gain_button.set_active (true);
-               current_canvas_cursor = cross_hair_cursor;
                break;
 
        case MouseZoom:
                mouse_zoom_button.set_active (true);
-               current_canvas_cursor = zoom_cursor;
                break;
 
        case MouseTimeFX:
                mouse_timefx_button.set_active (true);
-               current_canvas_cursor = time_fx_cursor; // just use playhead
                break;
 
        case MouseAudition:
                mouse_audition_button.set_active (true);
-               current_canvas_cursor = speaker_cursor;
                break;
        
        case MouseNote:
@@ -310,10 +354,8 @@ Editor::set_mouse_mode (MouseMode m, bool force)
                midi_toolbar_frame.hide();
 
        ignore_mouse_mode_toggle = false;
-
-       if (is_drawable()) {
-               track_canvas.get_window()->set_cursor(*current_canvas_cursor);
-       }
+       
+       set_canvas_cursor ();
 }
 
 void
@@ -375,6 +417,11 @@ Editor::midi_edit_mode_toggled (MidiEditMode m)
                        set_midi_edit_mode (m);
                break;
 
+       case MidiEditResize:
+               if (midi_tool_resize_button.get_active())
+                       set_midi_edit_mode (m);
+               break;
+
        case MidiEditErase:
                if (midi_tool_erase_button.get_active())
                        set_midi_edit_mode (m);
@@ -414,6 +461,10 @@ Editor::set_midi_edit_mode (MidiEditMode m, bool force)
                midi_tool_select_button.set_active (true);
                break;
 
+       case MidiEditResize:
+               midi_tool_resize_button.set_active (true);
+               break;
+
        case MidiEditErase:
                midi_tool_erase_button.set_active (true);
                break;
@@ -424,7 +475,7 @@ Editor::set_midi_edit_mode (MidiEditMode m, bool force)
        set_midi_edit_cursor (current_midi_edit_mode());
 
        if (is_drawable()) {
-               track_canvas.get_window()->set_cursor(*current_canvas_cursor);
+               track_canvas->get_window()->set_cursor(*current_canvas_cursor);
        }
 }
 
@@ -440,6 +491,10 @@ Editor::set_midi_edit_cursor (MidiEditMode m)
                current_canvas_cursor = midi_select_cursor;
                break;
 
+       case MidiEditResize:
+               current_canvas_cursor = midi_resize_cursor;
+               break;
+
        case MidiEditErase:
                current_canvas_cursor = midi_erase_cursor;
                break;
@@ -462,7 +517,7 @@ Editor::button_selection (ArdourCanvas::Item* item, GdkEvent* event, ItemType it
             (mouse_mode != MouseTimeFX || item_type != RegionItem) &&
             (mouse_mode != MouseRange)) ||
 
-           (event->type != GDK_BUTTON_PRESS && event->type != GDK_BUTTON_RELEASE || event->button.button > 3)) {
+           ((event->type != GDK_BUTTON_PRESS && event->type != GDK_BUTTON_RELEASE) || event->button.button > 3)) {
                
                return;
        }
@@ -542,7 +597,25 @@ Editor::button_selection (ArdourCanvas::Item* item, GdkEvent* event, ItemType it
 bool
 Editor::button_press_handler (ArdourCanvas::Item* item, GdkEvent* event, ItemType item_type)
 {
-       track_canvas.grab_focus();
+       Glib::RefPtr<Gdk::Window> canvas_window = const_cast<Editor*>(this)->track_canvas->get_window();
+       
+       if (canvas_window) {
+               Glib::RefPtr<const Gdk::Window> pointer_window;
+               int x, y;
+               double wx, wy;
+               Gdk::ModifierType mask;
+
+               pointer_window = canvas_window->get_pointer (x, y, mask);
+               
+               if (pointer_window == track_canvas->get_bin_window()) {
+                       track_canvas->window_to_world (x, y, wx, wy);
+                       allow_vertical_scroll = true;
+               } else {
+                       allow_vertical_scroll = false;
+               }
+       }
+
+       track_canvas->grab_focus();
 
        if (session && session->actively_recording()) {
                return true;
@@ -695,8 +768,8 @@ Editor::button_press_handler (ArdourCanvas::Item* item, GdkEvent* event, ItemTyp
                                        
                                case RegionViewName:
                                        /* rename happens on edit clicks */
-                                               start_trim (clicked_regionview->get_name_highlight(), event);
-                                               return true;
+                                       start_trim (clicked_regionview->get_name_highlight(), event);
+                                       return true;
                                        break;
 
                                case ControlPointItem:
@@ -812,7 +885,7 @@ Editor::button_press_handler (ArdourCanvas::Item* item, GdkEvent* event, ItemTyp
                        scrub_reverse_distance = 0;
                        last_scrub_x = event->button.x;
                        scrubbing_direction = 0;
-                       track_canvas.get_window()->set_cursor (*transparent_cursor);
+                       track_canvas->get_window()->set_cursor (*transparent_cursor);
                        /* rest handled in motion & release */
                        break;
 
@@ -903,7 +976,8 @@ Editor::button_press_handler (ArdourCanvas::Item* item, GdkEvent* event, ItemTyp
 bool
 Editor::button_release_handler (ArdourCanvas::Item* item, GdkEvent* event, ItemType item_type)
 {
-       nframes_t where = event_frame (event, 0, 0);
+       nframes64_t where = event_frame (event, 0, 0);
+       AutomationTimeAxisView* atv = 0;
 
        /* no action if we're recording */
                                                
@@ -1119,14 +1193,14 @@ Editor::button_release_handler (ArdourCanvas::Item* item, GdkEvent* event, ItemT
                case MouseObject:
                        switch (item_type) {
                        case AutomationTrackItem:
-                               dynamic_cast<AutomationTimeAxisView*>(clicked_axisview)->add_automation_event 
-                                       (item,
-                                        event,
-                                        where,
-                                        event->button.y);
+                               atv = dynamic_cast<AutomationTimeAxisView*>(clicked_routeview);
+                               if (atv) {
+                                       atv->add_automation_event (item, event, where, event->button.y);
+                               }
                                return true;
+                               
                                break;
-
+                               
                        default:
                                break;
                        }
@@ -1157,7 +1231,7 @@ Editor::button_release_handler (ArdourCanvas::Item* item, GdkEvent* event, ItemT
                        
                case MouseAudition:
                        _scrubbing = false;
-                       track_canvas.get_window()->set_cursor (*current_canvas_cursor);
+                       track_canvas->get_window()->set_cursor (*current_canvas_cursor);
                        if (scrubbing_direction == 0) {
                                /* no drag, just a click */
                                switch (item_type) {
@@ -1232,6 +1306,11 @@ Editor::enter_handler (ArdourCanvas::Item* item, GdkEvent* event, ItemType item_
        Marker * marker;
        double fraction;
        
+       if (last_item_entered != item) {
+               last_item_entered = item;
+               last_item_entered_n = 0;
+       }
+
        switch (item_type) {
        case ControlPointItem:
                if (mouse_mode == MouseGain || mouse_mode == MouseObject) {
@@ -1247,11 +1326,14 @@ Editor::enter_handler (ArdourCanvas::Item* item, GdkEvent* event, ItemType item_
 
                        fraction = 1.0 - (cp->get_y() / cp->line().height());
 
-                       set_verbose_canvas_cursor (cp->line().get_verbose_cursor_string (fraction), at_x, at_y);
-                       show_verbose_canvas_cursor ();
-
                        if (is_drawable() && !_scrubbing) {
-                               track_canvas.get_window()->set_cursor (*fader_cursor);
+                               track_canvas->get_window()->set_cursor (*fader_cursor);
+                       }
+
+                       last_item_entered_n++;
+                       set_verbose_canvas_cursor (cp->line().get_verbose_cursor_string (fraction), at_x, at_y);
+                       if (last_item_entered_n < 10) {
+                               show_verbose_canvas_cursor ();
                        }
                }
                break;
@@ -1262,7 +1344,7 @@ Editor::enter_handler (ArdourCanvas::Item* item, GdkEvent* event, ItemType item_
                        if (line)
                                line->property_fill_color_rgba() = ARDOUR_UI::config()->canvasvar_EnteredGainLine.get();
                        if (is_drawable()) {
-                               track_canvas.get_window()->set_cursor (*fader_cursor);
+                               track_canvas->get_window()->set_cursor (*fader_cursor);
                        }
                }
                break;
@@ -1275,14 +1357,14 @@ Editor::enter_handler (ArdourCanvas::Item* item, GdkEvent* event, ItemType item_
                                        line->property_fill_color_rgba() = ARDOUR_UI::config()->canvasvar_EnteredAutomationLine.get();
                        }
                        if (is_drawable()) {
-                               track_canvas.get_window()->set_cursor (*fader_cursor);
+                               track_canvas->get_window()->set_cursor (*fader_cursor);
                        }
                }
                break;
                
        case RegionViewNameHighlight:
                if (is_drawable() && mouse_mode == MouseObject) {
-                       track_canvas.get_window()->set_cursor (*trimmer_cursor);
+                       track_canvas->get_window()->set_cursor (*trimmer_cursor);
                }
                break;
 
@@ -1297,13 +1379,20 @@ Editor::enter_handler (ArdourCanvas::Item* item, GdkEvent* event, ItemType item_
 #endif
 
                if (is_drawable()) {
-                       track_canvas.get_window()->set_cursor (*trimmer_cursor);
+                       track_canvas->get_window()->set_cursor (*trimmer_cursor);
                }
                break;
 
        case PlayheadCursorItem:
                if (is_drawable()) {
-                       track_canvas.get_window()->set_cursor (*grabber_cursor);
+                       switch (_edit_point) {
+                       case EditAtMouse:
+                               track_canvas->get_window()->set_cursor (*grabber_edit_point_cursor);
+                               break;
+                       default:
+                               track_canvas->get_window()->set_cursor (*grabber_cursor);
+                               break;
+                       }
                }
                break;
 
@@ -1313,7 +1402,7 @@ Editor::enter_handler (ArdourCanvas::Item* item, GdkEvent* event, ItemType item_
 
                if (!reinterpret_cast<RegionView *> (item->get_data ("regionview"))->name_active()) {
                        if (mouse_mode == MouseObject && is_drawable()) {
-                               track_canvas.get_window()->set_cursor (*trimmer_cursor);
+                               track_canvas->get_window()->set_cursor (*trimmer_cursor);
                        }
                } 
                break;
@@ -1334,7 +1423,7 @@ Editor::enter_handler (ArdourCanvas::Item* item, GdkEvent* event, ItemType item_
                                break;
                        }
 
-                       track_canvas.get_window()->set_cursor (*cursor);
+                       track_canvas->get_window()->set_cursor (*cursor);
 
                        AutomationTimeAxisView* atv;
                        if ((atv = static_cast<AutomationTimeAxisView*>(item->get_data ("trackview"))) != 0) {
@@ -1351,7 +1440,7 @@ Editor::enter_handler (ArdourCanvas::Item* item, GdkEvent* event, ItemType item_
        case MeterBarItem:
        case TempoBarItem:
                if (is_drawable()) {
-                       time_canvas.get_window()->set_cursor (*timebar_cursor);
+                       track_canvas->get_window()->set_cursor (*timebar_cursor);
                }
                break;
 
@@ -1365,7 +1454,7 @@ Editor::enter_handler (ArdourCanvas::Item* item, GdkEvent* event, ItemType item_
        case MeterMarkerItem:
        case TempoMarkerItem:
                if (is_drawable()) {
-                       time_canvas.get_window()->set_cursor (*timebar_cursor);
+                       track_canvas->get_window()->set_cursor (*timebar_cursor);
                }
                break;
        case FadeInHandleItem:
@@ -1426,7 +1515,7 @@ Editor::leave_handler (ArdourCanvas::Item* item, GdkEvent* event, ItemType item_
                }
                
                if (is_drawable()) {
-                       track_canvas.get_window()->set_cursor (*current_canvas_cursor);
+                       track_canvas->get_window()->set_cursor (*current_canvas_cursor);
                }
 
                hide_verbose_canvas_cursor ();
@@ -1445,7 +1534,7 @@ Editor::leave_handler (ArdourCanvas::Item* item, GdkEvent* event, ItemType item_
 #endif
 
                if (is_drawable()) {
-                       track_canvas.get_window()->set_cursor (*current_canvas_cursor);
+                       track_canvas->get_window()->set_cursor (*current_canvas_cursor);
                }
                break;
 
@@ -1458,7 +1547,7 @@ Editor::leave_handler (ArdourCanvas::Item* item, GdkEvent* event, ItemType item_
                                line->property_fill_color_rgba() = al->get_line_color();
                }
                if (is_drawable()) {
-                       track_canvas.get_window()->set_cursor (*current_canvas_cursor);
+                       track_canvas->get_window()->set_cursor (*current_canvas_cursor);
                }
                break;
 
@@ -1466,7 +1555,7 @@ Editor::leave_handler (ArdourCanvas::Item* item, GdkEvent* event, ItemType item_
                /* see enter_handler() for notes */
                if (!reinterpret_cast<RegionView *> (item->get_data ("regionview"))->name_active()) {
                        if (is_drawable() && mouse_mode == MouseObject) {
-                               track_canvas.get_window()->set_cursor (*current_canvas_cursor);
+                               track_canvas->get_window()->set_cursor (*current_canvas_cursor);
                        }
                }
                break;
@@ -1478,7 +1567,7 @@ Editor::leave_handler (ArdourCanvas::Item* item, GdkEvent* event, ItemType item_
        case TempoBarItem:
        case MarkerBarItem:
                if (is_drawable()) {
-                       time_canvas.get_window()->set_cursor (*timebar_cursor);
+                       track_canvas->get_window()->set_cursor (*timebar_cursor);
                }
                break;
                
@@ -1495,7 +1584,7 @@ Editor::leave_handler (ArdourCanvas::Item* item, GdkEvent* event, ItemType item_
        case TempoMarkerItem:
                
                if (is_drawable()) {
-                       time_canvas.get_window()->set_cursor (*timebar_cursor);
+                       track_canvas->get_window()->set_cursor (*timebar_cursor);
                }
 
                break;
@@ -1514,7 +1603,7 @@ Editor::leave_handler (ArdourCanvas::Item* item, GdkEvent* event, ItemType item_
 
        case AutomationTrackItem:
                if (is_drawable()) {
-                       track_canvas.get_window()->set_cursor (*current_canvas_cursor);
+                       track_canvas->get_window()->set_cursor (*current_canvas_cursor);
                        clear_entered_track = true;
                        Glib::signal_idle().connect (mem_fun(*this, &Editor::left_automation_track));
                }
@@ -1537,6 +1626,85 @@ Editor::left_automation_track ()
        return false;
 }
 
+void
+Editor::scrub ()
+{
+       double delta;
+       
+       if (scrubbing_direction == 0) {
+               /* first move */
+               session->request_locate (drag_info.current_pointer_frame, false);
+               session->request_transport_speed (0.1);
+               scrubbing_direction = 1;
+               
+       } else {
+               
+               if (last_scrub_x > drag_info.current_pointer_x) {
+                       
+                       /* pointer moved to the left */
+                       
+                       if (scrubbing_direction > 0) {
+                               
+                               /* we reversed direction to go backwards */
+                               
+                               scrub_reversals++;
+                               scrub_reverse_distance += (int) (last_scrub_x - drag_info.current_pointer_x);
+                               
+                       } else {
+                               
+                               /* still moving to the left (backwards) */
+                               
+                               scrub_reversals = 0;
+                               scrub_reverse_distance = 0;
+                               
+                               delta = 0.01 * (last_scrub_x - drag_info.current_pointer_x);
+                               session->request_transport_speed (session->transport_speed() - delta);
+                       }
+                       
+               } else {
+                       /* pointer moved to the right */
+                       
+                       if (scrubbing_direction < 0) {
+                               /* we reversed direction to go forward */
+                               
+                               scrub_reversals++;
+                               scrub_reverse_distance += (int) (drag_info.current_pointer_x - last_scrub_x);
+                               
+                       } else {
+                               /* still moving to the right */
+                               
+                               scrub_reversals = 0;
+                               scrub_reverse_distance = 0;
+                               
+                               delta = 0.01 * (drag_info.current_pointer_x - last_scrub_x);
+                               session->request_transport_speed (session->transport_speed() + delta);
+                       }
+               }
+               
+               /* if there have been more than 2 opposite motion moves detected, or one that moves
+                  back more than 10 pixels, reverse direction
+               */
+               
+               if (scrub_reversals >= 2 || scrub_reverse_distance > 10) {
+                       
+                       if (scrubbing_direction > 0) {
+                               /* was forwards, go backwards */
+                               session->request_transport_speed (-0.1);
+                               scrubbing_direction = -1;
+                       } else {
+                               /* was backwards, go forwards */
+                               session->request_transport_speed (0.1);
+                               scrubbing_direction = 1;
+                       }
+                       
+                       scrub_reverse_distance = 0;
+                       scrub_reversals = 0;
+               }
+       }
+       
+       last_scrub_x = drag_info.current_pointer_x;
+}
+
 bool
 Editor::motion_handler (ArdourCanvas::Item* item, GdkEvent* event, ItemType item_type, bool from_autoscroll)
 {
@@ -1551,10 +1719,10 @@ Editor::motion_handler (ArdourCanvas::Item* item, GdkEvent* event, ItemType item
                   where DISPLAY = :0.0, and given the cost of what the motion
                   event might do, its a good tradeoff.  
                */
-               
-               track_canvas.get_pointer (x, y);
-       }
-       
+
+               track_canvas->get_pointer (x, y);
+       } 
+
        if (current_stepping_trackview) {
                /* don't keep the persistent stepped trackview if the mouse moves */
                current_stepping_trackview = 0;
@@ -1572,85 +1740,13 @@ Editor::motion_handler (ArdourCanvas::Item* item, GdkEvent* event, ItemType item
        drag_info.current_pointer_frame = event_frame (event, &drag_info.current_pointer_x,
                                                       &drag_info.current_pointer_y);
 
+       
        switch (mouse_mode) {
        case MouseAudition:
                if (_scrubbing) {
-
-                       double delta;
-
-                       if (scrubbing_direction == 0) {
-                               /* first move */
-                               session->request_locate (drag_info.current_pointer_frame, false);
-                               session->request_transport_speed (0.1);
-                               scrubbing_direction = 1;
-
-                       } else {
-                               
-                               if (last_scrub_x > drag_info.current_pointer_x) {
-
-                                       /* pointer moved to the left */
-                                       
-                                       if (scrubbing_direction > 0) {
-
-                                               /* we reversed direction to go backwards */
-
-                                               scrub_reversals++;
-                                               scrub_reverse_distance += (int) (last_scrub_x - drag_info.current_pointer_x);
-
-                                       } else {
-
-                                               /* still moving to the left (backwards) */
-                                               
-                                               scrub_reversals = 0;
-                                               scrub_reverse_distance = 0;
-
-                                               delta = 0.01 * (last_scrub_x - drag_info.current_pointer_x);
-                                               session->request_transport_speed (session->transport_speed() - delta);
-                                       }
-                                       
-                               } else {
-                                       /* pointer moved to the right */
-
-                                       if (scrubbing_direction < 0) {
-                                               /* we reversed direction to go forward */
-
-                                               scrub_reversals++;
-                                               scrub_reverse_distance += (int) (drag_info.current_pointer_x - last_scrub_x);
-
-                                       } else {
-                                               /* still moving to the right */
-
-                                               scrub_reversals = 0;
-                                               scrub_reverse_distance = 0;
-                                               
-                                               delta = 0.01 * (drag_info.current_pointer_x - last_scrub_x);
-                                               session->request_transport_speed (session->transport_speed() + delta);
-                                       }
-                               }
-
-                               /* if there have been more than 2 opposite motion moves detected, or one that moves
-                                  back more than 10 pixels, reverse direction
-                               */
-
-                               if (scrub_reversals >= 2 || scrub_reverse_distance > 10) {
-
-                                       if (scrubbing_direction > 0) {
-                                               /* was forwards, go backwards */
-                                               session->request_transport_speed (-0.1);
-                                               scrubbing_direction = -1;
-                                       } else {
-                                               /* was backwards, go forwards */
-                                               session->request_transport_speed (0.1);
-                                               scrubbing_direction = 1;
-                                       }
-                                       
-                                       scrub_reverse_distance = 0;
-                                       scrub_reversals = 0;
-                               }
-                       }
-
-                       last_scrub_x = drag_info.current_pointer_x;
+                       scrub ();
                }
+               break;
 
        default:
                break;
@@ -1683,6 +1779,9 @@ Editor::motion_handler (ArdourCanvas::Item* item, GdkEvent* event, ItemType item
        case PlayheadCursorItem:
        case MarkerItem:
        case ControlPointItem:
+       case RangeMarkerBarItem:
+       case TransportMarkerBarItem:
+       case CdMarkerBarItem:
        case TempoMarkerItem:
        case MeterMarkerItem:
        case RegionViewNameHighlight:
@@ -1704,13 +1803,13 @@ Editor::motion_handler (ArdourCanvas::Item* item, GdkEvent* event, ItemType item
          if (drag_info.item && (event->motion.state & Gdk::BUTTON1_MASK ||
                                 (event->motion.state & Gdk::BUTTON2_MASK))) {
                  if (!from_autoscroll) {
-                         maybe_autoscroll (event);
+                         maybe_autoscroll_horizontally (&event->motion);
                  }
                  (this->*(drag_info.motion_callback)) (item, event);
                  goto handled;
          }
          goto not_handled;
-         
+         break;
        default:
                break;
        }
@@ -1724,7 +1823,7 @@ Editor::motion_handler (ArdourCanvas::Item* item, GdkEvent* event, ItemType item
                if (drag_info.item && (event->motion.state & GDK_BUTTON1_MASK ||
                                       (event->motion.state & GDK_BUTTON2_MASK))) {
                        if (!from_autoscroll) {
-                               maybe_autoscroll (event);
+                               maybe_autoscroll (&event->motion);
                        }
                        (this->*(drag_info.motion_callback)) (item, event);
                        goto handled;
@@ -1745,6 +1844,48 @@ Editor::motion_handler (ArdourCanvas::Item* item, GdkEvent* event, ItemType item
        return false;
 }
 
+void
+Editor::break_drag ()
+{
+       stop_canvas_autoscroll ();
+       hide_verbose_canvas_cursor ();
+
+       if (drag_info.item) {
+               drag_info.item->ungrab (0);
+
+               /* put it back where it came from */
+
+               double cxw, cyw;
+               cxw = 0;
+               cyw = 0;
+               drag_info.item->i2w (cxw, cyw);
+               drag_info.item->move (drag_info.original_x - cxw, drag_info.original_y - cyw);
+       }
+
+       finalize_drag ();
+}
+
+void
+Editor::finalize_drag ()
+{
+       drag_info.item = 0;
+       drag_info.copy = false;
+       drag_info.motion_callback = 0;
+       drag_info.finished_callback = 0;
+       drag_info.dest_trackview = 0;
+       drag_info.source_trackview = 0;
+       drag_info.last_frame_position = 0;
+       drag_info.grab_frame = 0;
+       drag_info.last_pointer_frame = 0;
+       drag_info.current_pointer_frame = 0;
+       drag_info.brushing = false;
+
+       if (drag_info.copied_location) {
+               delete drag_info.copied_location;
+               drag_info.copied_location = 0;
+       }
+}
+
 void
 Editor::start_grab (GdkEvent* event, Gdk::Cursor *cursor)
 {
@@ -1755,7 +1896,7 @@ Editor::start_grab (GdkEvent* event, Gdk::Cursor *cursor)
        }
 
        if (cursor == 0) {
-               cursor = grabber_cursor;
+               cursor = which_grabber_cursor ();
        }
 
        // if dragging with button2, the motion is x constrained, with Alt-button2 it is y constrained
@@ -1789,6 +1930,10 @@ Editor::start_grab (GdkEvent* event, Gdk::Cursor *cursor)
        drag_info.brushing = false;
        drag_info.copied_location = 0;
 
+       drag_info.original_x = 0;
+       drag_info.original_y = 0;
+       drag_info.item->i2w (drag_info.original_x, drag_info.original_y);
+
        drag_info.item->grab (Gdk::POINTER_MOTION_MASK|Gdk::BUTTON_PRESS_MASK|Gdk::BUTTON_RELEASE_MASK,
                              *cursor,
                              event->button.time);
@@ -1818,7 +1963,7 @@ Editor::swap_grab (ArdourCanvas::Item* new_item, Gdk::Cursor* cursor, uint32_t t
        drag_info.item = new_item;
 
        if (cursor == 0) {
-               cursor = grabber_cursor;
+               cursor = which_grabber_cursor ();
        }
 
        drag_info.item->grab (Gdk::POINTER_MOTION_MASK|Gdk::BUTTON_PRESS_MASK|Gdk::BUTTON_RELEASE_MASK, *cursor, time);
@@ -1847,21 +1992,7 @@ Editor::end_grab (ArdourCanvas::Item* item, GdkEvent* event)
 
        hide_verbose_canvas_cursor();
 
-       drag_info.item = 0;
-       drag_info.copy = false;
-       drag_info.motion_callback = 0;
-       drag_info.finished_callback = 0;
-       drag_info.last_trackview = 0;
-       drag_info.last_frame_position = 0;
-       drag_info.grab_frame = 0;
-       drag_info.last_pointer_frame = 0;
-       drag_info.current_pointer_frame = 0;
-       drag_info.brushing = false;
-
-       if (drag_info.copied_location) {
-               delete drag_info.copied_location;
-               drag_info.copied_location = 0;
-       }
+       finalize_drag ();
 
        return did_drag;
 }
@@ -1882,15 +2013,16 @@ Editor::start_fade_in_grab (ArdourCanvas::Item* item, GdkEvent* event)
 
        AudioRegionView* arv = static_cast<AudioRegionView*>(drag_info.data);
 
-       drag_info.pointer_frame_offset = drag_info.grab_frame - ((nframes_t) arv->audio_region()->fade_in()->back()->when + arv->region()->position()); 
+
+       drag_info.pointer_frame_offset = drag_info.grab_frame - ((nframes64_t) arv->audio_region()->fade_in()->back()->when + arv->region()->position());       
 }
 
 void
 Editor::fade_in_drag_motion_callback (ArdourCanvas::Item* item, GdkEvent* event)
 {
        AudioRegionView* arv = static_cast<AudioRegionView*>(drag_info.data);
-       nframes_t pos;
-       nframes_t fade_length;
+       nframes64_t pos;
+       nframes64_t fade_length;
 
        if (drag_info.current_pointer_frame > drag_info.pointer_frame_offset) {
                pos = drag_info.current_pointer_frame - drag_info.pointer_frame_offset;
@@ -1932,8 +2064,8 @@ void
 Editor::fade_in_drag_finished_callback (ArdourCanvas::Item* item, GdkEvent* event)
 {
        AudioRegionView* arv = static_cast<AudioRegionView*>(drag_info.data);
-       nframes_t pos;
-       nframes_t fade_length;
+       nframes64_t pos;
+       nframes64_t fade_length;
 
        if (drag_info.first_move) return;
 
@@ -1965,6 +2097,7 @@ Editor::fade_in_drag_finished_callback (ArdourCanvas::Item* item, GdkEvent* even
                XMLNode &before = alist->get_state();
 
                tmp->audio_region()->set_fade_in_length (fade_length);
+               tmp->audio_region()->set_fade_in_active (true);
                
                XMLNode &after = alist->get_state();
                session->add_command(new MementoCommand<AutomationList>(*alist.get(), &before, &after));
@@ -1989,15 +2122,15 @@ Editor::start_fade_out_grab (ArdourCanvas::Item* item, GdkEvent* event)
 
        AudioRegionView* arv = static_cast<AudioRegionView*>(drag_info.data);
 
-       drag_info.pointer_frame_offset = drag_info.grab_frame - (arv->region()->length() - (nframes_t) arv->audio_region()->fade_out()->back()->when + arv->region()->position());      
+       drag_info.pointer_frame_offset = drag_info.grab_frame - (arv->region()->length() - (nframes64_t) arv->audio_region()->fade_out()->back()->when + arv->region()->position());    
 }
 
 void
 Editor::fade_out_drag_motion_callback (ArdourCanvas::Item* item, GdkEvent* event)
 {
        AudioRegionView* arv = static_cast<AudioRegionView*>(drag_info.data);
-       nframes_t pos;
-       nframes_t fade_length;
+       nframes64_t pos;
+       nframes64_t fade_length;
 
        if (drag_info.current_pointer_frame > drag_info.pointer_frame_offset) {
                pos = drag_info.current_pointer_frame - drag_info.pointer_frame_offset;
@@ -2043,8 +2176,8 @@ Editor::fade_out_drag_finished_callback (ArdourCanvas::Item* item, GdkEvent* eve
        if (drag_info.first_move) return;
 
        AudioRegionView* arv = static_cast<AudioRegionView*>(drag_info.data);
-       nframes_t pos;
-       nframes_t fade_length;
+       nframes64_t pos;
+       nframes64_t fade_length;
 
        if (drag_info.current_pointer_frame > drag_info.pointer_frame_offset) {
                pos = drag_info.current_pointer_frame - drag_info.pointer_frame_offset;
@@ -2081,6 +2214,7 @@ Editor::fade_out_drag_finished_callback (ArdourCanvas::Item* item, GdkEvent* eve
                XMLNode &before = alist->get_state();
                
                tmp->audio_region()->set_fade_out_length (fade_length);
+               tmp->audio_region()->set_fade_out_active (true);
 
                XMLNode &after = alist->get_state();
                session->add_command(new MementoCommand<AutomationList>(*alist.get(), &before, &after));
@@ -2126,7 +2260,7 @@ void
 Editor::cursor_drag_motion_callback (ArdourCanvas::Item* item, GdkEvent* event)
 {
        Cursor* cursor = (Cursor *) drag_info.data;
-       nframes_t adjusted_frame;
+       nframes64_t adjusted_frame;
        
        if (drag_info.current_pointer_frame > drag_info.pointer_frame_offset) {
                adjusted_frame = drag_info.current_pointer_frame - drag_info.pointer_frame_offset;
@@ -2219,7 +2353,7 @@ Editor::start_marker_grab (ArdourCanvas::Item* item, GdkEvent* event)
                // marker_drag_line->raise_to_top();
        } else {
                range_marker_drag_rect->show();
-               range_marker_drag_rect->raise_to_top();
+               //range_marker_drag_rect->raise_to_top();
        }
 
        if (is_start) {
@@ -2249,21 +2383,21 @@ Editor::start_marker_grab (ArdourCanvas::Item* item, GdkEvent* event)
 void
 Editor::marker_drag_motion_callback (ArdourCanvas::Item* item, GdkEvent* event)
 {
-       nframes_t f_delta;      
+       nframes64_t f_delta;    
        Marker* marker = (Marker *) drag_info.data;
        Location  *real_location;
        Location  *copy_location;
        bool is_start;
        bool move_both = false;
 
-       nframes_t newframe;
+       nframes64_t newframe;
        if (drag_info.pointer_frame_offset <= drag_info.current_pointer_frame) {
                newframe = drag_info.current_pointer_frame - drag_info.pointer_frame_offset;
        } else {
                newframe = 0;
        }
 
-       nframes_t next = newframe;
+       nframes64_t next = newframe;
 
        if (!Keyboard::modifier_state_contains (event->button.state, Keyboard::snap_modifier())) {
                snap_to (newframe, 0, true);
@@ -2345,8 +2479,8 @@ void
 Editor::marker_drag_finished_callback (ArdourCanvas::Item* item, GdkEvent* event)
 {
        if (drag_info.first_move) {
-               marker_drag_motion_callback (item, event);
-
+               /* just a click, do nothing but whatever selection occured */
+               return;
        }
 
        _dragging_edit_point = false;
@@ -2449,7 +2583,7 @@ void
 Editor::meter_marker_drag_motion_callback (ArdourCanvas::Item* item, GdkEvent* event)
 {
        MeterMarker* marker = (MeterMarker *) drag_info.data;
-       nframes_t adjusted_frame;
+       nframes64_t adjusted_frame;
 
        if (drag_info.current_pointer_frame > drag_info.pointer_frame_offset) {
                adjusted_frame = drag_info.current_pointer_frame - drag_info.pointer_frame_offset;
@@ -2581,7 +2715,7 @@ void
 Editor::tempo_marker_drag_motion_callback (ArdourCanvas::Item* item, GdkEvent* event)
 {
        TempoMarker* marker = (TempoMarker *) drag_info.data;
-       nframes_t adjusted_frame;
+       nframes64_t adjusted_frame;
        
        if (drag_info.current_pointer_frame > drag_info.pointer_frame_offset) {
                adjusted_frame = drag_info.current_pointer_frame - drag_info.pointer_frame_offset;
@@ -2693,8 +2827,9 @@ Editor::start_control_point_grab (ArdourCanvas::Item* item, GdkEvent* event)
        // the point doesn't 'jump' to the mouse after the first drag
        drag_info.grab_x = control_point->get_x();
        drag_info.grab_y = control_point->get_y();
+
        control_point->line().parent_group().i2w(drag_info.grab_x, drag_info.grab_y);
-       track_canvas.w2c(drag_info.grab_x, drag_info.grab_y, drag_info.grab_x, drag_info.grab_y);
+       track_canvas->w2c(drag_info.grab_x, drag_info.grab_y, drag_info.grab_x, drag_info.grab_y);
 
        drag_info.grab_frame = pixel_to_frame(drag_info.grab_x);
 
@@ -2752,7 +2887,7 @@ Editor::control_point_drag_motion_callback (ArdourCanvas::Item* item, GdkEvent*
        cy = min ((double) (cp->line().y_position() + cp->line().height()), cy);
 
        //translate cx to frames
-       nframes_t cx_frames = unit_to_frame (cx);
+       nframes64_t cx_frames = unit_to_frame (cx);
 
        if (!Keyboard::modifier_state_contains (event->button.state, Keyboard::snap_modifier()) && !drag_info.x_constrained) {
                snap_to (cx_frames);
@@ -2825,7 +2960,7 @@ Editor::start_line_grab (AutomationLine* line, GdkEvent* event)
 {
        double cx;
        double cy;
-       nframes_t frame_within_region;
+       nframes64_t frame_within_region;
 
        /* need to get x coordinate in terms of parent (TimeAxisItemView)
           origin.
@@ -2834,7 +2969,7 @@ Editor::start_line_grab (AutomationLine* line, GdkEvent* event)
        cx = event->button.x;
        cy = event->button.y;
        line->parent_group().w2i (cx, cy);
-       frame_within_region = (nframes_t) floor (cx * frames_per_unit);
+       frame_within_region = (nframes64_t) floor (cx * frames_per_unit);
 
        if (!line->control_points_adjacent (frame_within_region, current_line_drag_info.before, 
                                            current_line_drag_info.after)) {
@@ -2943,15 +3078,28 @@ Editor::start_region_grab (ArdourCanvas::Item* item, GdkEvent* event)
                speed = tv->get_diskstream()->speed();
        }
        
-       drag_info.last_frame_position = (nframes_t) (clicked_regionview->region()->position() / speed);
+       drag_info.last_frame_position = (nframes64_t) (clicked_regionview->region()->position() / speed);
        drag_info.pointer_frame_offset = drag_info.grab_frame - drag_info.last_frame_position;
-       drag_info.last_trackview = &clicked_regionview->get_time_axis_view();
+       drag_info.source_trackview = &clicked_regionview->get_time_axis_view();
+       drag_info.dest_trackview = drag_info.source_trackview;
        // we want a move threshold
        drag_info.want_move_threshold = true;
        
        show_verbose_time_cursor (drag_info.last_frame_position, 10);
 
        begin_reversible_command (_("move region(s)"));
+       /* 
+          the group containing moved regions may have been 
+          offset during autoscroll. reset its y offset
+          (we should really handle this in the same way 
+          we do with the x axis, but a simple way of achieving that 
+          eludes me right now). 
+       */
+
+       _region_motion_group->property_y() = 0;
+
+       /* sync the canvas to what we think is its current state */
+       track_canvas->update_now();
 }
 
 void
@@ -2960,7 +3108,8 @@ Editor::start_create_region_grab (ArdourCanvas::Item* item, GdkEvent* event)
        drag_info.copy = false;
        drag_info.item = item;
        drag_info.data = clicked_axisview;
-       drag_info.last_trackview = clicked_axisview;
+       drag_info.source_trackview = clicked_axisview;
+       drag_info.dest_trackview = drag_info.source_trackview;
        drag_info.motion_callback = &Editor::create_region_drag_motion_callback;
        drag_info.finished_callback = &Editor::create_region_drag_finished_callback;
 
@@ -2988,14 +3137,17 @@ Editor::start_region_copy_grab (ArdourCanvas::Item* item, GdkEvent* event)
                speed = rtv->get_diskstream()->speed();
        }
        
-       drag_info.last_trackview = &clicked_regionview->get_time_axis_view();
-       drag_info.last_frame_position = (nframes_t) (clicked_regionview->region()->position() / speed);
+       drag_info.source_trackview = &clicked_regionview->get_time_axis_view();
+       drag_info.dest_trackview = drag_info.source_trackview;
+       drag_info.last_frame_position = (nframes64_t) (clicked_regionview->region()->position() / speed);
        drag_info.pointer_frame_offset = drag_info.grab_frame - drag_info.last_frame_position;
        // we want a move threshold
        drag_info.want_move_threshold = true;
        drag_info.motion_callback = &Editor::region_drag_motion_callback;
        drag_info.finished_callback = &Editor::region_drag_finished_callback;
        show_verbose_time_cursor (drag_info.last_frame_position, 10);
+
+       _region_motion_group->property_y() = 0;
 }
 
 void
@@ -3021,9 +3173,10 @@ Editor::start_region_brush_grab (ArdourCanvas::Item* item, GdkEvent* event)
                speed = tv->get_diskstream()->speed();
        }
        
-       drag_info.last_frame_position = (nframes_t) (clicked_regionview->region()->position() / speed);
+       drag_info.last_frame_position = (nframes64_t) (clicked_regionview->region()->position() / speed);
        drag_info.pointer_frame_offset = drag_info.grab_frame - drag_info.last_frame_position;
-       drag_info.last_trackview = &clicked_regionview->get_time_axis_view();
+       drag_info.source_trackview = &clicked_regionview->get_time_axis_view();
+       drag_info.dest_trackview = drag_info.source_trackview;
        // we want a move threshold
        drag_info.want_move_threshold = true;
        drag_info.brushing = true;
@@ -3038,12 +3191,11 @@ Editor::possibly_copy_regions_during_grab (GdkEvent* event)
 
                drag_info.want_move_threshold = false; // don't copy again
 
-               /* duplicate the region(s) */
+               /* duplicate the regionview(s) and region(s) */
 
                vector<RegionView*> new_regionviews;
                
                for (list<RegionView*>::const_iterator i = selection->regions.by_layer().begin(); i != selection->regions.by_layer().end(); ++i) {
-
                        RegionView* rv;
                        RegionView* nrv;
 
@@ -3060,8 +3212,10 @@ Editor::possibly_copy_regions_during_grab (GdkEvent* event)
                                continue;
                        }
 
-                       nrv->get_canvas_group()->show ();
+                       const boost::shared_ptr<const Region> original = rv->region();
+                       boost::shared_ptr<Region> region_copy = RegionFactory::create (original);
 
+                       nrv->get_canvas_group()->show ();
                        new_regionviews.push_back (nrv);
                }
 
@@ -3069,15 +3223,29 @@ Editor::possibly_copy_regions_during_grab (GdkEvent* event)
                        return;
                }
 
-               /* reset selection to new regionviews */
+               /* reset selection to new regionviews. This will not set selection visual status for 
+                  these regionviews since they don't belong to a track, so do that by hand too.
+                */
 
                selection->set (new_regionviews);
-               
+
+               for (vector<RegionView*>::iterator i = new_regionviews.begin(); i != new_regionviews.end(); ++i) {
+                       (*i)->set_selected (true);
+               }
+
                /* reset drag_info data to reflect the fact that we are dragging the copies */
                
                drag_info.data = new_regionviews.front();
 
                swap_grab (new_regionviews.front()->get_canvas_group (), 0, event->motion.time);
+               /* 
+                  sync the canvas to what we think is its current state 
+                  without it, the canvas seems to 
+                  "forget" to update properly after the upcoming reparent() 
+                  ..only if the mouse is in rapid motion at the time of the grab. 
+                  something to do with regionview creation raking so long?
+                */
+               track_canvas->update_now();
        }
 }
 
@@ -3182,7 +3350,7 @@ Editor::region_drag_motion_callback (ArdourCanvas::Item* item, GdkEvent* event)
        double x_delta;
        double y_delta = 0;
        RegionView* rv = reinterpret_cast<RegionView*> (drag_info.data); 
-       nframes_t pending_region_position = 0;
+       nframes64_t pending_region_position = 0;
        int32_t pointer_y_span = 0, canvas_pointer_y_span = 0, original_pointer_order;
        int32_t visible_y_high = 0, visible_y_low = 512;  //high meaning higher numbered.. not the height on the screen
        bool clamp_y_axis = false;
@@ -3196,8 +3364,8 @@ Editor::region_drag_motion_callback (ArdourCanvas::Item* item, GdkEvent* event)
                return;
        }
 
-       original_pointer_order = drag_info.last_trackview->order;
-               
+       original_pointer_order = drag_info.dest_trackview->order;
+       
        /************************************************************
             Y-Delta Computation
        ************************************************************/   
@@ -3208,7 +3376,7 @@ Editor::region_drag_motion_callback (ArdourCanvas::Item* item, GdkEvent* event)
                goto y_axis_done;
        }
 
-       if ((pointer_y_span = (drag_info.last_trackview->order - tv->order)) != 0) {
+       if ((pointer_y_span = (drag_info.dest_trackview->order - tv->order)) != 0) {
 
                int32_t children = 0, numtracks = 0;
                // XXX hard coding track limit, oh my, so very very bad
@@ -3236,12 +3404,13 @@ Editor::region_drag_motion_callback (ArdourCanvas::Item* item, GdkEvent* event)
                                        tracks = tracks |= (0x01 << rtv2->order);
                                }
        
-                               height_list[rtv2->order] = (*i)->height;
+                               height_list[rtv2->order] = (*i)->current_height();
                                children = 1;
+
                                if ((children_list = rtv2->get_child_list()).size() > 0) {
                                        for (TimeAxisView::Children::iterator j = children_list.begin(); j != children_list.end(); ++j) { 
                                                tracks = tracks |= (0x01 << (rtv2->order + children));
-                                               height_list[rtv2->order + children] =  (*j)->height;                
+                                               height_list[rtv2->order + children] =  (*j)->current_height();
                                                numtracks++;
                                                children++;     
                                        }
@@ -3252,16 +3421,16 @@ Editor::region_drag_motion_callback (ArdourCanvas::Item* item, GdkEvent* event)
                /* find the actual span according to the canvas */
 
                canvas_pointer_y_span = pointer_y_span;
-               if (drag_info.last_trackview->order >= tv->order) {
+               if (drag_info.dest_trackview->order >= tv->order) {
                        int32_t y;
-                       for (y = tv->order; y < drag_info.last_trackview->order; y++) {
+                       for (y = tv->order; y < drag_info.dest_trackview->order; y++) {
                                if (height_list[y] == 0 ) {
                                        canvas_pointer_y_span--;
                                }
                        }
                } else {
                        int32_t y;
-                       for (y = drag_info.last_trackview->order;y <= tv->order; y++) {
+                       for (y = drag_info.dest_trackview->order;y <= tv->order; y++) {
                                if (    height_list[y] == 0 ) {
                                        canvas_pointer_y_span++;
                                }
@@ -3273,8 +3442,14 @@ Editor::region_drag_motion_callback (ArdourCanvas::Item* item, GdkEvent* event)
                        double ix1, ix2, iy1, iy2;
                        int32_t n = 0;
 
+                       if (rv2->region()->locked()) {
+                               continue;
+                       }
+
                        rv2->get_canvas_frame()->get_bounds (ix1, iy1, ix2, iy2);
                        rv2->get_canvas_group()->i2w (ix1, iy1);
+                       iy1 += vertical_adjustment.get_value() - canvas_timebars_vsize;
+
                        TimeAxisView* tvp2 = trackview_by_y_position (iy1);
                        RouteTimeAxisView* rtv2 = dynamic_cast<RouteTimeAxisView*>(tvp2);
 
@@ -3344,13 +3519,13 @@ Editor::region_drag_motion_callback (ArdourCanvas::Item* item, GdkEvent* event)
                        }
                }
 
-       } else  if (drag_info.last_trackview == tv) {
+       } else  if (drag_info.dest_trackview == tv) {
                clamp_y_axis = true;
        }         
 
   y_axis_done:
        if (!clamp_y_axis) {
-               drag_info.last_trackview = tv;        
+               drag_info.dest_trackview = tv;        
        }
          
        /************************************************************
@@ -3360,31 +3535,34 @@ Editor::region_drag_motion_callback (ArdourCanvas::Item* item, GdkEvent* event)
        /* compute the amount of pointer motion in frames, and where
           the region would be if we moved it by that much.
        */
+       if ( drag_info.move_threshold_passed ) {
 
-       if (drag_info.move_threshold_passed) {
-
-               if (drag_info.current_pointer_frame > drag_info.pointer_frame_offset) {
+               if (drag_info.current_pointer_frame >= drag_info.pointer_frame_offset) {
 
-                       nframes_t sync_frame;
-                       nframes_t sync_offset;
+                       nframes64_t sync_frame;
+                       nframes64_t sync_offset;
                        int32_t sync_dir;
-           
+
                        pending_region_position = drag_info.current_pointer_frame - drag_info.pointer_frame_offset;
 
                        sync_offset = rv->region()->sync_offset (sync_dir);
-                       sync_frame = rv->region()->adjust_to_sync (pending_region_position);
 
-                       /* we snap if the snap modifier is not enabled.
+                       /* we don't handle a sync point that lies before zero.
                         */
+                       if (sync_dir >= 0 || (sync_dir < 0 && pending_region_position >= sync_offset)) {
+                               sync_frame = pending_region_position + (sync_dir*sync_offset);
+
+                               /* we snap if the snap modifier is not enabled.
+                                */
            
-                       if (!Keyboard::modifier_state_contains (event->button.state, Keyboard::snap_modifier())) {
-                               snap_to (sync_frame);   
-                       }
+                               if (!Keyboard::modifier_state_contains (event->button.state, Keyboard::snap_modifier())) {
+                                       snap_to (sync_frame);   
+                               }
            
-                       if (sync_frame - sync_offset <= sync_frame) {
-                               pending_region_position = sync_frame + (sync_dir*sync_offset);
+                               pending_region_position = rv->region()->adjust_to_sync (sync_frame);
+
                        } else {
-                               pending_region_position = 0;
+                               pending_region_position = drag_info.last_frame_position;
                        }
            
                } else {
@@ -3394,10 +3572,23 @@ Editor::region_drag_motion_callback (ArdourCanvas::Item* item, GdkEvent* event)
                if (pending_region_position > max_frames - rv->region()->length()) {
                        pending_region_position = drag_info.last_frame_position;
                }
-         
+
                // printf ("3: pending_region_position= %lu    %lu\n", pending_region_position, drag_info.last_frame_position );
-         
-               if (pending_region_position != drag_info.last_frame_position && !drag_info.x_constrained) {
+
+               bool x_move_allowed;
+               
+               if (Config->get_edit_mode() == Lock) {
+                       if (drag_info.copy) {
+                               x_move_allowed = !drag_info.x_constrained;
+                       } else {
+                               /* in locked edit mode, reverse the usual meaning of x_constrained */
+                               x_move_allowed = drag_info.x_constrained;
+                       }
+               } else {
+                       x_move_allowed = !drag_info.x_constrained;
+               }
+
+               if (( pending_region_position != drag_info.last_frame_position) && x_move_allowed ) {
 
                        /* now compute the canvas unit distance we need to move the regionview
                           to make it appear at the new location.
@@ -3407,10 +3598,29 @@ Editor::region_drag_motion_callback (ArdourCanvas::Item* item, GdkEvent* event)
                                x_delta = ((double) (pending_region_position - drag_info.last_frame_position) / frames_per_unit);
                        } else {
                                x_delta = -((double) (drag_info.last_frame_position - pending_region_position) / frames_per_unit);
-                       }
+                               for (list<RegionView*>::const_iterator i = selection->regions.by_layer().begin(); i != selection->regions.by_layer().end(); ++i) {
+
+                                       RegionView* rv2 = (*i);
+
+                                       // If any regionview is at zero, we need to know so we can stop further leftward motion.
+       
+                                       double ix1, ix2, iy1, iy2;
+                                       rv2->get_canvas_frame()->get_bounds (ix1, iy1, ix2, iy2);
+                                       rv2->get_canvas_group()->i2w (ix1, iy1);
+                       
+                                       if (-x_delta > ix1 + horizontal_adjustment.get_value()) {
+                                               //      do_move = false;
+                                               cerr << "illegal move" << endl;
+                                               x_delta = 0;
+                                               pending_region_position = drag_info.last_frame_position;
+                                               break;
+                                       }
+                               }
 
+                       }
+               
                        drag_info.last_frame_position = pending_region_position;
-           
+
                } else {
                        x_delta = 0;
                }
@@ -3420,7 +3630,7 @@ Editor::region_drag_motion_callback (ArdourCanvas::Item* item, GdkEvent* event)
 
                x_delta = 0;
        }
-
+       
        /*************************************************************
            PREPARE TO MOVE
        ************************************************************/
@@ -3430,54 +3640,33 @@ Editor::region_drag_motion_callback (ArdourCanvas::Item* item, GdkEvent* event)
                   trackviews. nothing to do.
                */
                return;
-       } 
-
-
-       if (x_delta < 0) {
-               for (list<RegionView*>::const_iterator i = selection->regions.by_layer().begin(); i != selection->regions.by_layer().end(); ++i) {
-
-                       RegionView* rv2 = (*i);
-
-                       // If any regionview is at zero, we need to know so we can stop further leftward motion.
-                       
-                       double ix1, ix2, iy1, iy2;
-                       rv2->get_canvas_frame()->get_bounds (ix1, iy1, ix2, iy2);
-                       rv2->get_canvas_group()->i2w (ix1, iy1);
-
-                       if (ix1 <= 1) {
-                               x_delta = 0;
-                               break;
-                       }
-               }
        }
 
        /*************************************************************
            MOTION                                                                    
        ************************************************************/
-
-       bool do_move;
-
+       bool do_move = true;
        if (drag_info.first_move) {
-               if (drag_info.move_threshold_passed) {
-                       do_move = true;
-               } else {
+               if (!drag_info.move_threshold_passed) {
                        do_move = false;
                }
-       } else {
-               do_move = true;
        }
 
        if (do_move) {
 
                pair<set<boost::shared_ptr<Playlist> >::iterator,bool> insert_result;
                const list<RegionView*>& layered_regions = selection->regions.by_layer();
-
+               
                for (list<RegionView*>::const_iterator i = layered_regions.begin(); i != layered_regions.end(); ++i) {
            
                        RegionView* rv = (*i);
                        double ix1, ix2, iy1, iy2;
                        int32_t temp_pointer_y_span = pointer_y_span;
 
+                       if (rv->region()->locked()) {
+                               continue;
+                       }
+
                        /* get item BBox, which will be relative to parent. so we have
                           to query on a child, then convert to world coordinates using
                           the parent.
@@ -3485,6 +3674,30 @@ Editor::region_drag_motion_callback (ArdourCanvas::Item* item, GdkEvent* event)
 
                        rv->get_canvas_frame()->get_bounds (ix1, iy1, ix2, iy2);
                        rv->get_canvas_group()->i2w (ix1, iy1);
+
+                       if (drag_info.first_move) {
+
+                               // hide any dependent views 
+       
+                               rv->get_time_axis_view().hide_dependent_views (*rv);
+
+                               /* 
+                                  reparent to a non scrolling group so that we can keep the 
+                                  region selection above all time axis views.
+                                  reparenting means we have to move the rv as the two 
+                                  parent groups have different coordinates.
+                               */
+
+                               rv->get_canvas_group()->property_y() =  iy1 - 1;
+                               rv->get_canvas_group()->reparent(*_region_motion_group);
+
+                               rv->fake_set_opaque (true);
+                       }
+                       /* for evaluation of the track position of iy1, we have to adjust 
+                          to allow for the vertical scrolling adjustment and the height of the timebars.
+                       */
+                       iy1 += vertical_adjustment.get_value() - canvas_timebars_vsize;
+
                        TimeAxisView* tvp2 = trackview_by_y_position (iy1);
                        RouteTimeAxisView* canvas_rtv = dynamic_cast<RouteTimeAxisView*>(tvp2);
                        RouteTimeAxisView* temp_rtv;
@@ -3515,6 +3728,7 @@ Editor::region_drag_motion_callback (ArdourCanvas::Item* item, GdkEvent* event)
                                                        y_delta -= (*j);        
                                                        temp_pointer_y_span--;  
                                                }
+
                                                while (temp_pointer_y_span < 0) {                 
                                                        y_delta += (*j);
                                                        if (x != original_pointer_order) { 
@@ -3532,7 +3746,7 @@ Editor::region_drag_motion_callback (ArdourCanvas::Item* item, GdkEvent* event)
                  
                                                tvp2 = trackview_by_y_position (iy1 + y_delta);
                                                temp_rtv = dynamic_cast<RouteTimeAxisView*>(tvp2);
-                                               rv->set_y_position_and_height (0, temp_rtv->height);
+                                               rv->set_y_position_and_height (0, temp_rtv->current_height());
 
                                                /*   if you un-comment the following, the region colours will follow the track colours whilst dragging,
                                                     personally, i think this can confuse things, but never mind.
@@ -3545,7 +3759,7 @@ Editor::region_drag_motion_callback (ArdourCanvas::Item* item, GdkEvent* event)
                                        x++;
                                }
                        }
-         
+
                        /* prevent the regionview from being moved to before 
                           the zero position on the canvas.
                        */
@@ -3559,30 +3773,10 @@ Editor::region_drag_motion_callback (ArdourCanvas::Item* item, GdkEvent* event)
                                x_delta = max_frames - rv->region()->last_frame();
                        }
 
-
-                       if (drag_info.first_move) {
-
-                               /* hide any dependent views */
-                       
-                               rv->get_time_axis_view().hide_dependent_views (*rv);
-                       
-                               /* this is subtle. raising the regionview itself won't help,
-                                  because raise_to_top() just puts the item on the top of
-                                  its parent's stack. so, we need to put the trackview canvas_display group
-                                  on the top, since its parent is the whole canvas.
-                               */
-                       
-                               rv->get_canvas_group()->raise_to_top();
-                               rv->get_time_axis_view().canvas_display->raise_to_top();
-                               cursor_group->raise_to_top();
-
-                               rv->fake_set_opaque (true);
-                       }
-
                        if (drag_info.brushing) {
                                mouse_brush_insert_region (rv, pending_region_position);
                        } else {
-                               rv->move (x_delta, y_delta);                    
+                               rv->move (x_delta, y_delta);
                        }
 
                } /* foreach region */
@@ -3602,18 +3796,17 @@ Editor::region_drag_motion_callback (ArdourCanvas::Item* item, GdkEvent* event)
 void
 Editor::region_drag_finished_callback (ArdourCanvas::Item* item, GdkEvent* event)
 {
-       nframes_t where;
-       RegionView* rvdi = reinterpret_cast<RegionView *> (drag_info.data);
-       pair<set<boost::shared_ptr<Playlist> >::iterator,bool> insert_result;
        bool nocommit = true;
-       double speed;
-       RouteTimeAxisView* rtv;
-       bool regionview_y_movement;
-       bool regionview_x_movement;
        vector<RegionView*> copies;
-       list <boost::shared_ptr<Playlist > > used_playlists;
-       list <sigc::connection > used_connections;
-       bool preserve_selection = false;
+       RouteTimeAxisView* source_tv;
+       boost::shared_ptr<Diskstream> ds;
+       boost::shared_ptr<Playlist> from_playlist;
+       vector<RegionView*> new_selection;
+       typedef set<boost::shared_ptr<Playlist> > PlaylistSet;
+       PlaylistSet modified_playlists;
+       PlaylistSet frozen_playlists;
+       list <sigc::connection> modified_playlist_connections;
+       pair<PlaylistSet::iterator,bool> insert_result, frozen_insert_result;
 
        /* first_move is set to false if the regionview has been moved in the 
           motion handler. 
@@ -3626,11 +3819,12 @@ Editor::region_drag_finished_callback (ArdourCanvas::Item* item, GdkEvent* event
 
        nocommit = false;
 
-       /* The regionview has been moved at some stage during the grab so we need
+       /* XXX is this true??? i can''t tell the difference.
+          The regionview has been moved at some stage during the grab so we need
           to account for any mouse movement between this event and the last one. 
        */      
 
-       region_drag_motion_callback (item, event);
+       //region_drag_motion_callback (item, event);
 
        if (Config->get_edit_mode() == Splice && !pre_drag_region_selection.empty()) {
                selection->set (pre_drag_region_selection);
@@ -3649,21 +3843,15 @@ Editor::region_drag_finished_callback (ArdourCanvas::Item* item, GdkEvent* event
                goto out;
        }
 
-       /* adjust for track speed */
-       speed = 1.0;
-
-       rtv = dynamic_cast<RouteTimeAxisView*> (drag_info.last_trackview);
-       if (rtv && rtv->get_diskstream()) {
-               speed = rtv->get_diskstream()->speed();
-       }
-       
-       regionview_x_movement = (drag_info.last_frame_position != (nframes_t) (rvdi->region()->position()/speed));
-       regionview_y_movement = (drag_info.last_trackview != &rvdi->get_time_axis_view());
+       char* op_string;
 
-       //printf ("last_frame: %s position is %lu  %g\n", rv->get_time_axis_view().name().c_str(), drag_info.last_frame_position, speed); 
-       //printf ("last_rackview: %s \n", drag_info.last_trackview->name().c_str()); 
+       /* reverse this here so that we have the correct logic to finalize
+          the drag.
+       */
        
-       char* op_string;
+       if (Config->get_edit_mode() == Lock && !drag_info.copy) {
+               drag_info.x_constrained = !drag_info.x_constrained;
+       }
 
        if (drag_info.copy) {
                if (drag_info.x_constrained) {
@@ -3681,94 +3869,147 @@ Editor::region_drag_finished_callback (ArdourCanvas::Item* item, GdkEvent* event
 
        begin_reversible_command (op_string);
 
-       if (regionview_y_movement) {
+       for (list<RegionView*>::const_iterator i = selection->regions.by_layer().begin(); i != selection->regions.by_layer().end(); ) {
+                       
+               RegionView* rv = (*i);              
+               double ix1, ix2, iy1, iy2;
+               rv->get_canvas_frame()->get_bounds (ix1, iy1, ix2, iy2);
+               rv->get_canvas_group()->i2w (ix1, iy1);
+               iy1 += vertical_adjustment.get_value() - canvas_timebars_vsize;
+
+               TimeAxisView* dest_tv = trackview_by_y_position (iy1);
+               RouteTimeAxisView* dest_rtv = dynamic_cast<RouteTimeAxisView*>(dest_tv);
+               double speed;
+               bool changed_tracks, changed_position;
+               nframes64_t where;
+
+               if (rv->region()->locked()) {
+                       ++i;
+                       continue;
+               }
+
+               /* adjust for track speed */
 
-               /* moved to a different track. */
+               speed = 1.0;
                
-               vector<RegionView*> new_selection;
+               if (dest_rtv && dest_rtv->get_diskstream()) {
+                       speed = dest_rtv->get_diskstream()->speed();
+               }
                
-               for (list<RegionView*>::const_iterator i = selection->regions.by_layer().begin(); i != selection->regions.by_layer().end(); ) {
-                       
-                       RegionView* rv = (*i);      
+               changed_position = (drag_info.last_frame_position != (nframes64_t) (rv->region()->position()/speed));
+               changed_tracks = (dest_tv != &rv->get_time_axis_view());
 
-                       double ix1, ix2, iy1, iy2;
+               if (changed_position && !drag_info.x_constrained) {
+                       _master_group->w2i(ix1, iy1);
+                       where = (nframes64_t) (unit_to_frame (ix1) * speed);
+               } else {
+                       where = rv->region()->position();
+               }
                        
-                       rv->get_canvas_frame()->get_bounds (ix1, iy1, ix2, iy2);
-                       rv->get_canvas_group()->i2w (ix1, iy1);
+               boost::shared_ptr<Region> new_region;
 
-                       RouteTimeAxisView* rtv2 = dynamic_cast<RouteTimeAxisView*>(trackview_by_y_position (iy1));
 
-                       boost::shared_ptr<Playlist> from_playlist = rv->region()->playlist();
-                       boost::shared_ptr<Playlist> to_playlist = rtv2->playlist();
-                       
-                       where = (nframes_t) (unit_to_frame (ix1) * speed);
-                       boost::shared_ptr<Region> new_region (RegionFactory::create (rv->region()));
+               if (drag_info.copy) {
+                       /* we already made a copy */
+                       new_region = rv->region();
 
-                       if (! to_playlist->frozen()) {
-                               /* 
-                                  we haven't seen this playlist before. 
-                                  we want to freeze it because we don't want to relayer per-region. 
-                                  its much better to do that just once if the playlist is large. 
-                               */
+                       /* undo the previous hide_dependent_views so that xfades don't
+                          disappear on copying regions 
+                       */
+               
+                       //rv->get_time_axis_view().reveal_dependent_views (*rv);
+               
+               } else if (changed_tracks && dest_rtv->playlist()) {
+                       new_region = RegionFactory::create (rv->region());
+               }
 
-                               /*
-                                  connect so the selection is changed when the new regionview finally appears (after thaw). 
-                                  keep track of it so we can disconnect later. 
-                               */
+               if (changed_tracks || drag_info.copy) {
 
-                               sigc::connection c = rtv2->view()->RegionViewAdded.connect (mem_fun(*this, &Editor::collect_and_select_new_region_view));
-                               used_connections.push_back (c);
+                       boost::shared_ptr<Playlist> to_playlist = dest_rtv->playlist();
+                       if (!to_playlist) {
+                               ++i;
+                               continue;
+                       }
+
+                       latest_regionviews.clear ();
 
-                               /* undo */
+                       sigc::connection c = dest_rtv->view()->RegionViewAdded.connect (mem_fun(*this, &Editor::collect_new_region_view));
+                       
+                       insert_result = modified_playlists.insert (to_playlist);
+                       if (insert_result.second) {
                                session->add_command (new MementoCommand<Playlist>(*to_playlist, &to_playlist->get_state(), 0));
+                       }
 
-                               /* remember used playlists so we can thaw them later */ 
-                               used_playlists.push_back(to_playlist);
-                               to_playlist->freeze();
+                       to_playlist->add_region (new_region, where);
+
+                       c.disconnect ();
+                                                             
+                       if (!latest_regionviews.empty()) {
+                               // XXX why just the first one ? we only expect one
+                               // commented out in nick_m's canvas reworking. is that intended?
+                               //dest_atv->reveal_dependent_views (*latest_regionviews.front());
+                               new_selection.push_back (latest_regionviews.front());
                        }
-                       
-                       /* undo the previous hide_dependent_views so that xfades don't
-                          disappear on copying regions 
+
+               } else {
+                       /* 
+                          motion on the same track. plonk the previously reparented region 
+                          back to its original canvas group (its streamview).
+                          No need to do anything for copies as they are fake regions which will be deleted.
                        */
 
-                       rv->get_time_axis_view().reveal_dependent_views (*rv);
+                       rv->get_canvas_group()->reparent (*dest_rtv->view()->canvas_item());
+                       rv->get_canvas_group()->property_y() = 0;
+                 
+                       /* just change the model */
+                       
+                       boost::shared_ptr<Playlist> playlist = dest_rtv->playlist();
 
-                       if (!drag_info.copy) {
-                               
-                               /* the region that used to be in the old playlist is not
-                                  moved to the new one - we make a copy of it. as a result,
-                                  any existing editor for the region should no longer be
-                                  visible.
-                               */ 
-           
-                               rv->hide_region_editor();
-                               rv->fake_set_opaque (false);
+                       insert_result = modified_playlists.insert (playlist);
+                       if (insert_result.second) {
+                               session->add_command (new MementoCommand<Playlist>(*playlist, &playlist->get_state(), 0));
+                       }
+                       /* freeze to avoid lots of relayering in the case of a multi-region drag */
+                       frozen_insert_result = frozen_playlists.insert(playlist);
+                       if (frozen_insert_result.second) {
+                               playlist->freeze();
+                       }
 
-                               session->add_command (new MementoCommand<Playlist>(*from_playlist, &from_playlist->get_state(), 0));    
-                               from_playlist->remove_region ((rv->region()));
-                               session->add_command (new MementoCommand<Playlist>(*from_playlist, 0, &from_playlist->get_state()));    
+                       rv->region()->set_position (where, (void*) this);
+               }
 
-                       } else {
+               if (changed_tracks && !drag_info.copy) {
 
-                               /* the regionview we dragged around is a temporary copy, queue it for deletion */
+                       /* get the playlist where this drag started. we can't use rv->region()->playlist()
+                          because we may have copied the region and it has not been attached to a playlist.
+                       */
 
-                               copies.push_back (rv);
-                       }
+                       assert ((source_tv = dynamic_cast<RouteTimeAxisView*> (&rv->get_time_axis_view())));
+                       assert ((ds = source_tv->get_diskstream()));
+                       assert ((from_playlist = ds->playlist()));
 
-                       latest_regionviews.clear ();
-                       sigc::connection c = rtv2->view()->RegionViewAdded.connect (mem_fun(*this, &Editor::collect_new_region_view));
-                       session->add_command (new MementoCommand<Playlist>(*to_playlist, &to_playlist->get_state(), 0));        
+                       /* moved to a different audio track, without copying */
 
-                       to_playlist->add_region (new_region, where);
-                       session->add_command (new MementoCommand<Playlist>(*to_playlist, 0, &to_playlist->get_state()));        
-                       c.disconnect ();
-                                                             
-                       if (!latest_regionviews.empty()) {
-                               new_selection.insert (new_selection.end(), latest_regionviews.begin(), latest_regionviews.end());
+                       /* the region that used to be in the old playlist is not
+                          moved to the new one - we use a copy of it. as a result,
+                          any existing editor for the region should no longer be
+                          visible.
+                       */ 
+           
+                       rv->hide_region_editor();
+                       rv->fake_set_opaque (false);
+                       
+                       /* remove the region from the old playlist */
+
+                       insert_result = modified_playlists.insert (from_playlist);
+                       if (insert_result.second) {
+                               session->add_command (new MementoCommand<Playlist>(*from_playlist, &from_playlist->get_state(), 0));
                        }
 
+                       from_playlist->remove_region ((rv->region()));
+                       
                        /* OK, this is where it gets tricky. If the playlist was being used by >1 tracks, and the region
-                          was selected in all of them, then removing it from the playlist will have removed all
+                          was selected in all of them, then removing it from a playlist will have removed all
                           trace of it from the selection (i.e. there were N regions selected, we removed 1,
                           but since its the same playlist for N tracks, all N tracks updated themselves, removed the
                           corresponding regionview, and the selection is now empty).
@@ -3782,139 +4023,55 @@ Editor::region_drag_finished_callback (ArdourCanvas::Item* item, GdkEvent* event
 
                           EXCEPT .... if we are doing a copy drag, then the selection hasn't been modified and
                           we can just iterate.
-
                        */
 
-                       if (drag_info.copy) {
-                               ++i;
-                       } else {
-                               if (selection->regions.empty()) {
-                                       break;
-                               } else { 
-                                 /*
-                                   XXX see above .. but we just froze the playlists.. we have to keep iterating, right? 
-                                 */
-
-                                 //i = selection->regions.by_layer().begin();
-                                 ++i;
-                               }
-                       }
-               }
-
-       } else {
-
-               /* motion within a single track */
-
-               list<RegionView*> regions = selection->regions.by_layer();
-
-               for (list<RegionView*>::iterator i = regions.begin(); i != regions.end(); ++i) {
-
-                       RegionView* rv = (*i);
-                       boost::shared_ptr<Playlist> to_playlist = (*i)->region()->playlist();
-                       RouteTimeAxisView* from_rtv = dynamic_cast<RouteTimeAxisView*> (&(rv->get_time_axis_view()));
-
-                       if (!rv->region()->can_move()) {
-                               continue;
-                       }
-
-                       if (regionview_x_movement) {
-                               double ownspeed = 1.0;
-
-                               if (from_rtv && from_rtv->get_diskstream()) {
-                                       ownspeed = from_rtv->get_diskstream()->speed();
-                               }
-                               
-                               /* base the new region position on the current position of the regionview.*/
-                               
-                               double ix1, ix2, iy1, iy2;
-                               
-                               rv->get_canvas_frame()->get_bounds (ix1, iy1, ix2, iy2);
-                               rv->get_canvas_group()->i2w (ix1, iy1);
-                               where = (nframes_t) (unit_to_frame (ix1) * ownspeed);
-                               
-                       } else {
-                               
-                               where = rv->region()->position();
-                       }
-
-                       if (! to_playlist->frozen()) {
-                               sigc::connection c = from_rtv->view()->RegionViewAdded.connect (mem_fun(*this, &Editor::collect_and_select_new_region_view));
-                               used_connections.push_back (c);
-
-                               /* add the undo */      
-                               session->add_command (new MementoCommand<Playlist>(*to_playlist, &to_playlist->get_state(), 0));
-
-                               used_playlists.push_back(to_playlist);
-                               to_playlist->freeze();
-                       }
-
-                       if (drag_info.copy) {
-
-                               boost::shared_ptr<Region> newregion;
-                               boost::shared_ptr<Region> ar;
-                               boost::shared_ptr<Region> mr;
-
-                               if ((ar = boost::dynamic_pointer_cast<AudioRegion>(rv->region())) != 0) {
-                                       newregion = RegionFactory::create (ar);
-                               } else if ((mr = boost::dynamic_pointer_cast<MidiRegion>(rv->region())) != 0) {
-                                       newregion = RegionFactory::create (mr);
-                               }
-
-                               /* add it */
-
-                               latest_regionviews.clear ();
-                               sigc::connection c = rtv->view()->RegionViewAdded.connect (mem_fun(*this, &Editor::collect_new_region_view));
-                               to_playlist->add_region (newregion, (nframes_t) (where * rtv->get_diskstream()->speed()));
-                               c.disconnect ();
-                               
-                               if (!latest_regionviews.empty()) {
-                                       // XXX why just the first one ? we only expect one
-                                       rtv->reveal_dependent_views (*latest_regionviews.front());
-                                       selection->add (latest_regionviews);
-                               }
-                               
-                       } else {
-
-                               /* just change the model */
-
-                               rv->region()->set_position (where, (void*) this);
-                               preserve_selection = true;
-
+                       if (selection->regions.empty()) {
+                               break;
+                       } else { 
+                               i = selection->regions.by_layer().begin();
                        }
 
+               } else {
+                       ++i;
+               }
+               
+               if (drag_info.copy) {
+                       copies.push_back (rv);
                }
-
-       }
-       if (! preserve_selection) {
-         //selection->clear_regions();
        }
-       while (used_playlists.size() > 0) {
-
-               list <boost::shared_ptr<Playlist > >::iterator i = used_playlists.begin();
-               (*i)->thaw();
-
-               if (used_connections.size()) {
-                       sigc::connection c = used_connections.front();
-                       c.disconnect();
-                       used_connections.pop_front();
+       
+       if (new_selection.empty()) {
+               if (drag_info.copy) {
+                       /* the region(view)s that are selected and being dragged around
+                          are copies and do not belong to any track. remove them
+                          from the selection right here.
+                       */
+                       selection->clear_regions();
                }
-               /* add the redo */
-
-               session->add_command (new MementoCommand<Playlist>(*(*i), 0, &(*i)->get_state()));
-               used_playlists.pop_front();
+       } else {
+               /* this will clear any existing selection that would have been
+                  cleared in the other clause above
+               */
+               selection->set (new_selection);
        }
 
+       for (set<boost::shared_ptr<Playlist> >::iterator p = frozen_playlists.begin(); p != frozen_playlists.end(); ++p) {
+               (*p)->thaw();
+       }
+                       
   out:
-       
        if (!nocommit) {
+               for (set<boost::shared_ptr<Playlist> >::iterator p = modified_playlists.begin(); p != modified_playlists.end(); ++p) {
+                       session->add_command (new MementoCommand<Playlist>(*(*p), 0, &(*p)->get_state()));      
+               }
                commit_reversible_command ();
        }
 
        for (vector<RegionView*>::iterator x = copies.begin(); x != copies.end(); ++x) {
                delete *x;
        }
-}
 
+}
        
 void
 Editor::create_region_drag_motion_callback (ArdourCanvas::Item* item, GdkEvent* event)
@@ -3932,7 +4089,7 @@ Editor::create_region_drag_motion_callback (ArdourCanvas::Item* item, GdkEvent*
 void
 Editor::create_region_drag_finished_callback (ArdourCanvas::Item* item, GdkEvent* event)
 {
-       MidiTimeAxisView* mtv = dynamic_cast<MidiTimeAxisView*> (drag_info.last_trackview);
+       MidiTimeAxisView* mtv = dynamic_cast<MidiTimeAxisView*> (drag_info.dest_trackview);
        if (!mtv)
                return;
 
@@ -3948,7 +4105,7 @@ Editor::create_region_drag_finished_callback (ArdourCanvas::Item* item, GdkEvent
                begin_reversible_command (_("create region"));
                XMLNode &before = mtv->playlist()->get_state();
 
-               nframes_t start = drag_info.grab_frame;
+               nframes64_t start = drag_info.grab_frame;
                snap_to (start, -1);
                const Meter& m = session->tempo_map().meter_at(start);
                const Tempo& t = session->tempo_map().tempo_at(start);
@@ -3990,35 +4147,43 @@ Editor::region_view_item_click (AudioRegionView& rv, GdkEventButton* event)
 
                        if (Keyboard::modifier_state_equals (event->state, Keyboard::ModifierMask (Keyboard::PrimaryModifier|Keyboard::SecondaryModifier))) {
                                
-                               align_region (rv.region(), SyncPoint, (nframes_t) (where * speed));
+                               align_region (rv.region(), SyncPoint, (nframes64_t) (where * speed));
                                
                        } else if (Keyboard::modifier_state_equals (event->state, Keyboard::ModifierMask (Keyboard::PrimaryModifier|Keyboard::TertiaryModifier))) {
                                
-                               align_region (rv.region(), End, (nframes_t) (where * speed));
+                               align_region (rv.region(), End, (nframes64_t) (where * speed));
                                
                        } else {
                                
-                               align_region (rv.region(), Start, (nframes_t) (where * speed));
+                               align_region (rv.region(), Start, (nframes64_t) (where * speed));
                        }
                }
        }
 }
 
 void
-Editor::show_verbose_time_cursor (nframes_t frame, double offset, double xpos, double ypos) 
+Editor::show_verbose_time_cursor (nframes64_t frame, double offset, double xpos, double ypos) 
 {
        char buf[128];
        SMPTE::Time smpte;
        BBT_Time bbt;
        int hours, mins;
-       nframes_t frame_rate;
+       nframes64_t frame_rate;
        float secs;
 
        if (session == 0) {
                return;
        }
 
-       switch (Profile->get_small_screen() ? ARDOUR_UI::instance()->primary_clock.mode () : ARDOUR_UI::instance()->secondary_clock.mode ()) {
+       AudioClock::Mode m;
+
+       if (Profile->get_sae() || Profile->get_small_screen()) {
+               m = ARDOUR_UI::instance()->primary_clock.mode();
+       } else {
+               m = ARDOUR_UI::instance()->secondary_clock.mode();
+       }
+
+       switch (m) {
        case AudioClock::BBT:
                session->bbt_time (frame, bbt);
                snprintf (buf, sizeof (buf), "%02" PRIu32 "|%02" PRIu32 "|%02" PRIu32, bbt.bars, bbt.beats, bbt.ticks);
@@ -4041,7 +4206,7 @@ Editor::show_verbose_time_cursor (nframes_t frame, double offset, double xpos, d
                break;
 
        default:
-               snprintf (buf, sizeof(buf), "%u", frame);
+               snprintf (buf, sizeof(buf), "%" PRIi64, frame);
                break;
        }
 
@@ -4049,20 +4214,20 @@ Editor::show_verbose_time_cursor (nframes_t frame, double offset, double xpos, d
                set_verbose_canvas_cursor (buf, xpos + offset, ypos + offset);
        }
        else {
-               set_verbose_canvas_cursor (buf, drag_info.current_pointer_x + offset, drag_info.current_pointer_y + offset);
+               set_verbose_canvas_cursor (buf, drag_info.current_pointer_x + offset - horizontal_adjustment.get_value(), drag_info.current_pointer_y + offset - vertical_adjustment.get_value() + canvas_timebars_vsize);
        }
        show_verbose_canvas_cursor ();
 }
 
 void
-Editor::show_verbose_duration_cursor (nframes_t start, nframes_t end, double offset, double xpos, double ypos) 
+Editor::show_verbose_duration_cursor (nframes64_t start, nframes64_t end, double offset, double xpos, double ypos) 
 {
        char buf[128];
        SMPTE::Time smpte;
        BBT_Time sbbt;
        BBT_Time ebbt;
        int hours, mins;
-       nframes_t distance, frame_rate;
+       nframes64_t distance, frame_rate;
        float secs;
        Meter meter_at_start(session->tempo_map().meter_at(start));
 
@@ -4070,7 +4235,15 @@ Editor::show_verbose_duration_cursor (nframes_t start, nframes_t end, double off
                return;
        }
 
-       switch (ARDOUR_UI::instance()->secondary_clock.mode ()) {
+       AudioClock::Mode m;
+
+       if (Profile->get_sae() || Profile->get_small_screen()) {
+               m = ARDOUR_UI::instance()->primary_clock.mode ();
+       } else {
+               m = ARDOUR_UI::instance()->secondary_clock.mode ();
+       }
+
+       switch (m) {
        case AudioClock::BBT:
                session->bbt_time (start, sbbt);
                session->bbt_time (end, ebbt);
@@ -4115,7 +4288,7 @@ Editor::show_verbose_duration_cursor (nframes_t start, nframes_t end, double off
                break;
 
        default:
-               snprintf (buf, sizeof(buf), "%u", end - start);
+               snprintf (buf, sizeof(buf), "%" PRIi64, end - start);
                break;
        }
 
@@ -4125,6 +4298,7 @@ Editor::show_verbose_duration_cursor (nframes_t start, nframes_t end, double off
        else {
                set_verbose_canvas_cursor (buf, drag_info.current_pointer_x + offset, drag_info.current_pointer_y + offset);
        }
+
        show_verbose_canvas_cursor ();
 }
 
@@ -4150,7 +4324,7 @@ Editor::start_selection_grab (ArdourCanvas::Item* item, GdkEvent* event)
 
        /* lets try to create new Region for the selection */
 
-       vector<boost::shared_ptr<AudioRegion> > new_regions;
+       vector<boost::shared_ptr<Region> > new_regions;
        create_region_from_selection (new_regions);
 
        if (new_regions.empty()) {
@@ -4205,7 +4379,8 @@ Editor::start_selection_grab (ArdourCanvas::Item* item, GdkEvent* event)
 
        start_grab (event);
        
-       drag_info.last_trackview = clicked_axisview;
+       drag_info.source_trackview = clicked_routeview;
+       drag_info.dest_trackview = drag_info.source_trackview;
        drag_info.last_frame_position = latest_regionviews.front()->region()->position();
        drag_info.pointer_frame_offset = drag_info.grab_frame - drag_info.last_frame_position;
        
@@ -4225,8 +4400,8 @@ Editor::cancel_selection ()
 void
 Editor::start_selection_op (ArdourCanvas::Item* item, GdkEvent* event, SelectionOp op)
 {
-       nframes_t start = 0;
-       nframes_t end = 0;
+       nframes64_t start = 0;
+       nframes64_t end = 0;
 
        if (session == 0) {
                return;
@@ -4283,10 +4458,10 @@ Editor::start_selection_op (ArdourCanvas::Item* item, GdkEvent* event, Selection
 void
 Editor::drag_selection (ArdourCanvas::Item* item, GdkEvent* event)
 {
-       nframes_t start = 0;
-       nframes_t end = 0;
-       nframes_t length;
-       nframes_t pending_position;
+       nframes64_t start = 0;
+       nframes64_t end = 0;
+       nframes64_t length;
+       nframes64_t pending_position;
 
        if (drag_info.current_pointer_frame > drag_info.pointer_frame_offset) {
                pending_position = drag_info.current_pointer_frame - drag_info.pointer_frame_offset;
@@ -4391,7 +4566,7 @@ Editor::drag_selection (ArdourCanvas::Item* item, GdkEvent* event)
        }
        
        if (event->button.x >= horizontal_adjustment.get_value() + canvas_width) {
-               start_canvas_autoscroll (1);
+               start_canvas_autoscroll (1, 0);
        }
 
        if (start != end) {
@@ -4444,9 +4619,9 @@ Editor::start_trim (ArdourCanvas::Item* item, GdkEvent* event)
                speed = tv->get_diskstream()->speed();
        }
        
-       nframes_t region_start = (nframes_t) (clicked_regionview->region()->position() / speed);
-       nframes_t region_end = (nframes_t) (clicked_regionview->region()->last_frame() / speed);
-       nframes_t region_length = (nframes_t) (clicked_regionview->region()->length() / speed);
+       nframes64_t region_start = (nframes64_t) (clicked_regionview->region()->position() / speed);
+       nframes64_t region_end = (nframes64_t) (clicked_regionview->region()->last_frame() / speed);
+       nframes64_t region_length = (nframes64_t) (clicked_regionview->region()->length() / speed);
 
        //drag_info.item = clicked_regionview->get_name_highlight();
        drag_info.item = item;
@@ -4485,7 +4660,7 @@ void
 Editor::trim_motion_callback (ArdourCanvas::Item* item, GdkEvent* event)
 {
        RegionView* rv = clicked_regionview;
-       nframes_t frame_delta = 0;
+       nframes64_t frame_delta = 0;
        bool left_direction;
        bool obey_snap = !Keyboard::modifier_state_contains (event->button.state, Keyboard::snap_modifier());
 
@@ -4570,7 +4745,7 @@ Editor::trim_motion_callback (ArdourCanvas::Item* item, GdkEvent* event)
                }
                
        case EndTrim:
-               if ((left_direction == true) && (drag_info.current_pointer_frame > (nframes_t) (rv->region()->last_frame()/speed))) {
+               if ((left_direction == true) && (drag_info.current_pointer_frame > (nframes64_t) (rv->region()->last_frame()/speed))) {
                        break;
                } else {
                        for (list<RegionView*>::const_iterator i = selection->regions.by_layer().begin(); i != selection->regions.by_layer().end(); ++i) {
@@ -4598,10 +4773,10 @@ Editor::trim_motion_callback (ArdourCanvas::Item* item, GdkEvent* event)
 
        switch (trim_op) {
        case StartTrim:
-               show_verbose_time_cursor((nframes_t) (rv->region()->position()/speed), 10);     
+               show_verbose_time_cursor((nframes64_t) (rv->region()->position()/speed), 10);   
                break;
        case EndTrim:
-               show_verbose_time_cursor((nframes_t) (rv->region()->last_frame()/speed), 10);   
+               show_verbose_time_cursor((nframes64_t) (rv->region()->last_frame()/speed), 10); 
                break;
        case ContentsTrim:
                show_verbose_time_cursor(drag_info.current_pointer_frame, 10);  
@@ -4613,7 +4788,7 @@ Editor::trim_motion_callback (ArdourCanvas::Item* item, GdkEvent* event)
 }
 
 void
-Editor::single_contents_trim (RegionView& rv, nframes_t frame_delta, bool left_direction, bool swap_direction, bool obey_snap)
+Editor::single_contents_trim (RegionView& rv, nframes64_t frame_delta, bool left_direction, bool swap_direction, bool obey_snap)
 {
        boost::shared_ptr<Region> region (rv.region());
 
@@ -4621,7 +4796,7 @@ Editor::single_contents_trim (RegionView& rv, nframes_t frame_delta, bool left_d
                return;
        }
 
-       nframes_t new_bound;
+       nframes64_t new_bound;
 
        double speed = 1.0;
        TimeAxisView* tvp = clicked_axisview;
@@ -4633,27 +4808,27 @@ Editor::single_contents_trim (RegionView& rv, nframes_t frame_delta, bool left_d
        
        if (left_direction) {
                if (swap_direction) {
-                       new_bound = (nframes_t) (region->position()/speed) + frame_delta;
+                       new_bound = (nframes64_t) (region->position()/speed) + frame_delta;
                } else {
-                       new_bound = (nframes_t) (region->position()/speed) - frame_delta;
+                       new_bound = (nframes64_t) (region->position()/speed) - frame_delta;
                }
        } else {
                if (swap_direction) {
-                       new_bound = (nframes_t) (region->position()/speed) - frame_delta;
+                       new_bound = (nframes64_t) (region->position()/speed) - frame_delta;
                } else {
-                       new_bound = (nframes_t) (region->position()/speed) + frame_delta;
+                       new_bound = (nframes64_t) (region->position()/speed) + frame_delta;
                }
        }
 
        if (obey_snap) {
                snap_to (new_bound);
        }
-       region->trim_start ((nframes_t) (new_bound * speed), this);     
+       region->trim_start ((nframes64_t) (new_bound * speed), this);   
        rv.region_changed (StartChanged);
 }
 
 void
-Editor::single_start_trim (RegionView& rv, nframes_t frame_delta, bool left_direction, bool obey_snap)
+Editor::single_start_trim (RegionView& rv, nframes64_t frame_delta, bool left_direction, bool obey_snap)
 {
        boost::shared_ptr<Region> region (rv.region()); 
 
@@ -4661,7 +4836,7 @@ Editor::single_start_trim (RegionView& rv, nframes_t frame_delta, bool left_dire
                return;
        }
 
-       nframes_t new_bound;
+       nframes64_t new_bound;
 
        double speed = 1.0;
        TimeAxisView* tvp = clicked_axisview;
@@ -4672,22 +4847,22 @@ Editor::single_start_trim (RegionView& rv, nframes_t frame_delta, bool left_dire
        }
        
        if (left_direction) {
-               new_bound = (nframes_t) (region->position()/speed) - frame_delta;
+               new_bound = (nframes64_t) (region->position()/speed) - frame_delta;
        } else {
-               new_bound = (nframes_t) (region->position()/speed) + frame_delta;
+               new_bound = (nframes64_t) (region->position()/speed) + frame_delta;
        }
 
        if (obey_snap) {
                snap_to (new_bound, (left_direction ? 0 : 1));  
        }
 
-       region->trim_front ((nframes_t) (new_bound * speed), this);
+       region->trim_front ((nframes64_t) (new_bound * speed), this);
 
        rv.region_changed (Change (LengthChanged|PositionChanged|StartChanged));
 }
 
 void
-Editor::single_end_trim (RegionView& rv, nframes_t frame_delta, bool left_direction, bool obey_snap)
+Editor::single_end_trim (RegionView& rv, nframes64_t frame_delta, bool left_direction, bool obey_snap)
 {
        boost::shared_ptr<Region> region (rv.region());
 
@@ -4695,7 +4870,7 @@ Editor::single_end_trim (RegionView& rv, nframes_t frame_delta, bool left_direct
                return;
        }
 
-       nframes_t new_bound;
+       nframes64_t new_bound;
 
        double speed = 1.0;
        TimeAxisView* tvp = clicked_axisview;
@@ -4706,15 +4881,15 @@ Editor::single_end_trim (RegionView& rv, nframes_t frame_delta, bool left_direct
        }
        
        if (left_direction) {
-               new_bound = (nframes_t) ((region->last_frame() + 1)/speed) - frame_delta;
+               new_bound = (nframes64_t) ((region->last_frame() + 1)/speed) - frame_delta;
        } else {
-               new_bound = (nframes_t) ((region->last_frame() + 1)/speed) + frame_delta;
+               new_bound = (nframes64_t) ((region->last_frame() + 1)/speed) + frame_delta;
        }
 
        if (obey_snap) {
                snap_to (new_bound);
        }
-       region->trim_end ((nframes_t) (new_bound * speed), this);
+       region->trim_end ((nframes64_t) (new_bound * speed), this);
        rv.region_changed (LengthChanged);
 }
        
@@ -4754,7 +4929,7 @@ void
 Editor::point_trim (GdkEvent* event)
 {
        RegionView* rv = clicked_regionview;
-       nframes_t new_bound = drag_info.current_pointer_frame;
+       nframes64_t new_bound = drag_info.current_pointer_frame;
 
        if (!Keyboard::modifier_state_contains (event->button.state, Keyboard::snap_modifier())) {
                snap_to (new_bound);
@@ -4902,9 +5077,25 @@ Editor::start_range_markerbar_op (ArdourCanvas::Item* item, GdkEvent* event, Ran
 void
 Editor::drag_range_markerbar_op (ArdourCanvas::Item* item, GdkEvent* event)
 {
-       nframes_t start = 0;
-       nframes_t end = 0;
-       ArdourCanvas::SimpleRect *crect = (range_marker_op == CreateRangeMarker) ? range_bar_drag_rect: transport_bar_drag_rect;
+       nframes64_t start = 0;
+       nframes64_t end = 0;
+       ArdourCanvas::SimpleRect *crect;
+
+       switch (range_marker_op) {
+       case CreateRangeMarker:
+               crect = range_bar_drag_rect;
+               break;
+       case CreateTransportMarker:
+               crect = transport_bar_drag_rect;
+               break;
+       case CreateCDMarker:
+               crect = cd_marker_bar_drag_rect;
+               break;
+       default:
+               cerr << "Error: unknown range marker op passed to Editor::drag_range_markerbar_op ()" << endl;
+               return;
+               break;
+       }
        
        if (!Keyboard::modifier_state_contains (event->button.state, Keyboard::snap_modifier())) {
                snap_to (drag_info.current_pointer_frame);
@@ -4944,14 +5135,14 @@ Editor::drag_range_markerbar_op (ArdourCanvas::Item* item, GdkEvent* event)
 
                        update_marker_drag_item (temp_location);
                        range_marker_drag_rect->show();
-                       range_marker_drag_rect->raise_to_top();
+                       //range_marker_drag_rect->raise_to_top();
                        
                } 
                break;          
        }
        
        if (event->button.x >= horizontal_adjustment.get_value() + canvas_width) {
-               start_canvas_autoscroll (1);
+               start_canvas_autoscroll (1, 0);
        }
        
        if (start != end) {
@@ -4991,9 +5182,11 @@ Editor::end_range_markerbar_op (ArdourCanvas::Item* item, GdkEvent* event)
                        session->locations()->next_available_name(rangename,"unnamed");
                        if (range_marker_op == CreateCDMarker) {
                                flags =  Location::IsRangeMarker|Location::IsCDMarker;
+                               cd_marker_bar_drag_rect->hide();
                        }
                        else {
                                flags =  Location::IsRangeMarker;
+                               range_bar_drag_rect->hide();
                        }
                        newloc = new Location(temp_location->start(), temp_location->end(), rangename, (Location::Flags) flags);
                        session->locations()->add (newloc, true);
@@ -5001,7 +5194,6 @@ Editor::end_range_markerbar_op (ArdourCanvas::Item* item, GdkEvent* event)
                        session->add_command(new MementoCommand<Locations>(*(session->locations()), &before, &after));
                        commit_reversible_command ();
                        
-                       range_bar_drag_rect->hide();
                        range_marker_drag_rect->hide();
                        break;
                    }
@@ -5017,8 +5209,8 @@ Editor::end_range_markerbar_op (ArdourCanvas::Item* item, GdkEvent* event)
 
                if (Keyboard::no_modifier_keys_pressed (&event->button) && range_marker_op != CreateCDMarker) {
 
-                       nframes_t start;
-                       nframes_t end;
+                       nframes64_t start;
+                       nframes64_t end;
 
                        start = session->locations()->first_mark_before (drag_info.grab_frame);
                        end = session->locations()->first_mark_after (drag_info.grab_frame);
@@ -5068,8 +5260,8 @@ Editor::start_mouse_zoom (ArdourCanvas::Item* item, GdkEvent* event)
 void
 Editor::drag_mouse_zoom (ArdourCanvas::Item* item, GdkEvent* event)
 {
-       nframes_t start;
-       nframes_t end;
+       nframes64_t start;
+       nframes64_t end;
 
        if (!Keyboard::modifier_state_contains (event->button.state, Keyboard::snap_modifier())) {
                snap_to (drag_info.current_pointer_frame);
@@ -5129,7 +5321,7 @@ Editor::end_mouse_zoom (ArdourCanvas::Item* item, GdkEvent* event)
 }
 
 void
-Editor::reposition_zoom_rect (nframes_t start, nframes_t end)
+Editor::reposition_zoom_rect (nframes64_t start, nframes64_t end)
 {
        double x1 = frame_to_pixel (start);
        double x2 = frame_to_pixel (end);
@@ -5156,8 +5348,8 @@ Editor::start_rubberband_select (ArdourCanvas::Item* item, GdkEvent* event)
 void
 Editor::drag_rubberband_select (ArdourCanvas::Item* item, GdkEvent* event)
 {
-       nframes_t start;
-       nframes_t end;
+       nframes64_t start;
+       nframes64_t end;
        double y1;
        double y2;
 
@@ -5167,7 +5359,7 @@ Editor::drag_rubberband_select (ArdourCanvas::Item* item, GdkEvent* event)
                return;
        }
 
-       if (!Keyboard::modifier_state_contains (event->button.state, Keyboard::snap_modifier())) {
+       if (!Keyboard::modifier_state_contains (event->button.state, Keyboard::snap_modifier()) && Config->get_rubberbanding_snaps_to_grid()) {
                if (drag_info.first_move) {
                        snap_to (drag_info.grab_frame);
                } 
@@ -5224,8 +5416,7 @@ Editor::end_rubberband_select (ArdourCanvas::Item* item, GdkEvent* event)
                if (drag_info.current_pointer_y < drag_info.grab_y) {
                        y1 = drag_info.current_pointer_y;
                        y2 = drag_info.grab_y;
-               }
-               else {
+               } else {
                        y2 = drag_info.current_pointer_y;
                        y1 = drag_info.grab_y;
                }
@@ -5330,17 +5521,21 @@ Editor::end_time_fx (ArdourCanvas::Item* item, GdkEvent* event)
                return;
        }
        
-       nframes_t newlen = drag_info.last_pointer_frame - clicked_regionview->region()->position();
-#ifdef USE_RUBBERBAND
-       float percentage = (float) ((double) newlen / (double) clicked_regionview->region()->length());
-#else
-       float percentage = (float) ((double) newlen - (double) clicked_regionview->region()->length()) / ((double) newlen) * 100.0f;
-#endif 
+       nframes64_t newlen = drag_info.last_pointer_frame - clicked_regionview->region()->position();
 
+       float percentage = (double) newlen / (double) clicked_regionview->region()->length();
+       
+#ifndef USE_RUBBERBAND
+       // Soundtouch uses percentage / 100 instead of normal (/ 1) 
+       if (clicked_regionview->region()->data_type() == DataType::AUDIO) {
+               percentage = (float) ((double) newlen - (double) clicked_regionview->region()->length()) / ((double) newlen) * 100.0f;
+       }
+#endif 
+       
        begin_reversible_command (_("timestretch"));
-
+       
        // XXX how do timeFX on multiple regions ?
-
+       
        RegionSelection rs;
        rs.add (clicked_regionview);
 
@@ -5350,14 +5545,10 @@ Editor::end_time_fx (ArdourCanvas::Item* item, GdkEvent* event)
 }
 
 void
-Editor::mouse_brush_insert_region (RegionView* rv, nframes_t pos)
+Editor::mouse_brush_insert_region (RegionView* rv, nframes64_t pos)
 {
        /* no brushing without a useful snap setting */
 
-       // FIXME
-       AudioRegionView* arv = dynamic_cast<AudioRegionView*>(rv);
-       assert(arv);
-
        switch (snap_mode) {
        case SnapMagnetic:
                return; /* can't work because it allows region to be placed anywhere */
@@ -5379,7 +5570,7 @@ Editor::mouse_brush_insert_region (RegionView* rv, nframes_t pos)
                return;
        }
 
-       RouteTimeAxisView* rtv = dynamic_cast<RouteTimeAxisView*>(&arv->get_time_axis_view());
+       RouteTimeAxisView* rtv = dynamic_cast<RouteTimeAxisView*>(&rv->get_time_axis_view());
 
        if (rtv == 0 || !rtv->is_track()) {
                return;
@@ -5389,7 +5580,7 @@ Editor::mouse_brush_insert_region (RegionView* rv, nframes_t pos)
        double speed = rtv->get_diskstream()->speed();
        
        XMLNode &before = playlist->get_state();
-       playlist->add_region (boost::dynamic_pointer_cast<AudioRegion> (RegionFactory::create (arv->audio_region())), (nframes_t) (pos * speed));
+       playlist->add_region (RegionFactory::create (rv->region()), (nframes64_t) (pos * speed));
        XMLNode &after = playlist->get_state();
        session->add_command(new MementoCommand<Playlist>(*playlist.get(), &before, &after));
        
@@ -5401,13 +5592,7 @@ Editor::mouse_brush_insert_region (RegionView* rv, nframes_t pos)
 gint
 Editor::track_height_step_timeout ()
 {
-       struct timeval now;
-       struct timeval delta;
-       
-       gettimeofday (&now, 0);
-       timersub (&now, &last_track_height_step_timestamp, &delta);
-       
-       if (delta.tv_sec * 1000000 + delta.tv_usec > 250000) { /* milliseconds */
+       if (get_microseconds() - last_track_height_step_timestamp < 250000) {
                current_stepping_trackview = 0;
                return false;
        }