Skip to content

Fix handling gif encoding for global palettes. #2614

Merged
JimBobSquarePants merged 2 commits intorelease/3.1.xfrom
js/gif-global-256-no-trans-fix
Dec 11, 2023
Merged

Fix handling gif encoding for global palettes. #2614
JimBobSquarePants merged 2 commits intorelease/3.1.xfrom
js/gif-global-256-no-trans-fix

Conversation

@JimBobSquarePants
Copy link
Member

Prerequisites

  • I have written a descriptive pull-request title
  • I have verified that there are no overlapping pull-requests open
  • I have verified that I am following the existing coding patterns and practice as demonstrated in the repository. These follow strict Stylecop rules 👮.
  • I have provided test coverage for my change (where applicable)

Description

See SixLabors/ImageSharp.Web#344 (comment)

When an input gif has a full, non-transparent global palette we should use own to allow deduplication. Same image is reduced by 26% on encode.

@JimBobSquarePants JimBobSquarePants merged commit ad1b0e6 into release/3.1.x Dec 11, 2023
@JimBobSquarePants JimBobSquarePants deleted the js/gif-global-256-no-trans-fix branch December 11, 2023 12:01
@JimBobSquarePants JimBobSquarePants restored the js/gif-global-256-no-trans-fix branch December 11, 2023 12:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants