Skip to content

Show exactly one pager page at a time - #79

Merged
sanylax0 merged 1 commit into
mainfrom
claude/repo-review-improvements-yvk3t7-pager-page-bleed
Jul 18, 2026
Merged

sanylax0 merged 1 commit into
mainfrom
claude/repo-review-improvements-yvk3t7-pager-page-bleed

Conversation

@sanylax2

Copy link
Copy Markdown
Collaborator

What

On the vertical pager (Library ↑ / Now Playing / Up Next ↓), the status-bar and home-indicator bands showed slivers of the neighboring pages — the bottom of Library above Now Playing and the top of Up Next below it.

Cause: containerRelativeFrame(.vertical) sizes each page to the safe-area height, but a ScrollView's content draws edge-to-edge under the bars, so the safe-area bands render the adjacent pages' content.

How

Clip the pager ScrollView to its safe-area frame (.clipped()) and paint the exposed bands with a full-bleed black background — exactly one page is visible at rest and during the flip. Every page keeps its current internal layout untouched (nav bars, chevrons, list insets), which makes this the lowest-risk fix; the alternative (full-screen pages + re-injected safe-area padding) would re-layout all three pages.

Testing

Not buildable in this environment (iOS app target) — please verify on device: Now Playing should show no library/queue bleed at top/bottom, and the flip gesture should still page cleanly.

🤖 Generated with Claude Code

https://claude.ai/code/session_013TJoWkqg8bzkGdzxWhWjWP


Generated by Claude Code

Pager pages are sized to the safe-area height, but scroll content draws
edge-to-edge, so the status-bar and home-indicator bands showed slivers
of the neighboring pages (library bleeding above Now Playing, Up Next
below). Clip the pager to its safe-area frame and paint the exposed
bands black.
@sanylax0
sanylax0 marked this pull request as ready for review July 18, 2026 16:21
@sanylax0
sanylax0 merged commit bcd9034 into main Jul 18, 2026
2 of 4 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.

3 participants