Skip to content

fix: update Cyfrin icon [Fixes #15874]#15884

Merged
wackerow merged 2 commits into
ethereum:devfrom
IlyaSmiyukha:bugfix/update_cyfrin_icon
Jul 22, 2025
Merged

fix: update Cyfrin icon [Fixes #15874]#15884
wackerow merged 2 commits into
ethereum:devfrom
IlyaSmiyukha:bugfix/update_cyfrin_icon

Conversation

@IlyaSmiyukha
Copy link
Copy Markdown

Updated Cyfrin Updraft icon on https://ethereum.org/en/developers/learning-tools/ page

@netlify
Copy link
Copy Markdown

netlify Bot commented Jul 20, 2025

Deploy Preview for ethereumorg ready!

Name Link
🔨 Latest commit 68a2f32
🔍 Latest deploy log https://app.netlify.com/projects/ethereumorg/deploys/687e2592cd266600083a6dff
😎 Deploy Preview https://deploy-preview-15884--ethereumorg.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
7 paths audited
Performance: 51 (🔴 down 1 from production)
Accessibility: 94 (no change from production)
Best Practices: 92 (no change from production)
SEO: 99 (no change from production)
PWA: 59 (no change from production)
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify project configuration.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@konopkja Any reason we're using an app-icon form of their logo? This is a card banner, I would recommend not wrapping in the rounded box.

Would just use this transparent version:

image

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

agree this is better, can you update please?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

done

Copy link
Copy Markdown
Member

@wackerow wackerow left a comment

Choose a reason for hiding this comment

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

Looks great! Thanks @IlyaSmiyukha!

@wackerow wackerow merged commit 324d64b into ethereum:dev Jul 22, 2025
6 checks passed
@gitpoap-bot
Copy link
Copy Markdown

gitpoap-bot Bot commented Jul 22, 2025

Congrats, your important contribution to this open-source project has earned you a GitPOAP!

GitPOAP: 2025 Ethereum.org Contributor:

GitPOAP: 2025 Ethereum.org Contributor GitPOAP Badge

Join the [ethereum.org Discord server](https://ethereum.org/discord) to explore more ways to contribute to the project. Depending on the tasks you complete, you may also unlock additional rewards. Visit [ethereum.org/contributing](https://ethereum.org/contributing) to learn more.

Head to gitpoap.io & connect your GitHub account to mint!Keep buidling, keep learning, and let's grow the Ethereum open-source community together 🌱

Learn more about GitPOAPs here.

@wackerow
Copy link
Copy Markdown
Member

@all-contributors please add @IlyaSmiyukha for maintenance

@allcontributors
Copy link
Copy Markdown
Contributor

@wackerow

I've put up a pull request to add @IlyaSmiyukha! 🎉

This was referenced Jul 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants