Skip to content

小田急テーマのLineBoardにナンバリングアイコンを追加#5703

Merged
TinyKitten merged 2 commits intodevfrom
feature/odakyu-lineboard-numbering
Mar 27, 2026
Merged

小田急テーマのLineBoardにナンバリングアイコンを追加#5703
TinyKitten merged 2 commits intodevfrom
feature/odakyu-lineboard-numbering

Conversation

@TinyKitten
Copy link
Copy Markdown
Member

@TinyKitten TinyKitten commented Mar 27, 2026

Summary

  • 小田急テーマ使用時、LineBoardEastの駅名下にナンバリングアイコン(駅番号)を表示するよう実装
  • JR九州テーマ(LineBoardJRKyushu)と同等のNumberingIconViewコンポーネントを追加
  • ナンバリングアイコン表示時は駅名のmarginBottomを拡大してスペースを確保
  • 小田急テーマのプレビュー画像を更新

Closes #5702

Test plan

  • 小田急テーマでLineBoardにナンバリングアイコンが表示されること
  • ナンバリングがない駅ではアイコンが表示されないこと
  • タブレット/スマホ両方でレイアウトが崩れないこと
  • 他テーマ(東急等)に影響がないこと

🤖 Generated with Claude Code

Summary by CodeRabbit

リリースノート

  • 新機能

    • 小田急線ボードに駅番号アイコンを表示するようになりました(駅表示の視認性向上)。
  • スタイル

    • 駅名セルのレイアウトを調整し、番号表示がある場合の余白を最適化。
    • タブレット表示での番号配置と見た目を強化。

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@github-actions github-actions bot added the react label Mar 27, 2026
@TinyKitten TinyKitten self-assigned this Mar 27, 2026
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Mar 27, 2026

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: 124f66ce-63e4-4d9b-8183-e7a5e06915d7

📥 Commits

Reviewing files that changed from the base of the PR and between b8254ae and f929c85.

📒 Files selected for processing (1)
  • src/components/LineBoardEast.tsx
🚧 Files skipped from review as they are similar to previous changes (1)
  • src/components/LineBoardEast.tsx

📝 Walkthrough

Walkthrough

小田急テーマのLineBoardに駅のナンバリング表示を追加しました。NumberingIconViewを導入し、StationNameCellで条件付きにナンバリングを描画するようにレイアウト調整を行いました。

Changes

Cohort / File(s) Summary
Odakyu numbering UI
src/components/LineBoardEast.tsx
NumberingIconView を追加し、StationNameCell における hasDrawableNumberingnameCommonStyle を導入。小田急テーマ時に駅名下へナンバリングを条件付きで絶対位置で描画するよう変更。marginBottom を端末種別で調整。

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Poem

🐰 小田急の線路にぴょん、
駅名の下に数字が光るよ。
新しいアイコン、そっと乗せて、
ボードは今日もぴょんぴょん進む。

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed PR タイトルは小田急テーマの LineBoard にナンバリングアイコンを追加する主要な変更を明確に示しており、簡潔かつ具体的です。
Description check ✅ Passed PR 説明は概要、テスト計画、関連 Issue(#5702)への参照を含み、テンプレートの主要要件をおおむね満たしています。
Linked Issues check ✅ Passed PR は Issue #5702 の要件(小田急テーマで駅名下にナンバリングアイコンを表示、JR九州テーマと同様の実装)を完全に満たしています。
Out of Scope Changes check ✅ Passed すべてのコード変更は小田急テーマの LineBoard へのナンバリングアイコン追加という Issue #5702 の要件内に収まっており、スコープ外の変更はありません。
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
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feature/odakyu-lineboard-numbering

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

Copy link
Copy Markdown
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@src/components/LineBoardEast.tsx`:
- Around line 314-321: The current useMemo for nameCommonStyle replaces
styles.nameCommon when isOdakyu and station.stationNumbers exist, losing the
base styles; update it to merge the base style with the margin override (e.g.,
return { ...styles.nameCommon, marginBottom: isTablet ? 45 : 95 }) instead of
returning only marginBottom, and tighten the condition so it only applies when
there are drawable station numbers (e.g., verify station.stationNumbers.some(sn
=> sn && sn.icon) or another field that indicates an actual icon) rather than
just checking station.stationNumbers.length; adjust the same pattern at the
other occurrences (lines referenced by the reviewer) to preserve base styles
while applying the Odakyu-specific margin.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: ad24c289-9395-41e0-90f5-57dd423f9e94

📥 Commits

Reviewing files that changed from the base of the PR and between c535fba and b8254ae.

📒 Files selected for processing (3)
  • assets/images/themes/odakyu-sp.webp
  • assets/images/themes/odakyu-tablet.webp
  • src/components/LineBoardEast.tsx

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@TinyKitten TinyKitten merged commit 482bc46 into dev Mar 27, 2026
6 checks passed
@TinyKitten TinyKitten deleted the feature/odakyu-lineboard-numbering branch March 27, 2026 17:40
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.

小田急テーマ LineBoardにナンバリングがない

1 participant