Skip to content
This repository has been archived by the owner on Sep 27, 2023. It is now read-only.

fix: style bridged token icons #223

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from
Draft

fix: style bridged token icons #223

wants to merge 1 commit into from

Conversation

paouvrard
Copy link
Member

Fix to get icons from trustwallet: aurora-is-near/rainbow-bridge-client#31
Add styling to bridged icons
image
image
#159
#117

@heycorwin
Copy link

@paouvrard is my understanding correct that these style changes are currently limited to just the bridge UI, and that these icons are not yet being exported/added to the bridged contract token metadata?

@paouvrard
Copy link
Member Author

@corwinharrell Correct. This vaguely implements what was discussed here but feel free to make changes

@heycorwin
Copy link

heycorwin commented Apr 28, 2021

Gotcha. How would you feel about holding off on this for a bit? I'm not 100% set on any of the solutions that we've explored in the icon discussion, and would like to continue that exploration now that we have tokens getting displayed in the wallet, just to ensure that whatever we end up with is flexible enough to accommodate various use-cases.

I think for now it's fine just to use the original icons as long as nothing in the contract metadata is being changed right now.

@paouvrard paouvrard marked this pull request as draft May 17, 2021 02:03
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants