Skip to content

fix: z-index war in the asset viewer - #18091

Merged
jrasm91 merged 1 commit into
mainfrom
fix/z-index-war
May 9, 2025
Merged

jrasm91 merged 1 commit into
mainfrom
fix/z-index-war

Conversation

@danieldietzler

@danieldietzler danieldietzler commented May 5, 2025 •

Copy link
Copy Markdown
Member

Aims to fix #18039 (comment). I tried to check all UI elements, but it's possible I missed some in special scenarios.

@github-actions

github-actions Bot commented May 5, 2025

Copy link
Copy Markdown
Contributor

Deploying preview environment to https://pr-18091.preview.internal.immich.cloud/

@alextran1502 alextran1502 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Fixed the slideshow control

@danieldietzler
danieldietzler force-pushed the fix/z-index-war branch 3 times, most recently from 977a2a7 to 69537d1 Compare May 9, 2025 11:13
@jrasm91

jrasm91 commented May 9, 2025

Copy link
Copy Markdown
Member

The mobile sidebar is still broken (map view and/or memories). You can fix that in another PR if you want.

@jrasm91
jrasm91 merged commit 89551ed into main May 9, 2025
@jrasm91
jrasm91 deleted the fix/z-index-war branch May 9, 2025 14:17
@github-actions github-actions Bot removed the preview label May 9, 2025
savely-krasovsky pushed a commit to savely-krasovsky/immich that referenced this pull request Jun 8, 2025
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.

3 participants