Page MenuHomePhabricator

Optimize PNG images
ClosedPublic

Authored by Fabien on Tue, Jul 9, 06:16.

Details

Reviewers
deadalnix
Group Reviewers
Restricted Project
Commits
rABC2ef4a66327a3: Optimize PNG images
Summary

This is the result of the contrib/devtools/optimize-pngs.py script.
This saves the grand total of 14819 bytes.

Test Plan

Check the icons are still good-looking.

Diff Detail

Repository
rABC Bitcoin ABC
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

Fabien created this revision.Tue, Jul 9, 06:16
Herald added a reviewer: Restricted Project. · View Herald TranscriptTue, Jul 9, 06:16
deadalnix requested changes to this revision.Tue, Jul 9, 12:16

How come this is changing images we inherited from core ? Second, is this lossy ? If it is, it should not be done on source material.

This revision now requires changes to proceed.Tue, Jul 9, 12:16
Fabien requested review of this revision.Tue, Jul 9, 12:54

My understanding is that the script doesn't impact the image:
it will optimize the png files, remove various color profiles, remove ancillary chunks (alla) and text chunks (text).
Optimization is done by using the filter with the highest compression method by brute-forcing all the filters. Because PNG is lossless, this doesn't impact the image.

Tested on current core master, the HD images are not optimized, I guess they forgot to run the script.

deadalnix accepted this revision.Tue, Jul 9, 17:28
This revision is now accepted and ready to land.Tue, Jul 9, 17:28
This revision was automatically updated to reflect the committed changes.