Skip to content

LineBoardLEDの全Text要素にnumberOfLines制約を追加#5461

Merged
TinyKitten merged 1 commit intodevfrom
fix/led-marquee
Feb 26, 2026
Merged

LineBoardLEDの全Text要素にnumberOfLines制約を追加#5461
TinyKitten merged 1 commit intodevfrom
fix/led-marquee

Conversation

@TinyKitten
Copy link
Member

@TinyKitten TinyKitten commented Feb 26, 2026

Summary

  • LineBoardLEDの GreenText / OrangeText / CrimsonText コンポーネントおよび素の <Text> ラッパー全9箇所に numberOfLines={1} を追加
  • LineBoardLEDのMarqueeが2行に折り返される問題を修正 #5458flex: 1 を削除したが、<Text> 要素内部でのテキスト折り返しが依然として発生していたため、明示的に1行制約を追加
  • horizontal ScrollView 内では numberOfLines={1} はテキストを truncate せず、1行レイアウトを維持しつつ Marquee が正しくスクロールする

Test plan

  • LEDテーマで行き先が長い路線(例: 六本木・大門方面)を選択し、Marqueeが1行で横スクロールすることを確認
  • 短いテキストの場合にスクロールせず正常に表示されることを確認
  • ARRIVING / CURRENT / NEXT の各状態でテキストが1行表示になることを確認

🤖 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 Feb 26, 2026
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 26, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between db16518 and d720b4e.

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

📝 Walkthrough

Walkthrough

LineBoardLED.tsxにおいて、複数のTextコンポーネント(GreenText、OrangeText、CrimsonText)および駅名などのテキスト要素に対してnumberOfLines={1}プロパティを追加し、テキストの行折り返しを防止する変更。

Changes

Cohort / File(s) Summary
テキスト折り返し防止
src/components/LineBoardLED.tsx
色付きTextコンポーネントおよびArrivingContent、CurrentContent、NextStopContentの各テキスト要素にnumberOfLines={1}を設定し、単一行表示を強制。

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

Possibly related PRs

Suggested labels

react

Poem

🐰 折り返し防止の魔法かな
1行に収まる駅名たち
Textは今、わがままを言わず
画面の中で整列して
シンプルな美学、ウサギ推奨!

🚥 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 プルリクエストのタイトルは、LineBoardLED コンポーネント内のすべての Text 要素に numberOfLines 制約を追加するという主要な変更を正確に説明しています。
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 (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/led-marquee

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

@TinyKitten TinyKitten merged commit 61a3786 into dev Feb 26, 2026
7 checks passed
@TinyKitten TinyKitten deleted the fix/led-marquee branch February 26, 2026 22:36
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