add transparent background option to <Avatar type="account" />
#617
Conversation
🦋 Changeset detectedLatest commit: 56646cc The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
@jkbktl is attempting to deploy a commit to the Status Team on Vercel. A member of the Team first needs to authorize it. |
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
|
Can you point me to Figma where this is reflected? |
https://www.figma.com/design/WQZcp6S0EnzxdTL4taoKDv/Design-System-for-Mobile?node-id=483-19407&m=dev |
|
This is "Design System for Mobile". 🤔 |
Omg, didn't notice that, it's linked from Status Portfolio Figma though. https://www.figma.com/design/8IuYR751lsXg1F8bu8Cj8B/Status-Portfolio?node-id=1539-302342&m=dev |
|
Looking at figma, we can see that the design team are using a component from mobile's library. And then looking at the use case in the design file, we can see a very different implementation. Will the design implement this variant of the component in our library? The selected state is different from ours. Line should be dashed. |
We expect that design will put this in order so our design system (for web) is in use. It doesn't necessarily have to be dashed, it might be solid, it depends on type of wallet (watched vs connected) as visible here in Figma, should be also visible in portfolio PR on different addresses. This PR will be closed I guess. |
prichodko
changed the title
<AccountAvatar /> adds option to have transparent background<Avatar type="account" />
No description provided.