Skip to content

chore(design-system): widget-primitive Storybook stories (PR 3 of #18191)#18201

Merged
pettinarip merged 1 commit into
devfrom
chore/storybook-feature-widgets
May 19, 2026
Merged

chore(design-system): widget-primitive Storybook stories (PR 3 of #18191)#18201
pettinarip merged 1 commit into
devfrom
chore/storybook-feature-widgets

Conversation

@myelinated-wackerow
Copy link
Copy Markdown
Collaborator

Summary

PR 3 of #18191. Adds Storybook stories for six small widget components used across two or more areas (plus ListingMethodology, single-consumer today but explicitly designed for reuse per the issue).

Components covered:

  • ChainImages (server) -- chain logo row with name tooltips
  • ListingMethodology (async server) -- directory-page criteria section. Storied here because the design intent is reuse across other directory pages even though only wallets/find-wallet consumes it today.
  • MobileButtonDropdown (server) -- migrated; thin positioning wrapper over ButtonDropdown. Issue flagged this as borderline-storyable since the visual is essentially the underlying dropdown plus mobile placement; I've included it with a short Default + a mobile-viewport story for completeness, but a reviewer is welcome to drop it if it doesn't earn its keep.
  • Morpher (client) -- text-morph animation, exercises prefers-reduced-motion
  • SocialListItem (server) -- migrated; community-page social directory row
  • SupportedLanguagesTooltip (server) -- the + N chip with overflow tooltip

Each story uses the Components / Widgets / <Name> title pattern and opts out of Chromatic snapshots at the meta level per the #18121 convention.

Related to #18191. Does not close the issue.

Test plan

  • pnpm storybook -- the six new entries appear under Components / Widgets / * and render without console errors
  • a11y addon panel is clean on each story
  • Morpher cycles through words (animation skipped under Chromatic via MotionGlobalConfig.skipAnimations)
  • ListingMethodology renders end-to-end despite being an async server component (relies on experimentalRSC: true already set in .storybook/main.ts)
  • Consumer pages (wallets/find-wallet, layer-2 directories, community page) still resolve their @/components/Foo imports after the migrations

Claude (Opus 4.7) co-authored these changes.

PR 3 of #18191. Adds Storybook stories for six small widget components used across two or more areas (plus ListingMethodology, single-consumer today but explicitly designed for reuse).

Components covered: ChainImages, ListingMethodology (async server), MobileButtonDropdown (thin positioning wrapper over ButtonDropdown), Morpher (client text-morph animation), SocialListItem, and SupportedLanguagesTooltip. MobileButtonDropdown and SocialListItem migrated from loose .tsx into Foo/index.tsx so the story files can live alongside.

Each story uses the Components / Widgets / <Name> title pattern and opts out of Chromatic snapshots at the meta level per the #18121 convention.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Co-Authored-By: wackerow <54227730+wackerow@users.noreply.github.com>
@netlify
Copy link
Copy Markdown

netlify Bot commented May 15, 2026

Deploy Preview for ethereumorg ready!

Name Link
🔨 Latest commit 0cdf29d
🔍 Latest deploy log https://app.netlify.com/projects/ethereumorg/deploys/6a0714a4c27fc7000899cde9
😎 Deploy Preview https://deploy-preview-18201.ethereum.it
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
7 paths audited
Performance: 68 (🟢 up 3 from production)
Accessibility: 96 (no change from production)
Best Practices: 100 (no change from production)
SEO: 98 (🔴 down 1 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.

@pettinarip pettinarip merged commit 7b2296c into dev May 19, 2026
16 checks passed
@pettinarip pettinarip deleted the chore/storybook-feature-widgets branch May 19, 2026 15:13
@claude claude Bot mentioned this pull request May 22, 2026
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