-
Notifications
You must be signed in to change notification settings - Fork 6
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
Update PNGImage.as #381
base: master
Are you sure you want to change the base?
Update PNGImage.as #381
Conversation
Throw error message for unsupported PNG transparency mode instead of failing silently.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In issue #292 there is a link of the implementation for transparent PNG. It would be nice to have that in your pull request.
More details can also be found under this issue #186 |
Yes, it would be nice to have a fix. This is a major limitation of AlivePDF. A few notes on the topic:
I don't have the bandwidth to do the project of adding PNG transparency now unfortunately... |
We may have some available resource available in the coming days and we will look how to add the alpha lyer support for PNG. |
This is used in production, but being phased out. I have a few PNGs testcases I can contribute, if this helps. |
@teuf22 It would be nice if you can provide those PNG tastcases, it will help us for sure. Thanks. |
Throw error message for unsupported PNG transparency mode instead of failing silently.