Skip to content

Improve mobile search experience#9

Merged
kylemclaren merged 1 commit intomainfrom
mobile-search-improvements
Dec 28, 2025
Merged

Improve mobile search experience#9
kylemclaren merged 1 commit intomainfrom
mobile-search-improvements

Conversation

@kylemclaren
Copy link
Collaborator

@kylemclaren kylemclaren commented Dec 28, 2025

Summary

  • Fix background scroll not being disabled on mobile (iOS Safari) by using position: fixed with scroll position preservation
  • Make search button icon-only on mobile (no border/background, larger touch target)
  • Position search dialog closer to top on mobile with proper horizontal margins
  • Refactor scroll lock/unlock logic into reusable helper function

Test plan

  • Open on mobile device or responsive mode
  • Tap search icon - verify it's icon-only with no border
  • Verify search dialog appears near top of screen with margins
  • Scroll the background - verify it doesn't scroll while dialog is open
  • Close dialog - verify scroll position is preserved
  • Test on iOS Safari specifically for scroll lock

🤖 Generated with Claude Code

- Fix background scroll not being disabled on mobile (iOS Safari)
- Make search button icon-only on mobile (no border/background)
- Position search dialog closer to top on mobile with proper margins
- Refactor scroll lock/unlock into reusable helper

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@kylemclaren kylemclaren merged commit c117fa7 into main Dec 28, 2025
1 check passed
@kcmartin kcmartin deleted the mobile-search-improvements branch December 29, 2025 21:31
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