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

[Claim] Use blockies for wallet avatars instead of identicon #261

Closed
anxolin opened this issue Apr 2, 2022 · 1 comment
Closed

[Claim] Use blockies for wallet avatars instead of identicon #261

anxolin opened this issue Apr 2, 2022 · 1 comment
Labels
app:CowSwap CowSwap app Help_Wanted Low Severity indicator for defects. It won't cause any major break-down of the system

Comments

@anxolin
Copy link
Contributor

anxolin commented Apr 2, 2022

@alfetopito Tuning in here: I'd personally stick to the 'blockies' avatar for all wallets:

  • We can't detect the MetaMask setting the users has (e.g. you can enable/disable blockies in settings). So even with MM users there's a potential mismatch.
  • All other external block explorers/wallet managers use Blockies (Etherscan, Zerion, Gnosis Safe, Zapper, etc.).

Originally posted by @biocom in gnosis/cowswap#2161 (comment)

Original issue gnosis/cowswap#2197 by @alfetopito
@anxolin anxolin added Claim Help_Wanted Low Severity indicator for defects. It won't cause any major break-down of the system app:CowSwap CowSwap app labels Apr 2, 2022
@elena-zh
Copy link

elena-zh commented Apr 4, 2022

Closed as Claim page will be hidden based on the #349 task

@elena-zh elena-zh closed this as completed Apr 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
app:CowSwap CowSwap app Help_Wanted Low Severity indicator for defects. It won't cause any major break-down of the system
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@anxolin @elena-zh