# HG changeset patch
# User Chris Cannam
# Date 1406304136 -3600
# Node ID 5acfb11e8aef710a2b62286588a09003cf3e340b
# Parent  1fdd895063c5e1b9cdec2ab183f38cced9e8ee49
Do not move playhead on click, unless in "DAW mode". In centre-frame modes it's confusing and leads to random jumps when playing aligned material

diff -r 1fdd895063c5 -r 5acfb11e8aef view/Pane.cpp
--- a/view/Pane.cpp	Fri Jul 25 13:43:56 2014 +0100
+++ b/view/Pane.cpp	Fri Jul 25 17:02:16 2014 +0100
@@ -1333,10 +1333,13 @@
             m_dragStartMinValue = dmin;
         }
 
-        // Schedule a play-head move to the mouse frame location. This
-        // will happen only if nothing else of interest happens
-        // (double-click, drag) before the timeout.
-        schedulePlaybackFrameMove(getFrameForX(e->x()));
+        if (m_followPlay == PlaybackScrollPage) {
+            // Schedule a play-head move to the mouse frame
+            // location. This will happen only if nothing else of
+            // interest happens (double-click, drag) before the
+            // timeout.
+            schedulePlaybackFrameMove(getFrameForX(e->x()));
+        }
 
     } else if (mode == ViewManager::SelectMode) {
 
@@ -1381,11 +1384,13 @@
 
             m_resizing = false;
 
-            // Schedule a play-head move to the mouse frame
-            // location. This will happen only if nothing else of
-            // interest happens (double-click, drag) before the
-            // timeout.
-            schedulePlaybackFrameMove(mouseFrame);
+            if (m_followPlay == PlaybackScrollPage) {
+                // Schedule a play-head move to the mouse frame
+                // location. This will happen only if nothing else of
+                // interest happens (double-click, drag) before the
+                // timeout.
+                schedulePlaybackFrameMove(mouseFrame);
+            }
         }
 
         update();