Skip to content

feat: segmented pill tabs on search screen#554

Merged
barrydeen merged 1 commit into
barrydeen:mainfrom
dmnyc:feat/search-segmented-tabs
May 20, 2026
Merged

feat: segmented pill tabs on search screen#554
barrydeen merged 1 commit into
barrydeen:mainfrom
dmnyc:feat/search-segmented-tabs

Conversation

@dmnyc
Copy link
Copy Markdown
Contributor

@dmnyc dmnyc commented May 19, 2026

Summary

  • Replaces the People ▾ / Notes ▾ dropdown button in the search TopAppBar with a full-width segmented pill control matching the iOS design
  • Active tab gets an orange (colorScheme.primary) filled rounded pill with white text/icon; inactive tab is transparent
  • People tab uses AccountCircle icon, Notes tab uses Forum icon
  • Search field moves below the tab row with a leading search icon; the Tune filter toggle sits outside the field to the right
  • Removes the now-unused TopAppBar, DropdownMenu, ArrowDropDown, and defaultMinSize usages; adds Surface, background, statusBarsPadding, FontWeight, and ImageVector imports

Test plan

  • Built and installed on device (Samsung SM-A546U, Android 13)
  • Tap People tab → orange pill on People, search returns user results
  • Tap Notes tab → orange pill on Notes, relay/author filter rows appear on Tune toggle
  • Tune icon tint switches to orange when expanded, gray when collapsed
  • Search field focus, clear button, and keyboard search action all work

Redesigns the search screen header to match iOS — a two-tab pill
control (People | Notes) replaces the old ArrowDropDown button, and
the search field moves below the tabs in its own row alongside the
Tune filter toggle.
@dmnyc dmnyc force-pushed the feat/search-segmented-tabs branch from 330b02a to 4934338 Compare May 19, 2026 02:07
@barrydeen barrydeen merged commit d0ff54d into barrydeen:main May 20, 2026
@dmnyc dmnyc deleted the feat/search-segmented-tabs branch May 20, 2026 15:58
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