Skip to content

Conversation

@danielrafailov1
Copy link
Contributor

This PR sets up Storybook for the project with a basic HelloWorld component.

Closes #1

@crypto-a crypto-a requested a review from Copilot September 12, 2025 19:29
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Sets up Storybook infrastructure for the React component library with a basic HelloWorld component and proper configuration. This establishes the foundation for component development and documentation.

  • Configures Storybook with React-Vite framework and essential addons
  • Creates a simple HelloWorld component with corresponding story
  • Adds basic CSS styling and package.json dependencies

Reviewed Changes

Copilot reviewed 6 out of 13 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
package.json Adds Storybook dependencies and scripts for development and build
.storybook/main.ts Configures Storybook with React-Vite framework and addons
.storybook/preview.tsx Sets up global Storybook parameters and CSS imports
src/components/HelloWorld.tsx Creates basic HelloWorld React component
src/components/HelloWorld.stories.tsx Adds Storybook story for HelloWorld component
src/index.css Provides basic global CSS styles
Files not reviewed (4)
  • .idea/.gitignore: Language not supported
  • .idea/library-react.iml: Language not supported
  • .idea/modules.xml: Language not supported
  • .idea/vcs.xml: Language not supported

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@@ -0,0 +1,11 @@
import type { Preview } from '@storybook/react';
import '../src/index.css'; // if using Tailwind
Copy link

Copilot AI Sep 12, 2025

Choose a reason for hiding this comment

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

The comment mentions Tailwind but the CSS file contains basic styles, not Tailwind. Update the comment to reflect the actual purpose: '// Import global styles' or remove the Tailwind reference.

Suggested change
import '../src/index.css'; // if using Tailwind
import '../src/index.css'; // Import global styles

Copilot uses AI. Check for mistakes.
Copy link
Contributor

@crypto-a crypto-a left a comment

Choose a reason for hiding this comment

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

Solid initial Storybook setup.

  • Good use of Vite + type-safe config
  • Autodocs and global styles are set up correctly
  • Story structure is clean

Next steps (not blocking):

  • Use clearer commit messages
  • Commit more regularly in smaller chunks

Approved

@crypto-a crypto-a merged commit d4bc40e into main Sep 12, 2025
@crypto-a crypto-a deleted the 1/implement-storybook branch September 12, 2025 19:42
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.

Implement Storybook for library-react Components

3 participants