-
-
Notifications
You must be signed in to change notification settings - Fork 442
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
"Broken" welcome screen on first launch in small devices #5959
Comments
Thanks for the detailed issue description, I really appreciate the efforts you spent into finding solutions (without using AI to generate the description, that's so much more pleasant to work with for me) 👍
I think that this one sounds pretty interesting and fun to implement (I've already done something similar with Jetpack Compose some times, though LibreTube doesn't use Jetpack Compose unfortunately), so I'll probably see if I can make this work some time :) |
Thank you very much!! Looking forward to test this out! |
You can test it from the builds here: https://github.com/libre-tube/NightlyBuilds |
Steps to reproduce
Expected behavior
Any user should be able to properly navigate the instances list on the welcome screen.
Actual behavior
Only a small scrollable section is available.
In fact, when I encountered this error for the first time on version 0.22, it was impossible to select any instance. Thankfully I had my older phone, so I could generate a backup and restore it with the restore button (that is amazing that it can be done on the welcome screen, BTW, that's nice UX and I believe should not be changed).
LibreTube version
0.23.0
Android version
Android 13
Other details
I believe this is very easily solvable in many different ways.
I sadly don't have the Android skills to do it myself, but I believe that if the whole activity (I believe each "screen" is called like that) was scrollable and the list of instances was printed on the "root" itself... that would be fixed, and I don't see any downsides.
Some more alternatives would be:
Acknowledgements
The text was updated successfully, but these errors were encountered: