Skip to content

fix(mobile): one M3 search bar for every Android search field - #710

Merged
kipavy merged 2 commits into
devfrom
fix/mobile-search-radius
Oct 24, 2026
Merged

kipavy merged 2 commits into
devfrom
fix/mobile-search-radius

Conversation

@kipavy

@kipavy kipavy commented Oct 24, 2026

Copy link
Copy Markdown
Contributor

Android search fields had inconsistent shapes (visible on the Play Store screenshots): only Hosts used the 48 dp pill, other lists used a 12 px bordered box, and each bottom sheet had its own style.

  • MobileFilterBar is always the M3 search bar (48 dp, fully rounded, tonal fill, no outline); the pill prop is removed.
  • New props autoFocus, trailing, inset so sheets can use it: HostPickerSheet (SFTP / new session) and MobileSnippetTargetSheet now use it.
  • MobileProcessesScreen (plugin, cannot import app components) copies the same style and keeps its count.
  • Data hooks (data-…-search) unchanged.

Before/after emulator screenshots (Pixel 6, x86_64 debug build): https://claude.ai/artifact/HQXeUbvxcH1cKdcMqXE6U3. Known Hosts (empty list) and Processes (needs a live session) not captured.

tsc --noEmit passes.

🤖 Generated with Claude Code

https://claude.ai/code/session_015Ly4RP96nLUnj4FWtWthpE

kipavy and others added 2 commits October 24, 2026 10:40
Hosts already used the 48 dp pill; Snippets, Keychain, Known Hosts, Logs
and Port Forwarding used a 12 px bordered box, and the host picker,
snippet target sheet and Processes screen each had their own style.
MobileFilterBar is now always the pill (the pill prop is gone) and gains
autoFocus, trailing and inset props so the sheets can use it too.
The process-manager plugin copies the same style since it cannot import
app components.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_015Ly4RP96nLUnj4FWtWthpE
@kipavy
kipavy merged commit 5d74e64 into dev Oct 24, 2026
5 checks passed
@kipavy
kipavy deleted the fix/mobile-search-radius branch October 24, 2026 08:50
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.

1 participant