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.

Sign up for GitHub

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

Jump to bottom

Quo component Tags/ Summary Tag - design review #18410

Closed
J-Son89 opened this issue Jan 5, 2024 · 0 comments · Fixed by #18474
Closed

Quo component Tags/ Summary Tag - design review #18410

J-Son89 opened this issue Jan 5, 2024 · 0 comments · Fixed by #18474
Assignees
@ulisesmac
Labels
E:Mobile Wallet Release 1.27 all tasks for 1.27 related to wallet feature feature requests

Comments

@J-Son89
Copy link
Member

J-Son89 commented Jan 5, 2024

The Quo component Tags/ Summary Tag needs some design review
See:
https://www.figma.com/file/Tf5nfkYvpbnNCo4rKLK7lS/Feedback-for-Mobile?type=design&node-id=3415%3A214763&mode=design&t=ANYgMF9xQ4Gp2lOW-1

Adjust the component to the design comments.

Please check the use of this component on any screen using it, i.e preview page and transaction confirmation page.
@J-Son89 J-Son89 added the feature feature requests label Jan 5, 2024
@J-Son89 J-Son89 changed the title Wallet: Calculate Max fees on transaction confirmation page #18408 Quo component Tags/ Summary Tag - design review Jan 5, 2024
@J-Son89 J-Son89 added the E:Mobile Wallet Release 1.27 all tasks for 1.27 related to wallet label Jan 5, 2024
@J-Son89 J-Son89 mentioned this issue Jan 5, 2024
Merged
@ulisesmac ulisesmac self-assigned this Jan 11, 2024
@ulisesmac ulisesmac mentioned this issue Jan 12, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ulisesmac ulisesmac

Labels
E:Mobile Wallet Release 1.27 all tasks for 1.27 related to wallet feature feature requests
Projects
Do not use - tasks moved to unified Desktop / Mobile Board
Archived in project
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[#18410] - Fix summary-tag style status-im/status-mobile
2 participants
@J-Son89 @ulisesmac