Skip to content

Add Storybook coverage and fix mobile carousel hydration#148

Open
encryptedDegen wants to merge 2 commits into
mainfrom
storybook
Open

Add Storybook coverage and fix mobile carousel hydration#148
encryptedDegen wants to merge 2 commits into
mainfrom
storybook

Conversation

@encryptedDegen

Copy link
Copy Markdown
Collaborator

Summary

  • Add Storybook configuration for Next/Vite with docs and a11y addons.
  • Add a component inventory story covering all tracked app/shared component files plus a rendered UI overview story.
  • Fix the mobile home carousel hydration mismatch by deferring mobile layout offsets until the container is measured.

Verification

  • bun run typecheck
  • bun run lint (passes with one pre-existing .opencode warning)
  • bun run build-storybook

Created with Open-Inspect

@vercel

vercel Bot commented May 18, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
grails Ready Ready Preview, Comment May 18, 2026 1:09pm

Request Review

@open-inspect-constructor open-inspect-constructor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

I reviewed the Storybook setup, component inventory coverage, and the mobile carousel hydration change. I did not find any blocking PR-introduced issues.\n\nVerification performed: component inventory matches the intended tracked component files; lint passes; Storybook builds successfully. Typecheck currently fails in an untouched PremiumPriceControls StaticImageData typing issue, and I reproduced the same failure on main, so I am not treating it as a regression from this PR.

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.

1 participant