comparison view/View.cpp @ 511:455810e65543

* Do a _slightly_ better job of ensuring new panes come up showing the
  right region of the track (but not quite better enough)

 
     connect(this, SIGNAL(zoomLevelChanged(unsigned long, bool)),
 	    m_manager, SLOT(viewZoomLevelChanged(unsigned long, bool)));
 
     if (m_followPlay == PlaybackScrollPage) {
-//        std::cerr << "View::setViewManager: setting centre frame to global centre frame: " << m_manager->getGlobalCentreFrame() << std::endl;
+        std::cerr << "View::setViewManager: setting centre frame to global centre frame: " << m_manager->getGlobalCentreFrame() << std::endl;
         setCentreFrame(m_manager->getGlobalCentreFrame(), false);
     } else if (m_followPlay == PlaybackScrollContinuous) {
-//        std::cerr << "View::setViewManager: setting centre frame to playback frame: " << m_manager->getPlaybackFrame() << std::endl;
+        std::cerr << "View::setViewManager: setting centre frame to playback frame: " << m_manager->getPlaybackFrame() << std::endl;
         setCentreFrame(m_manager->getPlaybackFrame(), false);
     } else if (m_followPan) {
-//        std::cerr << "View::setViewManager: setting centre frame to global centre frame: " << m_manager->getGlobalCentreFrame() << std::endl;
+        std::cerr << "View::setViewManager: (follow pan) setting centre frame to global centre frame: " << m_manager->getGlobalCentreFrame() << std::endl;
         setCentreFrame(m_manager->getGlobalCentreFrame(), false);
     }
     if (m_followZoom) setZoomLevel(m_manager->getGlobalZoom());
+
+    movePlayPointer(getAlignedPlaybackFrame());
 
     toolModeChanged();
 }
 
 void

 	if (sender() != m_manager) return;
     }
 
     f = getAlignedPlaybackFrame();
 
-    if (m_playPointerFrame == f) return;
-    bool visible = (getXForFrame(m_playPointerFrame) != getXForFrame(f));
+    movePlayPointer(f);
+}
+
+void
+View::movePlayPointer(unsigned long newFrame)
+{
+    if (m_playPointerFrame == newFrame) return;
+    bool visibleChange =
+        (getXForFrame(m_playPointerFrame) != getXForFrame(newFrame));
     size_t oldPlayPointerFrame = m_playPointerFrame;
-    m_playPointerFrame = f;
-    if (!visible) return;
+    m_playPointerFrame = newFrame;
+    if (!visibleChange) return;
 
     switch (m_followPlay) {
 
     case PlaybackScrollContinuous:
 	if (QApplication::mouseButtons() == Qt::NoButton) {
-	    setCentreFrame(f, false);
+	    setCentreFrame(m_playPointerFrame, false);
 	}
 	break;
 
     case PlaybackScrollPage:
     { 
 	int xold = getXForFrame(oldPlayPointerFrame);
 	update(xold - 1, 0, 3, height());
 
 	long w = getEndFrame() - getStartFrame();
 	w -= w/5;
-	long sf = (f / w) * w - w/8;
+	long sf = (m_playPointerFrame / w) * w - w/8;
 
 	if (m_manager &&
 	    m_manager->isPlaying() &&
 	    m_manager->getPlaySelectionMode()) {
 	    MultiSelection::SelectionList selections = m_manager->getSelections();

 		}
 	    }
 	}
 
 #ifdef DEBUG_VIEW_WIDGET_PAINT
-	std::cerr << "PlaybackScrollPage: f = " << f << ", sf = " << sf << ", start frame "
+	std::cerr << "PlaybackScrollPage: f = " << m_playPointerFrame << ", sf = " << sf << ", start frame "
 		  << getStartFrame() << std::endl;
 #endif
 
 	// We don't consider scrolling unless the pointer is outside
 	// the clearly visible range already

 
 	break;
     }
 
     case PlaybackIgnore:
-	if (long(f) >= getStartFrame() && f < getEndFrame()) {
+	if (long(m_playPointerFrame) >= getStartFrame() &&
+            m_playPointerFrame < getEndFrame()) {
 	    update();
 	}
 	break;
     }
 }