Skip to content

feat: add Storybook stories for new UI components#76

Merged
Gitsack merged 2 commits intomainfrom
worktree-ham-97-storybook-stories
Apr 11, 2026
Merged

feat: add Storybook stories for new UI components#76
Gitsack merged 2 commits intomainfrom
worktree-ham-97-storybook-stories

Conversation

@Gitsack
Copy link
Copy Markdown
Owner

@Gitsack Gitsack commented Apr 4, 2026

Summary

  • Added Storybook stories for 5 components: ErrorBoundary, DeleteMediaDialog, ConfirmDialog, AddMediaDialog, and DownloadProgressCard
  • Covers multiple variants per component (error states, loading states, different media types, import vs download progress)
  • Follows existing story conventions (autodocs, actions, decorators)

Test plan

  • Run npm run storybook and verify all new stories render correctly
  • Check ErrorBoundary default/fullPage/customFallback/noError stories
  • Check DeleteMediaDialog remove/deleteFile modes across media types
  • Check ConfirmDialog default/loading states
  • Check AddMediaDialog movie/album/tvshow/author variants and loading/adding states
  • Check DownloadProgressCard single/multiple/importing/empty states

🤖 Generated with Claude Code

Add stories for ErrorBoundary, DeleteMediaDialog, ConfirmDialog,
AddMediaDialog, and DownloadProgressCard to improve visual regression
testing and enable isolated component review.

Co-Authored-By: Paperclip <noreply@paperclip.ing>
Copy link
Copy Markdown
Owner Author

@Gitsack Gitsack left a comment

Choose a reason for hiding this comment

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

Review: LGTM — Clean addition of Storybook stories for 5 components.

  • All component files exist and are properly imported
  • Stories follow existing conventions (autodocs, actions, decorators)
  • Good variant coverage: error states, loading states, media types, progress states
  • No test regressions (257/257 pass)
  • Storybook rendering requires manual verification (cannot run Storybook in CI headlessly here)

@Gitsack Gitsack merged commit d49ba78 into main Apr 11, 2026
@Gitsack Gitsack deleted the worktree-ham-97-storybook-stories branch April 11, 2026 10:36
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