Skip to content

Conversation

@empeig
Copy link
Contributor

@empeig empeig commented Oct 30, 2025

Thank you for your contribution! Before sending this Pull Request, please confirm the following:

  • I have read the contributing guidelines
  • I have run python check.py and received a ✅
  • The icon's size is under 1,024 Bytes
  • The layout of the SVG looks like this including newlines
<svg xmlns="http://www.w3.org/2000/svg"
aria-label="..." role="img"
viewBox="0 0 512 512"><path
d="m0 0H512V512H0"
fill="#fff"/> ... </svg>

If you have done the above, please send the Pull Request.

@empeig empeig mentioned this pull request Oct 30, 2025
@Eiim
Copy link
Collaborator

Eiim commented Oct 30, 2025

You can definitely reduce the file size by reducing the precision. Rounding to one decimal point is plenty fine, and you might be able to do 0 in places. I also wonder if it wouldn't be easier and smaller to incorporate the ellipse in the bucket path? It can be replicated with two arc commands.

@empeig
Copy link
Contributor Author

empeig commented Oct 30, 2025

Thank you for the feedback. I'll try to optimize a bit

@romain-dartigues
Copy link
Collaborator

You also can shrink a few bytes by using properties without the style; ie:
stroke-width="12" stroke="#fff" fill="none" instead of style="stroke-width:12px;stroke:#fff;fill:none" → saves 4

About the arcs commands empeig mentioned, you should give a try to the excellent https://yqnn.github.io/svg-path-editor/

@empeig empeig marked this pull request as draft November 3, 2025 10:22
Copy link
Contributor Author

@empeig empeig left a comment

Choose a reason for hiding this comment

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

I took into account all your suggestions:

  • replaced the ellipse by two arcs. It is difficult to have the same shape as the original which uses 4 arcs but I think it is close enough. See this file for reference (red is the original svg, white the SuperTiny svg shape):
    EditV2

  • Rounding to one decimal

  • Removed the style from properties

@empeig empeig marked this pull request as ready for review November 3, 2025 13:23
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