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

Adds optional 'quality' param to allow lossy jpeg output #2

Merged
merged 1 commit into from
Dec 4, 2013

Conversation

dforesman
Copy link
Contributor

Depending on formatting of the original image, slight lossy optimization can provide huge file-size savings over strictly lossless JPEG compression. This update adds an optional 'quality' param that can be sent along with the source image path.

Optimizer will default to lossless compression if no quality param is specified, this change will not break existing implementations of the optimizer. Updates jpegoptim command line calls to force Progressive JPG output, as they provide a better end-user experience when loading over a slow connection. PNG optimization untouched; PNGs will ignore the quality param. Readme updated to reflect new options and defaults.

…default to lossless compression. Updates jpegoptim command line calls to force Progressive JPG output, as they provide a better user experience when loading over a slow connection. Readme updated to reflect new options and defaults.
@coveralls
Copy link

Coverage Status

Coverage decreased (-2.35%) when pulling 647ce33 on dforesman:jpeg-quality into 90c5881 on jtescher:master.

jtescher added a commit that referenced this pull request Dec 4, 2013
Adds optional 'quality' param to allow lossy jpeg output
@jtescher jtescher merged commit 88d67ed into jtescher:master Dec 4, 2013
@jtescher
Copy link
Owner

jtescher commented Dec 4, 2013

Sounds good.

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.

None yet

3 participants