Closed sunidhi64 closed 2 years ago
Dope work. Super super cool.
I tested on my phone and everything worked well. I'll need to dig into the code a bit before I can give bigger comments. A few thoughts to start:
Dope work. Super super cool.
I tested on my phone and everything worked well. I'll need to dig into the code a bit before I can give bigger comments. A few thoughts to start:
- Can we put a button in the settings to allow people to reset the tutorials? As in, if they want to see it again?
- So the "tutorials" and the "drawer" buttons, I see you were able to just add a glowing circle, which indicates that they should check those out. The fact that they don't have text associated with them is because we didn't have resource IDs for those items right?
I'm moving the whole UI to Jetpack Compose, and so this PR will not be mergeable in its current state. But I'll keep that structure in mind and will link to this if I or others try to implement that feature in the future!
Thanks again for your great work @sunidhi64.
Refers to #136 When the user uses the app for the first time, there should be a quick tour of how to use the feature of the app. I have added a quick tour when a user creates a wallet for the first time.