Skip to content

Implement interlaced PNG support#1030

Merged
blikblum merged 6 commits into
foliojs:masterfrom
blikblum:support-png-interlaced
Nov 17, 2019
Merged

Implement interlaced PNG support#1030
blikblum merged 6 commits into
foliojs:masterfrom
blikblum:support-png-interlaced

Conversation

@blikblum

@blikblum blikblum commented Sep 29, 2019

Copy link
Copy Markdown
Member

What kind of change does this PR introduce?

Fix support for interlaced PNG files.

Fixes #926

Needs foliojs/png.js#48 to work

What is the current behavior?

Interlaced PNG files are draw with garbage

What is the new behavior?

Interlaced PNG files are draw correctly

Checklist:

  • Tests (preference for unit tests)
  • Documentation
  • Update CHANGELOG.md
  • Ready to be merged

@blikblum blikblum changed the title Implmenta interlaced PNG support Implment interlaced PNG support Sep 29, 2019
@blikblum blikblum changed the title Implment interlaced PNG support Implement interlaced PNG support Sep 29, 2019
@blikblum blikblum mentioned this pull request Sep 29, 2019
4 tasks
@blikblum

Copy link
Copy Markdown
Member Author

@devongovett Can you take a look into foliojs/png.js#48 ?

There's a few important fixes, together with this one, since last release.

@devongovett

Copy link
Copy Markdown
Member

There’s a conflict, but looks good

@devongovett

Copy link
Copy Markdown
Member

Released png-js to npm as v1.0.0. Will need to be bumped here.

@blikblum blikblum merged commit 4a7f0db into foliojs:master Nov 17, 2019
@blikblum

Copy link
Copy Markdown
Member Author

All done. Thanks

@canda

canda commented Nov 20, 2019

Copy link
Copy Markdown

when is it planned to release a new version including this fix?

@luke-robertson

Copy link
Copy Markdown

this gonna be released ?

@blikblum

Copy link
Copy Markdown
Member Author

Already released as v0.11

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.

Interlaced PNG files are not supported

4 participants