Skip to content

Tabsコンポーネントを追加 - #105

Merged
ore0 merged 3 commits into
mainfrom
feat/tabs
Jul 11, 2025
Merged

Tabsコンポーネントを追加#105
ore0 merged 3 commits into
mainfrom
feat/tabs

Conversation

@ore0

@ore0 ore0 commented Jul 10, 2025

Copy link
Copy Markdown

概要

Tabsコンポーネントを新規作成
image

変更内容

  • Tabsの追加
  • Tabsのテストケースを追加
  • TabsのStoryファイルを追加
  • item.jsonを新規作成し、コンポーネントのメタデータを追加

動作確認

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

関連 Issue

[SPARKLE-640] Tabsコンポーネントを開発時に使用できる goodpatch/sparkle-design-issue-archive#30

Close goodpatch/sparkle-design-issue-archive#30

- タブコンポーネントを新規作成
- タブのバリアント(solid, line, ghost)を実装
- タブの基本的なレンダリングテストを追加
- タブのユーザーインタラクションに関するテストを追加
- タブのアクセシビリティに関するテストを追加
@ore0
ore0 requested a review from touyou July 10, 2025 23:38
@vercel

vercel Bot commented Jul 10, 2025

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
sparkle-design ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 11, 2025 7:33am

- 不要なインポートを削除
- Reactのインポートを整理
- コードの可読性を向上

@touyou touyou left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

基本問題なさそうですが、一点だけコメントしました!

Comment on lines +18 to +27
const tabsTriggerVariants = cva(
[
"inline-flex items-center justify-center gap-1.5 whitespace-nowrap transition-colors duration-150",
"px-3 py-2 character-3-regular-pro",
"focus-visible:outline-none",
],
{
variants: {
variant: {
solid: [

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

カーソルの変更は指定してもいいかもと思いました🙋

- TabsTriggerのカーソルスタイルをポインタに変更
- 無効状態のカーソルスタイルを追加

@touyou touyou left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTMです!

@ore0
ore0 merged commit b3266c7 into main Jul 11, 2025
@ore0
ore0 deleted the feat/tabs branch July 11, 2025 07:40
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.

2 participants