-
Notifications
You must be signed in to change notification settings - Fork 2k
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
refactor(mobile): Use hooks to manage Chewie controller for video #7008
Conversation
Deploying with
|
Latest commit: |
a4128d3
|
Status: | ✅ Deploy successful! |
Preview URL: | https://761938a6.immich.pages.dev |
Branch Preview URL: | https://fix-video-delayed-loading-th.immich.pages.dev |
When you click on a video, I see that it is immediately in the immersive mode. Tapping on the screen, the control interface shows briefly and returns to the immersive mode. I think this is a bit inconvenient if you want to share a video, because you will have to aim to the middle of the screen to tap and then hit the pause button to get the control interface to show up |
That is definitely a bug. The intended behavior is for the controls to show for the |
…layed-loading-throbber
…hile playing and reverts hide controls timer to 5 seconds
f6ab532
to
985fcba
Compare
Rebased and working well on my end here! Ready for another round of review. |
…layed-loading-throbber
) * video loading delayed * Chewie controller implemented in hook * fixing look and feel * Finalizing delay and animations * Fixes issue with immersive mode showing immediately in videos * format fix * Fixes bug where video controls would hide immediately after showing while playing and reverts hide controls timer to 5 seconds * Fixed rebase issues --------- Co-authored-by: Alex Tran <alex.tran1502@gmail.com>
This was all to improve the asset video playback experience while navigating the main timeline.