fix: tweak StakeTransactions component for mobile#502
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
There was a problem hiding this comment.
Pull request overview
This PR improves the mobile responsiveness of the StakeTransactions component by implementing a mobile-first layout approach. The changes address issue #390 by reorganizing the flex layout to stack vertically on mobile devices and display horizontally on larger screens.
Changes:
- Converted layout from desktop-first to mobile-first responsive design using column layout with breakpoint overrides
- Added gap-based spacing for mobile and margin-based spacing for desktop (@bp2 breakpoint)
- Restructured button and amount display sections with additional Flex containers for better alignment control on mobile
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Jipperism
left a comment
There was a problem hiding this comment.
Overall, I think this could do with a bit more polish. Maybe align the amount to the left instead of right on mobile, or make it bold. I'd also underline or bolden the address on those tiles, so it's clear that it's an link, which it isn't on mobile atm. It does work, so if these designs are okay, feel free to merge.
Addressed in beecb0e |
Closes #390
Before
After