-
-
Notifications
You must be signed in to change notification settings - Fork 80
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
Replace AppIntro by Compose Pager #848
Conversation
Would probably be a good idea to mention in the git log message why AppIntro was replaced/removed. Two smaller issues:
|
@ArnyminerZ Please request a review from me again when ready :) |
Should be |
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.
Looking good. Minor comments :)
app/src/main/kotlin/at/bitfire/davdroid/ui/intro/IntroScreen.kt
Outdated
Show resolved
Hide resolved
app/src/main/kotlin/at/bitfire/davdroid/ui/composable/PositionIndicator.kt
Outdated
Show resolved
Hide resolved
app/src/main/kotlin/at/bitfire/davdroid/ui/intro/IntroActivity.kt
Outdated
Show resolved
Hide resolved
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.
Looking forward to remove the last non-Compose code :) Noticed some things:
- When you click through the pages (for instance, only 2 pages like in the video), DAVx5 quits instead of going to the Accounts overview:
Screen_recording_20240623_105103.webm
- I think it would look better if the position indicator circles would always have the same distance to each other.
Old app intro:
In contrast, the new app intro with the big distance between the circles:
I think we should define a fixed dp distance between the circles.
- I'd leave
ButtonWithIcon
andPositionIndicator
in the screen where it's needed. We don't need it somewhere else and it makes thecomposable
list long and at some time confusing. When we at some time need it somewhere else, we can extract it to thecomposables
package.
Signed-off-by: Arnau Mora Gras <arnyminerz@proton.me>
Signed-off-by: Arnau Mora Gras <arnyminerz@proton.me>
Signed-off-by: Arnau Mora Gras <arnyminerz@proton.me>
Signed-off-by: Arnau Mora Gras <arnyminerz@proton.me>
Signed-off-by: Arnau Mora <arnyminerz@proton.me>
Signed-off-by: Arnau Mora <arnyminerz@proton.me>
Signed-off-by: Arnau Mora Gras <arnyminerz@proton.me>
Signed-off-by: Arnau Mora Gras <arnyminerz@proton.me>
Signed-off-by: Arnau Mora Gras <arnyminerz@proton.me>
Signed-off-by: Arnau Mora Gras <arnyminerz@proton.me>
Signed-off-by: Arnau Mora Gras <arnyminerz@proton.me>
16cc3d0
to
c7b18b1
Compare
The PR should be in Draft state during development. As soon as it's finished, it should be marked as Ready for review and a reviewer should be chosen.
See also: Writing A Great Pull Request Description
Purpose
Migrates all AppIntro code to Compose Pager.
Short description
ButtonWithIcon
which is a circular button with an icon inside.PositionIndicator
which shows multiple circles indicating the current position of the slider.IntroScreen
which handles all the things AppIntro did before.Checklist