Move note creation code into a Drag; fix snapping of note

starts when drag-created (part of #4505).


git-svn-id: svn://localhost/ardour2/branches/3.0@10972 d708f5d6-7413-0410-9779-e7cbd77b26cf

1 files changed, 20 insertions, 123 deletions

diff --git a/gtk2_ardour/midi_region_view.cc b/gtk2_ardour/midi_region_view.cc
index 5ac243dd3e..fb598e5f28 100644
--- a/gtk2_ardour/midi_region_view.cc
+++ b/gtk2_ardour/midi_region_view.cc
@@ -97,7 +97,6 @@ MidiRegionView::MidiRegionView (ArdourCanvas::Group *parent, RouteTimeAxisView &
 	, _note_group(new ArdourCanvas::Group(*group))
 	, _note_diff_command (0)
 	, _ghost_note(0)
-	, _drag_rect (0)
 	, _step_edit_cursor (0)
 	, _step_edit_cursor_width (1.0)
 	, _step_edit_cursor_position (0.0)
@@ -134,7 +133,6 @@ MidiRegionView::MidiRegionView (ArdourCanvas::Group *parent, RouteTimeAxisView &
 	, _note_group(new ArdourCanvas::Group(*parent))
 	, _note_diff_command (0)
 	, _ghost_note(0)
-	, _drag_rect (0)
 	, _step_edit_cursor (0)
 	, _step_edit_cursor_width (1.0)
 	, _step_edit_cursor_position (0.0)
@@ -179,7 +177,6 @@ MidiRegionView::MidiRegionView (const MidiRegionView& other)
 	, _note_group(new ArdourCanvas::Group(*get_canvas_group()))
 	, _note_diff_command (0)
 	, _ghost_note(0)
-	, _drag_rect (0)
 	, _step_edit_cursor (0)
 	, _step_edit_cursor_width (1.0)
 	, _step_edit_cursor_position (0.0)
@@ -214,7 +211,6 @@ MidiRegionView::MidiRegionView (const MidiRegionView& other, boost::shared_ptr<M
 	, _note_group(new ArdourCanvas::Group(*get_canvas_group()))
 	, _note_diff_command (0)
 	, _ghost_note(0)
-	, _drag_rect (0)
 	, _step_edit_cursor (0)
 	, _step_edit_cursor_width (1.0)
 	, _step_edit_cursor_position (0.0)
@@ -382,7 +378,7 @@ MidiRegionView::enter_notify (GdkEventCrossing* ev)
 		_mouse_mode_connection, invalidator (*this), ui_bind (&MidiRegionView::mouse_mode_changed, this), gui_context ()
 		);
 
-	if (trackview.editor().current_mouse_mode() == MouseRange) {
+	if (trackview.editor().current_mouse_mode() == MouseRange && _mouse_state != AddDragging) {
 		create_ghost_note (ev->x, ev->y);
 	}
 
@@ -432,11 +428,6 @@ MidiRegionView::button_press (GdkEventButton* ev)
 		return false;
 	}
 
-	_last_x = ev->x;
-	_last_y = ev->y;
-
-	group->w2i (_last_x, _last_y);
-
 	if (_mouse_state != SelectTouchDragging) {
 
 		_pressed_button = ev->button;
@@ -491,7 +482,7 @@ MidiRegionView::button_release (GdkEventButton* ev)
 						beats = 1;
 					}
 
-					create_note_at (event_x, event_y, beats, true, true);
+					create_note_at (editor.pixel_to_frame (event_x), event_y, beats, true, true);
 				}
 
 				break;
@@ -505,7 +496,7 @@ MidiRegionView::button_release (GdkEventButton* ev)
 					beats = 1;
 				}
 
-				create_note_at (event_x, event_y, beats, true, true);
+				create_note_at (editor.pixel_to_frame (event_x), event_y, beats, true, true);
 
 				break;
 			}
@@ -516,30 +507,13 @@ MidiRegionView::button_release (GdkEventButton* ev)
 		_mouse_state = None;
 		break;
 
-	case SelectRectDragging: // Select drag done
+	case SelectRectDragging:
+	case AddDragging:
 		editor.drags()->end_grab ((GdkEvent *) ev);
 		_mouse_state = None;
+		create_ghost_note (ev->x, ev->y);
 		break;
 
-	case AddDragging: // Add drag done
-
-		_mouse_state = None;
-
-		if (Keyboard::is_insert_note_event(ev) || trackview.editor().current_mouse_mode() == MouseRange) {
-
-			if (_drag_rect->property_x2() > _drag_rect->property_x1() + 2) {
-
-				const double x  = _drag_rect->property_x1();
-				const double length = trackview.editor().pixel_to_frame (_drag_rect->property_x2() - _drag_rect->property_x1());
-
-				create_note_at (x, _drag_rect->property_y1(), region_frames_to_region_beats(length), true, false);
-			}
-		}
-
-		delete _drag_rect;
-		_drag_rect = 0;
-
-		create_ghost_note (ev->x, ev->y);
 
 	default:
 		break;
@@ -551,18 +525,7 @@ MidiRegionView::button_release (GdkEventButton* ev)
 bool
 MidiRegionView::motion (GdkEventMotion* ev)
 {
-	double event_x, event_y;
-	framepos_t event_frame = 0;
-
-	event_x = ev->x;
-	event_y = ev->y;
-	group->w2i(event_x, event_y);
-
 	PublicEditor& editor = trackview.editor ();
-	
-	// convert event_x to global frame
-	framecnt_t grid_frames;
-	event_frame = snap_frame_to_grid_underneath (editor.pixel_to_frame (event_x), grid_frames);
 
 	if (!_ghost_note && editor.current_mouse_mode() != MouseRange
 	    && Keyboard::modifier_state_contains (ev->state, Keyboard::insert_note_modifier())
@@ -575,8 +538,7 @@ MidiRegionView::motion (GdkEventMotion* ev)
 		update_ghost_note (ev->x, ev->y);
 	} else if (_ghost_note && editor.current_mouse_mode() != MouseRange) {
 
-		delete _ghost_note;
-		_ghost_note = 0;
+		remove_ghost_note ();
 
 		editor.verbose_cursor()->hide ();
 	} else if (_ghost_note && editor.current_mouse_mode() == MouseRange) {
@@ -590,12 +552,7 @@ MidiRegionView::motion (GdkEventMotion* ev)
 	}
 
 	switch (_mouse_state) {
-	case Pressed: // Maybe start a drag, if we've moved a bit
-
-		if (fabs (event_x - _last_x) < 1 && fabs (event_y - _last_y) < 1) {
-			/* no appreciable movement since the button was pressed */
-			return false;
-		}
+	case Pressed:
 
 		if (_pressed_button == 1 && editor.current_mouse_mode() == MouseObject
 		    && !Keyboard::modifier_state_contains (ev->state, Keyboard::insert_note_modifier())) {
@@ -605,33 +562,11 @@ MidiRegionView::motion (GdkEventMotion* ev)
 			return true;
 
 		} else if (editor.internal_editing()) {
-			// Add note drag start
-
-			group->grab(GDK_POINTER_MOTION_MASK | GDK_BUTTON_RELEASE_MASK,
-			            Gdk::Cursor(Gdk::FLEUR), ev->time);
-
-			_last_x = event_x;
-			_last_y = event_y;
-			_drag_start_x = event_x;
-			_drag_start_y = event_y;
-
-			_drag_rect = new ArdourCanvas::SimpleRect(*group);
-			_drag_rect->property_x1() = editor.frame_to_pixel(event_frame);
-
-			_drag_rect->property_y1() = midi_stream_view()->note_to_y(
-				midi_stream_view()->y_to_note(event_y));
-			_drag_rect->property_x2() = editor.frame_to_pixel(event_frame);
-			
-			_drag_rect->property_y2() = _drag_rect->property_y1()
-				+ floor(midi_stream_view()->note_height());
-			_drag_rect->property_outline_what() = 0xFF;
-			_drag_rect->property_outline_color_rgba() = 0xFFFFFF99;
-			_drag_rect->property_fill_color_rgba()    = 0xFFFFFF66;
 
+			editor.drags()->set (new NoteCreateDrag (dynamic_cast<Editor *> (&editor), group, this), (GdkEvent *) ev);
 			_mouse_state = AddDragging;
-			
-			delete _ghost_note;
-			_ghost_note = 0;
+
+			remove_ghost_note ();
 
 			editor.verbose_cursor()->hide ();
 
@@ -641,47 +576,10 @@ MidiRegionView::motion (GdkEventMotion* ev)
 		return false;
 
 	case SelectRectDragging:
+	case AddDragging:
 		editor.drags()->motion_handler ((GdkEvent *) ev, false);
 		break;
 		
-	case AddDragging: // Add note drag motion
-
-		if (ev->is_hint) {
-			int t_x;
-			int t_y;
-			GdkModifierType state;
-			gdk_window_get_pointer(ev->window, &t_x, &t_y, &state);
-			event_x = t_x;
-			event_y = t_y;
-		}
-
-		if (_mouse_state == AddDragging) {
-			event_x = editor.frame_to_pixel(event_frame);
-
-			if (editor.snap_mode() == SnapNormal) {
-				/* event_frame will have been snapped to the start of the note we are under;
-				   it's more intuitive if we use the end of that note here
-				*/
-				event_x = editor.frame_to_pixel (event_frame + grid_frames);
-			} else {
-				event_x = editor.frame_to_pixel (event_frame);
-			}
-			
-		}
-
-		if (_drag_rect) {
-
-			if (event_x > _drag_start_x) {
-				_drag_rect->property_x2() = event_x;
-			}
-			else {
-				_drag_rect->property_x1() = event_x;
-			}
-		}
-
-		_last_x = event_x;
-		_last_y = event_y;
-
 	case SelectTouchDragging:
 		return false;
 
@@ -878,14 +776,14 @@ MidiRegionView::show_list_editor ()
 }
 
 /** Add a note to the model, and the view, at a canvas (click) coordinate.
- * \param x horizontal position in pixels
+ * \param t time in frames relative to the position of the region
  * \param y vertical position in pixels
  * \param length duration of the note in beats, which will be snapped to the grid
  * \param sh true to make the note 1 frame shorter than the snapped version of \a length.
  * \param snap_x true to snap x to the grid, otherwise false.
  */
 void
-MidiRegionView::create_note_at (double x, double y, double length, bool sh, bool snap_x)
+MidiRegionView::create_note_at (framepos_t t, double y, double length, bool sh, bool snap_x)
 {
 	MidiTimeAxisView* const mtv = dynamic_cast<MidiTimeAxisView*>(&trackview);
 	MidiStreamView* const view = mtv->midi_view();
@@ -896,16 +794,16 @@ MidiRegionView::create_note_at (double x, double y, double length, bool sh, bool
 	assert(note <= 127.0);
 
 	// Start of note in frames relative to region start
-	framepos_t start_frames = trackview.editor().pixel_to_frame (x);
 	if (snap_x) {
 		framecnt_t grid_frames;
-		start_frames = snap_frame_to_grid_underneath (start_frames, grid_frames);
+		t = snap_frame_to_grid_underneath (t, grid_frames);
 	}
-	assert(start_frames >= 0);
+	assert (t >= 0);
 
 	// Snap length
 	length = region_frames_to_region_beats(
-		snap_frame_to_frame (start_frames + region_beats_to_region_frames(length)) - start_frames);
+		snap_frame_to_frame (t + region_beats_to_region_frames(length)) - t
+		);
 
 	assert (length != 0);
 
@@ -914,7 +812,7 @@ MidiRegionView::create_note_at (double x, double y, double length, bool sh, bool
 	}
 
 	const boost::shared_ptr<NoteType> new_note (new NoteType (mtv->get_channel_for_add (),
-	                                                          region_frames_to_region_beats(start_frames + _region->start()), length,
+	                                                          region_frames_to_region_beats(t + _region->start()), length,
 	                                                          (uint8_t)note, 0x40));
 
 	if (_model->contains (new_note)) {
@@ -3446,8 +3344,7 @@ MidiRegionView::update_ghost_note (double x, double y)
 void
 MidiRegionView::create_ghost_note (double x, double y)
 {
-	delete _ghost_note;
-	_ghost_note = 0;
+	remove_ghost_note ();
 
 	boost::shared_ptr<NoteType> g (new NoteType);
 	_ghost_note = new NoEventCanvasNote (*this, *_note_group, g);