Skip to content

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.

3 participants