Skip to content

Resource optimisations 2#5718

Open
otsako-vpk wants to merge 1 commit into
bloxstraplabs:mainfrom
otsako-vpk:optimize-resources
Open

Resource optimisations 2#5718
otsako-vpk wants to merge 1 commit into
bloxstraplabs:mainfrom
otsako-vpk:optimize-resources

Conversation

@otsako-vpk
Copy link
Copy Markdown
Contributor

@otsako-vpk otsako-vpk commented Aug 17, 2025

This PR saves an entire ~2MB!!!!! (we take what we can get)
As with the previous PR about this, the icon fonts inside of WPF.UI are about 25% of Bloxstrap's file size. They cannot be changed to a woff2 without it breaking. (it would've saved ~2MB too, what a shame).
Related PR: bloxstraplabs/wpfui#9

I've saved the branch before the force-push just incase.

Comment thread Bloxstrap/Bloxstrap.csproj
Comment thread Bloxstrap/Resources/Mods/Cursor/From2006/ArrowCursor.png
@otsako-vpk
Copy link
Copy Markdown
Contributor Author

otsako-vpk commented Aug 17, 2025

The FiraCode font inside of WPF.UI can be changed into a woff2, which would save about 190KB. Should I make a separate PR for that in WPF.UI?

The above does not change anything when compiled as the font never gets used. I'm dumb.

Comment thread Bloxstrap/Resources/Icon2022.ico
@otsako-vpk otsako-vpk force-pushed the optimize-resources branch from d79a1c2 to f09ec04 Compare March 1, 2026 11:23
@otsako-vpk
Copy link
Copy Markdown
Contributor Author

@bluepilledgreat Could you please add this to the main branch so I can include it in this PR? That'd be great.

@exurd
Copy link
Copy Markdown

exurd commented Mar 1, 2026

@/bluepilledgreat Could you please add this to the main branch so I can include it in this PR? That'd be great.

See #5621 (comment).

@otsako-vpk
Copy link
Copy Markdown
Contributor Author

@/bluepilledgreat Could you please add this to the main branch so I can include it in this PR? That'd be great.

See #5621 (comment).

I linked that comment myself.. (I was referring to the local branch in question.)

@exurd
Copy link
Copy Markdown

exurd commented Mar 1, 2026

Weird, it looked like it was linked to a different part of the conversation. Good to know.

@bluepilledgreat
Copy link
Copy Markdown
Member

@bluepilledgreat Could you please add this to the main branch so I can include it in this PR? That'd be great.

I'll look into this for the next update after 2.11.0. Going through all the PRs now.

@otsako-vpk
Copy link
Copy Markdown
Contributor Author

@ItzBloxxy #6405 (comment): Bad idea, cuz it doesn't natively support .webp, also it's better for websites and mobile apps not apps like this.

WebP works just fine as a static image without transparency since Windows 10 1809. Your second statement is also false, it's like saying HEIF is exclusive to iPhones. Please do more thorough research before making a comment.

@ItzBloxxy
Copy link
Copy Markdown

Like i said, WPF does not natively support WebP.

@ItzBloxxy
Copy link
Copy Markdown

Also android phones also has HEIFs.

@otsako-vpk
Copy link
Copy Markdown
Contributor Author

Like i said, WPF does not natively support WebP.

WPF relies on WIC to handle images, which supports WebP since Windows 10 1809. The codec required is pre installed on both Windows 10 and 11. We shouldn't account for custom Windows isos if that's what you're getting at. Have you tried building the PR locally to check if it's broken?

Also android phones also has HEIFs.

That's exactly my point?

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.

4 participants