Skip to content

feat: add tooltip to provisional flask icon in contest table#3397

Merged
KATO-Hiro merged 1 commit intostagingfrom
feature/provisional-flask-tooltip
Apr 11, 2026
Merged

feat: add tooltip to provisional flask icon in contest table#3397
KATO-Hiro merged 1 commit intostagingfrom
feature/provisional-flask-tooltip

Conversation

@river0525
Copy link
Copy Markdown
Collaborator

@river0525 river0525 commented Apr 11, 2026

フラスコにtipsを追加いたしました。
ご確認よろしくお願いいたします。

Summary by CodeRabbit

スタイル

  • 非公式グレード表示へのツールチップ追加
    • 投票に基づいて決定される非公式グレード表示にツールチップを追加し、ユーザーが機能についての詳細を確認しやすくしました。

Shows "非公式グレード(投票に基づく)" on hover via Flowbite Tooltip,
using a per-component ID to support multiple instances on the same page.
Updates aria-label to match the tooltip text.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Apr 11, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: 2ed26199-1604-4086-81d6-cbb328b80796

📥 Commits

Reviewing files that changed from the base of the PR and between f8b0bdc and 21cb28f.

📒 Files selected for processing (1)
  • src/features/votes/components/VotableGrade.svelte

📝 Walkthrough

Walkthrough

VotableGrade.svelteコンポーネントに対して、Tooltipコンポーネントをインポートし、暫定グレードを示すFlaskConicalアイコンに一意のIDを付与して、ツールチップで説明文を表示する機能を追加しました。

Changes

Cohort / File(s) Summary
UI強化
src/features/votes/components/VotableGrade.svelte
FlaskConicalアイコンに決定論的IDを付与し、アリアラベルを日本語で更新。Tooltipコンポーネントを追加して、アイコンホバー時に「非公式グレード(投票に基づく)」テキストを表示。

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

トイレットの先に光る
投票の声、集まりて
ツールチップ優し、真実を囁く 🔧✨
暫定から非公式へ、ユーザー喜ぶ

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed プルリクエストのタイトルは、コンテストテーブル内の仮チェックフラスコアイコンにツールチップを追加するという主要な変更を明確に要約している。
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feature/provisional-flask-tooltip

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown
Collaborator

@KATO-Hiro KATO-Hiro left a comment

Choose a reason for hiding this comment

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

ありがとうございます
LGTM

マージします

@KATO-Hiro KATO-Hiro merged commit 0abee29 into staging Apr 11, 2026
3 checks passed
@KATO-Hiro KATO-Hiro deleted the feature/provisional-flask-tooltip branch April 11, 2026 02:14
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