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

docs: updated images #11

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

docs: updated images #11

wants to merge 4 commits into from

Conversation

kremzli
Copy link
Contributor

@kremzli kremzli commented Dec 11, 2024

Fixes #10
One problem is they are big. And i dont know how you guys make them tiny. I tried full effort optipng and oxipng before doing full effort (-m 6) lossless webp.
But these new images show off the new (if #9 merged) background image (which is indeed part of the app already)

@kremzli kremzli changed the title Updates images Updated images Dec 11, 2024
@kremzli kremzli changed the title Updated images docs: updated images Dec 11, 2024
@iruzo
Copy link
Member

iruzo commented Dec 12, 2024

@kremzli

The way we upload images to the repo is using webp. There is no problem if they are a little big.

I will merge this as soon as #9 is merged, thanks for the PR !

@kremzli
Copy link
Contributor Author

kremzli commented Dec 12, 2024

They are webp, but they are still around 1.2mb while the ones currently in the repo are a bit over 200kb at similar resolution. So maybe they need a bit of compression magic that i dont know. Or are they not supposed to be lossless?

@iruzo
Copy link
Member

iruzo commented Dec 12, 2024

Maybe you can try with ImageMagick, that is basically the tool most of us use to transform from PNG to webp. You can specify even the quality like so:

magick wizard.png -quality 50 -define webp:lossless=true wizard.webp

This is the command I personally use to turn multiple images into webp:

for file in *.png; do convert -quality 100 "$file" "${file%.png}.webp"; done

@kremzli
Copy link
Contributor Author

kremzli commented Dec 13, 2024

I cant get them lower in size. Even after optimizing the PNGs (with oxipng -Z --fast) and then using full effort lossless webp. I'll keep looking into further optimizing but for now this is as low as they seem to go.

@kremzli
Copy link
Contributor Author

kremzli commented Dec 14, 2024

So I somehow didn't consider the very HD cat picture in the images that perhaps obviously wouldn't compress well with lossless. So I will make new screenshots without the kitty unfortunately. Either late Sunday or Monday.

@kremzli
Copy link
Contributor Author

kremzli commented Dec 15, 2024

Now it should be acceptable maybe. Another thing to consider is if the UI by default is on the bottom, but it shouldn't matter much.

@kremzli
Copy link
Contributor Author

kremzli commented Dec 15, 2024

Now this is as small as they will probably go.
Maybe squash this PR

@iruzo
Copy link
Member

iruzo commented Dec 16, 2024

Perfect, thanks for the optimization, I will merge this as soon as #9 is merged.

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.

Update images
2 participants