-
-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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
Migrate about activity to Jetpack Compose #11282
base: refactor
Are you sure you want to change the base?
Migrate about activity to Jetpack Compose #11282
Conversation
OMG finally a JC PR where there is a visible change in the UI. |
Dark red text for the tabs isn't quite ideal though, but I suppose the PR is still WIP. Can you please mark WIP PRs as draft next time? Great work again with regards to doing yet another Jetpack Compose migration. |
Now I looked a bit at the code and I think the list of libraries is definitely outdated, so that should be fixed as well. |
Regarding outdated libs: #11202 should be implemented once this is merged. |
The comments PR has visible changes as well. |
It's mostly completed, it's just that one detail remaining. Also, we might want to review how the theme should look. This should help: https://material-foundation.github.io/material-theme-builder/ |
8936de0
to
ca4759f
Compare
I have noticed 2 things:
|
The scrollbar is provided by a third-party library (Compose does not currently support scrollbars).
Sounds good. |
fa93e3a
to
ba2efde
Compare
Quality Gate failedFailed conditions |
That library includes a composable that automatically displays the external dependencies, so I used that in this PR itself. |
# Conflicts: # app/build.gradle # app/src/main/java/org/schabi/newpipe/ktx/Bundle.kt # build.gradle
d462fb5
to
c00c6c4
Compare
# Conflicts: # app/build.gradle # build.gradle
Quality Gate passedIssues Measures |
What is it?
Description of the changes in your PR
ScaffoldWithToolbar
composable for future reuse in refactoring.Before/After Screenshots/Screen Record
Before:
After:
Fixes the following issue(s)
Relies on the following changes
APK testing
The APK can be found by going to the "Checks" tab below the title. On the left pane, click on "CI", scroll down to "artifacts" and click "app" to download the zip file which contains the debug APK of this PR. You can find more info and a video demonstration on this wiki page.
Due diligence