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

refactor: (losslessly) compress images #924

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

j9t
Copy link

@j9t j9t commented Feb 7, 2025

Hi—spontaneous PR (working on updates to Imagemin Guard, testing them on somewhat random repos) that includes optimizations and a few savings for this repo’s images:

Compressed public/checkbox.png (0.79 KB → 0.65 KB)
Skipped public/chosen-sprite.png (already compressed more aggressively)
Compressed public/[email protected] (0.85 KB → 0.85 KB)
Compressed features/upload_files/img.png (7.74 KB → 4.51 KB)
Compressed public/images/email-top-banner.png (7.38 KB → 6.41 KB)
Compressed public/images/share-on-email-thin.png (2.36 KB → 2.11 KB)
Compressed public/images/share-on-fb-thin.png (4.38 KB → 2.63 KB)
Skipped public/images/share-on-fb.png (already compressed more aggressively)
Compressed public/images/share-on-g-thin.png (2.78 KB → 2.69 KB)
Skipped public/images/share-on-g.png (already compressed more aggressively)
Compressed public/images/share-on-tw-thin.png (3.65 KB → 2.43 KB)
Skipped public/images/share-on-tw.png (already compressed more aggressively)
Compressed app/assets/images/3dots.png (0.39 KB → 0.28 KB)
Compressed app/assets/images/EFF-logo-lockup-1color-white.png (5.80 KB → 4.51 KB)
Compressed app/assets/images/EFF_Monogram-red.png (0.94 KB → 0.51 KB)
Compressed app/assets/images/close.png (1.31 KB → 0.24 KB)
Skipped app/assets/images/empty.gif (already compressed)
Compressed app/assets/images/footer-cc.png (4.63 KB → 1.20 KB)
Compressed app/assets/images/footer-logo.png (3.60 KB → 3.03 KB)
Compressed app/assets/images/[email protected] (4.51 KB → 3.37 KB)
Compressed app/assets/images/missing.png (7.10 KB → 6.59 KB)
Skipped app/assets/images/mobile-apps.png (already compressed more aggressively)
Compressed app/assets/images/mobile-download-goog.png (2.36 KB → 2.36 KB)
Skipped app/assets/images/mobile-download-ios.png (already compressed more aggressively)
Compressed app/assets/images/og-image-default.png (7.10 KB → 6.59 KB)
Compressed app/assets/images/radio-checked.png (0.62 KB → 0.46 KB)
Compressed app/assets/images/radio-unchecked.png (0.49 KB → 0.28 KB)
Skipped app/assets/images/social-email.png (already compressed more aggressively)
Skipped app/assets/images/social-facebook.png (already compressed more aggressively)
Skipped app/assets/images/social-link.png (already compressed more aggressively)
Skipped app/assets/images/social-twitter.png (already compressed more aggressively)
Compressed app/assets/images/thankyou-circle.png (31.23 KB → 29.85 KB)
Compressed app/assets/images/twitter-card-homepage.png (7.10 KB → 6.59 KB)
Compressed spec/fixtures/files/test-image.png (5.41 KB → 3.51 KB)

Defensive base compression completed. You saved 20.89 KB.

The compression is lossless. Couldn’t detect any issues (which was the point of my own tests). Offering this if you like the improvements, otherwise feel free to discard!

Signed-off-by: Jens Oliver Meiert <[email protected]>
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.

1 participant