-
-
Notifications
You must be signed in to change notification settings - Fork 16
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
🥳[Feature] Redesign unlock view #50
Conversation
- Extended tap area of the balance animation
- Dev code fixes - Adjust layout
On the topic of the lock screen, I have not found a way to set up biometrics to login, Is that set up and Im missing it or is it something that we should create an issue for? @kcw-grunt |
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.
I think we should still have somewhere we can show the price after logging in, even if it's just in the menu or something. If you think not, we can push this down to a future redesign. @kcw-grunt
I do agree that it was confusing having current holdings and price right there though.
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.
LGTM
Agreed. This was just a |
Issue
Closes #7
Why?
The current price (in the header view) was confusing many users and the lock screen needed a refresh.
Screenshots