Skip to content

Improve error 500 page layout for desktop and mobile#605

Closed
mildabre wants to merge 18 commits intonette:masterfrom
mildabre:error-500-template-responsive-design
Closed

Improve error 500 page layout for desktop and mobile#605
mildabre wants to merge 18 commits intonette:masterfrom
mildabre:error-500-template-responsive-design

Conversation

@mildabre
Copy link
Copy Markdown

@mildabre mildabre commented Oct 9, 2025

Description:

switched from absolute to fixed positioning for reliable viewport anchoring
applied width: 100vw and height: 100vh for full coverage
added missing background-color: white
smaller font sizes and margins for mobile devices

This update needs also add missing viewport defined in my previous PR #604

@dg dg force-pushed the master branch 5 times, most recently from 32f8942 to cea50d6 Compare November 4, 2025 13:29
@dg dg force-pushed the master branch 3 times, most recently from c647740 to 7543389 Compare November 24, 2025 02:46
@dg dg force-pushed the master branch 4 times, most recently from e4c1d65 to 463af6a Compare December 29, 2025 22:28
@dg
Copy link
Copy Markdown
Member

dg commented Apr 12, 2026

Thanks for the effort! The viewport meta tag from #604 has been applied, which already makes the page render well on mobile (tested on iPhone SE width).

I'm not taking the rest of the layout changes though:

  • Switching to position: fixed + full-viewport white background would hide any partial output the app managed to render before the fatal error. I'd rather let users see whatever the app produced than cover it with a white overlay - even partial context is more valuable than none.
  • With the viewport meta fixed, the existing font sizes and margins are readable on typical mobile widths (~375-400px), so the extra media queries aren't needed.
  • The low-height landscape case is a rare edge case not worth the added CSS complexity.

Closing, but thanks again for spotting the mobile issue.

@dg dg closed this Apr 12, 2026
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.

4 participants