-
Notifications
You must be signed in to change notification settings - Fork 1.7k
feat: add Storybook stories for UI components #3061
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
Add new Storybook stories for the following components: - Alert: default, destructive, success, warning variants - Avatar: with image, fallback, sizes, and group - Badge: default, secondary, destructive, outline variants - Card: compound component with header, content, footer - Checkbox: checked, disabled, with label, group - HighlightText: search query highlighting - Input: text, password, email, disabled, with label - Progress: various progress states (0-100%) - Skeleton: card, avatar, list, form, table loading states - Switch: checked, disabled, with label, group - TemplateCard: with/without image, new badge, starred - Textarea: default, disabled, with label - Tooltip: positions, with/without arrow, delay All stories follow the Storybook best practices with: - Default story as first export - Proper argTypes with descriptions - Appropriate layout parameters - fn() for callback props Co-Authored-By: unknown <>
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
Note: I can only respond to comments from users who have write access to this repository. ⚙️ Control Options:
|
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Important Review skippedBot user detected. To trigger a single review, invoke the You can disable this status message by setting the Comment |
|
This pull request has been ignored for the connected project Preview Branches by Supabase. |
Co-Authored-By: unknown <>
Co-Authored-By: unknown <>
Description
Adds Storybook stories for 13 UI components that were missing coverage. This PR is the result of a Storybook audit to identify components without stories and create comprehensive documentation for them.
New stories added:
Each story includes:
Defaultstory as the first exportargTypeswith descriptions for controllable propsfn()for callback props where applicableRelated Issues
N/A - Storybook audit task
Type of Change
Testing
bun run lint- passedbun run typecheck- no story-related errors (pre-existing errors in middleware.ts/server.ts unrelated to this PR)Human Review Checklist
@storybook/nextjs-viteimports work (existing stories use@storybook/react)Additional Notes
apps/web/client/src/stories/to match existing repo structureLink to Devin run: https://app.devin.ai/sessions/6a9f4f5e29374a93bfd940c729afd479
Requested by: unknown ()
Important
Add Storybook stories for 13 UI components, enhancing documentation and testing with default and variant stories, argTypes, and callback props.
Input,Textarea,Checkbox,SwitchAlert,Progress,Skeleton,TooltipBadge,Card,AvatarHighlightText,TemplateCardDefaultstory and multiple variant stories.argTypeswith descriptions for controllable props.fn()used for callback props where applicable.apps/web/client/src/stories/to match existing repo structure.This description was created by
for 410a3d2. You can customize this summary. It will automatically update as commits are pushed.