Conversation
|
Hi Matej, thanks for this patch. I have a few suggestions about the screenshot images. Recently, I spent a few minutes on shrinking the screenshot file sizes, because I think 2.5 MB is way too large, and introduces recognizable delays when loading them, even on todays Internet's fast pipes. I usually use something around like this, using a combination of ImageMagick and oxipng, but I will be happy to learn more how to eventually improve the procedure. Other than this, I think reading flow and information conveyance can be tremendously improved by "zooming in to the feature at hand". As a reader, you can't recognize anything detailed anyway when looking at the full screenshot. Or is it just my bad eyes? On behalf a recent article, where I've demonstrated the Cloud Import feature, I just "zoomed in" to the import dialag, and I think it is much better like this. With kind regards, |
SStorm
left a comment
There was a problem hiding this comment.
A few minor points below 💯
Co-authored-by: Romanas Sonkinas <romanass@gmail.com>
Hi @amotl. Thank you for the suggestions. You're right, the sizes could be improved. I applied oxipng on all the pictures for an average 75% size reduction, so now coming in at around 0.5MB. I will do this for all the pictures in a separate PR and add it to my workflow. As for the Magick I'll go without it for now, hope that's okay. |
Summary of the changes / Why this is an improvement
Checklist