From: Ben Loftis <ben@harrisonconsoles.com>
Date: Fri, 10 Aug 2018 15:43:38 +0000 (-0500)
Subject: Likely fix for jittery snapped-cursor when dragging a region.
X-Git-Url: https://main.carlh.net/gitweb/?p=ardour.git;a=commitdiff_plain;h=9ba09caa9b9c0101e3d94a83d2cc57f637b5607c

Likely fix for jittery snapped-cursor when dragging a region.
---

diff --git a/gtk2_ardour/editor_mouse.cc b/gtk2_ardour/editor_mouse.cc
index 9bd882933b..5fc9524810 100644
--- a/gtk2_ardour/editor_mouse.cc
+++ b/gtk2_ardour/editor_mouse.cc
@@ -2106,17 +2106,17 @@ Editor::motion_handler (ArdourCanvas::Item* /*item*/, GdkEvent* event, bool from
 
 	update_join_object_range_location (event->motion.y);
 
-	/* snapped_cursor stuff (the snapped_cursor shows where an operation is going to occur) */
-	bool ignored;
-	MusicSample where (0, 0);
-	if (mouse_sample (where.sample, ignored)) {
-		snap_to_with_modifier (where, event);
-		set_snapped_cursor_position (where.sample);
-	}
-
-	/* drags might also change the snapped_cursor location, because we are snapping the thing being dragged, not the actual mouse cursor */
 	if (_drags->active ()) {
+		//drags change the snapped_cursor location, because we are snapping the thing being dragged, not the actual mouse cursor
 	 	return _drags->motion_handler (event, from_autoscroll);
+	} else {
+		//the snapped_cursor shows where an operation (like Split) is going to occur
+		bool ignored;
+		MusicSample where (0, 0);
+		if (mouse_sample (where.sample, ignored)) {
+			snap_to_with_modifier (where, event);
+			set_snapped_cursor_position (where.sample);
+		}
 	}
 
 	return false;