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

Hero image/screenshot 10x+ larger than nominal (4.5+ MB) due to apparent scaling issue and resulting poor compression efficiency #202

Closed
CAM-Gerlach opened this issue Sep 21, 2021 · 2 comments · Fixed by #225
Assignees
Labels

Comments

@CAM-Gerlach
Copy link
Member

Following PR #199 , particularly what appeared to be its addressing of #197 , the size of the hero image has increased by 1000% (!), to a whopping 4.5+ MB. This is a very big hit for users, especially newer ones loading the site for the first time or that don't have access to good internet. Given the that both the image itself and the screenshot and logotype within are the same size and similar in content, and optipng barely made any difference (<3 % reduction), I strongly suspect the primary cause is the screenshot being rescaled by a non-integer multiple, one if not multiple times before appearing in the final rendered PNG. This results in very poor compression efficiency, as well as slightly inferior crispness and quality.

In order to fix this, the screenshot in the final hero graphic layout should be retaken or adjusted to be at a 1:1 ratio (100% size) to the original source, by sizing the Spyder window and display rendered resolution according. For reference, in the original I made, the screenshot was approximately 1770 x 780 (dimensions which were roughly maintained in the output by the current version); it doesn't have to be exactly those dimensions but it should be close enough that it can be added to the final layout at exactly 1:1 size. If for some reason that isn't possible, at the very least it should be a 2:1 ratio with nearest neighbor downsampling, though encoding performance and (to a lesser extent) quality will suffer somewhat.

In PR #199 , @isabela-pf graciously agreed to take this on at some point, so assigning this to her.

@conradolandia
Copy link
Contributor

This is addressed by PR 224. Format of image changed from png to webp, new image size is 271 KB.

@CAM-Gerlach
Copy link
Member Author

This was fixed by #225 , so closing this. Thanks @conradolandia !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants