-
Notifications
You must be signed in to change notification settings - Fork 7
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
Update featured builds #23
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
For address.vision Andrea has prepared a new logo for Featured builds: have some doubts about whether the logo should match the real product "logo typography" or not. It might be a good idea for the product itself to choose one of the eyes and use a svg/png instead, so it is familiar for all users and doesn't look fishy when they see a different pair of eyes somewhere. |
Thanks @Pabl0cks !! I like the new logo, it's smoother (and consistent across OS!). I think we should use for the landing. And maybe change it on address.vision if @portdeveloper likes it. Another thing: should we just link to the tool instead of the intermediate BG build page (not very pretty... my fault haha)? the wallet recovery doesn't have the "Fork" link on the site, but maybe we could ask them to add it. Thoughts? |
Agree!
Yeah maybe right now is better to send to the Live project. If we can manage to get "Fork" to have repo access in wallet recovery would be great !! |
Ok! Let's do it:
Thanks Pablo! |
💯💯 !!
yup makes sense 🙌 |
Just pushed last changes, thanks for the feedback! 🙌♥ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tysm Pablo !!! Looking great !!!
Just pushed 3a7354e to remove todo but it was not related to this PR
We could update our featured builds to showoff more recent projects using Scaffold ETH-2. In this PR I'm proposing to have as Featured builds:
I've also added hover effect in all the links that were missing it (builds were one of them), so it's more intuitive for the user what is clickable and what not.
In future iterations we could also add more builds to this section, making it expandable to show more than the 3 featured builds.