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(mobile): add ability to force view original videos #15094

Merged
merged 3 commits into from
Jan 6, 2025

Conversation

ovidijusr
Copy link
Contributor

Hi there, I discovered some interesting use case with AV1 videos on older (iphone 14 and below) devices: #15048

If I transcode my library to av1, (which I do becouse browsers don't support original .MOV files) I lose ability to view videos on my iphone, it tries to open av1, but video doesn't work (shows a single frame with audio).

With this PR user still would be able to view original high quality HDR .MOV videos natively supported on iphone and also would be able to access transcoded videos in browsers.

This is my first PR so feel free to comment any uncertainties.

mobile/assets/i18n/en-US.json Outdated Show resolved Hide resolved
@mertalev
Copy link
Contributor

mertalev commented Jan 5, 2025

You should run the dart formatter to fix the failing check.

@ovidijusr ovidijusr requested a review from mertalev January 6, 2025 08:53
@github-actions github-actions bot added the documentation Improvements or additions to documentation label Jan 6, 2025
@ovidijusr ovidijusr force-pushed the feat/force-original-video branch from 4cab106 to ff7c9d8 Compare January 6, 2025 20:11
Copy link
Member

@danieldietzler danieldietzler left a comment

Choose a reason for hiding this comment

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

Your latest merge seems to have undone a bunch of changes, please fix that

@ovidijusr ovidijusr force-pushed the feat/force-original-video branch from 19f8db0 to 93194e2 Compare January 6, 2025 20:23
@ovidijusr ovidijusr force-pushed the feat/force-original-video branch from 93194e2 to fe247c6 Compare January 6, 2025 20:23
@mertalev mertalev merged commit a13b7b3 into immich-app:main Jan 6, 2025
33 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants