Skip to content

improved compression on png images, saving 1.15kb#121

Merged
RigleGit merged 19 commits into
RigleGit:v2.7.0from
Rhlp-Engineering:main
May 22, 2026
Merged

improved compression on png images, saving 1.15kb#121
RigleGit merged 19 commits into
RigleGit:v2.7.0from
Rhlp-Engineering:main

Conversation

@Rhlp-Engineering

Copy link
Copy Markdown
Contributor

this is my 5th time trying to submit this i dont understand githubb!!

anyways, i recompressed a few of the ui icons losslessly, meaning, they have the same name, and resolution, but are smaller. here! look!

home heres the old home icon, and here is my new smaller home icon!
home

i dont understand github im so sorry.

i have no way to test if this will cause issues, but because the files are the same size, and in the same place, it shouldent.

id also suggest getting rid of the back arrow, and replace it with a flipped forward arrow.

ok i think i did it right.

these icons are drastically smaller than the versions now in use. totalling a total change of  1.78kb less file size, wich means JUSSTT a bit more space for doing other things!
@Rhlp-Engineering

Copy link
Copy Markdown
Contributor Author

YES IT WORKED I SUBMITTED A THING YES!

@RigleGit

Copy link
Copy Markdown
Owner

I'll check it and add it to the next version! Thanks! 👍🏻

@RigleGit RigleGit added this to the v2.7.0 milestone May 21, 2026
@RigleGit RigleGit changed the base branch from main to v2.7.0 May 21, 2026 21:08
@Rhlp-Engineering

Copy link
Copy Markdown
Contributor Author

woaw thank you! do you mind if i keep finding little things? because i have a few text documents that are around 1.5mb, and if we can save that much space in ram, you could possibly preload a few more pages, and if you can do that, everything feels faster!

@RigleGit RigleGit changed the title improved compression on png images, saving 1.15mb improved compression on png images, saving 1.15kb May 22, 2026
@RigleGit

Copy link
Copy Markdown
Owner

Hey, thanks for the contribution! The size improvement is really small (I corrected the title from 1.15MB to around 1.15 KB). That's why I didn't priorize this optimization, because It doesn't make a meaningful differente in the app at runtime.

However, thanks for the improvement, everything counts!

Before I merge this, could you please remove the probably accidental assets/ui/icons/svg/ieuudr.txt file from the PR?

Once that’s cleaned up, I’m happy to merge it.

@Rhlp-Engineering

Copy link
Copy Markdown
Contributor Author

i dont know how toooo
i made that by accedent sorry

@Rhlp-Engineering

Copy link
Copy Markdown
Contributor Author

i think i did it?

@Rhlp-Engineering

Copy link
Copy Markdown
Contributor Author

could you rename it? because with the jpeg changes, i save an additional 107kb

@Rhlp-Engineering

Copy link
Copy Markdown
Contributor Author

ok i think everything is in order. and yeah it dosent save mutch, but i like compressing stuff so it's fun for me! and small improvements add up probably, thank you!!

@RigleGit

Copy link
Copy Markdown
Owner

could you rename it? because with the jpeg changes, i save an additional 107kb

That's just for the readme, not the code itself. Merging it for the next version!

@RigleGit RigleGit left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@RigleGit RigleGit merged commit 34a9723 into RigleGit:v2.7.0 May 22, 2026
5 checks passed
@Rhlp-Engineering

Copy link
Copy Markdown
Contributor Author

thank you soo much! im glad i could help! even tho it wasent much of a change.
thank you!!!

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.

2 participants