-
Notifications
You must be signed in to change notification settings - Fork 0
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
Use Material 3 #26
Use Material 3 #26
Conversation
androidApp/src/main/java/org/datepollsystems/waiterrobot/android/ui/billing/PaymentView.kt
Fixed
Show fixed
Hide fixed
androidApp/src/main/java/org/datepollsystems/waiterrobot/android/ui/billing/PaymentView.kt
Fixed
Show fixed
Hide fixed
androidApp/src/main/java/org/datepollsystems/waiterrobot/android/ui/billing/PaymentView.kt
Fixed
Show fixed
Hide fixed
androidApp/src/main/java/org/datepollsystems/waiterrobot/android/ui/common/SwipeableListItem.kt
Fixed
Show fixed
Hide fixed
androidApp/src/main/java/org/datepollsystems/waiterrobot/android/ui/common/SwipeableListItem.kt
Fixed
Show fixed
Hide fixed
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.
The whole UI refactoring looks generally speaking good, but I'm definitely not able to provide valuable insights by just reading compose code.
The only thing I have to add is that the Pull to refresh behaviour seems very complicated to do :D
androidApp/src/main/java/org/datepollsystems/waiterrobot/android/MainActivity.kt
Show resolved
Hide resolved
One more thing, is there a reason there is a left & right padding besides the chips in this screenshot? screenshot. Instead of it being aligned with the rest? |
Replace Material2 with Material3 and refactor improve UI of some screens.