Skip to content

Fix mobile viewport layout on song detail page - #147

Merged
tacticalnoot merged 1 commit into
mainfrom
codex/fix-mobile-screen-visibility-for-/id
May 16, 2026
Merged

Fix mobile viewport layout on song detail page#147
tacticalnoot merged 1 commit into
mainfrom
codex/fix-mobile-screen-visibility-for-/id

Conversation

@tacticalnoot

Copy link
Copy Markdown
Owner

Motivation

  • Mobile browsers with dynamic toolbars were hiding the song detail content because the page relied on 100vh; the container should prefer small-viewport units and top-align on mobile so content remains visible and scrollable.

Description

  • Update src/pages/[id].astro page wrapper to prefer svh with a vh fallback and change mobile alignment to justify-start while preserving lg:justify-center for large screens (layout-only change).

Testing

  • Ran pnpm check (Svelte typecheck); the command ran but failed due to pre-existing TypeScript/Svelte errors unrelated to this layout change (svelte-check reported 21 errors and 12 warnings across other components).

Codex Task

@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying smol-fe with  Cloudflare Pages  Cloudflare Pages

Latest commit: e2ea659
Status:⚡️  Build in progress...

View logs

@tacticalnoot
tacticalnoot merged commit 49686aa into main May 16, 2026
1 of 2 checks passed
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