-
Notifications
You must be signed in to change notification settings - Fork 21
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
Target SDK 35 (includes lots of library updates) #139
Conversation
@lucasmz-dev could I interest you in testing this PR? |
Can you set up CI to post the build results lol I don't wanna build stuff locally for security reasons |
Added a commit for uploading the APK, let's see if this works. |
worked, APK available for download here: https://github.com/onionshare/onionshare-android/actions/runs/13163328184?pr=139 |
Seems to look OK, the only thing I'm noticing is that, the plus icon at the bottom right is squared and not a circle.

|
Minor but the about page seems to not use edge to edge (has black bars at the bottom) |
I'm a bit confused on where the pure black is meant to apply, if at all, right now, everything seems to use it, except when you have some file added and then pretty much everywhere inside there uses other colours Pre-PR uses the pure black everywhere |
I think that's an automatic MD3 change: https://m3.material.io/components/floating-action-button/overview#fddf911f-14da-4d73-8601-59a6c2411c3c
should be fixed now
ok, so there's some areas that are not pure black. the file list should be fixed. anything else that was pure black before? |
Looks alright to me, think that was pretty much it |
do you have some official source for how the navbar is supposed to look now? I fixed the about page once again. |
No, I'm going after what looks normal and how Google does it in their apps |
Huh. Drive has this issue as well but not Maps, I guess it's fine, then? It'd look better but well /shrug |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good! Tested on API 21 and 34. It's concerning how much churn there's been in the UI libraries - thanks for dealing with that.
Thanks everyone for testing! |
Since it upgrades all the things, we should do smoke tests on Android 5 to 15.
Closes #119, #107