Skip to content

Conversation

@meesfrensel
Copy link
Collaborator

@meesfrensel meesfrensel commented Jan 6, 2026

Description

Updates the ErrorLayout page to use mostly Immich UI components. I think the page could do with a refresh in the future; this should make that easier.

  • Styling more closely matches modals
  • Logo in navbar is functional (clickable)
  • Alarm emoji is now icon
  • Layout is somewhat responsive
  • Code is somewhat cleaner
  • Page is a little more accessible

How Has This Been Tested?

  • With and without stack trace

Screenshots

image

Please describe to which degree, if any, an LLM was used in creating this pull request.

none

@github-actions
Copy link
Contributor

github-actions bot commented Jan 6, 2026

Deploying preview environment to https://pr-25094.preview.internal.immich.build/

@alextran1502
Copy link
Member

can't unseen this 😭

image

@meesfrensel
Copy link
Collaborator Author

Lol somehow I missed that completely when making the screenshot, will fix later

@meesfrensel meesfrensel force-pushed the refactor/error-layout branch from dd35bd7 to 8464bbd Compare January 7, 2026 12:26
@meesfrensel
Copy link
Collaborator Author

Fixed.

@alextran1502
Copy link
Member

One final nit, can you see if you can align the bottom on top-horizontally?

image

@meesfrensel
Copy link
Collaborator Author

Also fixed ;)

image

Copy link
Member

@danieldietzler danieldietzler left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice!

@alextran1502 alextran1502 merged commit ef4aec7 into main Jan 7, 2026
53 checks passed
@alextran1502 alextran1502 deleted the refactor/error-layout branch January 7, 2026 21:49
kiloomar pushed a commit to kiloomar/immich that referenced this pull request Jan 12, 2026
* chore: refactor ErrorLayout

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants