Skip to content
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

Improve gif quality by appropriately sizing gifs #2

Merged
merged 1 commit into from
Jun 25, 2014

Conversation

jkupferman
Copy link
Contributor

The existing code resized gifs down to 320x240 which twitter then scaled back up to 528x396 resulting in grainy gifs. This change r
It also tells gifsicle to limit gif colors to 256 which should reduce file size.

NOTE: This code should be tested to ensure that it generates gifs below the 3MB Twitter limit.

The existing code resized gifs down to 320x240 which twitter then scaled back up to 528x396 resulting in grainy gifs. This change r
It also tells gifsicle to limit gif colors to 256 which should reduce file size.
@jkupferman
Copy link
Contributor Author

While I think this will help with overall quality and size, I suspect you'd get much better results by posting an reasonably sized mp4 to twitter (as a video). You'd be able to achieve much higher video quality and stay well below the twitter video size limit.

@xdamman
Copy link
Owner

xdamman commented Jun 25, 2014

Thanks for this, really appreciate the contribution here. Let me test this first.

I also tweet the link to the mp4. Twitter doesn't let you upload a mp4 unfortunately. The only solution is to have my own player but they would first need to approve it.

@xdamman
Copy link
Owner

xdamman commented Jun 25, 2014

Works great, merging! Thanks again!

xdamman added a commit that referenced this pull request Jun 25, 2014
Improve gif quality by appropriately sizing gifs
@xdamman xdamman merged commit 8ad3acc into xdamman:master Jun 25, 2014
@FredericJacobs
Copy link

Or reverse the Vine/Instagram API? 0:)

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.

3 participants