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

Target SDK 35 (includes lots of library updates) #139

Merged
merged 6 commits into from
Feb 11, 2025

Conversation

grote
Copy link
Collaborator

@grote grote commented Jan 15, 2025

Since it upgrades all the things, we should do smoke tests on Android 5 to 15.

Closes #119, #107

@grote
Copy link
Collaborator Author

grote commented Feb 5, 2025

@lucasmz-dev could I interest you in testing this PR?

@lucasmz-dev
Copy link
Contributor

Can you set up CI to post the build results lol I don't wanna build stuff locally for security reasons

@grote
Copy link
Collaborator Author

grote commented Feb 5, 2025

Added a commit for uploading the APK, let's see if this works.

@grote
Copy link
Collaborator Author

grote commented Feb 5, 2025

worked, APK available for download here: https://github.com/onionshare/onionshare-android/actions/runs/13163328184?pr=139

@lucasmz-dev
Copy link
Contributor

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.

![Screenshot_20250205-211536_1](https://github.com/user-attachments/assets/cb1d51cd-3c18-4810-8903-dc8d22261544)

@lucasmz-dev
Copy link
Contributor

Minor but the about page seems to not use edge to edge (has black bars at the bottom)

@lucasmz-dev
Copy link
Contributor

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

@grote
Copy link
Collaborator Author

grote commented Feb 6, 2025

the only thing I'm noticing is that, the plus icon at the bottom right is squared and not a circle.

I think that's an automatic MD3 change: https://m3.material.io/components/floating-action-button/overview#fddf911f-14da-4d73-8601-59a6c2411c3c
However, we were cutting off too much off the edges, so I made them rounder again.

Minor but the about page seems to not use edge to edge (has black bars at the bottom)

should be fixed now

Pre-PR uses the pure black everywhere

ok, so there's some areas that are not pure black. the file list should be fixed. anything else that was pure black before?

@lucasmz-dev
Copy link
Contributor

Looks alright to me, think that was pretty much it

@lucasmz-dev
Copy link
Contributor

lucasmz-dev commented Feb 8, 2025

OK so I forgot to test how it looked in 3 button nav, and there a few things to fix there 🫠

  1. The nav bar is a different color (might be a semi transparent layer I'm not sure)

Screenshot_20250208-182855

@lucasmz-dev
Copy link
Contributor

lucasmz-dev commented Feb 8, 2025

  1. About page goes behind the nav bar at the end

Screenshot_20250208-183058

@grote
Copy link
Collaborator Author

grote commented Feb 10, 2025

The nav bar is a different color (might be a semi transparent layer I'm not sure)

do you have some official source for how the navbar is supposed to look now?

I fixed the about page once again.

@lucasmz-dev
Copy link
Contributor

do you have some official source for how the navbar is supposed to look now?

No, I'm going after what looks normal and how Google does it in their apps

@lucasmz-dev
Copy link
Contributor

Huh. Drive has this issue as well but not Maps, I guess it's fine, then? It'd look better but well /shrug

Copy link
Collaborator

@akwizgran akwizgran left a 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.

@grote grote merged commit 642f17e into onionshare:main Feb 11, 2025
1 check passed
@grote
Copy link
Collaborator Author

grote commented Feb 11, 2025

Thanks everyone for testing!

@grote grote deleted the target-35 branch February 11, 2025 11:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Material Design 3
3 participants