Skip to content

Android用のRefreshControl colors propを削除#5447

Merged
TinyKitten merged 1 commit intodevfrom
fix/p2r-android
Feb 26, 2026
Merged

Android用のRefreshControl colors propを削除#5447
TinyKitten merged 1 commit intodevfrom
fix/p2r-android

Conversation

@TinyKitten
Copy link
Member

@TinyKitten TinyKitten commented Feb 26, 2026

Summary

  • SelectLineScreenのRefreshControlからAndroid用のcolors propを削除

Test plan

  • iOS: Pull to RefreshのLEDテーマでスピナーが白く表示されること
  • Android: Pull to Refreshが正常に動作すること

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Refactor
    • リフレッシュコントロールの色設定ロジックを簡素化しました。条件付き色配列の処理を削除し、色の管理をより効率化しました。

Co-Authored-By: Claude Opus 4.6 <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

📝 Walkthrough

Walkthrough

SelectLineScreenで、RefreshControlのcolorsプロパティを削除しました。従来の条件付きカラー配列ロジック(LED テーマ時に['#fff']を設定)から、tintColorrefreshTintColorで制御)のみによるビジュアルスタイリングへ変更しました。

Changes

Cohort / File(s) Summary
RefreshControl スタイリング
src/screens/SelectLineScreen.tsx
colorsプロパティを削除し、tintColorのみによるカラー制御に統一。条件付きカラー配列ロジックを廃止。

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

  • SelectLineScreenにPull to Refreshを実装 #5444: SelectLineScreen.tsxのRefreshControl使用方法に関連。当該PRはtintColorとLED テーマ対応を追加し、本PRはそれに続いてcolorsプロパティを削除してtintColorのみ使用する方針に統一している。

Suggested labels

react, android

Poem

🐰 色の配列よ さようなら
tintColor一本で輝く
LED テーマも優しく統一
RefreshControl も シンプルに

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed タイトルは「Android用のRefreshControl colors propを削除」で、実際の変更内容(SelectLineScreenのRefreshControlからcolors propを削除)と完全に一致しており、明確で具体的。
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.

✏️ 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/p2r-android

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

@TinyKitten TinyKitten merged commit 15125b1 into dev Feb 26, 2026
7 checks passed
@TinyKitten TinyKitten deleted the fix/p2r-android branch February 26, 2026 00:45
@TinyKitten TinyKitten self-assigned this Feb 26, 2026
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