New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix: Handle start_time
changes in st.video
#7257
Conversation
frontend/cypress/snapshots/linux/2x/st_video.spec.js/video-updated-start.snap.png
Outdated
Show resolved
Hide resolved
@@ -49,6 +49,11 @@ export default function Video({ | |||
|
|||
if (videoNode) { | |||
videoNode.addEventListener("loadedmetadata", setStartTime) | |||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In this case, we will always change the current time to the start time, even if a rerun happens while playing video.
I think it is a good idea that the new behavior is consistent with st.audio
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Makes sense - I followed st.audio
pattern this time if you'd like to give it another look!
c25e7d5
to
529cd51
Compare
Adds additional check to useEffect to handle changes in start_time for st.video
Adds additional check to useEffect to handle changes in start_time for st.video
Adds additional check to useEffect to handle changes in start_time for st.video
Describe your changes
Adds additional check to
useEffect
to handle changes instart_time
forst.video
GitHub Issue Link (if applicable)
Closes #7126
Testing Plan