-
Notifications
You must be signed in to change notification settings - Fork 5
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
Shrink website/docs/assets/editor-screen.png ? #165
Comments
Oops, just saw this. I believe this shall resolve the issue. |
Closed by #155 (comment) |
Looks like 2f6c137 pulled in the 68 KB version from Flaki's comment, rather than the further reduction to 28 KB... since the asset is already there, might as well put in the Even Smaller one. |
Ah, I'd used that because I thought I'd understood from Flaki's comment that it was optimized for retina screens, so kind of a lim sup: the smallest size that would still look good on large/high res screens.
I'll definitely redo it if I misunderstood, ofc @callahad! |
Yep! The 28 KB version is exactly the same resolution as Flaki's (and thus equally suitable for retina screens), but compressed slightly more efficiently (8-bit pallet, etc.) |
Got i! |
See #155 (review):
I'm concerned that if it's smaller or lower res it'll be impossible to read. If it doesn't need to be easy to read, I could just remove it instead!
The text was updated successfully, but these errors were encountered: