Skip to content

[ShadCN]: migrate ConnectWeb3 Simulator Screen#14861

Merged
pettinarip merged 12 commits into
ethereum:devfrom
TylerAPfledderer:feat/shadcn-connectweb3-simulator-screen
Feb 11, 2025
Merged

[ShadCN]: migrate ConnectWeb3 Simulator Screen#14861
pettinarip merged 12 commits into
ethereum:devfrom
TylerAPfledderer:feat/shadcn-connectweb3-simulator-screen

Conversation

@TylerAPfledderer
Copy link
Copy Markdown
Contributor

Description

Migrate ConnectWeb3 simulator screen components to ShadCN/Tailwind.

Creates a set of stories using the main screen component.

@netlify
Copy link
Copy Markdown

netlify Bot commented Feb 10, 2025

Deploy Preview for ethereumorg ready!

Name Link
🔨 Latest commit 10950d0
🔍 Latest deploy log https://app.netlify.com/sites/ethereumorg/deploys/67ab128dd326e200081fdef7
😎 Deploy Preview https://deploy-preview-14861--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: 47 (🔴 down 7 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.

Comment thread src/components/ui/buttons/Button.tsx Outdated
Comment thread src/components/ui/buttons/Button.tsx Outdated
outline: "", // Base styling
ghost: "border-transparent hover:shadow-none",
link: "border-transparent hover:shadow-none underline py-0 px-1 active:text-primary",
link: "border-transparent hover:shadow-none underline !min-h-0 !py-0 !px-1 active:text-primary",
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.

👍🏼 weird case where we are doing sizing things in the variant... xd

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 I do not like it! 🤣

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.

Cool!

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.

Great @TylerAPfledderer. I cleaned up some console logs.

@pettinarip pettinarip merged commit cf716db into ethereum:dev Feb 11, 2025
@TylerAPfledderer TylerAPfledderer deleted the feat/shadcn-connectweb3-simulator-screen branch February 11, 2025 20:18
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