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

Feature request - Add more Bitcoin Icons #104

Open
danielnordh opened this issue Dec 13, 2024 · 3 comments
Open

Feature request - Add more Bitcoin Icons #104

danielnordh opened this issue Dec 13, 2024 · 3 comments

Comments

@danielnordh
Copy link
Contributor

When updating the Monday Wallet UI I was reaching for some of the icons in Bitcoin Icons but realised we have only added a handful.

@reez
Copy link
Owner

reez commented Dec 13, 2024

Here's my thinking about use of icons:

  • Use SF Symbols when there is an icon that represents what we want (because the upside of using a SF Symbol is massive compared to anything else)
  • Add a Bitcoin Icon when there is not

And I might even prefer a SF Symbol even if its slightly less representative.

Any thoughts/feedback on my framing/perspective?

@danielnordh
Copy link
Contributor Author

Completely agree.

We should only add icons where SF Symbols fail to produce a good option.
Some that I have trouble with at the moment: node, channels, seed, etc.

Not urgent, but created an issue to remember.

@reez
Copy link
Owner

reez commented Dec 13, 2024

Cool.

Yeah you mentioning those definitely reminds me I've had my same thoughts about node+channels icons, hadn't even thought about seed too that's a good one as well.

Thanks for adding the issue, was nice to discuss, and then also to remember too now.

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

No branches or pull requests

2 participants