Skip to content

[ShadCN]: Migrate MergeInfographic#14853

Merged
pettinarip merged 3 commits into
ethereum:devfrom
TylerAPfledderer:feat/mergeinfographic-tailwind
Feb 10, 2025
Merged

[ShadCN]: Migrate MergeInfographic#14853
pettinarip merged 3 commits into
ethereum:devfrom
TylerAPfledderer:feat/mergeinfographic-tailwind

Conversation

@TylerAPfledderer
Copy link
Copy Markdown
Contributor

No description provided.

@netlify
Copy link
Copy Markdown

netlify Bot commented Feb 9, 2025

Deploy Preview for ethereumorg ready!

Name Link
🔨 Latest commit 831a0e5
🔍 Latest deploy log https://app.netlify.com/sites/ethereumorg/deploys/67a9c6c515ea96000884e914
😎 Deploy Preview https://deploy-preview-14853--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: 43 (🔴 down 11 from production)
Accessibility: 95 (🟢 up 3 from production)
Best Practices: 89 (🔴 down 9 from production)
SEO: 98 (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 site configuration.

@TylerAPfledderer TylerAPfledderer force-pushed the feat/mergeinfographic-tailwind branch 2 times, most recently from 7c38c36 to 7acb8e6 Compare February 9, 2025 15:01
@TylerAPfledderer TylerAPfledderer force-pushed the feat/mergeinfographic-tailwind branch from 7acb8e6 to 1d1f65b Compare February 9, 2025 15:12
Copy link
Copy Markdown
Member

@pettinarip pettinarip left a comment

Choose a reason for hiding this comment

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

Nice job

...rest
}: SVGTextElementAttributes<SVGTextElement>) => (
<text
className={cn("fill-current [text-anchor:start]", className)}
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.

didn't know that you could do that (arbitrary props) TIL

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@pettinarip ah yes! Applies to any case where you need to explicitly write a CSS property and value. Of course, you lose the ability to apply a token.

This includes creating CSS variables as well.

@pettinarip pettinarip merged commit 663f745 into ethereum:dev Feb 10, 2025
@TylerAPfledderer TylerAPfledderer deleted the feat/mergeinfographic-tailwind branch February 10, 2025 14:11
This was referenced Feb 12, 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.

2 participants