immich icon indicating copy to clipboard operation
immich copied to clipboard

[BUG] photos should have a permanent url

Open uhthomas opened this issue 2 years ago • 6 comments

The bug

Photos don't have a permanent link when viewed, which means Immich will reset back to the main photos page when the page is refreshed. It also means the link can't be copied or opened in a new tab for whatever reason.

image

The OS that Immich Server is running on

N/A

Version of Immich Server

v1.62.1

Version of Immich Mobile App

N/A

Platform with the issue

  • [X] Server
  • [X] Web
  • [ ] Mobile

Your docker-compose.yml content

N/A

Your .env content

N/A

Reproduction steps

N/A

Additional information

No response

uhthomas avatar Jun 21 '23 21:06 uhthomas

Even though albums generate a full URL like /albums/:id/photos/:id, a refresh forces the user back to /albums/:id. Seems a bit pointless and should also be fixed.

uhthomas avatar Jun 25 '23 23:06 uhthomas

This is a more complicated behavior that we haven't gotten around to implementing. By navigating to the specific URL, the asset will need to be determined which time bucket it is in to allow proper navigation.

alextran1502 avatar Jun 26 '23 00:06 alextran1502

@alextran1502 I don't think it really matters? I don't think even Google Photos handles that. I would be happy if a direct link just handed the user back to the top of the page.

uhthomas avatar Jul 03 '23 17:07 uhthomas

Google does handle direct links to assets, and the previous/next arrows work within the context too.

jrasm91 avatar Jul 03 '23 17:07 jrasm91

Yes, sorry, I am aware that Google handles those cases correctly.

How does Google handle exiting from the asset?

uhthomas avatar Jul 03 '23 19:07 uhthomas

It goes back to the album/context as you would expect

jrasm91 avatar Jul 03 '23 22:07 jrasm91

With some of the recent changes in the asset grid implementation, I think it should be easier to add this now. For views that use the asset grid, we should be able to add this feature.

jrasm91 avatar Aug 06 '23 19:08 jrasm91

Implemented #8532

alextran1502 avatar Apr 27 '24 16:04 alextran1502