search

imagemin / gifsicle-bin

gifsicle bin-wrapper that makes it seamlessly available as a local dependency
http://www.lcdf.org/gifsicle/
MIT License
112 stars 52 forks source link

giflossy #62

Closed denji closed 8 years ago

denji commented 8 years ago

Based on gifsicle which implements lossy LZW compression. It can reduce animgif file sizes by 30%—50% at a cost of some dithering/noise.

sindresorhus commented 8 years ago

I don't have time to make a wrapper module for this, but maybe @shinnn or @kevva has time.

shinnn commented 8 years ago

I'm waiting for https://github.com/kohler/gifsicle/pull/16 to be merged.

kornelski commented 8 years ago

The PR has been open for so long now, that I lost hope it'll ever be merged.

shinnn commented 8 years ago

OK I'll create imagemin/giflossy-bin.

shinnn commented 8 years ago

@denji First, can you help fixing merge conflict of the @pornel's pull request?

https://github.com/kohler/gifsicle/pull/16#issuecomment-181295112

If someone wants to help, there's a merge conflict that needs to be resolved.

Then if the PR is still ignored by the Gifsicle author for a long time, I'll create a giflossy wrapper. Maintaining two wrappers for almost the same functionality is not the best solution. I hope Gifsicle obtains --lossy option even now.

denji commented 8 years ago

If someone wants to help, there's a merge conflict that needs to be resolved.

I'm not sure that after resolving merge conflict will it immediately in the upstream and review. It will be reviewed?

Produced semantic merge based on the C https://github.com/kohler/gifsicle/pull/72