BrawlCrate icon indicating copy to clipboard operation
BrawlCrate copied to clipboard

png palette import fix

Open MichaelHinrichs opened this issue 4 years ago • 1 comments

This is an attempt to consolidate some commits that were left behind in BrawlBox, with changes that have been made to BrawlCrate since it branched off. These changes are based off of libertyernie/png-palette-import-fix, which was branched and merged with the BrawlBox's master branch after BrawlCrate.

The Brawlbox commits read If PngBitmapDecoder can't load a PNG, give up on importing palette and fall back to loading at a higher level.

Adjust for DPI when importing indexed PNG.

Merge pull request #232 from libertyernie/png-palette-import-fix PNG import fixes

MichaelHinrichs avatar Nov 23 '21 00:11 MichaelHinrichs

What precisely does this fix? Crate already had fixes in place for the DPI and at least one bug with PNG palettes. I didn't merge in the code when it was pushed to Box initially due to some bugs I found when testing it (though it was long enough ago that these may have been fixed and I don't recall what they were)

soopercool101 avatar Apr 11 '22 23:04 soopercool101