ETA数字を拡大しPadArchに単位ラベルを追加、AndroidのTLC文字縦位置を補正#6442
Conversation
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01DLC4eDQuUAfo1FRde7SHYJ
📝 WalkthroughWalkthroughETAバッジと単位ラベルの配置・文字スタイル、単位ラベルの可読性判定、AndroidにおけるTLC文字の位置補正が更新された。 Changes表示調整
Estimated code review effort: 3 (Moderate) | ~20 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Comment |
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01DLC4eDQuUAfo1FRde7SHYJ
|
@coderabbitai review |
✅ Action performedReview finished.
|
There was a problem hiding this comment.
🧹 Nitpick comments (1)
src/components/NumberingIconSquare.tsx (1)
71-78: 📐 Maintainability & Code Quality | 🔵 Trivial | ⚡ Quick winAndroid向け補正をプラットフォーム別テストで固定してください。
src/components/NumberingIconSquare.test.tsxはTKYの描画確認のみで、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
📒 Files selected for processing (3)
src/components/LineBoardJO.tsxsrc/components/NumberingIconSquare.tsxsrc/components/PadArch.tsx
概要
ETA(到着予想時分)数字を拡大して視認性を高め、PadArch には数字の意味を示す単位ラベル「(分)/(min)」を追加しました。あわせて Android で TLC(スリーレターコード)の文字がバッジ内で下寄りに見える問題を修正しています。
変更の種類
変更内容
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