Replace png image format with webp #20
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Lighthouse estimates a second's worth of savings on load time but this is based on lossless settings. I used
|
* Used `cwebp -near_lossless` for everything in `src/img` so that they're virtually indistinguishable from the png's but smaller. This reduced the directory size from 6.6 MB to 4.1 MB (~38% smaller). * Used `cwebp -af` for `vils-required2.png` because it's way too big for near-lossless compression and should be converted to HTML anyway. I also didn't find any references to it in the history. * Left `background.jpg` alone because I gave up looking for a `cwebp` option that wasn't noticeably blurrier but was smaller than the jpeg.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
While it's possible to make it fall back to older image formats, I don't believe it's worth the effort because webp has been widely supported for a long time: https://caniuse.com/webp
Tested by launching my local server, verifying that no PNG's existed on the pages (in case something in the page generation made
git grepandgit ls-filesfail), and checked the console for 404's. I also eyeballed the differences between the images and they are virtually none.From the commit message:
cwebp -near_losslessfor everything insrc/imgso that they'revirtually indistinguishable from the png's but smaller. This reduced the
directory size from 6.6 MB to 4.1 MB (~38% smaller).
cwebp -afforvils-required2.pngbecause it's way too big fornear-lossless compression and should be converted to HTML anyway.
I also didn't find any references to it in the history.
background.jpgalone because I gave up looking for acwebpoptionthat wasn't noticeably blurrier but was smaller than the jpeg.