immich-app/immich
1
0
Fork 0
mirror of https://github.com/immich-app/immich.git synced 2026-02-05 00:30:57 +03:00
Code Issues 11 Packages Projects Releases 10 Wiki Activity

[BUG] web: thumbnails load fails when Album page is reloaded #1626

New Issue
Closed
opened 2026-02-05 02:43:45 +03:00 by OVERLORD · 7 comments
OVERLORD commented 2026-02-05 02:43:45 +03:00
Owner
Copy Link

Originally created by @waclaw66 on GitHub (Nov 16, 2023).

The bug

Loading of album list is slower when there is high number of albums. Those are shown as squares with their ids for a while.
It would be nice to load album list with thumbhashes and show them instantly, because album covers are loaded in separate requests afterwards.
The same applies when albums are scrolled down and new thumbnails are loaded, it takes few seconds and empty squares with ids are shown.

The real cause is that WEBP thumbnails load fails when Albums page is reloaded by F5.
I'm not sure why the key parameter is passed while F5 reload, there is no key when page navigation to Albums is used.
Because of that, it takes a while to fetch JPEG thumbnails and album icons are for few seconds without image.

obrazek

The OS that Immich Server is running on

Fedora 38

Version of Immich Server

1.86.0

Version of Immich Mobile App

not releavant

Platform with the issue

  • Server
  • Web
  • Mobile

Your docker-compose.yml content

not relevant

Your .env content

not relevant

Reproduction steps

1. hit F5 on Albums page

Additional information

No response

Originally created by @waclaw66 on GitHub (Nov 16, 2023). ### The bug ~~Loading of album list is slower when there is high number of albums. Those are shown as squares with their ids for a while. It would be nice to load album list with thumbhashes and show them instantly, because album covers are loaded in separate requests afterwards. The same applies when albums are scrolled down and new thumbnails are loaded, it takes few seconds and empty squares with ids are shown.~~ The real cause is that WEBP thumbnails load fails when Albums page is reloaded by F5. I'm not sure why the `key` parameter is passed while F5 reload, there is no `key` when page navigation to Albums is used. Because of that, it takes a while to fetch JPEG thumbnails and album icons are for few seconds without image. ![obrazek](https://github.com/immich-app/immich/assets/15554561/a11e55b5-6bcd-4640-ae2f-e450a18b7943) ### The OS that Immich Server is running on Fedora 38 ### Version of Immich Server 1.86.0 ### Version of Immich Mobile App not releavant ### Platform with the issue - [ ] Server - [X] Web - [ ] Mobile ### Your docker-compose.yml content ```YAML not relevant ``` ### Your .env content ```Shell not relevant ``` ### Reproduction steps ```bash 1. hit F5 on Albums page ``` ### Additional information _No response_
OVERLORD commented 2026-02-05 02:43:50 +03:00
Author
Owner
Copy Link

@jrasm91 commented on GitHub (Nov 17, 2023):

Hmm, I can't reproduce it. The only time the key is set is on the initial page load. This should only happen if you are on the share/[key] route. Do you have any reproduce steps? Did you navigate to the albums page from somewhere else?

  if ($page.route.id?.startsWith('/(user)/share/[key]')) {
    api.setKey($page.params.key);
  }
@jrasm91 commented on GitHub (Nov 17, 2023): Hmm, I can't reproduce it. The only time the key is set is on the initial page load. This should only happen if you are on the share/[key] route. Do you have any reproduce steps? Did you navigate to the albums page from somewhere else? ```ts if ($page.route.id?.startsWith('/(user)/share/[key]')) { api.setKey($page.params.key); } ```
OVERLORD commented 2026-02-05 02:43:54 +03:00
Author
Owner
Copy Link

@waclaw66 commented on GitHub (Nov 17, 2023):

I can reproduce it on any account, just hit F5 on Albums page.
It's weird that the key is the same for all users. Where does it come from?

@waclaw66 commented on GitHub (Nov 17, 2023): I can reproduce it on any account, just hit F5 on Albums page. It's weird that the key is the same for all users. Where does it come from?
OVERLORD commented 2026-02-05 02:43:58 +03:00
Author
Owner
Copy Link

@jrasm91 commented on GitHub (Nov 17, 2023):

Can you reproduce it on the demo site? In an incognito window (without any additional extensions)

@jrasm91 commented on GitHub (Nov 17, 2023): Can you reproduce it on the demo site? In an incognito window (without any additional extensions)
OVERLORD commented 2026-02-05 02:44:03 +03:00
Author
Owner
Copy Link

@waclaw66 commented on GitHub (Nov 17, 2023):

Will do on Sunday, currently away from home.

@waclaw66 commented on GitHub (Nov 17, 2023): Will do on Sunday, currently away from home.
OVERLORD commented 2026-02-05 02:44:08 +03:00
Author
Owner
Copy Link

@waclaw66 commented on GitHub (Nov 18, 2023):

I've noticed that 1.87.0 has been releases, thus I checked it and everything seems to be working fine, key parameter is not used anymore. Nothing else has been changed on my side.

@waclaw66 commented on GitHub (Nov 18, 2023): I've noticed that 1.87.0 has been releases, thus I checked it and everything seems to be working fine, `key` parameter is not used anymore. Nothing else has been changed on my side.
OVERLORD commented 2026-02-05 02:44:11 +03:00
Author
Owner
Copy Link

@jrasm91 commented on GitHub (Nov 18, 2023):

Interesting. Maybe a regression in the prior release. Everything is working as expected now then?

@jrasm91 commented on GitHub (Nov 18, 2023): Interesting. Maybe a regression in the prior release. Everything is working as expected now then?
OVERLORD commented 2026-02-05 02:44:15 +03:00
Author
Owner
Copy Link

@waclaw66 commented on GitHub (Nov 20, 2023):

Interesting. Maybe a regression in the prior release. Everything is working as expected now then?

Everything works fine so far. Closing for now.

@waclaw66 commented on GitHub (Nov 20, 2023): > Interesting. Maybe a regression in the prior release. Everything is working as expected now then? Everything works fine so far. Closing for now.
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
accessibility
changelog:enhancement
changelog:security
changelog:translation
cli
date-time
documentation
external-library
format
good first issue
needs-answer
nice to have
sharing
tech-debt
📱mobile
🖥️web
🗄️server
🧠machine-learning
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
OVERLORD
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: immich-app/immich#1626
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?