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 sizes of ten icons #646

Merged
merged 10 commits into from
Apr 24, 2022
Merged

Reduce sizes of ten icons #646

merged 10 commits into from
Apr 24, 2022

Conversation

Eiim
Copy link
Collaborator

@Eiim Eiim commented Apr 17, 2022

  • Messenger 824 -> 699 bytes
  • Open Source 261 -> 243 bytes
  • Typescript 1014 -> 623 bytes
  • Laravel 523 -> 404 bytes
  • Jellyfin 1003 -> 676 bytes
  • Gojek 376 -> 299 bytes
  • SOGo 1021 -> 851 bytes
  • Codeberg 837 -> 597 bytes
  • Udemy 537 -> 306 bytes
  • StumbleUpon 598 -> 364 bytes

Total bytes saved: 1,934

Also fixes #547 and makes #548 redundant (has further optimizations than in #548)

I made a quick tool to help with some of these by changing the start node, which is fairly difficult to do with other tools like https://yqnn.github.io/svg-path-editor/ or Inkscape. It's still slightly buggy, but it mostly works: https://eiim.github.io/SVGPathStartMover/

If I do more groups of these optimizations in the future, let me know if I should do it differently in any way, like if you want them all as separate PRs, or if I should only change the README once at the end, or some such.

@edent
Copy link
Owner

edent commented Apr 24, 2022

Amazing! Keep doing what you're doing :-)

I'm granting you commit access to the Repo. This is per the advice of The Pull Request Hack.

Please use this privilege wisely.

@edent edent merged commit f7aba65 into edent:master Apr 24, 2022
@Eiim
Copy link
Collaborator Author

Eiim commented Apr 24, 2022

Many thanks! I'll certainly be sure to use the privilege with care.

@Eiim Eiim deleted the assorted-golfing branch April 24, 2022 18:32
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.

jellyfin.svg contains useless commands
2 participants