Skip to content

feat: implement waiting for video finish#2001

Merged
ebiggz merged 6 commits intocrowbartools:v5from
dennisrijsdijk:wait-for-video-to-finish
Jan 12, 2023
Merged

feat: implement waiting for video finish#2001
ebiggz merged 6 commits intocrowbartools:v5from
dennisrijsdijk:wait-for-video-to-finish

Conversation

@dennisrijsdijk
Copy link
Copy Markdown
Collaborator

@dennisrijsdijk dennisrijsdijk commented Dec 25, 2022

Description of the Change

This Pull Request implements waiting for a Local video to complete.

Applicable Issues

#1958
#1859
#1783 (Partly, would rely on waiting for video)

Testing

I've tested to ensure that the effect will always continue execution after the video finishes playing. In order to ensure this, loop is disabled when wait for video to finish is enabled.

Screenshots

Play Video Effect Duration, Wait for video to finish is not ticked.
image
Play Video Effect Duration, Wait for video to finish is ticked and Loop is automatically unticked.
image

Copy link
Copy Markdown
Member

@ebiggz ebiggz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks great! Awesome work!

@ebiggz ebiggz changed the title Implement waiting for Local Video (File & Random From Folder) feat: implement waiting for video finish Jan 12, 2023
@ebiggz ebiggz enabled auto-merge (squash) January 12, 2023 15:28
@ebiggz ebiggz merged commit 4c08d6b into crowbartools:v5 Jan 12, 2023
@dennisrijsdijk dennisrijsdijk deleted the wait-for-video-to-finish branch July 31, 2024 10:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants