-
Notifications
You must be signed in to change notification settings - Fork 439
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
fix: Adjust overview widget on dashboard for new design #2782
Conversation
Branch preview✅ Deploy successful! https://dashboard_overview--walletweb.review-wallet-web.5afe.dev |
ESLint Summary View Full Report
Report generated by eslint-plus-action |
Coverage report
Test suite run success1105 tests passing in 155 suites. Report generated by 🧪jest coverage report action from a66b7d2 |
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!
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 just realized that in Dark mode the icons do not adjust their color.
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 app is also listed now on staging and the button works :)
The skeletons need some adjustments, it jumps twice when loading. Screen.Recording.2023-11-09.at.16.24.32.mov |
Can you make Send and Receive be on the same line, and Buy crypto on a different line? |
What it solves
Updates the dashboard overview widget to support easy on-ramping
Figma: https://www.figma.com/file/ptTs6lDBeUuLNySroJ5PiF/Web-Master-File?node-id=8786%3A68251&mode=dev
How this PR fixes it
How to test it
Screenshots
Checklist