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

feat(web,mobile) Allow videos to be looped in the detail viewer #8615

Merged
merged 21 commits into from
May 14, 2024

Conversation

kleinMaggus
Copy link
Contributor

This PR adds a setting to the web and mobile version to make videos automatically loop in the detail viewer. The user can decide if he wants this behaviour or not (default currently enabled).

Web setting:

Mobile setting:

It is my first contribution and I am complete new to flutter and have only little svelte experience, so I am open for constructive feedback :)

@jrasm91 jrasm91 requested a review from alextran1502 April 10, 2024 03:19
@alextran1502 alextran1502 self-assigned this Apr 15, 2024
Copy link
Contributor

@jrasm91 jrasm91 left a comment

Choose a reason for hiding this comment

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

LGTM

@jrasm91
Copy link
Contributor

jrasm91 commented May 13, 2024

@alextran1502 can you test this when you have a second?

@zackpollard
Copy link
Contributor

I'll take a look at this and test it tomorrow

@alextran1502 alextran1502 merged commit d62e904 into immich-app:main May 14, 2024
22 checks passed
@aminomaboul
Copy link

aminomaboul commented May 15, 2024

i update both the mobile app and the server (1.105.0 build 139 and 1.105.1) but still cannot see the option in the Asset Viewer settings section on the mobile App (Android)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

6 participants