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

m3u-player: prevent flickering on track-transitions #734

Merged
merged 7 commits into from
Oct 17, 2021

Conversation

ArneBab
Copy link
Contributor

@ArneBab ArneBab commented May 5, 2021

prevent flickering on track-transitions by keeping the size fixed and overlaying an image of the video

…ze fixed and overlaying an image of the video
@desyncr
Copy link
Contributor

desyncr commented Jul 1, 2021

🎉 I'll try this tonight! 👍

@ArneBab
Copy link
Contributor Author

ArneBab commented Jul 1, 2021

Yay!

This is totally crazy by the way — actually taking snapshots of the video to smooth over breaks — but it’s what works.

The next step in quality would be split-second fade-out and fade-in for audio.

(and these small polishing steps are what’s needed to make some feature — almost any feature — actually enjoyable, just like my gms tweaks work towards making the streams just work, even in pure friend-to-friend mode)

@desyncr
Copy link
Contributor

desyncr commented Jul 1, 2021

It's actually a very clever idea! :)

@desyncr
Copy link
Contributor

desyncr commented Jul 4, 2021

Got to test it finally. Although there's the sound flickering that you mention the video is smoother now. 👍

@ArneBab
Copy link
Contributor Author

ArneBab commented Jul 5, 2021

Thank you for testing!

@Bombe
Copy link
Contributor

Bombe commented Sep 22, 2021

@ArneBab can you please pay a bit attention to what you commit on which branch? Both the bookmark updates and the version number increase do not belong into this pull request.

@ArneBab
Copy link
Contributor Author

ArneBab commented Sep 23, 2021

@ArneBab can you please pay a bit attention to what you commit on which branch? Both the bookmark updates and the version number increase do not belong into this pull request.

arg, sorry. I will fix this here. And there’s still a bit of work to be done: adjusting the canvas to the video size does not yet work in all cases.

@ArneBab ArneBab force-pushed the m3u-player-prevent-flickering branch from 1d737d7 to 0e55657 Compare September 30, 2021 21:52
@ArneBab
Copy link
Contributor Author

ArneBab commented Sep 30, 2021

@Bombe the branch is cleaned up from stuff not belonging here now.

@ArneBab
Copy link
Contributor Author

ArneBab commented Sep 30, 2021

@desyncr the latest commit should kill every flickering during transition. The only side-effect should be that the video stops dynamic size changes from resizing of the browser window while it is showing a loading spinner. It restarts them once the next part is loaded.

It would be great if you could give this another review to see whether it can be merged.

@ArneBab
Copy link
Contributor Author

ArneBab commented Oct 1, 2021

@desyncr so much for „last“: I found simplifications. Now it should be more robust and work well if the video element size is fixed to something that does not match the video aspect ratio.

@desyncr
Copy link
Contributor

desyncr commented Oct 1, 2021

I'll take a look tonight! :) The dynamic resizing sounds look a very specific use-case so I'm not worried about that!

@ArneBab
Copy link
Contributor Author

ArneBab commented Oct 3, 2021

Did you get to look into it?

@desyncr
Copy link
Contributor

desyncr commented Oct 4, 2021

@ArneBab I could test it just now and looks way better! Both mobile and desktop.

@ArneBab
Copy link
Contributor Author

ArneBab commented Oct 4, 2021

Thank you!

@ArneBab ArneBab merged commit 3bbd0e4 into next Oct 17, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants