Tweak temporal playback behavior

nyalldawson · nyalldawson · commit 25e24e7d2ab9 · 2020-05-22T17:22:28.000+10:00
If a non-looping animation is paused at the end and the user hits the play
button again, auto rewind and play the animation from the start
diff --git a/src/core/qgstemporalnavigationobject.cpp b/src/core/qgstemporalnavigationobject.cpp
@@ -218,12 +218,24 @@ void QgsTemporalNavigationObject::pause()
 
 void QgsTemporalNavigationObject::playForward()
 {
+  if ( mPlayBackMode == Idle &&  mCurrentFrameNumber >= totalFrameCount() - 1 )
+  {
+    // if we are paused at the end of the video, and the user hits play, we automatically rewind and play again
+    rewindToStart();
+  }
+
   setAnimationState( AnimationState::Forward );
   play();
 }
 
 void QgsTemporalNavigationObject::playBackward()
 {
+  if ( mPlayBackMode == Idle &&  mCurrentFrameNumber <= 0 )
+  {
+    // if we are paused at the start of the video, and the user hits play, we automatically skip to end and play in reverse again
+    skipToEnd();
+  }
+
   setAnimationState( AnimationState::Reverse );
   play();
 }

Original file line number	Diff line number	Diff line change
`@@ -218,12 +218,24 @@ void QgsTemporalNavigationObject::pause()`
`218`	`218`
`219`	`219`	`void QgsTemporalNavigationObject::playForward()`
`220`	`220`	`{`
	`221`	`+ if ( mPlayBackMode == Idle && mCurrentFrameNumber >= totalFrameCount() - 1 )`
	`222`	`+ {`
	`223`	`+ // if we are paused at the end of the video, and the user hits play, we automatically rewind and play again`
	`224`	`+ rewindToStart();`
	`225`	`+ }`
	`226`	`+`
`221`	`227`	`setAnimationState( AnimationState::Forward );`
`222`	`228`	`play();`
`223`	`229`	`}`
`224`	`230`
`225`	`231`	`void QgsTemporalNavigationObject::playBackward()`
`226`	`232`	`{`
	`233`	`+ if ( mPlayBackMode == Idle && mCurrentFrameNumber <= 0 )`
	`234`	`+ {`
	`235`	`+ // if we are paused at the start of the video, and the user hits play, we automatically skip to end and play in reverse again`
	`236`	`+ skipToEnd();`
	`237`	`+ }`
	`238`	`+`
`227`	`239`	`setAnimationState( AnimationState::Reverse );`
`228`	`240`	`play();`
`229`	`241`	`}`