feat(mobile): full-screen search with messages, channels, and people - #402
Merged
Conversation
Replace the old _BrowseChannelsSheet bottom sheet with a unified full-screen search experience. Search is accessible from both the bottom nav (new Search tab) and the channels page search bar. - Messages: global full-text search via /api/search with debounce - Channels: client-side filtering of joined + discoverable channels - People: user search via /api/users/search, tap to open DM - Filter chips: All / Messages / Channels / People - Deep-linking: forum posts (kind 45001) → ForumThreadPage, stream messages → ChannelDetailPage - Progressive rendering: 3 parallel lookups update state independently - Removed dead code: browseChannels(), _BrowseChannelsSheet, _BrowseTile, _MiniHeader (~220 lines) Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
… chips - Remove search pill from channels_page app bar (dedicated Search tab handles this now), replace with "Sprout" title + profile avatar - Remove onSearchTap callback from ChannelsPage (no longer needed) - Restyle filter chips: compact custom containers instead of bulky Material FilterChip with oversized checkmarks - Style search TextField with pill container matching the app's visual language (rounded, surfaceContainerHighest background) Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Add horizontal padding (Grid.xs) to search bar in AppBar - Even out filter chip row padding (symmetric xxs top/bottom) - Add 🌱 emoji to "Sprout" title on Home tab Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Tab order: Home → Activity → Search (was Home → Search → Activity) - Reduce top padding on filter chip row to visually balance the gap between search bar and chip row vs chip row and results Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Test was looking for old "Search" pill text, now expects "🌱 Sprout". Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Bare https:// URLs are now converted to tappable markdown links, matching the desktop behavior (remark-gfm autolinks). Existing markdown links, image syntax, and backtick-quoted code are preserved. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Collaborator
Author
|
6 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
/api/searchendpoint (Typesense); channels filter client-side; people search uses the user directory APITest plan
flutter analyze— no issuesflutter test— 256 passed, 1 skipped, 0 failed🤖 Generated with Claude Code