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

Fix generation for already supported image formats #460

Merged
merged 1 commit into from Nov 27, 2022
Merged

Fix generation for already supported image formats #460

merged 1 commit into from Nov 27, 2022

Conversation

Bungeefan
Copy link
Contributor

This PR fixes the support for the already supported image formats by the underlying library.
The existing logic of copying the background image has been changed to convert it to PNG and only copy it if it is already ".png".

https://github.com/brendan-duncan/image#supported-image-formats

  • PNG / Animated APNG
  • JPEG
  • Targa
  • GIF / Animated GIF
  • ICO
  • BMP

@Bungeefan
Copy link
Contributor Author

This would at least partially help #245.

@Bungeefan Bungeefan changed the title Fixes generation for already supported image formats Fix generation for already supported image formats Nov 24, 2022
@jonbhanson
Copy link
Owner

@Bungeefan this is great. Can you also integrate your change into the Android part of the package?

@Bungeefan
Copy link
Contributor Author

Thanks.
This should already be done as _createBackground is being called from the Android part, or did I miss something?

@jonbhanson
Copy link
Owner

Oh, yes, you are correct. Thanks!

@jonbhanson jonbhanson merged commit 2f6e090 into jonbhanson:master Nov 27, 2022
jonbhanson added a commit that referenced this pull request Nov 27, 2022
…Severin for PR #459.  Support more image formats.  Thanks Severin for PR #460.
@shahmirzali49
Copy link

@jonbhanson @Bungeefan now the package is support GIF ?

@Bungeefan
Copy link
Contributor Author

Yes and no, the package now just converts your GIF file to a PNG, it doesn't animate it or anything.
AFAIK it only takes the first frame, if there are multiple.

@shahmirzali49
Copy link

sad :/ . thanks for replying

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants