Skip to content
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

nicer fading #740

Merged
merged 1 commit into from
Sep 3, 2023
Merged

nicer fading #740

merged 1 commit into from
Sep 3, 2023

Conversation

barbeque-squared
Copy link
Collaborator

fixes #86
make fading from sing screen to score screen much nicer by unloading the video later

maybe it'll still show the background image for a fraction of a second if you disabled both the background + video by pressing V during a song. did not test visualizations.

demo video:

simplescreenrecorder-2023-08-30_10.40.34.mp4

for at least videos or background images it will just use whatever was on the last frame.

@basisbit
Copy link
Member

Please make sure that this doesn't cause issues with the transition audio playback, when using a video file with audio integrated and using that audio as source.
If I remember correctly, OnHide gets called only after the transition is done and the next screen is already loaded, so there also is a chance that this here change introduces a memory leak when playing some plugins or the multiple-songs-in-a-row feature and maybe also for medley.

@barbeque-squared
Copy link
Collaborator Author

@basisbit I retested this with multiple webm's (audio+video from the same webm) as both individual songs as well as a medley, did not notice any issues.
I think the transitions within a medley were always hard. I can't really test this on Windows but anyone can download the PR build artifact and quickly verify that themselves. If it turns out there is a path somewhere that doesn't call onHide at all, I think that's more of a bug in that path and less in this item.

I don't know what you mean with the "multiple-songs-in-a-row feature"

@barbeque-squared
Copy link
Collaborator Author

Considering there's been no replies I'm merging this. If something substantially does break from this we can always just revert it (or fix it) and do a bugfix release.

@barbeque-squared barbeque-squared merged commit 6264b36 into master Sep 3, 2023
2 checks passed
@barbeque-squared barbeque-squared deleted the nicer-fading branch September 3, 2023 08:18
@barbeque-squared barbeque-squared mentioned this pull request Sep 3, 2023
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.

Song screen fade out shows image even if video was used
2 participants