fix: improve portfolio value display on mobile - #549
Conversation
|
@billcode124 is attempting to deploy a commit to the David Dada's projects Team on Vercel. A member of the Team first needs to authorize it. |
|
@billcode124 Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits. You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀 |
sshdopey
left a comment
There was a problem hiding this comment.
Thanks for tackling this! The responsive stacking for the portfolio value on mobile looks like a solid approach, and the tests and CSS updates show real effort. I appreciate that you kept the desktop layout intact. Nice work! ❤️
|
Approved! 🎉 I couldn't auto-merge this just yet (it may need a rebase or have a check still running). A maintainer can merge it whenever it's ready. |
Summary
Fixes the portfolio value display on mobile screens so the full amount and currency can be viewed without horizontal scrolling.
Changes
Testing
Validation
npm test npm run lint npm run typecheckScope
The change is limited to the portfolio value responsive display and does not alter unrelated portfolio functionality.
closes #453