Skip to content

fix: enable scrolling to bottom on detail pages on mobile#13

Merged
joekiller merged 2 commits into
mainfrom
claude/details-scroll-fix-0x6kg7
Jun 28, 2026
Merged

fix: enable scrolling to bottom on detail pages on mobile#13
joekiller merged 2 commits into
mainfrom
claude/details-scroll-fix-0x6kg7

Conversation

@joekiller

Copy link
Copy Markdown
Owner

Two root causes prevented scrolling to the bottom of detail pages
(e.g. Eggs, Pets) on mobile browsers when content was only slightly
taller than the visible viewport:

  1. CollectionsLayout used 100vh for the inner window on mobile.
    On Android/iOS, 100vh includes browser chrome (address bar +
    nav buttons), so when content fills 100vh but not the actual
    visible area, the scrollable range is too small to reach the
    chrome-hide threshold leaving the bottom content permanently
    obscured. Changed to 100dvh (Dynamic Viewport Height), which
    always equals the current visible area and adjusts as chrome
    shows or hides.

  2. Every detail component (EggsComponent, PetsComponent, etc.) sets
    height: 100% on its root div. That resolves against the scroll
    container clientHeight, clamping the component layout box to that
    fixed height. Overflowing content (overflow: visible) may not
    register in the scroll container scrollHeight across all browsers,
    making the page appear non-scrollable. Added a plain div wrapper
    inside the scroll container in DynamicCollectionConfigData: since
    the wrapper has no explicit height, child components height: 100%
    resolves to auto (natural content height), so scrollHeight
    correctly reflects full content.

Affects all detail routes (/collections/:collection/:item).

Co-Authored-By: Claude Sonnet 4.6 noreply@anthropic.com
Claude-Session: https://claude.ai/code/session_01Tp4RJb9XQn3JzDFu2Fnht4

@joekiller joekiller force-pushed the claude/details-scroll-fix-0x6kg7 branch from cadc5c5 to 61e5866 Compare June 28, 2026 23:49
@joekiller joekiller merged commit 45f9a69 into main Jun 28, 2026
1 check passed
@joekiller joekiller deleted the claude/details-scroll-fix-0x6kg7 branch June 28, 2026 23:54
@github-actions

Copy link
Copy Markdown

🎉 This PR is included in version 2.9.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant