Skip to content
This repository has been archived by the owner on Apr 26, 2024. It is now read-only.

Move from ethereum-blockies-base64 to blo #229

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

bpierre
Copy link

@bpierre bpierre commented Sep 26, 2023

Move from ethereum-blockies-base64 to blo, for a faster (~4.7x) and more precise rendering of the blockies identicons, while also being smaller (2.75kb => 0.67 KB).

Disclaimer: I am the author of blo.

I couldn’t find how to generate the complete Storybook as mentionned in the README (https://components.safe.global/), the one in the repo only has 3 components so I took screenshots of the EthHashInfo component.

Before:
image

After:
image

Before (browser zoom):
image

After (browser zoom):
image

@github-actions
Copy link

github-actions bot commented Sep 26, 2023

CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

@bpierre
Copy link
Author

bpierre commented Sep 26, 2023

I have read the CLA Document and I hereby sign the CLA

Copy link
Member

@katspaugh katspaugh left a comment

Choose a reason for hiding this comment

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

🙏

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