Skip to content

Compress images #84

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

Closed
wants to merge 2 commits into from
Closed

Compress images #84

wants to merge 2 commits into from

Conversation

balcsida
Copy link

@balcsida balcsida commented Sep 1, 2016

JPEGs: 2171 KB -> 844 KB (JPEGMini)
PNGs: 7 MB savings (TinyPNG)

@lachlanjc
Copy link
Member

Hey @balcsida, thanks so much for contributing! I'm all for optimization, but it's making these photos look visibly worse. For example, number 10 looks especially bad:

screen shot 2016-09-01 at 4 50 46 pm

Maybe try lossless optimization?

@mrmrs
Copy link
Member

mrmrs commented Sep 1, 2016

Image optimization is on the list of todos, but agree this doesn't look like the solution. Definitely needs to be done. Thanks for the reminder!

@mrmrs mrmrs closed this Sep 1, 2016
@balcsida
Copy link
Author

balcsida commented Sep 2, 2016

Hi @lachlanjc, I compared the both image and haven't noticed any visible changes.
Displays I used:

  • My ASUS K55VD's built in display
  • A color calibrated Dell U2311H
  • AMOLED display of my Lumia 930

I'm just curious where to look the differences.
Also, these are JPEG files, so there is no such thing as "lossless optimization" as JPEG is a lossy compression "by nature" and the software I used is kind of "industry standard" (used by ex. Netflix and 9gag).

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