Skip to content

Tooltipコンポーネントを追加 - #110

Merged
ore0 merged 7 commits into
mainfrom
feat/tooltip
Jul 18, 2025
Merged

Tooltipコンポーネントを追加#110
ore0 merged 7 commits into
mainfrom
feat/tooltip

Conversation

@ore0

@ore0 ore0 commented Jul 17, 2025

Copy link
Copy Markdown

概要

Tooltipコンポーネントを追加しました

image

変更内容

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

動作確認

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

関連 Issue

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

ore0 added 3 commits July 17, 2025 19:09
- ツールチップコンポーネントの実装
- ツールチップのトリガーとコンテンツのテストを追加
- ツールチップの位置指定機能を追加
- ツールチップのJSONメタデータを追加
- Tooltipコンポーネントの概要を明確に記述
- 使用例を追加して利用方法を示す
- Tooltipコンポーネントの説明文を簡略化しました。
@ore0
ore0 requested a review from touyou July 17, 2025 10:19
@vercel

vercel Bot commented Jul 17, 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 18, 2025 6:55am

@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です!

side: {
control: { type: "select" },
options: ["top", "right", "bottom", "left"],
description: "TooltipContentの表示位置",

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.

nits: ここ元のものが上書きされちゃうのと二重管理になるのでdescriptionだけ削除いただくがいいかもです!

- TooltipContentの表示位置の説明を削除
- サイドバー関連のファイルを削除しました。
- TooltipContentの型定義を修正
- ストーリーでのTooltipコンポーネントの使用を更新
- 不要なargTypesの削除
- Tooltipコンポーネントの概要を追加
- 使用例を追加
- TooltipContentの説明を簡潔に更新

@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 991490b into main Jul 18, 2025
@ore0
ore0 deleted the feat/tooltip branch July 18, 2025 07:06
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