Skip to content

fix(logs): FlatListにflex:1を追加しフィルターのスクロール追従を修正#33

Closed
TinyKitten wants to merge 2 commits intomainfrom
claude/fix-timeline-flicker-k9rBq
Closed

fix(logs): FlatListにflex:1を追加しフィルターのスクロール追従を修正#33
TinyKitten wants to merge 2 commits intomainfrom
claude/fix-timeline-flicker-k9rBq

Conversation

@TinyKitten
Copy link
Copy Markdown
Member

@TinyKitten TinyKitten commented Mar 3, 2026

Summary

  • FlatList に flex: 1 を追加し、スティッキーヘッダー下の残りスペースに制約されるように修正
  • これにより ListHeaderComponent 内のフィルターアコーディオンと件数表示がリストと一緒にスクロールするようになった

Test plan

  • ログ画面でスクロールし、フィルターと件数がリストと一緒に流れることを確認
  • タイトルと検索ボックスが固定表示のままであることを確認
  • フィルター操作が正常に動作することを確認

https://claude.ai/code/session_01MPkqpw4PNwHcRH6Gb74HD9

Summary by CodeRabbit

リリースノート

  • 改善
    • ログ画面のフィルター UI を再構成しました。フィルターがスクロール可能なリスト内に統合され、より効率的なレイアウトになりました。検索機能と基本的な操作性は変わりません。

claude added 2 commits March 3, 2026 02:32
フィルターアコーディオンと件数表示をstickyHeaderからFlatListの
ListHeaderComponentに移動。タイトルと検索ボックスのみがスクロール
追従で固定表示され、フィルターと件数はリストと一緒にスクロールする。

https://claude.ai/code/session_01MPkqpw4PNwHcRH6Gb74HD9
FlatListにflex:1がないと残りスペースに制約されず、
ListHeaderComponent内のフィルターと件数表示が固定表示のままになっていた。

https://claude.ai/code/session_01MPkqpw4PNwHcRH6Gb74HD9
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Mar 3, 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 7e6ab6c and 8370e9f.

📒 Files selected for processing (1)
  • app/(tabs)/logs.tsx

📝 Walkthrough

ウォークスルー

ログビューのUI構造を再構成し、フィルター アコーディオンを独立したコンポーネントからFlatListのListHeaderComponentに変更。検索とタイトルは固定的に保持されながら、フィルターUIと件数情報はスクロール可能なリストヘッダー領域に統合されました。

変更内容

コホート / ファイル 概要
ログビュー再構成
app/(tabs)/logs.tsx
FlatListベースのレンダリングを導入し、フィルターアコーディオンとカウント情報をListHeaderComponentとして埋め込み。スタイルフック(list, listHeaderContent)を追加し、スクロール位置のWebフォールバック動作を維持しながら、ヘッダー領域の構造を変更。

推定コードレビュー所要時間

🎯 2 (Simple) | ⏱️ ~12 分

関連の可能性があるPR

ウサギからのお祝いの詩

🐰 ログの流れを整えて
FlatList の中へ
フィルターが踊る、
スクロールの波に乗って
見やすさが花開く! 🌸

🚥 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 PR のタイトルはフィルターのスクロール追従修正という実際の主要な変更を明確に説明しており、目的セクションの内容と一致しています。
Docstring Coverage ✅ Passed Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

✏️ 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 claude/fix-timeline-flicker-k9rBq

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

@TinyKitten
Copy link
Copy Markdown
Member Author

#34

@TinyKitten TinyKitten closed this Mar 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants