Skip to content

Fix collapsed Now Playing page: make homeLayout greedy again - #98

Merged
sanylax0 merged 1 commit into
mainfrom
claude/repo-review-improvements-yvk3t7-home-layout-fill
Jul 18, 2026
Merged

sanylax0 merged 1 commit into
mainfrom
claude/repo-review-improvements-yvk3t7-home-layout-fill

Conversation

@sanylax2

Copy link
Copy Markdown
Collaborator

What

Fixes the squashed Now Playing screen (content collapsed to a centered band with the Library/Up Next chevrons overlapping the title and play disc).

Why

When the backdrop moved out of homeLayout and into the pager background (#96), the layout lost its only greedy layer — the backdrop was what stretched the ZStack to full page height. Without it, the ZStack hugs the title/transport column, containerRelativeFrame centers that small view in the page, and the top/bottom chevron overlays attach to the collapsed frame, piling onto the content.

How

.frame(maxWidth: .infinity, maxHeight: .infinity) on the home ZStack, with a comment explaining why it must stay greedy. The chevrons return to the screen edges and the column re-centers on the full page.

Testing

On device: Now Playing content vertically centered on the full screen, Library chevron at the very top, Up Next at the very bottom, gradient/hybrid backdrop behind everything.

🤖 Generated with Claude Code

https://claude.ai/code/session_013TJoWkqg8bzkGdzxWhWjWP


Generated by Claude Code

Removing the embedded backdrop from homeLayout also removed its only
greedy layer — the ZStack then hugged the title/transport column, the
page collapsed to a centered band, and the chevron overlays piled onto
the content. Fill the available space explicitly.
@sanylax0
sanylax0 merged commit 82bbadc 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