Skip to content

Optimize Images#3539

Open
ann0see wants to merge 1 commit intojamulussoftware:mainfrom
ann0see:imgbot
Open

Optimize Images#3539
ann0see wants to merge 1 commit intojamulussoftware:mainfrom
ann0see:imgbot

Conversation

@ann0see
Copy link
Copy Markdown
Member

@ann0see ann0see commented Sep 17, 2025

Small improvements in image sizes (non critical and only parts should be applied in prod e.g. SVG should not be minified). The shrinkage is minimal - but imo still worth it. We'll need to shrink some of the images - but that's for a later day, I think.

Done by ImgBot.

Changelog: Optimize images for slightly smaller binary size.

Co-authored-by: ann0see <20726856+ann0see@users.noreply.github.com>
@ann0see
Copy link
Copy Markdown
Member Author

ann0see commented Apr 10, 2026

Since this is a small change, I'd like to get it into the upcoming release, if possible. @softins @pljones ?

@ann0see ann0see moved this from Triage to Waiting on Team in Tracking Apr 10, 2026
@ann0see ann0see marked this pull request as ready for review April 10, 2026 20:35
@pljones
Copy link
Copy Markdown
Collaborator

pljones commented Apr 11, 2026

The byte saving looks trivial.

The colours have changed in some cases, too. Are they still okay in Fancy skin and normal?

(I've no idea why the LEDs need to be that big -- even on high definition displays, given the size of the other images.)

@ann0see
Copy link
Copy Markdown
Member Author

ann0see commented Apr 11, 2026

I think they are just the original files. So it's probably fine.

I'll check the colors later.

@softins
Copy link
Copy Markdown
Member

softins commented Apr 12, 2026

Since this is a small change, I'd like to get it into the upcoming release, if possible. @softins @pljones ?

I have no objection in principle, but I can't check anything as I am away from home for the next week.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Waiting on Team

Development

Successfully merging this pull request may close these issues.

4 participants