Merged with trunk (painfully)
[ardour.git] / gtk2_ardour / editor_mouse.cc
index 0bfd36c6365afa3a235257ed756ef6d7ecaf156f..cc171617a631737925c8694c07c313e2c6d918a3 100644 (file)
@@ -43,7 +43,6 @@
 #include "keyboard.h"
 #include "editing.h"
 #include "rgb_macros.h"
-#include "extra_bind.h"
 
 #include <ardour/types.h>
 #include <ardour/route.h>
@@ -86,14 +85,18 @@ Editor::event_frame (GdkEvent* event, double* pcx, double* pcy)
        case GDK_BUTTON_PRESS:
        case GDK_2BUTTON_PRESS:
        case GDK_3BUTTON_PRESS:
-               gnome_canvas_w2c_d (GNOME_CANVAS(track_gnome_canvas), event->button.x, event->button.y, pcx, pcy);
+               track_canvas.w2c(event->button.x, event->button.y, *pcx, *pcy);
                break;
        case GDK_MOTION_NOTIFY:
-               gnome_canvas_w2c_d (GNOME_CANVAS(track_gnome_canvas), event->motion.x, event->motion.y, pcx, pcy);
+               track_canvas.w2c(event->motion.x, event->motion.y, *pcx, *pcy);
                break;
        case GDK_ENTER_NOTIFY:
        case GDK_LEAVE_NOTIFY:
-               gnome_canvas_w2c_d (GNOME_CANVAS(track_gnome_canvas), 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);
                break;
        default:
                warning << string_compose (_("Editor::event_frame() used on unhandled event type %1"), event->type) << endmsg;
@@ -104,7 +107,7 @@ Editor::event_frame (GdkEvent* event, double* pcx, double* pcy)
           position is negative (as can be the case with motion events in particular),
           the frame location is always positive.
        */
-
+       
        return pixel_to_frame (*pcx);
 }
 
@@ -164,7 +167,7 @@ Editor::set_mouse_mode (MouseMode m, bool force)
                return;
        }
 
-       if (m == mouse_mode && !force) {
+       if (!force && m == mouse_mode) {
                return;
        }
        
@@ -187,13 +190,10 @@ Editor::set_mouse_mode (MouseMode m, bool force)
 
        } else {
 
-               /* in range mode, hide object (region) selection, and show the 
-                  range selection.
+               /* 
+                  in range mode,show the range selection.
                */
 
-               for (AudioRegionSelection::iterator i = selection->audio_regions.begin(); i != selection->audio_regions.end(); ++i) {
-                       (*i)->set_should_show_selection (false);
-               }
                for (TrackSelection::iterator i = selection->tracks.begin(); i != selection->tracks.end(); ++i) {
                        if ((*i)->selected()) {
                                (*i)->show_selection (selection->time);
@@ -243,7 +243,7 @@ Editor::set_mouse_mode (MouseMode m, bool force)
        ignore_mouse_mode_toggle = false;
 
        if (is_drawable()) {
-               gdk_window_set_cursor (track_canvas_scroller.get_window(), current_canvas_cursor);
+               track_canvas.get_window()->set_cursor(*current_canvas_cursor);
        }
 }
 
@@ -283,63 +283,68 @@ Editor::step_mouse_mode (bool next)
        }
 }
 
-gint
-Editor::button_press_handler (GnomeCanvasItem* item, GdkEvent* event, ItemType item_type)
+void
+Editor::button_selection (ArdourCanvas::Item* item, GdkEvent* event, ItemType item_type)
 {
-       jack_nframes_t where = event_frame (event, 0, 0);
-
-       if (session && session->actively_recording()) {
-               return TRUE;
-       }
+       bool commit;
+       bool c1; 
+       bool c2;
 
        /* in object/audition/timefx mode, any button press sets
           the selection if the object can be selected. this is a
           bit of hack, because we want to avoid this if the
           mouse operation is a region alignment.
-       */
 
-       if (((mouse_mode == MouseObject) ||
-            (mouse_mode == MouseAudition && item_type == RegionItem) ||
-            (mouse_mode == MouseTimeFX && item_type == RegionItem)) &&
-           event->type == GDK_BUTTON_PRESS && 
-           event->button.button <= 3) {
-
-               AudioRegionView* rv;
-               ControlPoint* cp;
-
-               /* not dbl-click or triple-click */
-
-               switch (item_type) {
-               case RegionItem:
-                       set_selected_regionview_from_click (Keyboard::modifier_state_equals (event->button.state, Keyboard::Shift), true);
-                       break;
-                       
-               case AudioRegionViewNameHighlight:
-               case AudioRegionViewName:
-                       if ((rv = reinterpret_cast<AudioRegionView *> (gtk_object_get_data(GTK_OBJECT(item), "regionview"))) != 0) {
-                               set_selected_regionview_from_click (Keyboard::modifier_state_equals (event->button.state, Keyboard::Shift), true);
-                       }
-                       break;
-                       
-               case GainAutomationControlPointItem:
-               case PanAutomationControlPointItem:
-               case RedirectAutomationControlPointItem:
-                       if ((cp = reinterpret_cast<ControlPoint *> (gtk_object_get_data(GTK_OBJECT(item), "control_point"))) != 0) {
-                               set_selected_control_point_from_click (Keyboard::modifier_state_equals (event->button.state, Keyboard::Shift), true);
-                       }
-                       break;
+          note: not dbl-click or triple-click
+       */
 
-               case StreamItem:
-                       break;
+       if (((mouse_mode != MouseObject) &&
+            (mouse_mode != MouseAudition || item_type != RegionItem) &&
+            (mouse_mode != MouseTimeFX || item_type != RegionItem)) ||
+           (event->type != GDK_BUTTON_PRESS && event->type != GDK_BUTTON_RELEASE || event->button.button > 3)) {
+               
+               return;
+       }
+           
+       Selection::Operation op = Keyboard::selection_type (event->button.state);
+       bool press = (event->type == GDK_BUTTON_PRESS);
 
-               case AutomationTrackItem:
-                       break;
+       begin_reversible_command (_("select on click"));
 
-               default:
-                       break;
-               }
+       switch (item_type) {
+       case RegionItem:
+               c1 = set_selected_track_from_click (press, op, true, true);
+               c2 = set_selected_regionview_from_click (press, op, true);
+               commit = (c1 || c2);
+               break;
+               
+       case AudioRegionViewNameHighlight:
+       case AudioRegionViewName:
+               c1 = set_selected_track_from_click (press, op, true, true);
+               c2 = set_selected_regionview_from_click (press, op, true);
+               commit = (c1 || c2);
+               break;
+               
+       case GainAutomationControlPointItem:
+       case PanAutomationControlPointItem:
+       case RedirectAutomationControlPointItem:
+               c1 = set_selected_track_from_click (press, op, true, true);
+               c2 = set_selected_control_point_from_click (press, op, false);
+               commit = (c1 || c2);
+               break;
+               
+       case StreamItem:
+               commit = set_selected_track_from_click (press, op, true, true);
+               break;
+                   
+       case AutomationTrackItem:
+               commit = set_selected_track_from_click (press, op, true, true);
+               break;
+               
+       default:
+               break;
        }
-
+       
 #define SELECT_TRACK_FROM_CANVAS_IN_RANGE_MODE
 #ifdef  SELECT_TRACK_FROM_CANVAS_IN_RANGE_MODE
        /* in range mode, button 1/2/3 press potentially selects a track */
@@ -348,31 +353,43 @@ Editor::button_press_handler (GnomeCanvasItem* item, GdkEvent* event, ItemType i
            event->type == GDK_BUTTON_PRESS && 
            event->button.button <= 3) {
                
-               AudioRegionView* rv;
-
                switch (item_type) {
                case StreamItem:
                case RegionItem:
                case AutomationTrackItem:
-                       set_selected_track_from_click (Keyboard::modifier_state_equals (event->button.state, Keyboard::Shift), true, true);
+                       commit = set_selected_track_from_click (press, op, true, true);
                        break;
 
-               case AudioRegionViewNameHighlight:
-               case AudioRegionViewName:
-                       rv = reinterpret_cast<AudioRegionView *> (gtk_object_get_data(GTK_OBJECT(item), "regionview"));
                default:
                        break;
                }
        }
 #endif
+       if (commit) {
+               commit_reversible_command ();
+       }
+}
+
+bool
+Editor::button_press_handler (ArdourCanvas::Item* item, GdkEvent* event, ItemType item_type)
+{
+       jack_nframes_t where = event_frame (event, 0, 0);
+
+       track_canvas.grab_focus();
+
+       if (session && session->actively_recording()) {
+               return true;
+       }
 
+       button_selection (item, event, item_type);
+       
        if (drag_info.item == 0 &&
            (Keyboard::is_delete_event (&event->button) ||
             Keyboard::is_context_menu_event (&event->button) ||
             Keyboard::is_edit_event (&event->button))) {
                
                /* handled by button release */
-               return TRUE;
+               return true;
        }
 
        switch (event->button.button) {
@@ -381,7 +398,7 @@ Editor::button_press_handler (GnomeCanvasItem* item, GdkEvent* event, ItemType i
                if (event->type == GDK_BUTTON_PRESS) {
 
                        if (drag_info.item) {
-                               gnome_canvas_item_ungrab (drag_info.item, event->button.time);
+                               drag_info.item->ungrab (event->button.time);
                        }
 
                        /* single mouse clicks on any of these item types operate
@@ -394,7 +411,7 @@ Editor::button_press_handler (GnomeCanvasItem* item, GdkEvent* event, ItemType i
                        case EditCursorItem:
                        case PlayheadCursorItem:
                                start_cursor_grab (item, event);
-                               return TRUE;
+                               return true;
 
                        case MarkerItem:
                                if (Keyboard::modifier_state_equals (event->button.state, 
@@ -403,29 +420,38 @@ Editor::button_press_handler (GnomeCanvasItem* item, GdkEvent* event, ItemType i
                                } else {
                                        start_marker_grab (item, event);
                                }
-                               return TRUE;
+                               return true;
 
                        case TempoMarkerItem:
-                               start_tempo_marker_grab (item, event);
-                               return TRUE;
+                               if (Keyboard::modifier_state_contains (event->button.state, Keyboard::Control)) {
+                                       start_tempo_marker_copy_grab (item, event);
+                               } else {
+                                       start_tempo_marker_grab (item, event);
+                               }
+                               return true;
 
                        case MeterMarkerItem:
-                               start_meter_marker_grab (item, event);
-                               return TRUE;
+                               if (Keyboard::modifier_state_contains (event->button.state, Keyboard::Control)) {
+                                       start_meter_marker_copy_grab (item, event);
+                               } else {
+                                       start_meter_marker_grab (item, event);
+                               }
+                               return true;
 
                        case TempoBarItem:
-                               return TRUE;
+                               return true;
 
                        case MeterBarItem:
-                               return TRUE;
+                               return true;
                                
                        case RangeMarkerBarItem:
                                start_range_markerbar_op (item, event, CreateRangeMarker); 
-                               return TRUE;
+                               return true;
                                break;
+
                        case TransportMarkerBarItem:
                                start_range_markerbar_op (item, event, CreateTransportMarker); 
-                               return TRUE;
+                               return true;
                                break;
 
                        default:
@@ -463,7 +489,7 @@ Editor::button_press_handler (GnomeCanvasItem* item, GdkEvent* event, ItemType i
                        default:
                                start_selection_op (item, event, CreateSelection);
                        }
-                       return TRUE;
+                       return true;
                        break;
                        
                case MouseObject:
@@ -478,11 +504,11 @@ Editor::button_press_handler (GnomeCanvasItem* item, GdkEvent* event, ItemType i
                                switch (item_type) {
                                case FadeInHandleItem:
                                        start_fade_in_grab (item, event);
-                                       return TRUE;
+                                       return true;
                                        
                                case FadeOutHandleItem:
                                        start_fade_out_grab (item, event);
-                                       return TRUE;
+                                       return true;
 
                                case RegionItem:
                                        if (Keyboard::modifier_state_contains (event->button.state, Keyboard::Control)) {
@@ -496,27 +522,27 @@ Editor::button_press_handler (GnomeCanvasItem* item, GdkEvent* event, ItemType i
                                        
                                case AudioRegionViewNameHighlight:
                                        start_trim (item, event);
-                                       return TRUE;
+                                       return true;
                                        break;
                                        
                                case AudioRegionViewName:
                                        /* rename happens on edit clicks */
                                                start_trim (clicked_regionview->get_name_highlight(), event);
-                                               return TRUE;
+                                               return true;
                                        break;
 
                                case GainAutomationControlPointItem:
                                case PanAutomationControlPointItem:
                                case RedirectAutomationControlPointItem:
                                        start_control_point_grab (item, event);
-                                       return TRUE;
+                                       return true;
                                        break;
                                        
                                case GainAutomationLineItem:
                                case PanAutomationLineItem:
                                case RedirectAutomationLineItem:
                                        start_line_grab_from_line (item, event);
-                                       return TRUE;
+                                       return true;
                                        break;
 
                                case StreamItem:
@@ -527,19 +553,19 @@ Editor::button_press_handler (GnomeCanvasItem* item, GdkEvent* event, ItemType i
                                /* <CMT Additions> */
                                case ImageFrameHandleStartItem:
                                        imageframe_start_handle_op(item, event) ;
-                                       return(TRUE) ;
+                                       return(true) ;
                                        break ;
                                case ImageFrameHandleEndItem:
                                        imageframe_end_handle_op(item, event) ;
-                                       return(TRUE) ;
+                                       return(true) ;
                                        break ;
                                case MarkerViewHandleStartItem:
                                        markerview_item_start_handle_op(item, event) ;
-                                       return(TRUE) ;
+                                       return(true) ;
                                        break ;
                                case MarkerViewHandleEndItem:
                                        markerview_item_end_handle_op(item, event) ;
-                                       return(TRUE) ;
+                                       return(true) ;
                                        break ;
                                /* </CMT Additions> */
                                
@@ -552,11 +578,15 @@ Editor::button_press_handler (GnomeCanvasItem* item, GdkEvent* event, ItemType i
                                        break ;
                                /* </CMT Additions> */
 
+                               case MarkerBarItem:
+                                       
+                                       break;
+
                                default:
                                        break;
                                }
                        }
-                       return TRUE;
+                       return true;
                        break;
                        
                case MouseGain:
@@ -567,23 +597,23 @@ Editor::button_press_handler (GnomeCanvasItem* item, GdkEvent* event, ItemType i
 
                        case GainControlPointItem:
                                start_control_point_grab (item, event);
-                               return TRUE;
+                               return true;
                                
                        case GainLineItem:
                                start_line_grab_from_line (item, event);
-                               return TRUE;
+                               return true;
 
                        case GainAutomationControlPointItem:
                        case PanAutomationControlPointItem:
                        case RedirectAutomationControlPointItem:
                                start_control_point_grab (item, event);
-                               return TRUE;
+                               return true;
                                break;
 
                        default:
                                break;
                        }
-                       return TRUE;
+                       return true;
                        break;
 
                        switch (item_type) {
@@ -608,7 +638,7 @@ Editor::button_press_handler (GnomeCanvasItem* item, GdkEvent* event, ItemType i
                        default:
                                break;
                        }
-                       return TRUE;
+                       return true;
                        break;
 
                case MouseZoom:
@@ -616,7 +646,7 @@ Editor::button_press_handler (GnomeCanvasItem* item, GdkEvent* event, ItemType i
                                start_mouse_zoom (item, event);
                        }
 
-                       return TRUE;
+                       return true;
                        break;
 
                case MouseTimeFX:
@@ -651,7 +681,7 @@ Editor::button_press_handler (GnomeCanvasItem* item, GdkEvent* event, ItemType i
                                case PanAutomationControlPointItem:
                                case RedirectAutomationControlPointItem:
                                        start_control_point_grab (item, event);
-                                       return TRUE;
+                                       return true;
                                        break;
                                        
                                default:
@@ -663,12 +693,12 @@ Editor::button_press_handler (GnomeCanvasItem* item, GdkEvent* event, ItemType i
                        switch (item_type) {
                        case AudioRegionViewNameHighlight:
                                start_trim (item, event);
-                               return TRUE;
+                               return true;
                                break;
                                
                        case AudioRegionViewName:
                                start_trim (clicked_regionview->get_name_highlight(), event);
-                               return TRUE;
+                               return true;
                                break;
                                
                        default:
@@ -681,7 +711,7 @@ Editor::button_press_handler (GnomeCanvasItem* item, GdkEvent* event, ItemType i
                        if (event->type == GDK_BUTTON_PRESS) {
                                /* relax till release */
                        }
-                       return TRUE;
+                       return true;
                        break;
                                        
                                
@@ -691,7 +721,7 @@ Editor::button_press_handler (GnomeCanvasItem* item, GdkEvent* event, ItemType i
                        } else {
                                temporal_zoom_to_frame (true, event_frame(event));
                        }
-                       return TRUE;
+                       return true;
                        break;
 
                default:
@@ -718,6 +748,7 @@ Editor::button_press_handler (GnomeCanvasItem* item, GdkEvent* event, ItemType i
                        
                        if (Keyboard::modifier_state_contains (event->button.state, Keyboard::ModifierMask(Keyboard::Alt))) {
                                scroll_backward (0.6f);
+                               return true;
                        }
                        else if (Keyboard::no_modifier_keys_pressed (&event->button)) {
                                scroll_tracks_up_line ();
@@ -725,8 +756,7 @@ Editor::button_press_handler (GnomeCanvasItem* item, GdkEvent* event, ItemType i
                                if (Keyboard::modifier_state_equals (event->button.state, Keyboard::Shift)) {
                                        if (clicked_trackview) {
                                                if (!current_stepping_trackview) {
-                                                       TimeoutSig t;
-                                                       step_timeout = t.connect (mem_fun(*this, &Editor::track_height_step_timeout), 500);
+                                                 step_timeout = Glib::signal_timeout().connect (mem_fun(*this, &Editor::track_height_step_timeout), 500);
                                                        current_stepping_trackview = clicked_trackview;
                                                }
                                                gettimeofday (&last_track_height_step_timestamp, 0);
@@ -755,6 +785,7 @@ Editor::button_press_handler (GnomeCanvasItem* item, GdkEvent* event, ItemType i
 
                        if (Keyboard::modifier_state_contains (event->button.state, Keyboard::ModifierMask(Keyboard::Alt))) {
                                scroll_forward (0.6f);
+                               return true;
                        }
                        else if (Keyboard::no_modifier_keys_pressed (&event->button)) {
                                scroll_tracks_down_line ();
@@ -762,8 +793,7 @@ Editor::button_press_handler (GnomeCanvasItem* item, GdkEvent* event, ItemType i
                                if (Keyboard::modifier_state_equals (event->button.state, Keyboard::Shift)) {
                                        if (clicked_trackview) {
                                                if (!current_stepping_trackview) {
-                                                       TimeoutSig t;
-                                                       step_timeout = t.connect (mem_fun(*this, &Editor::track_height_step_timeout), 500);
+                                                 step_timeout = Glib::signal_timeout().connect (mem_fun(*this, &Editor::track_height_step_timeout), 500);
                                                        current_stepping_trackview = clicked_trackview;
                                                }
                                                gettimeofday (&last_track_height_step_timestamp, 0);
@@ -780,18 +810,19 @@ Editor::button_press_handler (GnomeCanvasItem* item, GdkEvent* event, ItemType i
                break;
 
        }
-       return FALSE;
+
+       return false;
 }
 
-gint
-Editor::button_release_handler (GnomeCanvasItem* item, GdkEvent* event, ItemType item_type)
+bool
+Editor::button_release_handler (ArdourCanvas::Item* item, GdkEvent* event, ItemType item_type)
 {
        jack_nframes_t where = event_frame (event, 0, 0);
 
        /* no action if we're recording */
                                                
        if (session && session->actively_recording()) {
-               return TRUE;
+               return true;
        }
 
        /* first, see if we're finishing a drag ... */
@@ -799,10 +830,12 @@ Editor::button_release_handler (GnomeCanvasItem* item, GdkEvent* event, ItemType
        if (drag_info.item) {
                if (end_grab (item, event)) {
                        /* grab dragged, so do nothing else */
-                       return TRUE;
+                       return true;
                }
        }
        
+       button_selection (item, event, item_type);
+
        /* edit events get handled here */
        
        if (drag_info.item == 0 && Keyboard::is_edit_event (&event->button)) {
@@ -828,7 +861,7 @@ Editor::button_release_handler (GnomeCanvasItem* item, GdkEvent* event, ItemType
                default:
                        break;
                }
-               return TRUE;
+               return true;
        }
 
        /* context menu events get handled here */
@@ -911,7 +944,7 @@ Editor::button_release_handler (GnomeCanvasItem* item, GdkEvent* event, ItemType
                                break;
                        }
 
-                       return TRUE;
+                       return true;
                }
        }
 
@@ -929,7 +962,7 @@ Editor::button_release_handler (GnomeCanvasItem* item, GdkEvent* event, ItemType
                        break;
 
                case MarkerItem:
-                       remove_marker (item, event);
+                       remove_marker (*item, event);
                        break;
 
                case RegionItem:
@@ -953,7 +986,7 @@ Editor::button_release_handler (GnomeCanvasItem* item, GdkEvent* event, ItemType
                default:
                        break;
                }
-               return TRUE;
+               return true;
        }
 
        switch (event->button.button) {
@@ -970,25 +1003,25 @@ Editor::button_release_handler (GnomeCanvasItem* item, GdkEvent* event, ItemType
                case RedirectAutomationLineItem:
                case StartSelectionTrimItem:
                case EndSelectionTrimItem:
-                       return TRUE;
+                       return true;
 
                case MarkerBarItem:
                        if (!Keyboard::modifier_state_contains (event->button.state, Keyboard::snap_modifier())) {
                                snap_to (where, 0, true);
                        }
                        mouse_add_new_marker (where);
-                       return TRUE;
+                       return true;
 
                case TempoBarItem:
                        if (!Keyboard::modifier_state_contains (event->button.state, Keyboard::snap_modifier())) {
                                snap_to (where);
                        }
                        mouse_add_new_tempo_event (where);
-                       return TRUE;
+                       return true;
                        
                case MeterBarItem:
                        mouse_add_new_meter_event (pixel_to_frame (event->button.x));
-                       return TRUE;
+                       return true;
                        break;
 
                default:
@@ -999,12 +1032,12 @@ Editor::button_release_handler (GnomeCanvasItem* item, GdkEvent* event, ItemType
                case MouseObject:
                        switch (item_type) {
                        case AutomationTrackItem:
-                               dynamic_cast<AutomationTimeAxisView*>(clicked_trackview)->signal_add_automation_event() 
+                               dynamic_cast<AutomationTimeAxisView*>(clicked_trackview)->add_automation_event 
                                        (item,
                                         event,
                                         where,
                                         event->button.y);
-                               return TRUE;
+                               return true;
                                break;
 
                        default:
@@ -1015,14 +1048,14 @@ Editor::button_release_handler (GnomeCanvasItem* item, GdkEvent* event, ItemType
                case MouseGain:
                        switch (item_type) {
                        case RegionItem:
-                               clicked_regionview->signal_add_gain_point_event() (item, event);
-                               return TRUE;
+                               clicked_regionview->add_gain_point_event (item, event);
+                               return true;
                                break;
                                
                        case AutomationTrackItem:
                                dynamic_cast<AutomationTimeAxisView*>(clicked_trackview)->
                                        add_automation_event (item, event, where, event->button.y);
-                               return TRUE;
+                               return true;
                                break;
                        default:
                                break;
@@ -1044,7 +1077,7 @@ Editor::button_release_handler (GnomeCanvasItem* item, GdkEvent* event, ItemType
 
                }
 
-               return TRUE;
+               return true;
                break;
 
 
@@ -1061,7 +1094,7 @@ Editor::button_release_handler (GnomeCanvasItem* item, GdkEvent* event, ItemType
                                } else {
                                        // Button2 click is unused
                                }
-                               return TRUE;
+                               return true;
                                
                                break;
                                
@@ -1073,7 +1106,7 @@ Editor::button_release_handler (GnomeCanvasItem* item, GdkEvent* event, ItemType
                case MouseRange:
                        
                        // x_style_paste (where, 1.0);
-                       return TRUE;
+                       return true;
                        break;
                        
                default:
@@ -1088,36 +1121,11 @@ Editor::button_release_handler (GnomeCanvasItem* item, GdkEvent* event, ItemType
        default:
                break;
        }
-       return FALSE;
+       return false;
 }
 
-void
-Editor::maybe_autoscroll (GdkEvent* event)
-{
-       jack_nframes_t one_page = (jack_nframes_t) rint (canvas_width * frames_per_unit);
-       jack_nframes_t rightmost_frame = leftmost_frame + one_page;
-
-       jack_nframes_t frame = drag_info.current_pointer_frame;
-
-       if (autoscroll_timeout_tag < 0) {
-               if (frame > rightmost_frame) {
-                       if (rightmost_frame < max_frames) {
-                               start_canvas_autoscroll (1);
-                       }
-               } else if (frame < leftmost_frame) {
-                       if (leftmost_frame > 0) {
-                               start_canvas_autoscroll (-1);
-                       }
-               } 
-       } else {
-               if (frame >= leftmost_frame && frame < rightmost_frame) {
-                       stop_canvas_autoscroll ();
-               }
-       }
-}
-
-gint
-Editor::enter_handler (GnomeCanvasItem* item, GdkEvent* event, ItemType item_type)
+bool
+Editor::enter_handler (ArdourCanvas::Item* item, GdkEvent* event, ItemType item_type)
 {
        ControlPoint* cp;
        Marker * marker;
@@ -1126,13 +1134,13 @@ Editor::enter_handler (GnomeCanvasItem* item, GdkEvent* event, ItemType item_typ
        switch (item_type) {
        case GainControlPointItem:
                if (mouse_mode == MouseGain) {
-                       cp = reinterpret_cast<ControlPoint*>(gtk_object_get_data (GTK_OBJECT(item), "control_point"));
+                       cp = static_cast<ControlPoint*>(item->get_data ("control_point"));
                        cp->set_visible (true);
 
                        double at_x, at_y;
                        at_x = cp->get_x();
                        at_y = cp->get_y ();
-                       gnome_canvas_item_i2w (cp->item, &at_x, &at_y);
+                       cp->item->i2w (at_x, at_y);
                        at_x += 20.0;
                        at_y += 20.0;
 
@@ -1142,7 +1150,7 @@ Editor::enter_handler (GnomeCanvasItem* item, GdkEvent* event, ItemType item_typ
                        show_verbose_canvas_cursor ();
 
                        if (is_drawable()) {
-                               gdk_window_set_cursor (track_canvas_scroller.get_window(), fader_cursor);
+                               track_canvas.get_window()->set_cursor (*fader_cursor);
                        }
                }
                break;
@@ -1150,13 +1158,13 @@ Editor::enter_handler (GnomeCanvasItem* item, GdkEvent* event, ItemType item_typ
        case GainAutomationControlPointItem:
        case PanAutomationControlPointItem:
        case RedirectAutomationControlPointItem:
-               cp = reinterpret_cast<ControlPoint*>(gtk_object_get_data (GTK_OBJECT(item), "control_point"));
+               cp = static_cast<ControlPoint*>(item->get_data ("control_point"));
                cp->set_visible (true);
                
                double at_x, at_y;
                at_x = cp->get_x();
                at_y = cp->get_y ();
-               gnome_canvas_item_i2w (cp->item, &at_x, &at_y);
+               cp->item->i2w (at_x, at_y);
                at_x += 20.0;
                at_y += 20.0;
 
@@ -1166,15 +1174,17 @@ Editor::enter_handler (GnomeCanvasItem* item, GdkEvent* event, ItemType item_typ
                show_verbose_canvas_cursor ();
                
                if (is_drawable()) {
-                       gdk_window_set_cursor (track_canvas_scroller.get_window(), fader_cursor);
+                       track_canvas.get_window()->set_cursor (*fader_cursor);
                }
                break;
                
        case GainLineItem:
                if (mouse_mode == MouseGain) {
-                       gnome_canvas_item_set (item, "fill_color_rgba", color_map[cEnteredGainLine], NULL);
+                       ArdourCanvas::Line *line = dynamic_cast<ArdourCanvas::Line *> (item);
+                       if (line)
+                               line->property_fill_color_rgba() = color_map[cEnteredGainLine];
                        if (is_drawable()) {
-                               gdk_window_set_cursor (track_canvas_scroller.get_window(), fader_cursor);
+                               track_canvas.get_window()->set_cursor (*fader_cursor);
                        }
                }
                break;
@@ -1182,15 +1192,19 @@ Editor::enter_handler (GnomeCanvasItem* item, GdkEvent* event, ItemType item_typ
        case GainAutomationLineItem:
        case RedirectAutomationLineItem:
        case PanAutomationLineItem:
-               gnome_canvas_item_set (item, "fill_color_rgba", color_map[cEnteredAutomationLine], NULL);
+               {
+                       ArdourCanvas::Line *line = dynamic_cast<ArdourCanvas::Line *> (item);
+                       if (line)
+                               line->property_fill_color_rgba() = color_map[cEnteredAutomationLine];
+               }
                if (is_drawable()) {
-                       gdk_window_set_cursor (track_canvas_scroller.get_window(), fader_cursor);
+                       track_canvas.get_window()->set_cursor (*fader_cursor);
                }
                break;
                
        case AudioRegionViewNameHighlight:
                if (is_drawable() && mouse_mode == MouseObject) {
-                       gdk_window_set_cursor (track_canvas_scroller.get_window(), trimmer_cursor);
+                       track_canvas.get_window()->set_cursor (*trimmer_cursor);
                }
                break;
 
@@ -1204,14 +1218,14 @@ Editor::enter_handler (GnomeCanvasItem* item, GdkEvent* event, ItemType item_typ
        /* </CMT Additions> */
 
                if (is_drawable()) {
-                       gdk_window_set_cursor (track_canvas_scroller.get_window(), trimmer_cursor);
+                       track_canvas.get_window()->set_cursor (*trimmer_cursor);
                }
                break;
 
        case EditCursorItem:
        case PlayheadCursorItem:
                if (is_drawable()) {
-                       gdk_window_set_cursor (track_canvas_scroller.get_window(), grabber_cursor);
+                       track_canvas.get_window()->set_cursor (*grabber_cursor);
                }
                break;
 
@@ -1219,9 +1233,9 @@ Editor::enter_handler (GnomeCanvasItem* item, GdkEvent* event, ItemType item_typ
                
                /* when the name is not an active item, the entire name highlight is for trimming */
 
-               if (!reinterpret_cast<AudioRegionView *> (gtk_object_get_data(GTK_OBJECT(item), "regionview"))->name_active()) {
+               if (!reinterpret_cast<AudioRegionView *> (item->get_data ("regionview"))->name_active()) {
                        if (mouse_mode == MouseObject && is_drawable()) {
-                               gdk_window_set_cursor (track_canvas_scroller.get_window(), trimmer_cursor);
+                               track_canvas.get_window()->set_cursor (*trimmer_cursor);
                        }
                } 
                break;
@@ -1229,7 +1243,7 @@ Editor::enter_handler (GnomeCanvasItem* item, GdkEvent* event, ItemType item_typ
 
        case AutomationTrackItem:
                if (is_drawable()) {
-                       GdkCursor *cursor;
+                       Gdk::Cursor *cursor;
                        switch (mouse_mode) {
                        case MouseRange:
                                cursor = selector_cursor;
@@ -1242,10 +1256,10 @@ Editor::enter_handler (GnomeCanvasItem* item, GdkEvent* event, ItemType item_typ
                                break;
                        }
 
-                       gdk_window_set_cursor (track_canvas_scroller.get_window(), cursor);
+                       track_canvas.get_window()->set_cursor (*cursor);
 
                        AutomationTimeAxisView* atv;
-                       if ((atv = static_cast<AutomationTimeAxisView*>(gtk_object_get_data(GTK_OBJECT(item), "trackview"))) != 0) {
+                       if ((atv = static_cast<AutomationTimeAxisView*>(item->get_data ("trackview"))) != 0) {
                                clear_entered_track = false;
                                set_entered_track (atv);
                        }
@@ -1258,12 +1272,12 @@ Editor::enter_handler (GnomeCanvasItem* item, GdkEvent* event, ItemType item_typ
        case MeterBarItem:
        case TempoBarItem:
                if (is_drawable()) {
-                       gdk_window_set_cursor (time_canvas_scroller.get_window(), timebar_cursor);
+                       time_canvas.get_window()->set_cursor (*timebar_cursor);
                }
                break;
 
        case MarkerItem:
-               if ((marker = static_cast<Marker *> (gtk_object_get_data (GTK_OBJECT(item), "marker"))) == 0) {
+               if ((marker = static_cast<Marker *> (item->get_data ("marker"))) == 0) {
                        break;
                }
                marker->set_color_rgba (color_map[cEnteredMarker]);
@@ -1271,13 +1285,17 @@ Editor::enter_handler (GnomeCanvasItem* item, GdkEvent* event, ItemType item_typ
        case MeterMarkerItem:
        case TempoMarkerItem:
                if (is_drawable()) {
-                       gdk_window_set_cursor (time_canvas_scroller.get_window(), timebar_cursor);
+                       time_canvas.get_window()->set_cursor (*timebar_cursor);
                }
                break;
        case FadeInHandleItem:
        case FadeOutHandleItem:
                if (mouse_mode == MouseObject) {
-                       gnome_canvas_item_set (item, "fill_color_rgba", 0, "outline_pixels", 1, NULL);
+                       ArdourCanvas::SimpleRect *rect = dynamic_cast<ArdourCanvas::SimpleRect *> (item);
+                       if (rect) {
+                               rect->property_fill_color_rgba() = 0;
+                               rect->property_outline_pixels() = 1;
+                       }
                }
                break;
 
@@ -1310,11 +1328,11 @@ Editor::enter_handler (GnomeCanvasItem* item, GdkEvent* event, ItemType item_typ
                break;
        }
 
-       return FALSE;
+       return false;
 }
 
-gint
-Editor::leave_handler (GnomeCanvasItem* item, GdkEvent* event, ItemType item_type)
+bool
+Editor::leave_handler (ArdourCanvas::Item* item, GdkEvent* event, ItemType item_type)
 {
        AutomationLine* al;
        ControlPoint* cp;
@@ -1328,7 +1346,7 @@ Editor::leave_handler (GnomeCanvasItem* item, GdkEvent* event, ItemType item_typ
        case GainAutomationControlPointItem:
        case PanAutomationControlPointItem:
        case RedirectAutomationControlPointItem:
-               cp = reinterpret_cast<ControlPoint*>(gtk_object_get_data (GTK_OBJECT(item), "control_point"));
+               cp = reinterpret_cast<ControlPoint*>(item->get_data ("control_point"));
                if (cp->line.npoints() > 1) {
                        if (!cp->selected) {
                                cp->set_visible (false);
@@ -1336,7 +1354,7 @@ Editor::leave_handler (GnomeCanvasItem* item, GdkEvent* event, ItemType item_typ
                }
                
                if (is_drawable()) {
-                       gdk_window_set_cursor (track_canvas_scroller.get_window(), current_canvas_cursor);
+                       track_canvas.get_window()->set_cursor (*current_canvas_cursor);
                }
 
                hide_verbose_canvas_cursor ();
@@ -1354,7 +1372,7 @@ Editor::leave_handler (GnomeCanvasItem* item, GdkEvent* event, ItemType item_typ
        case MarkerViewHandleEndItem:
        /* </CMT Additions> */
                if (is_drawable()) {
-                       gdk_window_set_cursor (track_canvas_scroller.get_window(), current_canvas_cursor);
+                       track_canvas.get_window()->set_cursor (*current_canvas_cursor);
                }
                break;
 
@@ -1362,18 +1380,22 @@ Editor::leave_handler (GnomeCanvasItem* item, GdkEvent* event, ItemType item_typ
        case GainAutomationLineItem:
        case RedirectAutomationLineItem:
        case PanAutomationLineItem:
-               al = reinterpret_cast<AutomationLine*> (gtk_object_get_data (GTK_OBJECT(item),"line"));
-               gnome_canvas_item_set (item, "fill_color_rgba", al->get_line_color(), NULL);
+               al = reinterpret_cast<AutomationLine*> (item->get_data ("line"));
+               {
+                       ArdourCanvas::Line *line = dynamic_cast<ArdourCanvas::Line *> (item);
+                       if (line)
+                               line->property_fill_color_rgba() = al->get_line_color();
+               }
                if (is_drawable()) {
-                       gdk_window_set_cursor (track_canvas_scroller.get_window(), current_canvas_cursor);
+                       track_canvas.get_window()->set_cursor (*current_canvas_cursor);
                }
                break;
 
        case AudioRegionViewName:
                /* see enter_handler() for notes */
-               if (!reinterpret_cast<AudioRegionView *> (gtk_object_get_data(GTK_OBJECT(item), "regionview"))->name_active()) {
+               if (!reinterpret_cast<AudioRegionView *> (item->get_data ("regionview"))->name_active()) {
                        if (is_drawable() && mouse_mode == MouseObject) {
-                               gdk_window_set_cursor (track_canvas_scroller.get_window(), current_canvas_cursor);
+                               track_canvas.get_window()->set_cursor (*current_canvas_cursor);
                        }
                }
                break;
@@ -1384,12 +1406,12 @@ Editor::leave_handler (GnomeCanvasItem* item, GdkEvent* event, ItemType item_typ
        case TempoBarItem:
        case MarkerBarItem:
                if (is_drawable()) {
-                       gdk_window_set_cursor (time_canvas_scroller.get_window(), timebar_cursor);
+                       time_canvas.get_window()->set_cursor (*timebar_cursor);
                }
                break;
                
        case MarkerItem:
-               if ((marker = static_cast<Marker *> (gtk_object_get_data (GTK_OBJECT(item), "marker"))) == 0) {
+               if ((marker = static_cast<Marker *> (item->get_data ("marker"))) == 0) {
                        break;
                }
                loc = find_location_from_marker (marker, is_start);
@@ -1399,23 +1421,28 @@ Editor::leave_handler (GnomeCanvasItem* item, GdkEvent* event, ItemType item_typ
        case TempoMarkerItem:
                
                if (is_drawable()) {
-                       gdk_window_set_cursor (time_canvas_scroller.get_window(), timebar_cursor);
+                       time_canvas.get_window()->set_cursor (*timebar_cursor);
                }
 
                break;
 
        case FadeInHandleItem:
        case FadeOutHandleItem:
-               rv = static_cast<AudioRegionView*>(gtk_object_get_data (GTK_OBJECT(item), "regionview"));
-               gnome_canvas_item_set (item, "fill_color_rgba", rv->get_fill_color(), "outline_pixels", 0, NULL);
+               rv = static_cast<AudioRegionView*>(item->get_data ("regionview"));
+               {
+                       ArdourCanvas::SimpleRect *rect = dynamic_cast<ArdourCanvas::SimpleRect *> (item);
+                       if (rect) {
+                               rect->property_fill_color_rgba() = rv->get_fill_color();
+                               rect->property_outline_pixels() = 0;
+                       }
+               }
                break;
 
        case AutomationTrackItem:
                if (is_drawable()) {
-                       gdk_window_set_cursor (track_canvas_scroller.get_window(), current_canvas_cursor);
-
+                       track_canvas.get_window()->set_cursor (*current_canvas_cursor);
                        clear_entered_track = true;
-                       Main::idle.connect (mem_fun(*this, &Editor::left_automation_track));
+                       Glib::signal_idle().connect (mem_fun(*this, &Editor::left_automation_track));
                }
                break;
                
@@ -1423,7 +1450,7 @@ Editor::leave_handler (GnomeCanvasItem* item, GdkEvent* event, ItemType item_typ
                break;
        }
 
-       return FALSE;
+       return false;
 }
 
 gint
@@ -1433,11 +1460,11 @@ Editor::left_automation_track ()
                set_entered_track (0);
                clear_entered_track = false;
        }
-       return FALSE;
+       return false;
 }
 
-gint
-Editor::motion_handler (GnomeCanvasItem* item, GdkEvent* event, ItemType item_type)
+bool
+Editor::motion_handler (ArdourCanvas::Item* item, GdkEvent* event, ItemType item_type, bool from_autoscroll)
 {
        gint x, y;
        
@@ -1450,7 +1477,7 @@ Editor::motion_handler (GnomeCanvasItem* item, GdkEvent* event, ItemType item_ty
           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 */
@@ -1460,21 +1487,23 @@ Editor::motion_handler (GnomeCanvasItem* item, GdkEvent* event, ItemType item_ty
 
        if (session && session->actively_recording()) {
                /* Sorry. no dragging stuff around while we record */
-               return TRUE;
+               return true;
        }
 
+       drag_info.item_type = item_type;
        drag_info.current_pointer_frame = event_frame (event, &drag_info.current_pointer_x,
                                                       &drag_info.current_pointer_y);
-       if (drag_info.item) {
-               /* item != 0 is the best test i can think of for 
-                  dragging.
+
+       if (!from_autoscroll && drag_info.item) {
+               /* item != 0 is the best test i can think of for dragging.
                */
-               if (!drag_info.move_threshold_passsed)
-               {
-                       drag_info.move_threshold_passsed = (abs ((int) (drag_info.current_pointer_x - drag_info.grab_x)) > 4);
+               if (!drag_info.move_threshold_passed) {
 
+                       drag_info.move_threshold_passed = (abs ((int) (drag_info.current_pointer_x - drag_info.grab_x)) > 4);
+                       
                        // and change the initial grab loc/frame if this drag info wants us to
-                       if (drag_info.want_move_threshold && drag_info.move_threshold_passsed) {
+
+                       if (drag_info.want_move_threshold && drag_info.move_threshold_passed) {
                                drag_info.grab_frame = drag_info.current_pointer_frame;
                                drag_info.grab_x = drag_info.current_pointer_x;
                                drag_info.grab_y = drag_info.current_pointer_y;
@@ -1510,14 +1539,16 @@ Editor::motion_handler (GnomeCanvasItem* item, GdkEvent* event, ItemType item_ty
        case MarkerViewHandleStartItem:
        case MarkerViewHandleEndItem:
        /* </CMT Additions> */
-               if (drag_info.item && (event->motion.state & GDK_BUTTON1_MASK ||
-                                      (event->motion.state & GDK_BUTTON2_MASK))) {
-                       maybe_autoscroll (event);
-                       (this->*(drag_info.motion_callback)) (item, event);
-                       goto handled;
-               }
-               goto not_handled;
-
+         if (drag_info.item && (event->motion.state & Gdk::BUTTON1_MASK ||
+                                (event->motion.state & Gdk::BUTTON2_MASK))) {
+                 if (!from_autoscroll) {
+                         maybe_autoscroll (event);
+                 }
+                 (this->*(drag_info.motion_callback)) (item, event);
+                 goto handled;
+         }
+         goto not_handled;
+         
        default:
                break;
        }
@@ -1529,7 +1560,9 @@ Editor::motion_handler (GnomeCanvasItem* item, GdkEvent* event, ItemType item_ty
        case MouseTimeFX:
                if (drag_info.item && (event->motion.state & GDK_BUTTON1_MASK ||
                                       (event->motion.state & GDK_BUTTON2_MASK))) {
-                       maybe_autoscroll (event);
+                       if (!from_autoscroll) {
+                               maybe_autoscroll (event);
+                       }
                        (this->*(drag_info.motion_callback)) (item, event);
                        goto handled;
                }
@@ -1541,15 +1574,16 @@ Editor::motion_handler (GnomeCanvasItem* item, GdkEvent* event, ItemType item_ty
        }
 
   handled:
-       track_canvas_motion (item, event);
-       return TRUE;
+       track_canvas_motion (event);
+       // drag_info.last_pointer_frame = drag_info.current_pointer_frame;
+       return true;
        
   not_handled:
-       return FALSE;
+       return false;
 }
 
 void
-Editor::start_grab (GdkEvent* event, GdkCursor *cursor)
+Editor::start_grab (GdkEvent* event, Gdk::Cursor *cursor)
 {
        if (drag_info.item == 0) {
                fatal << _("programming error: start_grab called without drag item") << endmsg;
@@ -1564,9 +1598,16 @@ Editor::start_grab (GdkEvent* event, GdkCursor *cursor)
         // if dragging with button2, the motion is x constrained, with Alt-button2 it is y constrained
 
        if (event->button.button == 2) {
-               drag_info.x_constrained = true;
+               if (Keyboard::modifier_state_equals (event->button.state, Keyboard::Alt)) {
+                       drag_info.y_constrained = true;
+                       drag_info.x_constrained = false;
+               } else {
+                       drag_info.y_constrained = false;
+                       drag_info.x_constrained = true;
+               }
        } else {
                drag_info.x_constrained = false;
+               drag_info.y_constrained = false;
        }
 
        drag_info.grab_frame = event_frame(event, &drag_info.grab_x, &drag_info.grab_y);
@@ -1577,15 +1618,14 @@ Editor::start_grab (GdkEvent* event, GdkCursor *cursor)
        drag_info.cumulative_x_drag = 0;
        drag_info.cumulative_y_drag = 0;
        drag_info.first_move = true;
-       drag_info.move_threshold_passsed = false;
+       drag_info.move_threshold_passed = false;
        drag_info.want_move_threshold = false;
        drag_info.pointer_frame_offset = 0;
        drag_info.brushing = false;
        drag_info.copied_location = 0;
 
-       gnome_canvas_item_grab (drag_info.item,
-                             Gdk::POINTER_MOTION_MASK|Gdk::BUTTON_PRESS_MASK|Gdk::BUTTON_RELEASE_MASK,
-                             cursor,
+       drag_info.item->grab (Gdk::POINTER_MOTION_MASK|Gdk::BUTTON_PRESS_MASK|Gdk::BUTTON_RELEASE_MASK,
+                             *cursor,
                              event->button.time);
 
        if (session && session->transport_rolling()) {
@@ -1606,8 +1646,21 @@ Editor::start_grab (GdkEvent* event, GdkCursor *cursor)
        }
 }
 
+void
+Editor::swap_grab (ArdourCanvas::Item* new_item, Gdk::Cursor* cursor, uint32_t time)
+{
+       drag_info.item->ungrab (0);
+       drag_info.item = new_item;
+
+       if (cursor == 0) {
+               cursor = grabber_cursor;
+       }
+
+       drag_info.item->grab (Gdk::POINTER_MOTION_MASK|Gdk::BUTTON_PRESS_MASK|Gdk::BUTTON_RELEASE_MASK, *cursor, time);
+}
+
 bool
-Editor::end_grab (GnomeCanvasItem* item, GdkEvent* event)
+Editor::end_grab (ArdourCanvas::Item* item, GdkEvent* event)
 {
        bool did_drag = false;
 
@@ -1617,7 +1670,7 @@ Editor::end_grab (GnomeCanvasItem* item, GdkEvent* event)
                return false;
        }
        
-       gnome_canvas_item_ungrab (drag_info.item, event->button.time);
+       drag_info.item->ungrab (event->button.time);
 
        if (drag_info.finished_callback) {
                (this->*(drag_info.finished_callback)) (item, event);
@@ -1676,7 +1729,7 @@ Editor::set_playhead_cursor (GdkEvent* event)
 }
 
 void
-Editor::start_fade_in_grab (GnomeCanvasItem* item, GdkEvent* event)
+Editor::start_fade_in_grab (ArdourCanvas::Item* item, GdkEvent* event)
 {
        drag_info.item = item;
        drag_info.motion_callback = &Editor::fade_in_drag_motion_callback;
@@ -1684,7 +1737,7 @@ Editor::start_fade_in_grab (GnomeCanvasItem* item, GdkEvent* event)
 
        start_grab (event);
 
-       if ((drag_info.data = (gtk_object_get_data (GTK_OBJECT(item), "regionview"))) == 0) {
+       if ((drag_info.data = (item->get_data ("regionview"))) == 0) {
                fatal << _("programming error: fade in canvas item has no regionview data pointer!") << endmsg;
                /*NOTREACHED*/
        }
@@ -1695,7 +1748,7 @@ Editor::start_fade_in_grab (GnomeCanvasItem* item, GdkEvent* event)
 }
 
 void
-Editor::fade_in_drag_motion_callback (GnomeCanvasItem* item, GdkEvent* event)
+Editor::fade_in_drag_motion_callback (ArdourCanvas::Item* item, GdkEvent* event)
 {
        AudioRegionView* arv = static_cast<AudioRegionView*>(drag_info.data);
        jack_nframes_t pos;
@@ -1728,7 +1781,7 @@ Editor::fade_in_drag_motion_callback (GnomeCanvasItem* item, GdkEvent* event)
 }
 
 void
-Editor::fade_in_drag_finished_callback (GnomeCanvasItem* item, GdkEvent* event)
+Editor::fade_in_drag_finished_callback (ArdourCanvas::Item* item, GdkEvent* event)
 {
        if (drag_info.first_move) return;
 
@@ -1766,7 +1819,7 @@ Editor::fade_in_drag_finished_callback (GnomeCanvasItem* item, GdkEvent* event)
 }
 
 void
-Editor::start_fade_out_grab (GnomeCanvasItem* item, GdkEvent* event)
+Editor::start_fade_out_grab (ArdourCanvas::Item* item, GdkEvent* event)
 {
        drag_info.item = item;
        drag_info.motion_callback = &Editor::fade_out_drag_motion_callback;
@@ -1774,7 +1827,7 @@ Editor::start_fade_out_grab (GnomeCanvasItem* item, GdkEvent* event)
 
        start_grab (event);
 
-       if ((drag_info.data = (gtk_object_get_data (GTK_OBJECT(item), "regionview"))) == 0) {
+       if ((drag_info.data = (item->get_data ("regionview"))) == 0) {
                fatal << _("programming error: fade out canvas item has no regionview data pointer!") << endmsg;
                /*NOTREACHED*/
        }
@@ -1785,7 +1838,7 @@ Editor::start_fade_out_grab (GnomeCanvasItem* item, GdkEvent* event)
 }
 
 void
-Editor::fade_out_drag_motion_callback (GnomeCanvasItem* item, GdkEvent* event)
+Editor::fade_out_drag_motion_callback (ArdourCanvas::Item* item, GdkEvent* event)
 {
        AudioRegionView* arv = static_cast<AudioRegionView*>(drag_info.data);
        jack_nframes_t pos;
@@ -1820,7 +1873,7 @@ Editor::fade_out_drag_motion_callback (GnomeCanvasItem* item, GdkEvent* event)
 }
 
 void
-Editor::fade_out_drag_finished_callback (GnomeCanvasItem* item, GdkEvent* event)
+Editor::fade_out_drag_finished_callback (ArdourCanvas::Item* item, GdkEvent* event)
 {
        if (drag_info.first_move) return;
 
@@ -1859,7 +1912,7 @@ Editor::fade_out_drag_finished_callback (GnomeCanvasItem* item, GdkEvent* event)
 }
 
 void
-Editor::start_cursor_grab (GnomeCanvasItem* item, GdkEvent* event)
+Editor::start_cursor_grab (ArdourCanvas::Item* item, GdkEvent* event)
 {
        drag_info.item = item;
        drag_info.motion_callback = &Editor::cursor_drag_motion_callback;
@@ -1867,7 +1920,7 @@ Editor::start_cursor_grab (GnomeCanvasItem* item, GdkEvent* event)
 
        start_grab (event);
 
-       if ((drag_info.data = (gtk_object_get_data (GTK_OBJECT(item), "cursor"))) == 0) {
+       if ((drag_info.data = (item->get_data ("cursor"))) == 0) {
                fatal << _("programming error: cursor canvas item has no cursor data pointer!") << endmsg;
                /*NOTREACHED*/
        }
@@ -1886,7 +1939,7 @@ Editor::start_cursor_grab (GnomeCanvasItem* item, GdkEvent* event)
 }
 
 void
-Editor::cursor_drag_motion_callback (GnomeCanvasItem* item, GdkEvent* event)
+Editor::cursor_drag_motion_callback (ArdourCanvas::Item* item, GdkEvent* event)
 {
        Cursor* cursor = (Cursor *) drag_info.data;
        jack_nframes_t adjusted_frame;
@@ -1919,17 +1972,17 @@ Editor::cursor_drag_motion_callback (GnomeCanvasItem* item, GdkEvent* event)
 }
 
 void
-Editor::cursor_drag_finished_callback (GnomeCanvasItem* item, GdkEvent* event)
+Editor::cursor_drag_finished_callback (ArdourCanvas::Item* item, GdkEvent* event)
 {
        if (drag_info.first_move) return;
        
        cursor_drag_motion_callback (item, event);
        
-       if (item == playhead_cursor->canvas_item) {
+       if (item == &playhead_cursor->canvas_item) {
                if (session) {
                        session->request_locate (playhead_cursor->current_frame, drag_info.was_rolling);
                }
-       } else if (item == edit_cursor->canvas_item) {
+       } else if (item == &edit_cursor->canvas_item) {
                edit_cursor->set_position (edit_cursor->current_frame);
                edit_cursor_clock.set (edit_cursor->current_frame);
        } 
@@ -1942,21 +1995,22 @@ Editor::update_marker_drag_item (Location *location)
        double x2 = frame_to_pixel (location->end());
 
        if (location->is_mark()) {
-               marker_drag_line_points->coords[0] = x1;
-               marker_drag_line_points->coords[2] = x1;
-               gnome_canvas_item_set (marker_drag_line, "points", marker_drag_line_points, NULL);
+               marker_drag_line_points.front().set_x(x1);
+               marker_drag_line_points.back().set_x(x1);
+               marker_drag_line->property_points() = marker_drag_line_points;
        }
        else {
-               gnome_canvas_item_set (range_marker_drag_rect, "x1", x1, "x2", x2, NULL);
+               range_marker_drag_rect->property_x1() = x1;
+               range_marker_drag_rect->property_x2() = x2;
        }
 }
 
 void
-Editor::start_marker_grab (GnomeCanvasItem* item, GdkEvent* event)
+Editor::start_marker_grab (ArdourCanvas::Item* item, GdkEvent* event)
 {
        Marker* marker;
 
-       if ((marker = static_cast<Marker *> (gtk_object_get_data (GTK_OBJECT(item), "marker"))) == 0) {
+       if ((marker = static_cast<Marker *> (item->get_data ("marker"))) == 0) {
                fatal << _("programming error: marker canvas item has no marker object pointer!") << endmsg;
                /*NOTREACHED*/
        }
@@ -1978,12 +2032,12 @@ Editor::start_marker_grab (GnomeCanvasItem* item, GdkEvent* event)
        update_marker_drag_item (location);
 
        if (location->is_mark()) {
-               gnome_canvas_item_show (marker_drag_line);
-               gnome_canvas_item_raise_to_top (marker_drag_line);
+               marker_drag_line->show();
+               marker_drag_line->raise_to_top();
        }
        else {
-               gnome_canvas_item_show (range_marker_drag_rect);
-               gnome_canvas_item_raise_to_top (range_marker_drag_rect);
+               range_marker_drag_rect->show();
+               range_marker_drag_rect->raise_to_top();
        }
        
        if (is_start) show_verbose_time_cursor (location->start(), 10);
@@ -1991,7 +2045,7 @@ Editor::start_marker_grab (GnomeCanvasItem* item, GdkEvent* event)
 }
 
 void
-Editor::marker_drag_motion_callback (GnomeCanvasItem* item, GdkEvent* event)
+Editor::marker_drag_motion_callback (ArdourCanvas::Item* item, GdkEvent* event)
 {
        jack_nframes_t f_delta; 
        Marker* marker = (Marker *) drag_info.data;
@@ -2000,6 +2054,7 @@ Editor::marker_drag_motion_callback (GnomeCanvasItem* item, GdkEvent* event)
        bool is_start;
        bool move_both = false;
 
+
        jack_nframes_t newframe;
        if (drag_info.pointer_frame_offset <= (long) drag_info.current_pointer_frame) {
                newframe = drag_info.current_pointer_frame - drag_info.pointer_frame_offset;
@@ -2007,14 +2062,16 @@ Editor::marker_drag_motion_callback (GnomeCanvasItem* item, GdkEvent* event)
        else {
                newframe = 0;
        }
-               
+
        jack_nframes_t next = newframe;
 
        if (!Keyboard::modifier_state_contains (event->button.state, Keyboard::snap_modifier())) {
                snap_to (newframe, 0, true);
        }
        
-       if (drag_info.current_pointer_frame == drag_info.last_pointer_frame) return;
+       if (drag_info.current_pointer_frame == drag_info.last_pointer_frame) { 
+               return;
+       }
 
        /* call this to find out if its the start or end */
        
@@ -2070,7 +2127,7 @@ Editor::marker_drag_motion_callback (GnomeCanvasItem* item, GdkEvent* event)
 }
 
 void
-Editor::marker_drag_finished_callback (GnomeCanvasItem* item, GdkEvent* event)
+Editor::marker_drag_finished_callback (ArdourCanvas::Item* item, GdkEvent* event)
 {
        if (drag_info.first_move) {
                marker_drag_motion_callback (item, event);
@@ -2079,22 +2136,31 @@ Editor::marker_drag_finished_callback (GnomeCanvasItem* item, GdkEvent* event)
        
        Marker* marker = (Marker *) drag_info.data;
        bool is_start;
+
+
+       begin_reversible_command ( _("move marker") );
+       session->add_undo( session->locations()->get_memento() );
+       
        Location * location = find_location_from_marker (marker, is_start);
+       
        if (location) {
                location->set (drag_info.copied_location->start(), drag_info.copied_location->end());
        }
+
+       session->add_redo_no_execute( session->locations()->get_memento() );
+       commit_reversible_command ();
        
-       gnome_canvas_item_hide (marker_drag_line);
-       gnome_canvas_item_hide (range_marker_drag_rect);
+       marker_drag_line->hide();
+       range_marker_drag_rect->hide();
 }
 
 void
-Editor::start_meter_marker_grab (GnomeCanvasItem* item, GdkEvent* event)
+Editor::start_meter_marker_grab (ArdourCanvas::Item* item, GdkEvent* event)
 {
        Marker* marker;
        MeterMarker* meter_marker;
 
-       if ((marker = reinterpret_cast<Marker *> (gtk_object_get_data (GTK_OBJECT(item), "marker"))) == 0) {
+       if ((marker = reinterpret_cast<Marker *> (item->get_data ("marker"))) == 0) {
                fatal << _("programming error: meter marker canvas item has no marker object pointer!") << endmsg;
                /*NOTREACHED*/
        }
@@ -2120,7 +2186,40 @@ Editor::start_meter_marker_grab (GnomeCanvasItem* item, GdkEvent* event)
 }
 
 void
-Editor::meter_marker_drag_motion_callback (GnomeCanvasItem* item, GdkEvent* event)
+Editor::start_meter_marker_copy_grab (ArdourCanvas::Item* item, GdkEvent* event)
+{
+       Marker* marker;
+       MeterMarker* meter_marker;
+
+       if ((marker = reinterpret_cast<Marker *> (item->get_data ("marker"))) == 0) {
+               fatal << _("programming error: meter marker canvas item has no marker object pointer!") << endmsg;
+               /*NOTREACHED*/
+       }
+
+       meter_marker = dynamic_cast<MeterMarker*> (marker);
+       
+       // create a dummy marker for visual representation of moving the copy.
+       // The actual copying is not done before we reach the finish callback.
+       char name[64];
+       snprintf (name, sizeof(name), "%g/%g", meter_marker->meter().beats_per_bar(), meter_marker->meter().note_divisor ());
+       MeterMarker* new_marker = new MeterMarker(*this, *meter_group, color_map[cMeterMarker], name, 
+                                                 *new MeterSection(meter_marker->meter()));
+
+       drag_info.item = &new_marker->the_item();
+       drag_info.copy = true;
+       drag_info.data = new_marker;
+       drag_info.motion_callback = &Editor::meter_marker_drag_motion_callback;
+       drag_info.finished_callback = &Editor::meter_marker_drag_finished_callback;
+
+       start_grab (event);
+
+       drag_info.pointer_frame_offset = drag_info.grab_frame - meter_marker->meter().frame();  
+
+       show_verbose_time_cursor (drag_info.current_pointer_frame, 10);
+}
+
+void
+Editor::meter_marker_drag_motion_callback (ArdourCanvas::Item* item, GdkEvent* event)
 {
        MeterMarker* marker = (MeterMarker *) drag_info.data;
        jack_nframes_t adjusted_frame;
@@ -2148,11 +2247,11 @@ Editor::meter_marker_drag_motion_callback (GnomeCanvasItem* item, GdkEvent* even
 }
 
 void
-Editor::meter_marker_drag_finished_callback (GnomeCanvasItem* item, GdkEvent* event)
+Editor::meter_marker_drag_finished_callback (ArdourCanvas::Item* item, GdkEvent* event)
 {
        if (drag_info.first_move) return;
 
-       meter_marker_drag_motion_callback (item, event);
+       meter_marker_drag_motion_callback (drag_info.item, event);
        
        MeterMarker* marker = (MeterMarker *) drag_info.data;
        BBT_Time when;
@@ -2160,20 +2259,32 @@ Editor::meter_marker_drag_finished_callback (GnomeCanvasItem* item, GdkEvent* ev
        TempoMap& map (session->tempo_map());
        map.bbt_time (drag_info.last_pointer_frame, when);
        
-       begin_reversible_command (_("move meter mark"));
-       session->add_undo (map.get_memento());
-       map.move_meter (marker->meter(), when);
-       session->add_redo_no_execute (map.get_memento());
-       commit_reversible_command ();
+       if (drag_info.copy == true) {
+               begin_reversible_command (_("copy meter mark"));
+               session->add_undo (map.get_memento());
+               map.add_meter (marker->meter(), when);
+               session->add_redo_no_execute (map.get_memento());
+               commit_reversible_command ();
+               
+               // delete the dummy marker we used for visual representation of copying.
+               // a new visual marker will show up automatically.
+               delete marker;
+       } else {
+               begin_reversible_command (_("move meter mark"));
+               session->add_undo (map.get_memento());
+               map.move_meter (marker->meter(), when);
+               session->add_redo_no_execute (map.get_memento());
+               commit_reversible_command ();
+       }
 }
 
 void
-Editor::start_tempo_marker_grab (GnomeCanvasItem* item, GdkEvent* event)
+Editor::start_tempo_marker_grab (ArdourCanvas::Item* item, GdkEvent* event)
 {
        Marker* marker;
        TempoMarker* tempo_marker;
 
-       if ((marker = reinterpret_cast<Marker *> (gtk_object_get_data (GTK_OBJECT(item), "tempo_marker"))) == 0) {
+       if ((marker = reinterpret_cast<Marker *> (item->get_data ("marker"))) == 0) {
                fatal << _("programming error: tempo marker canvas item has no marker object pointer!") << endmsg;
                /*NOTREACHED*/
        }
@@ -2201,7 +2312,43 @@ Editor::start_tempo_marker_grab (GnomeCanvasItem* item, GdkEvent* event)
 }
 
 void
-Editor::tempo_marker_drag_motion_callback (GnomeCanvasItem* item, GdkEvent* event)
+Editor::start_tempo_marker_copy_grab (ArdourCanvas::Item* item, GdkEvent* event)
+{
+       Marker* marker;
+       TempoMarker* tempo_marker;
+
+       if ((marker = reinterpret_cast<Marker *> (item->get_data ("marker"))) == 0) {
+               fatal << _("programming error: tempo marker canvas item has no marker object pointer!") << endmsg;
+               /*NOTREACHED*/
+       }
+
+       if ((tempo_marker = dynamic_cast<TempoMarker *> (marker)) == 0) {
+               fatal << _("programming error: marker for tempo is not a tempo marker!") << endmsg;
+               /*NOTREACHED*/
+       }
+
+       // create a dummy marker for visual representation of moving the copy.
+       // The actual copying is not done before we reach the finish callback.
+       char name[64];
+       snprintf (name, sizeof (name), "%.2f", tempo_marker->tempo().beats_per_minute());
+       TempoMarker* new_marker = new TempoMarker(*this, *tempo_group, color_map[cTempoMarker], name, 
+                                                 *new TempoSection(tempo_marker->tempo()));
+
+       drag_info.item = &new_marker->the_item();
+       drag_info.copy = true;
+       drag_info.data = new_marker;
+       drag_info.motion_callback = &Editor::tempo_marker_drag_motion_callback;
+       drag_info.finished_callback = &Editor::tempo_marker_drag_finished_callback;
+
+       start_grab (event);
+
+       drag_info.pointer_frame_offset = drag_info.grab_frame - tempo_marker->tempo().frame();
+
+       show_verbose_time_cursor (drag_info.current_pointer_frame, 10);
+}
+
+void
+Editor::tempo_marker_drag_motion_callback (ArdourCanvas::Item* item, GdkEvent* event)
 {
        TempoMarker* marker = (TempoMarker *) drag_info.data;
        jack_nframes_t adjusted_frame;
@@ -2230,11 +2377,11 @@ Editor::tempo_marker_drag_motion_callback (GnomeCanvasItem* item, GdkEvent* even
 }
 
 void
-Editor::tempo_marker_drag_finished_callback (GnomeCanvasItem* item, GdkEvent* event)
+Editor::tempo_marker_drag_finished_callback (ArdourCanvas::Item* item, GdkEvent* event)
 {
        if (drag_info.first_move) return;
        
-       tempo_marker_drag_motion_callback (item, event);
+       tempo_marker_drag_motion_callback (drag_info.item, event);
        
        TempoMarker* marker = (TempoMarker *) drag_info.data;
        BBT_Time when;
@@ -2242,19 +2389,31 @@ Editor::tempo_marker_drag_finished_callback (GnomeCanvasItem* item, GdkEvent* ev
        TempoMap& map (session->tempo_map());
        map.bbt_time (drag_info.last_pointer_frame, when);
        
-       begin_reversible_command (_("move tempo mark"));
-       session->add_undo (map.get_memento());
-       map.move_tempo (marker->tempo(), when);
-       session->add_redo_no_execute (map.get_memento());
-       commit_reversible_command ();
+       if (drag_info.copy == true) {
+               begin_reversible_command (_("copy tempo mark"));
+               session->add_undo (map.get_memento());
+               map.add_tempo (marker->tempo(), when);
+               session->add_redo_no_execute (map.get_memento());
+               commit_reversible_command ();
+               
+               // delete the dummy marker we used for visual representation of copying.
+               // a new visual marker will show up automatically.
+               delete marker;
+       } else {
+               begin_reversible_command (_("move tempo mark"));
+               session->add_undo (map.get_memento());
+               map.move_tempo (marker->tempo(), when);
+               session->add_redo_no_execute (map.get_memento());
+               commit_reversible_command ();
+       }
 }
 
 void
-Editor::remove_gain_control_point (GnomeCanvasItem*item, GdkEvent* event)
+Editor::remove_gain_control_point (ArdourCanvas::Item*item, GdkEvent* event)
 {
        ControlPoint* control_point;
 
-       if ((control_point = reinterpret_cast<ControlPoint *> (gtk_object_get_data (GTK_OBJECT(item), "control_point"))) == 0) {
+       if ((control_point = reinterpret_cast<ControlPoint *> (item->get_data ("control_point"))) == 0) {
                fatal << _("programming error: control point canvas item has no control point object pointer!") << endmsg;
                /*NOTREACHED*/
        }
@@ -2269,11 +2428,11 @@ Editor::remove_gain_control_point (GnomeCanvasItem*item, GdkEvent* event)
 }
 
 void
-Editor::remove_control_point (GnomeCanvasItem*item, GdkEvent* event)
+Editor::remove_control_point (ArdourCanvas::Item*item, GdkEvent* event)
 {
        ControlPoint* control_point;
 
-       if ((control_point = reinterpret_cast<ControlPoint *> (gtk_object_get_data (GTK_OBJECT(item), "control_point"))) == 0) {
+       if ((control_point = reinterpret_cast<ControlPoint *> (item->get_data ("control_point"))) == 0) {
                fatal << _("programming error: control point canvas item has no control point object pointer!") << endmsg;
                /*NOTREACHED*/
        }
@@ -2282,11 +2441,11 @@ Editor::remove_control_point (GnomeCanvasItem*item, GdkEvent* event)
 }
 
 void
-Editor::start_control_point_grab (GnomeCanvasItem* item, GdkEvent* event)
+Editor::start_control_point_grab (ArdourCanvas::Item* item, GdkEvent* event)
 {
        ControlPoint* control_point;
        
-       if ((control_point = reinterpret_cast<ControlPoint *> (gtk_object_get_data (GTK_OBJECT(item), "control_point"))) == 0) {
+       if ((control_point = reinterpret_cast<ControlPoint *> (item->get_data ("control_point"))) == 0) {
                fatal << _("programming error: control point canvas item has no control point object pointer!") << endmsg;
                /*NOTREACHED*/
        }
@@ -2308,7 +2467,7 @@ Editor::start_control_point_grab (GnomeCanvasItem* item, GdkEvent* event)
 }
 
 void
-Editor::control_point_drag_motion_callback (GnomeCanvasItem* item, GdkEvent* event)
+Editor::control_point_drag_motion_callback (ArdourCanvas::Item* item, GdkEvent* event)
 {
        ControlPoint* cp = reinterpret_cast<ControlPoint *> (drag_info.data);
 
@@ -2318,29 +2477,23 @@ Editor::control_point_drag_motion_callback (GnomeCanvasItem* item, GdkEvent* eve
        drag_info.cumulative_x_drag = cx - drag_info.grab_x ;
        drag_info.cumulative_y_drag = cy - drag_info.grab_y ;
 
-       bool x_constrained = false;
-
        if (drag_info.x_constrained) {
-               if (fabs(drag_info.cumulative_x_drag) < fabs(drag_info.cumulative_y_drag)) {
-                       cx = drag_info.grab_x;
-                       x_constrained = true;
-
-               } else {
-                       cy = drag_info.grab_y;
-               }
-       
-       } 
+               cx = drag_info.grab_x;
+       }
+       if (drag_info.y_constrained) {
+               cy = drag_info.grab_y;
+       }
 
-       gnome_canvas_item_w2i (cp->line.parent_group(), &cx, &cy);
+       cp->line.parent_group().w2i (cx, cy);
 
        cx = max (0.0, cx);
        cy = max (0.0, cy);
        cy = min ((double) cp->line.height(), cy);
 
        //translate cx to frames
-       jack_nframes_t cx_frames = (jack_nframes_t) floor (cx * frames_per_unit);
+       jack_nframes_t cx_frames = unit_to_frame (cx);
 
-       if (!Keyboard::modifier_state_contains (event->button.state, Keyboard::snap_modifier()) && !x_constrained) {
+       if (!Keyboard::modifier_state_contains (event->button.state, Keyboard::snap_modifier()) && !drag_info.x_constrained) {
                snap_to (cx_frames);
        }
 
@@ -2357,18 +2510,31 @@ Editor::control_point_drag_motion_callback (GnomeCanvasItem* item, GdkEvent* eve
        cp->line.point_drag (*cp, cx_frames , fraction, push);
        
        set_verbose_canvas_cursor_text (cp->line.get_verbose_cursor_string (fraction));
+
+       drag_info.first_move = false;
 }
 
 void
-Editor::control_point_drag_finished_callback (GnomeCanvasItem* item, GdkEvent* event)
+Editor::control_point_drag_finished_callback (ArdourCanvas::Item* item, GdkEvent* event)
 {
        ControlPoint* cp = reinterpret_cast<ControlPoint *> (drag_info.data);
-       control_point_drag_motion_callback (item, event);
+
+       if (drag_info.first_move) {
+
+               /* just a click */
+               
+               if ((event->type == GDK_BUTTON_RELEASE) && (event->button.button == 1) && Keyboard::modifier_state_equals (event->button.state, Keyboard::Shift)) {
+                       reset_point_selection ();
+               }
+
+       } else {
+               control_point_drag_motion_callback (item, event);
+       }
        cp->line.end_drag (cp);
 }
 
 void
-Editor::start_line_grab_from_regionview (GnomeCanvasItem* item, GdkEvent* event)
+Editor::start_line_grab_from_regionview (ArdourCanvas::Item* item, GdkEvent* event)
 {
        switch (mouse_mode) {
        case MouseGain:
@@ -2380,11 +2546,11 @@ Editor::start_line_grab_from_regionview (GnomeCanvasItem* item, GdkEvent* event)
 }
 
 void
-Editor::start_line_grab_from_line (GnomeCanvasItem* item, GdkEvent* event)
+Editor::start_line_grab_from_line (ArdourCanvas::Item* item, GdkEvent* event)
 {
        AutomationLine* al;
        
-       if ((al = reinterpret_cast<AutomationLine*> (gtk_object_get_data (GTK_OBJECT(item), "line"))) == 0) {
+       if ((al = reinterpret_cast<AutomationLine*> (item->get_data ("line"))) == 0) {
                fatal << _("programming error: line canvas item has no line pointer!") << endmsg;
                /*NOTREACHED*/
        }
@@ -2405,7 +2571,7 @@ Editor::start_line_grab (AutomationLine* line, GdkEvent* event)
 
        cx = event->button.x;
        cy = event->button.y;
-       gnome_canvas_item_w2i (line->parent_group(), &cx, &cy);
+       line->parent_group().w2i (cx, cy);
        frame_within_region = (jack_nframes_t) floor (cx * frames_per_unit);
 
        if (!line->control_points_adjacent (frame_within_region, current_line_drag_info.before, 
@@ -2414,7 +2580,7 @@ Editor::start_line_grab (AutomationLine* line, GdkEvent* event)
                return;
        }
 
-       drag_info.item = line->grab_item();
+       drag_info.item = &line->grab_item();
        drag_info.data = line;
        drag_info.motion_callback = &Editor::line_drag_motion_callback;
        drag_info.finished_callback = &Editor::line_drag_finished_callback;
@@ -2431,13 +2597,13 @@ Editor::start_line_grab (AutomationLine* line, GdkEvent* event)
 }
 
 void
-Editor::line_drag_motion_callback (GnomeCanvasItem* item, GdkEvent* event)
+Editor::line_drag_motion_callback (ArdourCanvas::Item* item, GdkEvent* event)
 {
        AutomationLine* line = reinterpret_cast<AutomationLine *> (drag_info.data);
        double cx = drag_info.current_pointer_x;
        double cy = drag_info.current_pointer_y;
 
-       gnome_canvas_item_w2i (line->parent_group(), &cx, &cy);
+       line->parent_group().w2i (cx, cy);
        
        double fraction;
        fraction = 1.0 - (cy / line->height());
@@ -2456,7 +2622,7 @@ Editor::line_drag_motion_callback (GnomeCanvasItem* item, GdkEvent* event)
 }
 
 void
-Editor::line_drag_finished_callback (GnomeCanvasItem* item, GdkEvent* event)
+Editor::line_drag_finished_callback (ArdourCanvas::Item* item, GdkEvent* event)
 {
        AutomationLine* line = reinterpret_cast<AutomationLine *> (drag_info.data);
        line_drag_motion_callback (item, event);
@@ -2464,7 +2630,7 @@ Editor::line_drag_finished_callback (GnomeCanvasItem* item, GdkEvent* event)
 }
 
 void
-Editor::start_region_grab (GnomeCanvasItem* item, GdkEvent* event)
+Editor::start_region_grab (ArdourCanvas::Item* item, GdkEvent* event)
 {
        if (selection->audio_regions.empty() || clicked_regionview == 0) {
                return;
@@ -2498,65 +2664,15 @@ Editor::start_region_grab (GnomeCanvasItem* item, GdkEvent* event)
 }
 
 void
-Editor::start_region_copy_grab (GnomeCanvasItem* item, GdkEvent* event)
+Editor::start_region_copy_grab (ArdourCanvas::Item* item, GdkEvent* event)
 {
        if (selection->audio_regions.empty() || clicked_regionview == 0) {
                return;
        }
 
-       /* this is committed in the grab finished callback. */
-
-       begin_reversible_command (_("Drag region copy"));
-                               
-       /* duplicate the region(s) */
-
-       vector<AudioRegionView*> new_regionviews;
-
-       for (list<AudioRegionView*>::const_iterator i = selection->audio_regions.by_layer().begin(); i != selection->audio_regions.by_layer().end(); ++i) {
-               AudioRegionView* rv;
-
-               rv = (*i);
-
-               Playlist* to_playlist = rv->region.playlist();
-               AudioTimeAxisView* atv = dynamic_cast<AudioTimeAxisView*>(&rv->get_time_axis_view());
-               
-               session->add_undo (to_playlist->get_memento ());
-               latest_regionview = 0;
-
-               sigc::connection c = atv->view->AudioRegionViewAdded.connect (mem_fun(*this, &Editor::collect_new_region_view));
-               
-               /* create a new region with the same name.
-                */
-
-               AudioRegion* newregion = new AudioRegion (rv->region);
-
-               /* if the original region was locked, we don't care */
-
-               newregion->set_locked (false);
-
-               to_playlist->add_region (*newregion, (jack_nframes_t) (rv->region.position() * atv->get_diskstream()->speed()));
-               
-               c.disconnect ();
-               
-               if (latest_regionview) {
-                       new_regionviews.push_back (latest_regionview);
-               }
-               
-       }
-
-       if (new_regionviews.empty()) {
-               return;
-       }
-
-       /* reset selection to new regionviews */
-
-       selection->set (new_regionviews);
-
-       drag_info.item = new_regionviews.front()->get_canvas_group ();
        drag_info.copy = true;
-       drag_info.data = new_regionviews.front();
-       drag_info.motion_callback = &Editor::region_drag_motion_callback;
-       drag_info.finished_callback = &Editor::region_drag_finished_callback;
+       drag_info.item = item;
+       drag_info.data = clicked_regionview;    
 
        start_grab(event);
 
@@ -2573,14 +2689,12 @@ Editor::start_region_copy_grab (GnomeCanvasItem* item, GdkEvent* event)
        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;
-
-       show_verbose_time_cursor (drag_info.last_frame_position, 10);
-
-       begin_reversible_command (_("copy region(s)"));
+       drag_info.motion_callback = &Editor::region_drag_motion_callback;
+       drag_info.finished_callback = &Editor::region_drag_finished_callback;
 }
 
 void
-Editor::start_region_brush_grab (GnomeCanvasItem* item, GdkEvent* event)
+Editor::start_region_brush_grab (ArdourCanvas::Item* item, GdkEvent* event)
 {
        if (selection->audio_regions.empty() || clicked_regionview == 0) {
                return;
@@ -2613,7 +2727,7 @@ Editor::start_region_brush_grab (GnomeCanvasItem* item, GdkEvent* event)
 }
 
 void
-Editor::region_drag_motion_callback (GnomeCanvasItem* item, GdkEvent* event)
+Editor::region_drag_motion_callback (ArdourCanvas::Item* item, GdkEvent* event)
 {
        double x_delta;
        double y_delta = 0;
@@ -2625,6 +2739,72 @@ Editor::region_drag_motion_callback (GnomeCanvasItem* item, GdkEvent* event)
        vector<int32_t>  height_list(512) ;
        vector<int32_t>::iterator j;
 
+       show_verbose_time_cursor (drag_info.last_frame_position, 10);
+
+       if (drag_info.copy && drag_info.move_threshold_passed && drag_info.want_move_threshold) {
+
+               drag_info.want_move_threshold = false; // don't copy again
+
+               /* this is committed in the grab finished callback. */
+               
+               begin_reversible_command (_("Drag region copy"));
+               
+               /* duplicate the region(s) */
+               
+               vector<AudioRegionView*> new_regionviews;
+               
+               set<Playlist*> affected_playlists;
+               pair<set<Playlist*>::iterator,bool> insert_result;
+               
+               for (list<AudioRegionView*>::const_iterator i = selection->audio_regions.by_layer().begin(); i != selection->audio_regions.by_layer().end(); ++i) {
+                       AudioRegionView* rv;
+                       
+                       rv = (*i);
+                       
+                       Playlist* to_playlist = rv->region.playlist();
+                       AudioTimeAxisView* atv = dynamic_cast<AudioTimeAxisView*>(&rv->get_time_axis_view());
+                       
+                       insert_result = affected_playlists.insert (to_playlist);
+                       if (insert_result.second) {
+                               session->add_undo (to_playlist->get_memento ());
+                       }
+                       
+                       latest_regionview = 0;
+                       
+                       sigc::connection c = atv->view->AudioRegionViewAdded.connect (mem_fun(*this, &Editor::collect_new_region_view));
+                       
+                       /* create a new region with the same name.
+                        */
+                       
+                       AudioRegion* newregion = new AudioRegion (rv->region);
+                       
+                       /* if the original region was locked, we don't care */
+                       
+                       newregion->set_locked (false);
+                       
+                       to_playlist->add_region (*newregion, (jack_nframes_t) (rv->region.position() * atv->get_diskstream()->speed()));
+                       
+                       c.disconnect ();
+                       
+                       if (latest_regionview) {
+                               new_regionviews.push_back (latest_regionview);
+                       }
+               }
+               
+               if (new_regionviews.empty()) {
+                       return;
+               }
+               
+               /* reset selection to new regionviews */
+               
+               selection->set (new_regionviews);
+               
+               /* 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);
+       }
+
        /* Which trackview is this ? */
 
        TimeAxisView* tvp = trackview_by_y_position (drag_info.current_pointer_y);
@@ -2657,7 +2837,8 @@ Editor::region_drag_motion_callback (GnomeCanvasItem* item, GdkEvent* event)
        if ((pointer_y_span = (drag_info.last_trackview->order - tv->order)) != 0) {
 
                int32_t children = 0, numtracks = 0;
-               bitset <512> tracks (0x00);
+               // XXX hard coding track limit, oh my, so very very bad
+               bitset <1024> tracks (0x00);
                /* get a bitmask representing the visible tracks */
 
                for (TrackViewList::iterator i = track_views.begin(); i != track_views.end(); ++i) {
@@ -2719,8 +2900,8 @@ Editor::region_drag_motion_callback (GnomeCanvasItem* item, GdkEvent* event)
                        double ix1, ix2, iy1, iy2;
                        int32_t n = 0;
 
-                       gnome_canvas_item_get_bounds (rv2->get_canvas_frame(), &ix1, &iy1, &ix2, &iy2);
-                       gnome_canvas_item_i2w (rv2->get_canvas_group(), &ix1, &iy1);
+                       rv2->get_canvas_frame()->get_bounds (ix1, iy1, ix2, iy2);
+                       rv2->get_canvas_group()->i2w (ix1, iy1);
                        TimeAxisView* tvp2 = trackview_by_y_position (iy1);
                        AudioTimeAxisView* atv2 = dynamic_cast<AudioTimeAxisView*>(tvp2);
 
@@ -2807,7 +2988,7 @@ Editor::region_drag_motion_callback (GnomeCanvasItem* item, GdkEvent* event)
           the region would be if we moved it by that much.
        */
 
-       if (drag_info.move_threshold_passsed) {
+       if (drag_info.move_threshold_passed) {
 
                if ((int32_t)drag_info.current_pointer_frame > drag_info.pointer_frame_offset) {
 
@@ -2889,8 +3070,8 @@ Editor::region_drag_motion_callback (GnomeCanvasItem* item, GdkEvent* event)
                        */
                        
                        double ix1, ix2, iy1, iy2;
-                       gnome_canvas_item_get_bounds (rv2->get_canvas_frame(), &ix1, &iy1, &ix2, &iy2);
-                       gnome_canvas_item_i2w (rv2->get_canvas_group(), &ix1, &iy1);
+                       rv2->get_canvas_frame()->get_bounds (ix1, iy1, ix2, iy2);
+                       rv2->get_canvas_group()->i2w (ix1, iy1);
 
                        if (ix1 <= 1) {
                                x_delta = 0;
@@ -2904,8 +3085,9 @@ Editor::region_drag_motion_callback (GnomeCanvasItem* item, GdkEvent* event)
        ************************************************************/
 
        pair<set<Playlist*>::iterator,bool> insert_result;
+       const list<AudioRegionView*>& layered_regions = selection->audio_regions.by_layer();
 
-       for (list<AudioRegionView*>::const_iterator i = selection->audio_regions.by_layer().begin(); i != selection->audio_regions.by_layer().end(); ++i) {
+       for (list<AudioRegionView*>::const_iterator i = layered_regions.begin(); i != layered_regions.end(); ++i) {
            
                AudioRegionView* rv;
                rv = (*i);
@@ -2917,8 +3099,8 @@ Editor::region_drag_motion_callback (GnomeCanvasItem* item, GdkEvent* event)
                   the parent.
                */
 
-               gnome_canvas_item_get_bounds (rv->get_canvas_frame(), &ix1, &iy1, &ix2, &iy2);
-               gnome_canvas_item_i2w (rv->get_canvas_group(), &ix1, &iy1);
+               rv->get_canvas_frame()->get_bounds (ix1, iy1, ix2, iy2);
+               rv->get_canvas_group()->i2w (ix1, iy1);
                TimeAxisView* tvp2 = trackview_by_y_position (iy1);
                AudioTimeAxisView* canvas_atv = dynamic_cast<AudioTimeAxisView*>(tvp2);
                AudioTimeAxisView* temp_atv;
@@ -3005,9 +3187,9 @@ Editor::region_drag_motion_callback (GnomeCanvasItem* item, GdkEvent* event)
                           on the top, since its parent is the whole canvas.
                        */
 
-                       gnome_canvas_item_raise_to_top (rv->get_canvas_group());
-                       gnome_canvas_item_raise_to_top (rv->get_time_axis_view().canvas_display);
-                       gnome_canvas_item_raise_to_top (cursor_group);
+                       rv->get_canvas_group()->raise_to_top();
+                       rv->get_time_axis_view().canvas_display->raise_to_top();
+                       cursor_group->raise_to_top();
 
                        /* freeze the playlists from notifying till
                           the motion is done.
@@ -3036,7 +3218,7 @@ Editor::region_drag_motion_callback (GnomeCanvasItem* item, GdkEvent* event)
        }
                
        if (drag_info.first_move) {
-               gnome_canvas_item_raise_to_top (cursor_group);
+               cursor_group->raise_to_top();
        }
                
        drag_info.first_move = false;
@@ -3048,7 +3230,7 @@ Editor::region_drag_motion_callback (GnomeCanvasItem* item, GdkEvent* event)
 } 
 
 void
-Editor::region_drag_finished_callback (GnomeCanvasItem* item, GdkEvent* event)
+Editor::region_drag_finished_callback (ArdourCanvas::Item* item, GdkEvent* event)
 {
        jack_nframes_t where;
        AudioRegionView* rv = reinterpret_cast<AudioRegionView *> (drag_info.data);
@@ -3129,8 +3311,8 @@ Editor::region_drag_finished_callback (GnomeCanvasItem* item, GdkEvent* event)
                                
                        double ix1, ix2, iy1, iy2;
            
-                       gnome_canvas_item_get_bounds ((*i)->get_canvas_frame(), &ix1, &iy1, &ix2, &iy2);
-                       gnome_canvas_item_i2w ((*i)->get_canvas_group(), &ix1, &iy1);
+                       (*i)->get_canvas_frame()->get_bounds (ix1, iy1, ix2, iy2);
+                       (*i)->get_canvas_group()->i2w (ix1, iy1);
                        TimeAxisView* tvp2 = trackview_by_y_position (iy1);
                        AudioTimeAxisView* atv2 = dynamic_cast<AudioTimeAxisView*>(tvp2);
            
@@ -3164,8 +3346,8 @@ Editor::region_drag_finished_callback (GnomeCanvasItem* item, GdkEvent* event)
                                
                        double ix1, ix2, iy1, iy2;
            
-                       gnome_canvas_item_get_bounds ((*i)->get_canvas_frame(), &ix1, &iy1, &ix2, &iy2);
-                       gnome_canvas_item_i2w ((*i)->get_canvas_group(), &ix1, &iy1);
+                       (*i)->get_canvas_frame()->get_bounds (ix1, iy1, ix2, iy2);
+                       (*i)->get_canvas_group()->i2w (ix1, iy1);
                        TimeAxisView* tvp2 = trackview_by_y_position (iy1);
                        AudioTimeAxisView* atv2 = dynamic_cast<AudioTimeAxisView*>(tvp2);
            
@@ -3191,7 +3373,7 @@ Editor::region_drag_finished_callback (GnomeCanvasItem* item, GdkEvent* event)
        } else {
 
                /* motion within a single track */
-
+               
                for (list<AudioRegionView*>::const_iterator i = selection->audio_regions.by_layer().begin(); i != selection->audio_regions.by_layer().end(); ++i) {
 
                        rv = (*i);
@@ -3212,8 +3394,8 @@ Editor::region_drag_finished_callback (GnomeCanvasItem* item, GdkEvent* event)
                                
                                double ix1, ix2, iy1, iy2;
                                
-                               gnome_canvas_item_get_bounds (rv->get_canvas_frame(), &ix1, &iy1, &ix2, &iy2);
-                               gnome_canvas_item_i2w (rv->get_canvas_group(), &ix1, &iy1);
+                               rv->get_canvas_frame()->get_bounds (ix1, iy1, ix2, iy2);
+                               rv->get_canvas_group()->i2w (ix1, iy1);
                                where = (jack_nframes_t) (unit_to_frame (ix1) * ownspeed);
                                
                        } else {
@@ -3222,8 +3404,9 @@ Editor::region_drag_finished_callback (GnomeCanvasItem* item, GdkEvent* event)
                        }
 
                        rv->get_time_axis_view().reveal_dependent_views (*rv);
+
+                       /* no need to add an undo here, we did that when we added this playlist to motion_frozen playlists */
                        
-                       session->add_undo (rv->region.playlist()->get_memento());
                        rv->region.set_position (where, (void *) this);
                }
        }
@@ -3275,7 +3458,7 @@ void
 Editor::show_verbose_time_cursor (jack_nframes_t frame, double offset, double xpos, double ypos) 
 {
        char buf[128];
-       SMPTE_Time smpte;
+       SMPTE::Time smpte;
        BBT_Time bbt;
        float secs;
 
@@ -3319,7 +3502,7 @@ void
 Editor::show_verbose_duration_cursor (jack_nframes_t start, jack_nframes_t end, double offset, double xpos, double ypos) 
 {
        char buf[128];
-       SMPTE_Time smpte;
+       SMPTE::Time smpte;
        BBT_Time sbbt;
        BBT_Time ebbt;
        float secs;
@@ -3365,7 +3548,7 @@ Editor::show_verbose_duration_cursor (jack_nframes_t start, jack_nframes_t end,
                /* XXX fix this to compute min/sec properly */
                session->smpte_duration (end - start, smpte);
                secs = smpte.seconds + ((float) smpte.frames / session->smpte_frames_per_second);
-               snprintf (buf, sizeof (buf), "%02ld:%02ld:%.4f", smpte.hours, smpte.minutes, secs);
+               snprintf (buf, sizeof (buf), "%02" PRId32 ":%02" PRId32 ":%.4f", smpte.hours, smpte.minutes, secs);
                break;
 
        default:
@@ -3389,7 +3572,7 @@ Editor::collect_new_region_view (AudioRegionView* rv)
 }
 
 void
-Editor::start_selection_grab (GnomeCanvasItem* item, GdkEvent* event)
+Editor::start_selection_grab (ArdourCanvas::Item* item, GdkEvent* event)
 {
        if (clicked_regionview == 0) {
                return;
@@ -3460,15 +3643,17 @@ Editor::start_selection_grab (GnomeCanvasItem* item, GdkEvent* event)
 void
 Editor::cancel_selection ()
 {
-       for (TrackViewList::iterator i = track_views.begin(); i != track_views.end(); ++i) {
+        for (TrackViewList::iterator i = track_views.begin(); i != track_views.end(); ++i) {
                (*i)->hide_selection ();
        }
+       begin_reversible_command (_("cancel selection"));
        selection->clear ();
        clicked_selection = 0;
+       commit_reversible_command ();
 }      
 
 void
-Editor::start_selection_op (GnomeCanvasItem* item, GdkEvent* event, SelectionOp op)
+Editor::start_selection_op (ArdourCanvas::Item* item, GdkEvent* event, SelectionOp op)
 {
        jack_nframes_t start = 0;
        jack_nframes_t end = 0;
@@ -3485,7 +3670,6 @@ Editor::start_selection_op (GnomeCanvasItem* item, GdkEvent* event, SelectionOp
 
        switch (op) {
        case CreateSelection:
-               
                if (Keyboard::modifier_state_equals (event->button.state, Keyboard::Shift)) {
                        drag_info.copy = true;
                } else {
@@ -3495,14 +3679,18 @@ Editor::start_selection_op (GnomeCanvasItem* item, GdkEvent* event, SelectionOp
                break;
 
        case SelectionStartTrim:
-               clicked_trackview->order_selection_trims (item, true);
+               if (clicked_trackview) {
+                       clicked_trackview->order_selection_trims (item, true);
+               } 
                start_grab (event, trimmer_cursor);
                start = selection->time[clicked_selection].start;
                drag_info.pointer_frame_offset = drag_info.grab_frame - start;  
                break;
                
        case SelectionEndTrim:
-               clicked_trackview->order_selection_trims (item, false);
+               if (clicked_trackview) {
+                       clicked_trackview->order_selection_trims (item, false);
+               }
                start_grab (event, trimmer_cursor);
                end = selection->time[clicked_selection].end;
                drag_info.pointer_frame_offset = drag_info.grab_frame - end;    
@@ -3523,7 +3711,7 @@ Editor::start_selection_op (GnomeCanvasItem* item, GdkEvent* event, SelectionOp
 }
 
 void
-Editor::drag_selection (GnomeCanvasItem* item, GdkEvent* event)
+Editor::drag_selection (ArdourCanvas::Item* item, GdkEvent* event)
 {
        jack_nframes_t start = 0;
        jack_nframes_t end = 0;
@@ -3633,8 +3821,7 @@ Editor::drag_selection (GnomeCanvasItem* item, GdkEvent* event)
                break;
        }
        
-       
-       if (event->button.x >= track_canvas_scroller.get_hadjustment()->get_value() + canvas_width) {
+       if (event->button.x >= horizontal_adjustment.get_value() + canvas_width) {
                start_canvas_autoscroll (1);
        }
 
@@ -3653,7 +3840,7 @@ Editor::drag_selection (GnomeCanvasItem* item, GdkEvent* event)
 }
 
 void
-Editor::end_selection_op (GnomeCanvasItem* item, GdkEvent* event)
+Editor::end_selection_op (ArdourCanvas::Item* item, GdkEvent* event)
 {
        if (!drag_info.first_move) {
                drag_selection (item, event);
@@ -3678,7 +3865,7 @@ Editor::end_selection_op (GnomeCanvasItem* item, GdkEvent* event)
 }
 
 void
-Editor::start_trim (GnomeCanvasItem* item, GdkEvent* event)
+Editor::start_trim (ArdourCanvas::Item* item, GdkEvent* event)
 {
        double speed = 1.0;
        TimeAxisView* tvp = clicked_trackview;
@@ -3691,6 +3878,8 @@ Editor::start_trim (GnomeCanvasItem* item, GdkEvent* event)
        jack_nframes_t region_start = (jack_nframes_t) (clicked_regionview->region.position() / speed);
        jack_nframes_t region_end = (jack_nframes_t) (clicked_regionview->region.last_frame() / speed);
        jack_nframes_t region_length = (jack_nframes_t) (clicked_regionview->region.length() / speed);
+
+       motion_frozen_playlists.clear();
        
        //drag_info.item = clicked_regionview->get_name_highlight();
        drag_info.item = item;
@@ -3723,12 +3912,10 @@ Editor::start_trim (GnomeCanvasItem* item, GdkEvent* event)
                show_verbose_time_cursor(drag_info.current_pointer_frame, 10);  
                break;
        }
-       
-       flush_track_canvas ();
 }
 
 void
-Editor::trim_motion_callback (GnomeCanvasItem* item, GdkEvent* event)
+Editor::trim_motion_callback (ArdourCanvas::Item* item, GdkEvent* event)
 {
        AudioRegionView* rv = clicked_regionview;
        jack_nframes_t frame_delta = 0;
@@ -3743,6 +3930,7 @@ Editor::trim_motion_callback (GnomeCanvasItem* item, GdkEvent* event)
        double speed = 1.0;
        TimeAxisView* tvp = clicked_trackview;
        AudioTimeAxisView* tv = dynamic_cast<AudioTimeAxisView*>(tvp);
+       pair<set<Playlist*>::iterator,bool> insert_result;
 
        if (tv && tv->is_audio_track()) {
                speed = tv->get_diskstream()->speed();
@@ -3783,7 +3971,12 @@ Editor::trim_motion_callback (GnomeCanvasItem* item, GdkEvent* event)
                for (list<AudioRegionView*>::const_iterator i = selection->audio_regions.by_layer().begin(); i != selection->audio_regions.by_layer().end(); ++i) {
                        (*i)->region.freeze ();
                        (*i)->temporarily_hide_envelope ();
-                       session->add_undo ((*i)->region.playlist()->get_memento());
+
+                       Playlist * pl = (*i)->region.playlist();
+                       insert_result = motion_frozen_playlists.insert (pl);
+                       if (insert_result.second) {
+                               session->add_undo (pl->get_memento());
+                       }
                }
        }
 
@@ -3954,7 +4147,7 @@ Editor::single_end_trim (AudioRegionView& rv, jack_nframes_t frame_delta, bool l
 }
        
 void
-Editor::trim_finished_callback (GnomeCanvasItem* item, GdkEvent* event)
+Editor::trim_finished_callback (ArdourCanvas::Item* item, GdkEvent* event)
 {
        if (!drag_info.first_move) {
                trim_motion_callback (item, event);
@@ -3969,13 +4162,19 @@ Editor::trim_finished_callback (GnomeCanvasItem* item, GdkEvent* event)
                                thaw_region_after_trim (**i);
                        }
                }
+               
+               for (set<Playlist*>::iterator p = motion_frozen_playlists.begin(); p != motion_frozen_playlists.end(); ++p) {
+                       //(*p)->thaw ();
+                       session->add_redo_no_execute ((*p)->get_memento());
+               }
+               
+               motion_frozen_playlists.clear ();
+
                commit_reversible_command();
        } else {
                /* no mouse movement */
                point_trim (event);
        }
-       
-       flush_track_canvas ();
 }
 
 void
@@ -4066,12 +4265,12 @@ Editor::thaw_region_after_trim (AudioRegionView& rv)
 }
 
 void
-Editor::hide_marker (GnomeCanvasItem* item, GdkEvent* event)
+Editor::hide_marker (ArdourCanvas::Item* item, GdkEvent* event)
 {
        Marker* marker;
        bool is_start;
 
-       if ((marker = static_cast<Marker *> (gtk_object_get_data (GTK_OBJECT(item), "marker"))) == 0) {
+       if ((marker = static_cast<Marker *> (item->get_data ("marker"))) == 0) {
                fatal << _("programming error: marker canvas item has no marker object pointer!") << endmsg;
                /*NOTREACHED*/
        }
@@ -4082,9 +4281,8 @@ Editor::hide_marker (GnomeCanvasItem* item, GdkEvent* event)
 
 
 void
-Editor::start_range_markerbar_op (GnomeCanvasItem* item, GdkEvent* event, RangeMarkerOp op)
+Editor::start_range_markerbar_op (ArdourCanvas::Item* item, GdkEvent* event, RangeMarkerOp op)
 {
-
        if (session == 0) {
                return;
        }
@@ -4117,12 +4315,11 @@ Editor::start_range_markerbar_op (GnomeCanvasItem* item, GdkEvent* event, RangeM
 }
 
 void
-Editor::drag_range_markerbar_op (GnomeCanvasItem* item, GdkEvent* event)
+Editor::drag_range_markerbar_op (ArdourCanvas::Item* item, GdkEvent* event)
 {
        jack_nframes_t start = 0;
        jack_nframes_t end = 0;
-       
-       GnomeCanvasItem * crect = (range_marker_op == CreateRangeMarker) ? range_bar_drag_rect: transport_bar_drag_rect;
+       ArdourCanvas::SimpleRect *crect = (range_marker_op == CreateRangeMarker) ? range_bar_drag_rect: transport_bar_drag_rect;
        
        if (!Keyboard::modifier_state_contains (event->button.state, Keyboard::snap_modifier())) {
                snap_to (drag_info.current_pointer_frame);
@@ -4150,25 +4347,24 @@ Editor::drag_range_markerbar_op (GnomeCanvasItem* item, GdkEvent* event)
                }
                
                /* first drag: Either add to the selection
-                  or create a new selection->
+                  or create a new selection.
                */
                
                if (drag_info.first_move) {
                        
                        temp_location->set (start, end);
                        
-                       gnome_canvas_item_show (crect);
+                       crect->show ();
 
                        update_marker_drag_item (temp_location);
-                       gnome_canvas_item_show (range_marker_drag_rect);
-                       gnome_canvas_item_raise_to_top (range_marker_drag_rect);
+                       range_marker_drag_rect->show();
+                       range_marker_drag_rect->raise_to_top();
                        
                } 
                break;          
        }
        
-       
-       if (event->button.x >= track_canvas_scroller.get_hadjustment()->get_value() + canvas_width) {
+       if (event->button.x >= horizontal_adjustment.get_value() + canvas_width) {
                start_canvas_autoscroll (1);
        }
        
@@ -4177,7 +4373,8 @@ Editor::drag_range_markerbar_op (GnomeCanvasItem* item, GdkEvent* event)
 
                double x1 = frame_to_pixel (start);
                double x2 = frame_to_pixel (end);
-               gnome_canvas_item_set (crect, "x1", x1, "x2", x2, NULL);
+               crect->property_x1() = x1;
+               crect->property_x2() = x2;
 
                update_marker_drag_item (temp_location);
        }
@@ -4190,25 +4387,24 @@ Editor::drag_range_markerbar_op (GnomeCanvasItem* item, GdkEvent* event)
 }
 
 void
-Editor::end_range_markerbar_op (GnomeCanvasItem* item, GdkEvent* event)
+Editor::end_range_markerbar_op (ArdourCanvas::Item* item, GdkEvent* event)
 {
        Location * newloc = 0;
        
        if (!drag_info.first_move) {
                drag_range_markerbar_op (item, event);
 
-               switch (range_marker_op)
-               {
+               switch (range_marker_op) {
                case CreateRangeMarker:
                        begin_reversible_command (_("new range marker"));
                        session->add_undo (session->locations()->get_memento());
-                       newloc = new Location(temp_location->start(), temp_location->end(), "unnamed");
+                       newloc = new Location(temp_location->start(), temp_location->end(), "unnamed", Location::IsRangeMarker);
                        session->locations()->add (newloc, true);
                        session->add_redo_no_execute (session->locations()->get_memento());
                        commit_reversible_command ();
                        
-                       gnome_canvas_item_hide (range_bar_drag_rect);
-                       gnome_canvas_item_hide (range_marker_drag_rect);
+                       range_bar_drag_rect->hide();
+                       range_marker_drag_rect->hide();
                        break;
 
                case CreateTransportMarker:
@@ -4218,12 +4414,39 @@ Editor::end_range_markerbar_op (GnomeCanvasItem* item, GdkEvent* event)
                        break;
                }
        } else {
-               /* just a click, no pointer movement.*/
+               /* just a click, no pointer movement. remember that context menu stuff was handled elsewhere */
 
                if (Keyboard::no_modifier_keys_pressed (&event->button)) {
 
-                       // nothing yet
+                       jack_nframes_t start;
+                       jack_nframes_t end;
+
+                       start = session->locations()->first_mark_before (drag_info.grab_frame);
+                       end = session->locations()->first_mark_after (drag_info.grab_frame);
+                       
+                       if (end == max_frames) {
+                               end = session->current_end_frame ();
+                       }
+
+                       if (start == 0) {
+                               start = session->current_start_frame ();
+                       }
 
+                       switch (mouse_mode) {
+                       case MouseObject:
+                               /* find the two markers on either side and then make the selection from it */
+                               cerr << "select between " << start << " .. " << end << endl;
+                               select_all_within (start, end, 0.0f, FLT_MAX, Selection::Set);
+                               break;
+
+                       case MouseRange:
+                               /* find the two markers on either side of the click and make the range out of it */
+                               selection->set (0, start, end);
+                               break;
+
+                       default:
+                               break;
+                       }
                } 
        }
 
@@ -4233,7 +4456,7 @@ Editor::end_range_markerbar_op (GnomeCanvasItem* item, GdkEvent* event)
 
 
 void
-Editor::start_mouse_zoom (GnomeCanvasItem* item, GdkEvent* event)
+Editor::start_mouse_zoom (ArdourCanvas::Item* item, GdkEvent* event)
 {
        drag_info.item = item;
        drag_info.motion_callback = &Editor::drag_mouse_zoom;
@@ -4245,7 +4468,7 @@ Editor::start_mouse_zoom (GnomeCanvasItem* item, GdkEvent* event)
 }
 
 void
-Editor::drag_mouse_zoom (GnomeCanvasItem* item, GdkEvent* event)
+Editor::drag_mouse_zoom (ArdourCanvas::Item* item, GdkEvent* event)
 {
        jack_nframes_t start;
        jack_nframes_t end;
@@ -4272,8 +4495,8 @@ Editor::drag_mouse_zoom (GnomeCanvasItem* item, GdkEvent* event)
        if (start != end) {
 
                if (drag_info.first_move) {
-                       gnome_canvas_item_show (zoom_rect);
-                       gnome_canvas_item_raise_to_top (zoom_rect);
+                       zoom_rect->show();
+                       zoom_rect->raise_to_top();
                }
 
                reposition_zoom_rect(start, end);
@@ -4286,7 +4509,7 @@ Editor::drag_mouse_zoom (GnomeCanvasItem* item, GdkEvent* event)
 }
 
 void
-Editor::end_mouse_zoom (GnomeCanvasItem* item, GdkEvent* event)
+Editor::end_mouse_zoom (ArdourCanvas::Item* item, GdkEvent* event)
 {
        if (!drag_info.first_move) {
                drag_mouse_zoom (item, event);
@@ -4304,7 +4527,7 @@ Editor::end_mouse_zoom (GnomeCanvasItem* item, GdkEvent* event)
                */
        }
 
-       gnome_canvas_item_hide (zoom_rect);
+       zoom_rect->hide();
 }
 
 void
@@ -4314,16 +4537,14 @@ Editor::reposition_zoom_rect (jack_nframes_t start, jack_nframes_t end)
        double x2 = frame_to_pixel (end);
        double y2 = canvas_height - 2;
 
-       gtk_object_set (GTK_OBJECT(zoom_rect), 
-                       "x1", x1,
-                       "y1", 1.0,
-                       "x2", x2,
-                       "y2", y2,
-                       NULL);
+       zoom_rect->property_x1() = x1;
+       zoom_rect->property_y1() = 1.0;
+       zoom_rect->property_x2() = x2;
+       zoom_rect->property_y2() = y2;
 }
 
 void
-Editor::start_rubberband_select (GnomeCanvasItem* item, GdkEvent* event)
+Editor::start_rubberband_select (ArdourCanvas::Item* item, GdkEvent* event)
 {
        drag_info.item = item;
        drag_info.motion_callback = &Editor::drag_rubberband_select;
@@ -4335,7 +4556,7 @@ Editor::start_rubberband_select (GnomeCanvasItem* item, GdkEvent* event)
 }
 
 void
-Editor::drag_rubberband_select (GnomeCanvasItem* item, GdkEvent* event)
+Editor::drag_rubberband_select (ArdourCanvas::Item* item, GdkEvent* event)
 {
        jack_nframes_t start;
        jack_nframes_t end;
@@ -4381,15 +4602,13 @@ Editor::drag_rubberband_select (GnomeCanvasItem* item, GdkEvent* event)
                double x1 = frame_to_pixel (start);
                double x2 = frame_to_pixel (end);
                
-               gtk_object_set (GTK_OBJECT(rubberband_rect), 
-                               "x1", x1,
-                               "y1", y1,
-                               "x2", x2,
-                               "y2", y2,
-                               NULL);
-
-               gnome_canvas_item_show (rubberband_rect);
-               gnome_canvas_item_raise_to_top (rubberband_rect);
+               rubberband_rect->property_x1() = x1;
+               rubberband_rect->property_y1() = y1;
+               rubberband_rect->property_x2() = x2;
+               rubberband_rect->property_y2() = y2;
+
+               rubberband_rect->show();
+               rubberband_rect->raise_to_top();
                
                drag_info.last_pointer_frame = drag_info.current_pointer_frame;
                drag_info.first_move = false;
@@ -4399,7 +4618,7 @@ Editor::drag_rubberband_select (GnomeCanvasItem* item, GdkEvent* event)
 }
 
 void
-Editor::end_rubberband_select (GnomeCanvasItem* item, GdkEvent* event)
+Editor::end_rubberband_select (ArdourCanvas::Item* item, GdkEvent* event)
 {
        if (!drag_info.first_move) {
 
@@ -4416,15 +4635,15 @@ Editor::end_rubberband_select (GnomeCanvasItem* item, GdkEvent* event)
                }
 
 
-               bool add = Keyboard::modifier_state_contains (event->button.state, Keyboard::Shift);
+               Selection::Operation op = Keyboard::selection_type (event->button.state);
                bool commit;
 
                begin_reversible_command (_("select regions"));
 
                if (drag_info.grab_frame < drag_info.last_pointer_frame) {
-                       commit = select_all_within (drag_info.grab_frame, drag_info.last_pointer_frame, y1, y2, add);
+                       commit = select_all_within (drag_info.grab_frame, drag_info.last_pointer_frame, y1, y2, op);
                } else {
-                       commit = select_all_within (drag_info.last_pointer_frame, drag_info.grab_frame, y1, y2, add);
+                       commit = select_all_within (drag_info.last_pointer_frame, drag_info.grab_frame, y1, y2, op);
                }               
 
                if (commit) {
@@ -4437,12 +4656,12 @@ Editor::end_rubberband_select (GnomeCanvasItem* item, GdkEvent* event)
                selection->clear_lines ();
        }
 
-       gnome_canvas_item_hide (rubberband_rect);
+       rubberband_rect->hide();
 }
 
 
 gint
-Editor::mouse_rename_region (GnomeCanvasItem* item, GdkEvent* event)
+Editor::mouse_rename_region (ArdourCanvas::Item* item, GdkEvent* event)
 {
        using namespace Gtkmm2ext;
 
@@ -4450,25 +4669,23 @@ Editor::mouse_rename_region (GnomeCanvasItem* item, GdkEvent* event)
 
        prompter.set_prompt (_("Name for region:"));
        prompter.set_initial_text (clicked_regionview->region.name());
+       prompter.add_button (_("Rename"), Gtk::RESPONSE_ACCEPT);
+       prompter.set_response_sensitive (Gtk::RESPONSE_ACCEPT, false);
        prompter.show_all ();
-       prompter.done.connect (Main::quit.slot());
-       
-       Main::run ();
-
-       if (prompter.status == Prompter::cancelled) {
-               return TRUE;
+       switch (prompter.run ()) {
+       case Gtk::RESPONSE_ACCEPT:
+        string str;
+               prompter.get_result(str);
+               if (str.length()) {
+               clicked_regionview->region.set_name (str);
+               }
+               break;
        }
-
-       string str;
-
-       prompter.get_result(str);
-       clicked_regionview->region.set_name (str);
-
-       return TRUE;
+       return true;
 }
 
 void
-Editor::start_time_fx (GnomeCanvasItem* item, GdkEvent* event)
+Editor::start_time_fx (ArdourCanvas::Item* item, GdkEvent* event)
 {
        drag_info.item = item;
        drag_info.motion_callback = &Editor::time_fx_motion;
@@ -4480,7 +4697,7 @@ Editor::start_time_fx (GnomeCanvasItem* item, GdkEvent* event)
 }
 
 void
-Editor::time_fx_motion (GnomeCanvasItem *item, GdkEvent* event)
+Editor::time_fx_motion (ArdourCanvas::Item *item, GdkEvent* event)
 {
        AudioRegionView* rv = clicked_regionview;
 
@@ -4503,7 +4720,7 @@ Editor::time_fx_motion (GnomeCanvasItem *item, GdkEvent* event)
 }
 
 void
-Editor::end_time_fx (GnomeCanvasItem* item, GdkEvent* event)
+Editor::end_time_fx (ArdourCanvas::Item* item, GdkEvent* event)
 {
        clicked_regionview->get_time_axis_view().hide_timestretch ();
 
@@ -4578,7 +4795,7 @@ Editor::track_height_step_timeout ()
        
        if (delta.tv_sec * 1000000 + delta.tv_usec > 250000) { /* milliseconds */
                current_stepping_trackview = 0;
-               return FALSE;
+               return false;
        }
-       return TRUE;
+       return true;
 }