Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

'Add a location' feature in Immich web doesn't work on Safari #9594

Closed
1 of 3 tasks
eon-pixel opened this issue May 19, 2024 · 4 comments · Fixed by #9756
Closed
1 of 3 tasks

'Add a location' feature in Immich web doesn't work on Safari #9594

eon-pixel opened this issue May 19, 2024 · 4 comments · Fixed by #9756
Labels
bug Something isn't working 🖥️web

Comments

@eon-pixel
Copy link

The bug

The dialog box/menu for the 'Add a location' feature does not appear correctly on Safari.

I opened an image, then clicked info, then 'Add a location'. The dialog box opens for a fraction of a second before appearing to vanish, leaving the rest of the web interface in slight greyscale.

I am using macOS 14.5 and Safari 17.5.

Below is an example of the problem:
screenshot

The OS that Immich Server is running on

Ubuntu 22.04

Version of Immich Server

v1.105.1

Version of Immich Mobile App

N/A

Platform with the issue

  • Server
  • Web
  • Mobile

Your docker-compose.yml content

Not relevant

Your .env content

Not relevant

Reproduction steps

1. Open an image on Immich web 
2. Click on info
3. Click on 'Add a location'
4. Dialog box opens then vanishes

Relevant log output

No response

Additional information

No response

@bo0tzz bo0tzz added bug Something isn't working 🖥️web labels May 19, 2024
@danieldietzler
Copy link
Member

Duplicate of #8223

@danieldietzler danieldietzler marked this as a duplicate of #8223 May 20, 2024
@danieldietzler danieldietzler closed this as not planned Won't fix, can't repro, duplicate, stale May 20, 2024
@Xenthio
Copy link

Xenthio commented May 25, 2024

This isn't a duplicate issue, the two issues seem to be different.
When you specify z-index on the details panel you are creating a new stacking context, so the fullscreen zindex 9999 popup doesn't overlay anything outside of it.
image

you are able to fix this by removing the highlighted class which specifies the z-index. or you could move the popup from inside the detail panel outside of it.

@Xenthio
Copy link

Xenthio commented May 25, 2024

Or for consistent behaviour that other browsers show, you can specify overflow:visible on the detail panel

@bo0tzz
Copy link
Member

bo0tzz commented May 25, 2024

This is indeed a different issue

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working 🖥️web
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants