Skip to content

feat: add icons for nano contract transactions [8] #455

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

Merged
merged 3 commits into from
May 28, 2024

Conversation

alexruzenhack
Copy link
Contributor

Motivation

This is the first PR of a serie to make the review easier.

Acceptance Criteria

  • Should add all the icons to be used on nano contract transactions screen context

Security Checklist

  • Make sure you do not include new dependencies in the project unless strictly necessary and do not include dev-dependencies as production ones. More dependencies increase the possibility of one of them being hijacked and affecting us.

@andreabadesso andreabadesso mentioned this pull request Apr 8, 2024
1 task
@alexruzenhack alexruzenhack changed the title feat: add icons for nano contract transactions feat: add icons for nano contract transactions [8] Apr 8, 2024
@alexruzenhack alexruzenhack force-pushed the feat/nano-contract-list-component branch from 243c126 to bf4e720 Compare April 12, 2024 14:57
@alexruzenhack alexruzenhack force-pushed the feat/nano-contract-list-component branch from bf4e720 to fea4259 Compare May 7, 2024 21:45
@alexruzenhack alexruzenhack force-pushed the feat/nc-add-icons branch 2 times, most recently from 37fa2b3 to 06e563d Compare May 13, 2024 18:23
@alexruzenhack alexruzenhack requested review from andreabadesso, tuliomir and r4mmer and removed request for tuliomir May 13, 2024 18:41
r4mmer
r4mmer previously approved these changes May 14, 2024
Copy link
Member

@r4mmer r4mmer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would these also be useful on the desktop wallet?

@alexruzenhack
Copy link
Contributor Author

Would these also be useful on the desktop wallet?

I believe it would be useful there, yes. Aninha is preparing a Design System, which means standardization for our components.

@alexruzenhack alexruzenhack force-pushed the feat/nano-contract-list-component branch from 090211f to 65b0c55 Compare May 23, 2024 16:20
@alexruzenhack alexruzenhack force-pushed the feat/nc-add-icons branch 2 times, most recently from 1e1e19b to 6c9c73e Compare May 23, 2024 16:30
@alexruzenhack alexruzenhack requested review from r4mmer and removed request for r4mmer May 23, 2024 16:35
@pedroferreira1 pedroferreira1 dismissed r4mmer’s stale review May 23, 2024 16:36

He is not the reviewer anymore

Base automatically changed from feat/nano-contract-list-component to master May 24, 2024 13:34
@alexruzenhack alexruzenhack merged commit 4326760 into master May 28, 2024
2 checks passed
@alexruzenhack alexruzenhack deleted the feat/nc-add-icons branch May 28, 2024 14:24
andreabadesso pushed a commit that referenced this pull request Jun 24, 2024
* feat(nc): add icons
* refactor: move icons to /Icons and add typed properties
@tuliomir tuliomir mentioned this pull request Oct 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

4 participants