Skip to content

PadArchで直通先路線の色が隣接駅から継承されない不具合を修正#5345

Merged
TinyKitten merged 1 commit intodevfrom
fix/padarch
Feb 19, 2026
Merged

PadArchで直通先路線の色が隣接駅から継承されない不具合を修正#5345
TinyKitten merged 1 commit intodevfrom
fix/padarch

Conversation

@TinyKitten
Copy link
Member

@TinyKitten TinyKitten commented Feb 19, 2026

Summary by CodeRabbit

リリースノート

  • 改善
    • ステーション色の割り当てロジックを最適化し、隣接するステーション情報を活用することで、より一貫した色表示を実現しました。

@TinyKitten TinyKitten self-assigned this Feb 19, 2026
@github-actions github-actions bot added the react label Feb 19, 2026
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 19, 2026

No actionable comments were generated in the recent review. 🎉


📝 Walkthrough

Walkthrough

駅の色決定ロジックをリファクタリング。直接的な色解決から2段階のnull充填アプローチに変更し、隣接駅の色を伝播することでより滑らかな色の割り当てを実現。

Changes

Cohort / File(s) Summary
駅色の決定ロジック
src/components/PadArch.tsx
駅の色決定をnull値を含む中間配列から始める2段階処理に変更。trainTypeLines マッチ時に非マッチ駅はnullを返し、その後隣接駅の色をフォワード・バックワードで伝播して最終的な色を決定。

推定コードレビュー工数

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

関連する可能性のあるPR

Poem

🐰 隣の駅の色を借りて
すいすいと流れる虹色の路線
nullな駅も、たちまち輝く✨
前へ、後ろへ、色が踊る
より優しい色の物語

🚥 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 タイトルはPadArchコンポーネントで直通先路線の色が隣接駅から継承されない不具合を修正する内容を明確に要約しており、変更内容の主要ポイントと一致している。
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 docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/padarch

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

@TinyKitten TinyKitten merged commit 27f7cbd into dev Feb 19, 2026
7 checks passed
@TinyKitten TinyKitten deleted the fix/padarch branch February 19, 2026 12:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant