comparison view/Pane.cpp @ 790:7555b8b17e6e tonioni

Move play pointer on single-click in pane in navigate mode

 
     if (m_clickedInRange) {
         mouseMoveEvent(e);
     }
 
+    int mouseFrame = e ? getFrameForX(e->x()) : 0;
+    if (mouseFrame < 0) mouseFrame == 0;
+
     if (m_navigating || mode == ViewManager::NavigateMode) {
 
         m_navigating = false;
 
         if (mode != ViewManager::NavigateMode) {

 
             int y0 = std::min(m_clickPos.y(), m_mousePos.y());
             int y1 = std::max(m_clickPos.y(), m_mousePos.y());
 
             emit regionOutlined(QRect(x0, y0, x1 - x0, y1 - y0));
+
+        } else if (m_manager && m_dragMode == UnresolvedDrag) {
+
+            // Simple click, no drag made: move play head to the mouse
+            // frame location
+            m_manager->setPlaybackFrame(mouseFrame);
         }
 
     } else if (mode == ViewManager::SelectMode) {
 
         if (!hasTopLayerTimeXAxis()) {

             if (exclusive) {
                 m_manager->setSelection(selection);
             } else {
                 m_manager->addSelection(selection);
             }
-        }
-        else if (m_manager && !m_manager->haveInProgressSelection()) {
+
+        } else if (m_manager && !m_manager->haveInProgressSelection()) {
             
-            //cerr << "JTEST: release without selection" << endl;
-            // Get frame location of mouse
-            int mouseFrame = getFrameForX(e->x());
-            //cerr << "JTEST: frame location of click is " << mouseFrame << endl;
-            // Move play head to that frame location
-            int playbackFrame = fmax(0,mouseFrame);
-            m_manager->setPlaybackFrame(playbackFrame);
+            // Simple click, no selection made: move play head to the
+            // mouse frame location
+            m_manager->setPlaybackFrame(mouseFrame);
         }
     
         update();
 
     } else if (mode == ViewManager::DrawMode) {