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

Support webp format #1785

Merged
merged 6 commits into from Sep 11, 2023
Merged

Support webp format #1785

merged 6 commits into from Sep 11, 2023

Conversation

Phlogi
Copy link
Contributor

@Phlogi Phlogi commented Nov 13, 2022

This allows users to easily add further formats.

Also these formats should not be hard-coded there anyway :)

Update: multiple changes now to support serving webp files.

This allows users to easily add further formats.
@Phlogi Phlogi changed the title Use configured picture extensions instead of hardcoded. Support webp format Nov 13, 2022
@NeiRib
Copy link

NeiRib commented May 16, 2023

Thumbnails don't work

@edison23
Copy link

Hi, works great for me (used the steps described here - https://edison23.net/blog/posts/piwigo-webp-support#implement-the-pr-into-your-piwigo ), but I'm missing EXIF support.

Were you, @Phlogi , able to look into that, if it's possible to solve somehow? Because currently, even though Piwigo does display the WebP images without an issue, it sorts them wrong because it can't see the EXIF tags, so it's rather unusable :(

@plegall
Copy link
Member

plegall commented Sep 11, 2023

[...] even though Piwigo does display the WebP images without an issue, it sorts them
wrong because it can't see the EXIF tags, so it's rather unusable :(

I will work on it (after this pull-request is merged), and other things related to webP.

@plegall plegall merged commit 79e91e3 into Piwigo:master Sep 11, 2023
@plegall plegall self-assigned this Sep 11, 2023
@plegall plegall added this to the 14.0.0beta3 milestone Sep 11, 2023
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.

None yet

4 participants