5
votes
Accepted
Compose Grid With Lists
Looks quite good already, I haven't found any serious issues.
The are some minor improvements that I would suggest, though.
Small bugs
I'm not sure how your data source looks like, but you import <...
- 498
4
votes
Accepted
Logic to check if app was launched for the first time using DataStore
This already looks quite good.
Apart from some minor formatting issues I think the following things can be improved:
In the MainActivity the property dataStore can ...
- 498
4
votes
Accepted
Jetpack Compose: Length-Units Converter
Regarding making this more idiomatic Kotlin code you can take advantage of direct value assignment like so:
...
- 186
3
votes
Kotlin Compose authentication with Google and Facebook using a CircularProgressIndicator for UI loading
First, GoogleSignIn is deprecated. You now have to use Credential Manager.
That being said...
I recommend you to move all login logic to the ViewModel or data layer:...
- 31
2
votes
Jetpack Compose registration screen
Readability
Although the composable is pretty long, there is no complex logic. It is properly formatted and straight-forward to read; I would probably keep it as it is for now. When it grows bigger ...
- 498
2
votes
Handling View State & Side Effects with Lifecycle Awareness in Jetpack Compose MVI Composable
I have a composable for handling lifecycle events
...
1
vote
Android: Navigation with Hilt + Jetpack Compose
I pass in a delegate and handle navigation at the top. I find this better practice
- 11
Only top scored, non community-wiki answers of a minimum length are eligible