Skip to content

[デザインチェック後マージ] Storybookのセクション分けを調整 - #112

Merged
touyou merged 4 commits into
mainfrom
refact/storybook-section
Sep 26, 2025
Merged

[デザインチェック後マージ] Storybookのセクション分けを調整#112
touyou merged 4 commits into
mainfrom
refact/storybook-section

Conversation

@touyou

@touyou touyou commented Jul 25, 2025

Copy link
Copy Markdown
Member
  • Badge, Breadcrumb, Button, Card, Checkbox, Dialog, Divider, IconButton, Icon, InlineMessage, InputPassword, Input, Link, Modal, Radio, Select, Skeleton, Slider, Spinner, Switch, Tabs, Tag, Textarea, Tooltipのタイトルを更新し、適切なカテゴリに分類しました。
  • MCPドキュメントのタイトルと説明を更新しました。

概要

StorybookとFigmaの構造を合わせることでメンテナンス性と一覧のしやすさを向上する試みです。

変更内容

  • Figmaと同じセクション分けにする
image

動作確認

  • pnpm lint を実行
  • pnpm build を実行してプロジェクトが正常にビルドされることを確認
  • pnpm test を実行してテストがすべて成功することを確認

関連 Issue

- Badge, Breadcrumb, Button, Card, Checkbox, Dialog, Divider, IconButton, Icon, InlineMessage, InputPassword, Input, Link, Modal, Radio, Select, Skeleton, Slider, Spinner, Switch, Tabs, Tag, Textarea, Tooltipのタイトルを更新し、適切なカテゴリに分類しました。
- MCPドキュメントのタイトルと説明を更新しました。
@touyou
touyou requested a review from ore0 July 25, 2025 05:09
@touyou touyou self-assigned this Jul 25, 2025
Copilot AI review requested due to automatic review settings July 25, 2025 05:09
@vercel

vercel Bot commented Jul 25, 2025

Copy link
Copy Markdown

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

Project Deployment Preview Comments Updated (UTC)
sparkle-design Ready Ready Preview Comment Sep 26, 2025 8:43am

Copilot AI 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.

Pull Request Overview

This PR reorganizes Storybook component categories to align with Figma's structure for better maintainability and discoverability. The changes move components from a flat "Components/" structure to semantic categories like "Form/", "Data Display/", "Navigation/", etc.

  • Restructures 24 UI components into semantic categories (Form, Data Display, Navigation, Overlay, Feedback, Disclosure, Media)
  • Updates MCP documentation title and adds bilingual description
  • Standardizes component organization between Storybook and Figma design system

Reviewed Changes

Copilot reviewed 25 out of 25 changed files in this pull request and generated no comments.

File Description
src/docs/mcp.mdx Updates documentation title and adds English translation
src/components/ui/*/index.stories.tsx Reorganizes 24 components into semantic categories instead of flat "Components/" structure
Comments suppressed due to low confidence (3)

src/components/ui/input-password/index.stories.tsx:7

  • [nitpick] The title "Form/Input Password" uses inconsistent capitalization compared to other components. Consider using "Form/InputPassword" to match the component name pattern used elsewhere.
  title: "Form/Input Password",

src/components/ui/inline-message/index.stories.tsx:12

  • [nitpick] The title "Feedback/Inline Message" uses inconsistent capitalization compared to other components. Consider using "Feedback/InlineMessage" to match the component name pattern used elsewhere.
  title: "Feedback/Inline Message",

src/components/ui/icon-button/index.stories.tsx:7

  • [nitpick] The title "Form/Icon Button" uses inconsistent capitalization compared to other components. Consider using "Form/IconButton" to match the component name pattern used elsewhere.
  title: "Form/Icon Button",

@ore0

ore0 commented Aug 26, 2025

Copy link
Copy Markdown

Figmaのコンポーネントの分類が適切ではないと考えているので、このPRも承認していいもいいものかと悩んでいました。
とはいえ一旦Figmaと揃えてFigmaを修正するタイミングでこちらも修正するという方針もアリかなと考えています。

また、shadcn MCPの正式名称はshadcn/ui Registry MCPかもしれないです。
実態はSparkle Designのコンポーネント配信基盤なので、Sparkle Design Registry MCPとしてもいいかもしれないです。

@touyou

touyou commented Aug 27, 2025

Copy link
Copy Markdown
Member Author

@ore0
分類に関してはこれがいいというよりはFigmaと一致させておくという点が参照のしやすさという観点でいいかなという意図でした🙆‍♂️なので

一旦Figmaと揃えてFigmaを修正するタイミングでこちらも修正するという方針

になるかと思います

shadcnのMCP名称に関しては本当にこことTwitterしか情報がなく正解がわからなかったので画像を素直に読んだ場合といった抜き出し方をしていたというところでした。

image

とはいえSparkleのMCPという扱いになるという点はアグリーなのでshadcn側の正式名称はぼかしてSparkle Design Registry MCPにしようかと思います!

- ボタンのテーマを「secondary」から「neutral」に変更
@touyou

touyou commented Aug 27, 2025

Copy link
Copy Markdown
Member Author

@ore0 上記コミットでMCPドキュメントのみ調整してます!

@ore0 ore0 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.

LGTM!!

@touyou

touyou commented Aug 27, 2025

Copy link
Copy Markdown
Member Author

ありがとうございます!多分URLが変わっちゃうので一旦今だしている修正完了確認終わってからマージします🙆‍♂️

@touyou touyou changed the title Storybookのセクション分けを調整 [デザインチェック後マージ] Storybookのセクション分けを調整 Sep 19, 2025
@touyou
touyou merged commit eeaa91e into main Sep 26, 2025
3 checks passed
@touyou
touyou deleted the refact/storybook-section branch September 26, 2025 08:45
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.

3 participants