Skip to content
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

Wallet page - buttons #445

Closed
babalicious-io opened this issue Dec 5, 2024 · 2 comments · Fixed by #498
Closed

Wallet page - buttons #445

babalicious-io opened this issue Dec 5, 2024 · 2 comments · Fixed by #498
Assignees
Labels
design revision revision to match design

Comments

@babalicious-io
Copy link
Collaborator

Wallet page - and Stamps Overview, Token Overview, Collections Overview

Screenshot 2024-11-24 at 11 57 56

Screenshot 2024-11-24 at 11 58 54

All button icons need to be checked, cause their size is inconsistent, both the button and icons
Some buttons use png others svg icons

There should be two sizes of buttons - mobileSm/Md and mobileLg/tablet/desktop
mobileSm/Md - 28px button // 14px icon
mobileLg/tablet/desktop - 36px button // 18px icon
All icons are bold version

Phosphor Icons

To be found here
https://www.figma.com/design/Os9UawIAetoFAqwkNKHN9j/Stampchain-web?node-id=706-2870&m=dev&t=dvAvnM9JSBN0moXZ-1
Have just updated the colors (for those components), so figma design is in sync with dev site

@babalicious-io
Copy link
Collaborator Author

There is a Button.tsx file with some button components and we have some button styling hiding in Styles.ts
Would be great to get all buttons in one place !

@itttm127 itttm127 linked a pull request Dec 14, 2024 that will close this issue
@reinamora137 reinamora137 added the design revision revision to match design label Dec 15, 2024
@babalicious-io
Copy link
Collaborator Author

Nicely done !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
design revision revision to match design
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants