Skip to content

Conversation

@krusty-agent
Copy link
Collaborator

Fixes weird scrolling behavior on mobile app.

Changes

This PR addresses the mobile scrolling issues by implementing several key improvements:

iOS Momentum Scrolling

  • Added -webkit-overflow-scrolling: touch to all scrollable containers
  • This enables smooth, native-feeling momentum scrolling on iOS devices

Better Overscroll Behavior

  • Replaced overscroll-contain with overscroll-behavior-y: contain
  • Provides better scroll boundaries without feeling janky

Fixed Scroll Context

  • Fixed body/root height to prevent multiple competing scroll contexts
  • Removed overflow-hidden from main app container
  • Ensured only the main content area handles scrolling

Layout Improvements

  • Updated main container to use min-h-screen-safe (with dvh support)
  • Added flex-shrink-0 to header to prevent collapse during scroll
  • Cleaned up Panel component scroll behavior

Testing

  • ✅ Build passes successfully
  • Ready for mobile device testing

Related

Fixes item from Poo App feature list: "The scrolling feels a bit weird especially in the mobile app"

- Add -webkit-overflow-scrolling: touch for smooth momentum scrolling on iOS
- Replace overscroll-contain with overscroll-behavior-y: contain for better scroll behavior
- Fix body/root height to prevent multiple scroll contexts
- Remove overflow-hidden from main container to allow proper scrolling
- Update main content to use min-h-screen-safe instead of min-h-screen
- Add flex-shrink-0 to header to prevent it from collapsing
- Remove unnecessary inline style and overscroll-contain from main content
- Update Panel component to use proper overscroll behavior

These changes ensure smooth, native-feeling scrolling on mobile devices,
particularly on iOS where -webkit-overflow-scrolling: touch is crucial
for momentum scrolling.
@brianorwhatever brianorwhatever merged commit 7ff9b21 into aviarytech:main Feb 10, 2026
3 checks passed
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