Skip to content

初期画面のPull to Refresh対象を路線リストに限定した変更をリバートして元のスクロール挙動に戻す#6416

Merged
TinyKitten merged 1 commit into
devfrom
claude/pull-refresh-home-scope-s3bvp5
Jul 13, 2026
Merged

初期画面のPull to Refresh対象を路線リストに限定した変更をリバートして元のスクロール挙動に戻す#6416
TinyKitten merged 1 commit into
devfrom
claude/pull-refresh-home-scope-s3bvp5

Conversation

@TinyKitten

@TinyKitten TinyKitten commented Jul 13, 2026

Copy link
Copy Markdown
Member

概要

初期画面の Pull to Refresh 対象を路線リストに限定した変更(#6412)を差し戻し、SelectLineScreen を変更前の状態へ復元します。

差戻しの理由: 元々の画面設計が Pull to Refresh を前提としておらず、「一部分(路線リスト)だけを PTR 対象にする」にはデザイン(構造)の変更が必要になる可能性が高いと判断したため。一旦挙動を元に戻し、設計方針は別途検討します。

変更の種類

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

変更内容

テスト

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

関連Issue

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


Generated by Claude Code

Summary by CodeRabbit

  • 新機能

    • 近隣駅の読み込み中に、専用のローディング表示を追加しました。
    • プリセット表示をリスト内に統合し、更新操作の対象に含めました。
  • 改善

    • リスト下部の余白とスクロール表示を調整しました。
    • 更新中のインジケーター表示を改善しました。

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

coderabbitai Bot commented Jul 13, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

📝 Walkthrough

Walkthrough

SelectLineScreenのプリセットをScrollView内へ移動し、リスト余白とローディング表示を更新しました。関連するローディング状態テストの期待値も変更されています。

Changes

SelectLineScreen表示フロー

Layer / File(s) Summary
リスト余白とローディング表示
src/screens/SelectLineScreen.tsx
リスト用スタイルを追加し、NearbyStationLoaderを導入しました。下端余白はスタイルのpaddingBottomとfooterHeightから計算されます。
ScrollViewへのプリセット統合
src/screens/SelectLineScreen.tsx, src/screens/SelectLineScreen.render.test.tsx
プリセットをAnimated.ScrollViewのコンテンツ内へ移動し、contentContainerStyleとRefreshControlを再構成しました。ローディング中はプリセットが非表示になる期待値へテストを更新しました。

Estimated code review effort: 2 (Simple) | ~10 minutes

Poem

ぴょんと跳ねるうさぎが見る
スクロールの中へ preset が行く
余白を測って、 loader も待つ
更新の流れはすっきり進む
にんじん色のテストも笑う

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed 変更内容の差し戻しとスクロール挙動の復元を簡潔に要約しており、PRの主旨に合っています。
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 claude/pull-refresh-home-scope-s3bvp5

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

@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.

Actionable comments posted: 1

🤖 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.

Inline comments:
In `@src/screens/SelectLineScreen.tsx`:
- Around line 46-51: Remove paddingHorizontal from the listContainerStyle
definition in SelectLineScreen, keeping the root SafeAreaView horizontal padding
and existing list bottom padding unchanged.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 0d1f5c7c-a4cb-4c5a-9ee0-0b069a48819f

📥 Commits

Reviewing files that changed from the base of the PR and between 392ba14 and 49d16e8.

📒 Files selected for processing (2)
  • src/screens/SelectLineScreen.render.test.tsx
  • src/screens/SelectLineScreen.tsx

Comment thread src/screens/SelectLineScreen.tsx
@TinyKitten TinyKitten merged commit 86f1490 into dev Jul 13, 2026
7 checks passed
@TinyKitten TinyKitten deleted the claude/pull-refresh-home-scope-s3bvp5 branch July 13, 2026 03:46
@TinyKitten TinyKitten mentioned this pull request Jul 13, 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.

2 participants