SelectBoundModalで読み込み中は設定ボタン以外は無効化

[TinyKitten]

Summary by CodeRabbit

バグ修正

• ローディング状態中、ルート情報、列車タイプ、ルート保存操作に関連するボタンが無効化されるようになりました。ユーザーインターフェースの応答性が向上し、データ読み込み中のユーザー操作がより適切に処理されるようになります。

[coderabbitai]

📝 Walkthrough

ウォークスルー

SelectBoundModal.tsx のUI制御にローディング状態の処理が追加されました。RouteInfo ボタンがローディング中に無効化され、TrainType と Save Route アクションもローディング状態を考慮するように拡張されました。

変更内容

コホート / ファイル	概要
ローディング状態の UI 制御 src/components/SelectBoundModal.tsx	ローディング中にボタンを無効化するロジックを追加。RouteInfo ボタン、TrainType、Save Route アクションがローディング状態を条件として考慮されるようになりました。

コード確認の推定工数

🎯 2 (Simple) | ⏱️ ~8 分

関連する可能性のあるPR

• SelectBoundModalボタン整理 #4636: SelectBoundModal.tsx 内のボタンとその動作を修正した同じファイルへの変更
• 経路検索改良 #4721: SelectBoundModal.tsx のローディング・無効化 UI ロジックを修正し、モーダルのローディング/エラーソースを更新
• SelectBoundModalでboundStationsがある場合はOUTBOUNDボタンを表示するように修正 #4985: SelectBoundModal.tsx のボタンレンダリング/動作を修正（ローディングベースの無効化状態の追加）

推奨ラベル

react

詩

🐰 ローディング状態で ボタンたちは
静かに眠り 待つがいい
ユーザーの手を 優しく止めて
完了を待つ その時まで ✨

🚥 Pre-merge checks | ✅ 3 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Merge Conflict Detection	⚠️ Warning	⚠️ Unable to check for merge conflicts: Failed to fetch base branch: From https://github.com/TrainLCD/MobileApp ! [rejected] dev -> dev (non-fast-forward) + f709c9c...3ec5d89 dev -> origin/dev (forced update)

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	プルリクエストのタイトルは、SelectBoundModalコンポーネントで読み込み中に設定ボタン以外を無効化するという主な変更内容を明確かつ簡潔に説明しており、変更内容と完全に関連している。
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

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch feature/disable-if-loading

⚔️ Resolve merge conflicts (beta)

• Auto-commit resolved conflicts to branch feature/disable-if-loading
• Create stacked PR with resolved conflicts
• Post resolved changes as copyable diffs in a comment

---

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

[coderabbitai]

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)

src/components/SelectBoundModal.tsx (1)

586-591: ⚠️ Potential issue | 🟡 Minor

読み込み中は「閉じる」も無効化が必要です。
PR目的「設定ボタン以外は無効化」に対して、閉じるボタンが有効のままです。

修正案

           <Button
             style={styles.closeButton}
             textStyle={styles.closeButtonText}
             onPress={onClose}
+            disabled={loading}
           >
             {translate('close')}
           </Button>