fix: Review Stacks のスクロール連動をホバーからクリックに変更 (#281)

[coji]

Summary

• ホバーで他カラムを自動スクロールしていた挙動を廃止し、PR クリック時のみスクロール
• ハイライト/ディミングは従来どおりホバーで動作
• クリックするたびに再スクロール（tick で effect 再発火、クリック元と同一カラムはスキップ）

Closes #281

Test plan

• /$orgSlug/workload で PR にホバー → ハイライトのみ、スクロールしない
• PR クリック → 反対カラムの該当行へスクロール
• 同じ PR を再クリック → 再びスクロール
• クリック元と同じカラムはスクロールしない

Summary by CodeRabbit

リリースノート

• 新機能
  • スタックチャートのインタラクション方式を改善 - ホバーベースからクリックドリブンの選択トラッキングへ移行
  • スクロール動作をより正確に制御し、ユーザーエクスペリエンスを向上
  • 関連するプルリクエストのハイライト機能を強化
  • PRコンポーネントにクリックハンドラーを追加

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 85c17d01-0e2b-4823-a434-af5556511358

📥 Commits

Reviewing files that changed from the base of the PR and between 4eb2a2f and 1dd0e2f.

📒 Files selected for processing (2)

• app/routes/$orgSlug/+components/pr-block.tsx
• app/routes/$orgSlug/workload/+components/team-stacks-chart.tsx

---

📝 Walkthrough

ウォークスルー

PRBlock コンポーネントに onClick ハンドラのサポートを追加し、team-stacks-chart.tsx ではホバー駆動のスクロール動作をクリック駆動に変更。選択状態を管理する新しいコンテキストを導入し、クリック回数を追跡する tick カウンターで効果の再実行を制御。

変更内容

コホート / ファイル(s)	概要
PRBlock 拡張 app/routes/$orgSlug/+components/pr-block.tsx	onClick ハンドラの任意オプションプロップを追加。マウスイベントの型付けされたコールバックを受け取り、ボタン要素に直接渡す。
スクロール挙動の再実装 app/routes/$orgSlug/workload/+components/team-stacks-chart.tsx	ホバー駆動スクロールをクリック駆動に切り替え。HoverSourceColumnContext を削除し、SelectedSourceColumnContext と SelectedContext を導入。tick インクリメント機構で効果の再実行を強制。ホバーはハイライトのみに限定し、クリック時のみスクロール実行。

シーケンス図

sequenceDiagram
    actor User
    participant PRBlock
    participant team-stacks-chart
    participant SelectedContext
    participant useScrollIntoColumn

    rect rgba(100, 150, 200, 0.5)
    Note over User,useScrollIntoColumn: 旧動作: ホバー時スクロール（問題あり）
    User->>PRBlock: onMouseEnter
    PRBlock->>team-stacks-chart: ホバー状態を通知
    team-stacks-chart->>useScrollIntoColumn: スクロール実行
    useScrollIntoColumn->>team-stacks-chart: スクロール完了
    end

    rect rgba(100, 200, 150, 0.5)
    Note over User,useScrollIntoColumn: 新動作: クリック時スクロール
    User->>PRBlock: onClick
    PRBlock->>SelectedContext: 選択状態を更新 (tick++)
    SelectedContext->>team-stacks-chart: 選択通知＋tick更新
    team-stacks-chart->>useScrollIntoColumn: tick依存でスクロール実行
    useScrollIntoColumn->>team-stacks-chart: スクロール完了
    end

    User->>PRBlock: onMouseEnter
    PRBlock->>team-stacks-chart: ホバー状態（ハイライトのみ）
    team-stacks-chart->>team-stacks-chart: 関連行をハイライト（スクロール無し）

Loading

推定レビュー工数

🎯 3 (中程度) | ⏱️ ~20 分

ウサギの詩

クリック時代の幕開けよ 🐰✨
ホバーの誤爆はもうさよなら
選択された PR は確かに選ばれし者
tick が刻む新しいリズム
スクロール、ハイライト、完璧な調和 🎯

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	PR のタイトルは、ホバー駆動のスクロール挙動からクリック駆動への変更という主要な変更点を明確に表現しており、変更セットの主な目的を正確に要約しています。
Linked Issues check	✅ Passed	PR の実装は Issue #281 の全ての要件を満たしています。ホバーでのスクロール廃止、クリックによるスクロール機能、ホバー時のハイライト機能維持が実装されています。
Out of Scope Changes check	✅ Passed	PR の変更は Issue #281 の要件にのみ集中しており、スコープ外の変更は見当たりません。PRBlock への onClick ハンドラ追加とスクロール制御ロジックの変更が適切に実装されています。

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

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch fix/review-stacks-click-to-scroll

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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