Skip to content

ETA数字を拡大しPadArchに単位ラベルを追加、AndroidのTLC文字縦位置を補正#6442

Merged
TinyKitten merged 2 commits into
devfrom
feature/eta-display-and-tlc-adjustments
Jul 17, 2026
Merged

ETA数字を拡大しPadArchに単位ラベルを追加、AndroidのTLC文字縦位置を補正#6442
TinyKitten merged 2 commits into
devfrom
feature/eta-display-and-tlc-adjustments

Conversation

@TinyKitten

@TinyKitten TinyKitten commented Jul 17, 2026

Copy link
Copy Markdown
Member

概要

ETA(到着予想時分)数字を拡大して視認性を高め、PadArch には数字の意味を示す単位ラベル「(分)/(min)」を追加しました。あわせて Android で TLC(スリーレターコード)の文字がバッジ内で下寄りに見える問題を修正しています。

変更の種類

  • バグ修正
  • 新機能
  • リファクタリング
  • ドキュメント
  • CI/CD
  • その他

変更内容

  • LineBoardJO.tsx: ETA 数字のフォントを拡大(スマホ 22→24px、タブレット 30→38px)。ドット幅を超えるサイズになるため、明示 width の絶対配置コンテナでドット中心に重ねる構成に変更(width 未指定の絶対配置子は親=ドット幅上限で測られて数字が折り返されるため)
  • PadArch.tsx: ETA 数字を 28→42px に拡大し、最上段ドットの右横に単位ラベル「(分)/(min)」を追加。ラベルはバンド色の上に重なるため、readableColor で暗色バンドのときのみ文字色を白(既定は駅名と同じ #333)に切り替え
  • NumberingIconSquare.tsx: Android では TLC のグリフが行ボックス下寄りに描画されて下がって見えるため、レイアウトに影響しない transform で文字だけを上方向に補正(iOS は補正不要)

テスト

  • npm run lint が通ること
  • npm test が通ること
  • npm run typecheck が通ること

関連Issue

スクリーンショット(任意)

Summary by CodeRabbit

  • 改善
    • 未通過駅のETAバッジをドット中央に重ねて表示し、視認性を向上しました。
    • ETAの単位(分/min)を表示するようになりました。
    • 背景色に応じて単位ラベルの文字色を調整し、読みやすくしました。
    • Androidで駅番号テキストの上下位置を補正し、表示のずれを軽減しました。
    • ETA数値のサイズや配置を調整し、全体のバランスを改善しました。

@TinyKitten TinyKitten self-assigned this Jul 17, 2026
@github-actions github-actions Bot added the react label Jul 17, 2026
@coderabbitai

coderabbitai Bot commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

📝 Walkthrough

Walkthrough

ETAバッジと単位ラベルの配置・文字スタイル、単位ラベルの可読性判定、AndroidにおけるTLC文字の位置補正が更新された。

Changes

表示調整

Layer / File(s) Summary
ETA表示レイアウト
src/components/LineBoardJO.tsx, src/components/PadArch.tsx
ETA数値をドット上に重ねるためのコンテナ、サイズ、配置、文字スタイルが追加・更新された。
ETA単位ラベル描画
src/components/LineBoardJO.tsx, src/components/PadArch.tsx
ETA単位ラベルの色計算と条件付き描画が追加され、LineBoardJOのETAバッジが絶対配置コンテナ内で描画されるよう変更された。
TLC文字のプラットフォーム補正
src/components/NumberingIconSquare.tsx
AndroidではTLC文字を上方向へ補正し、iOSでは補正しないスタイル分岐が追加された。

Estimated code review effort: 3 (Moderate) | ~20 minutes

Possibly related PRs

Poem

ぴょんと跳ねればETAが見える
ドットの真ん中、数字が踊る
(分)も色よく寄り添って
Androidの文字も背筋を伸ばす
うさぎは満足、耳をふるわせる

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed 主要な変更であるETA拡大、PadArchの単位ラベル追加、AndroidのTLC補正を簡潔に要約できています。
Description check ✅ Passed テンプレートの主要セクションが揃っており、概要・変更内容・テストも具体的で十分に埋まっています。
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ 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/eta-display-and-tlc-adjustments

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

@TinyKitten

Copy link
Copy Markdown
Member Author

@coderabbitai review

@coderabbitai

coderabbitai Bot commented Jul 17, 2026

Copy link
Copy Markdown
Contributor
✅ Action performed

Review finished.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🧹 Nitpick comments (1)
src/components/NumberingIconSquare.tsx (1)

71-78: 📐 Maintainability & Code Quality | 🔵 Trivial | ⚡ Quick win

Android向け補正をプラットフォーム別テストで固定してください。

src/components/NumberingIconSquare.test.tsxTKY の描画確認のみで、Android の translateY 補正と iOS の未補正を検証していません。今回の変更目的に合わせて、Platform.OS を切り替えたスタイル assertion を追加し、将来のスタイル変更で補正が失われても検知できるようにしてください。

🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@src/components/NumberingIconSquare.tsx` around lines 71 - 78, Update the
platform-specific tests in NumberingIconSquare.test.tsx to switch Platform.OS
and assert that NumberingIconSquare applies the Android translateY correction
while leaving the iOS transform unmodified. Preserve the existing TKY rendering
coverage and verify both platform outcomes so future style changes are detected.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Nitpick comments:
In `@src/components/NumberingIconSquare.tsx`:
- Around line 71-78: Update the platform-specific tests in
NumberingIconSquare.test.tsx to switch Platform.OS and assert that
NumberingIconSquare applies the Android translateY correction while leaving the
iOS transform unmodified. Preserve the existing TKY rendering coverage and
verify both platform outcomes so future style changes are detected.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: bdba584c-9af2-4669-b932-77ce22595e1d

📥 Commits

Reviewing files that changed from the base of the PR and between d7f8944 and 97afb7c.

📒 Files selected for processing (3)
  • src/components/LineBoardJO.tsx
  • src/components/NumberingIconSquare.tsx
  • src/components/PadArch.tsx

@TinyKitten
TinyKitten merged commit b1ccbae into dev Jul 17, 2026
9 checks passed
@TinyKitten
TinyKitten deleted the feature/eta-display-and-tlc-adjustments branch July 17, 2026 06:38
@TinyKitten TinyKitten mentioned this pull request Jul 17, 2026
9 tasks
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