-
Notifications
You must be signed in to change notification settings - Fork 569
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
Episode synopsis bottom dialog not dismissing properly #816
Comments
i am not able to reproduce this , it gets dismissed in once again and again |
Click the synopsis, dismiss it, it works. Play an episode, go back, click the synopsis again, and you have to click twice to dismiss it. |
ezgif-1-619935eb48.mp4 |
ya, reproductive now after player opening |
This is particularly annoying when you click the synopsis for each episode after playing an episode you have to dismiss it however many times you have clicked the synopsis then played an episode, which is quite annoying. I do think it is related to dismiss callback not working properly though, I just can't seem to get CloudStream Debug to test properly right now |
i got this, the log starts when i open the synospsi after playing and close it, i should be getting binder got null and it closes once, but when suspended twice it generates this, |
also this bro there is something wrong with sorastream, when we close the player, the extension and extractor generate high amount of requests to sites and use data even when player is closed few seconds ago |
@IndusAryan Now that's interesting... I wonder if that has anything to do with #530 as well? |
@fire-light42 @LagradOst @Blatzar also ^ should I report this on hexated repo or is this an issue with CloudStream? |
This did give me an idea for what causes #530 if it isn't that exactly though. |
Instead of using a BottomSheetDialog, which can be buggy (recloudstream#816) and modifies the UI more then we need to, which is just a little bit of poor UI, this way it just expands the description/synopsis if it is clicked, which is nicer UI. It only only this for phone and emulator layouts though, on TV having the same popup is nicer so it puts the text in a bit bigger view also. This method is also nice, because if there is nothing to expand it doesn't created an unneeded dialog or do anything at all, and if you accidentally click it it doesn't end up getting in your way. showBottomDialogText in SingleSelectionHelper could probably be removed after this if you want as it is now unused again, however I decided to leave it as I figured maybe it would be useful in the future but not sure if you want to or not...
Steps to reproduce
Expected behavior
It should only need to be closed once
Actual behavior
Discribed above.
Cloudstream version and commit hash
4.3.0 6d51c59
Android version
Android 14
Logcat
No response
Other details
I believe this is caused by the dismiss callback not being triggered this way, and so viewmodel.releaseEpisodeSynopsis is never triggered. I could be wrong though and my PC hasn't been playing well lately so can't test and fix myself
Acknowledgements
The text was updated successfully, but these errors were encountered: