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.

Sign up for GitHub

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

Jump to bottom

Add support for AVIF #159

Merged
merged 1 commit into from
Mar 7, 2024
Merged

Add support for AVIF #159

merged 1 commit into from
Mar 7, 2024

Conversation

jlHertel
Copy link
Contributor

@jlHertel jlHertel commented Mar 6, 2024

What is it?

  • Bugfix
  • Feature
  • Codebase improvement

Description of the changes in your PR

  • Add support for AVIF images in non-standard sizes like 3000x4000.

Fixes the following issue(s)

Acknowledgement

@jlHertel
Copy link
Contributor Author

jlHertel commented Mar 6, 2024

Note for reviewers: I have a full gallery with a ton of photos I can share for testing if anybody needs them.
Here two photos that were failling to open previously but open correctly after this PR.
1, 2

@naveensingh
Copy link
Member

Looks good to me!

I tested on a couple of devices and AVIF images with alpha channel are still problematic but that's a known issue so we'll worry about it later.

Thanks :)

@naveensingh naveensingh merged commit 64e631f into FossifyOrg:master Mar 7, 2024
1 check passed
@naveensingh naveensingh mentioned this pull request Sep 24, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Add support to AVIF
2 participants
@jlHertel @naveensingh