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

Add instructions to compress images #22

Closed
agarrharr opened this issue Jan 25, 2017 · 3 comments
Closed

Add instructions to compress images #22

agarrharr opened this issue Jan 25, 2017 · 3 comments

Comments

@agarrharr
Copy link
Owner

We should compress images (like @bfarayev did in #21). Add instructions to contributing.md.

It would be nice if it could be done with a command line tool. If not, add instructions to use a tool like ImageOptim.

@mischah
Copy link
Contributor

mischah commented Mar 13, 2019

There is a ImageOptim CLI which needs to be installed via Homebrew.

So I guess the best thing would be to recommend an online webapp like:

What do you think?

@agarrharr
Copy link
Owner Author

There's also ImageMagick. I'd be open to a PR suggesting any of these things.

@agarrharr
Copy link
Owner Author

agarrharr commented Mar 15, 2019

By the way, I have this alias for resizing/optimizing for the web with ImageMagick:

# $1 input_image
# $2 width
# $3 output_dir
# https://www.smashingmagazine.com/2015/06/efficient-image-resizing-with-imagemagick/
smartresize() {
   mogrify -path $3 -filter Triangle -define filter:support=2 -thumbnail $2 -unsharp 0.25x0.08+8.3+0.045 -dither None -posterize 136 -quality 82 -define jpeg:fancy-upsampling=off -define png:compression-filter=5 -define png:compression-level=9 -define png:compression-strategy=1 -define png:exclude-chunk=all -interlace none -colorspace sRGB $1
}

@agarrharr agarrharr closed this as not planned Won't fix, can't repro, duplicate, stale Oct 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants