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

Service File should not change GIF to JPG #1570

Open
DonRichards opened this issue Jul 27, 2020 · 3 comments
Open

Service File should not change GIF to JPG #1570

DonRichards opened this issue Jul 27, 2020 · 3 comments
Labels
Type: bug identifies a problem in the software with clear steps to reproduce

Comments

@DonRichards
Copy link
Member

DonRichards commented Jul 27, 2020

Uploading an animated GIF the "Service File" generator ignores the that the original might be animated and should respect if animated but currently does not.

gif_services_file

Either create a lower res GIF or convert to a more appropriate format.

Suggested Alternatives:

APNG Animated Portable Network Graphics
APNG Animated Portable Network Graphics

WebP Modern image format developed by Google
WebP Modern image format developed by Google

@seth-shaw-unlv
Copy link
Contributor

Huh, there isn't a clear winner between APNG and WebP based on browser support. It looks WebP has the widest support but it has a gaping hole without Safari (for now, at least).

On the other hand, you can get ImageMagick/GraphicsMagick to produce WebP files but not ANPG as far as I can discern. We would need a separate Crayfish service to wrap gif2apng to support it.

@seth-shaw-unlv
Copy link
Contributor

MP4 may be even better... and we can use FFMPEG for the job.

Alternatively, we could just not create service files for Gifs. You can mark them as both Original File and Service File....

@DonRichards
Copy link
Member Author

@seth-shaw-unlv I agree MP4 is probably the best option from what I've seen online.

@kstapelfeldt kstapelfeldt added Type: bug identifies a problem in the software with clear steps to reproduce and removed bug labels Sep 25, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: bug identifies a problem in the software with clear steps to reproduce
Projects
Development

No branches or pull requests

3 participants