Skip to content

TrainTypeBox下の次の種別テキストが1行で収まるようにadjustsFontSizeToFitを追加#5712

Merged
TinyKitten merged 4 commits intodevfrom
claude/fix-train-type-wrapping-jjpYi
Mar 28, 2026
Merged

TrainTypeBox下の次の種別テキストが1行で収まるようにadjustsFontSizeToFitを追加#5712
TinyKitten merged 4 commits intodevfrom
claude/fix-train-type-wrapping-jjpYi

Conversation

@TinyKitten
Copy link
Copy Markdown
Member

@TinyKitten TinyKitten commented Mar 28, 2026

https://claude.ai/code/session_01AjvvoDzBxFXE7offduqWWm

Summary by CodeRabbit

バグ修正

  • 次の列車タイプ表示のレイアウトと描画領域を最適化しました。

@github-actions github-actions bot added the react label Mar 28, 2026
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Mar 28, 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: 35192454-9e6e-4780-a850-7383c53f9403

📥 Commits

Reviewing files that changed from the base of the PR and between 660a4dc and 880867f.

📒 Files selected for processing (1)
  • src/components/TrainTypeBox.tsx

📝 Walkthrough

Walkthrough

TrainTypeBox.tsx の "次の列車タイプ" レンダリング処理をリファクタリング。Typography要素を新しい絶対配置View(nextTrainTypeWrapper)でラップし、位置制御をテキスト要素からラッパーコンポーネントに移行。

Changes

Cohort / File(s) Summary
レイアウト構造の再編成
src/components/TrainTypeBox.tsx
Typography要素の位置制御を新たに導入したnextTrainTypeWrapperView要素に委譲。topmarginTopposition: 'absolute'widthなどのポジショニングプロパティをnextTrainTypeスタイルから削除し、ラッパーでalignItems: 'flex-start'overflow: 'visible'を設定。

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Poem

🐇 ビューを積み重ね、要素は踊る
位置の責務、今はラッパーの手に
レイアウト整いて、次の電車の表示
シンプルなる変化、美しき秩序
コードは流れて、更に清き

🚥 Pre-merge checks | ✅ 1 | ❌ 2

❌ Failed checks (2 warnings)

Check name Status Explanation Resolution
Title check ⚠️ Warning プルリクエストのタイトルは、TrainTypeBoxコンポーネント下のテキスト表示を1行に収まるように調整する変更内容と関連していますが、実際の変更内容(View ラッパーの追加と位置指定プロパティの再構成)とは異なり、adjustsFontSizeToFitの追加というアプローチを示唆しています。 タイトルを実際の実装内容に合わせて修正してください。例:「TrainTypeBoxの次の種別テキストを絶対位置指定で1行表示するようレイアウトを調整」
Description check ⚠️ Warning プルリクエストの説明がClaudeの外部セッションリンクのみであり、リポジトリのテンプレートで要求されている「概要」「変更の種類」「変更内容」「テスト」などの重要なセクションが完全に不足しています。 テンプレートに従い、変更の概要、カテゴリ(リファクタリング等)、詳細な変更内容、実施したテストを記載してください。
✅ Passed checks (1 passed)
Check name Status Explanation
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 claude/fix-train-type-wrapping-jjpYi

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

@TinyKitten TinyKitten merged commit 417b14f into dev Mar 28, 2026
7 checks passed
@TinyKitten TinyKitten deleted the claude/fix-train-type-wrapping-jjpYi branch March 28, 2026 11:23
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.

2 participants