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

Reduce logo sizes even more #9211

Merged
merged 2 commits into from
Jan 2, 2024

Conversation

mmlb
Copy link
Contributor

@mmlb mmlb commented Jan 2, 2024

While going through commits to master for my favorite editor I noticed #9106 and wondered if the relatively-new-to-me-svg-optimizer svgo was used for this or not, it was not. SVGO is pretty neat and considering the logo.svg is "a minified file we're basically treating as binary" I figured might as well go all the way and also do the other svg files in the repo. So here is my tiny drive by contribution.

Before:

wc -c *.svg
 2665 logo.svg
 7253 logo_dark.svg
 6922 logo_light.svg
16840 total

After:

wc -c *.svg
1554 logo.svg
3073 logo_dark.svg
2981 logo_light.svg
7608 total

A >50% reduction.

While reading through commits to helix I saw
helix-editor#9106 and wondered if the
relatively-new-to-me svgo would do better than -95B diff, indeed it
does. Seeing as this file is "a minified file we're basically treating
as binary" anyway I figured might as well minify it further.
@pascalkuthe pascalkuthe merged commit efc4865 into helix-editor:master Jan 2, 2024
6 checks passed
@mmlb mmlb deleted the reduce-logo-size-even-more branch January 2, 2024 22:49
woojiq pushed a commit to woojiq/helix that referenced this pull request Jan 13, 2024
* Reduce logo.svg even more

While reading through commits to helix I saw
helix-editor#9106 and wondered if the
relatively-new-to-me svgo would do better than -95B diff, indeed it
does. Seeing as this file is "a minified file we're basically treating
as binary" anyway I figured might as well minify it further.

* Minimize all the svg logos
dgkf pushed a commit to dgkf/helix that referenced this pull request Jan 30, 2024
* Reduce logo.svg even more

While reading through commits to helix I saw
helix-editor#9106 and wondered if the
relatively-new-to-me svgo would do better than -95B diff, indeed it
does. Seeing as this file is "a minified file we're basically treating
as binary" anyway I figured might as well minify it further.

* Minimize all the svg logos
mtoohey31 pushed a commit to mtoohey31/helix that referenced this pull request Jun 2, 2024
* Reduce logo.svg even more

While reading through commits to helix I saw
helix-editor#9106 and wondered if the
relatively-new-to-me svgo would do better than -95B diff, indeed it
does. Seeing as this file is "a minified file we're basically treating
as binary" anyway I figured might as well minify it further.

* Minimize all the svg logos
Vulpesx pushed a commit to Vulpesx/helix that referenced this pull request Jun 7, 2024
* Reduce logo.svg even more

While reading through commits to helix I saw
helix-editor#9106 and wondered if the
relatively-new-to-me svgo would do better than -95B diff, indeed it
does. Seeing as this file is "a minified file we're basically treating
as binary" anyway I figured might as well minify it further.

* Minimize all the svg logos
smortime pushed a commit to smortime/helix that referenced this pull request Jul 10, 2024
* Reduce logo.svg even more

While reading through commits to helix I saw
helix-editor#9106 and wondered if the
relatively-new-to-me svgo would do better than -95B diff, indeed it
does. Seeing as this file is "a minified file we're basically treating
as binary" anyway I figured might as well minify it further.

* Minimize all the svg logos
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.

3 participants