-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
[convertPath]: combine arcs and convert full arcs to circles #1748
Comments
@SethFalco I have a working version here, code is not in the shape svgo expects and a bit of a mess tho. Example: https://lucide-site-git-feat-added-edit-page-jguddas.vercel.app/edit |
@KTibow Looks like you found a bug, did a quick test, that bug only exists in the deployed version. I haven't had the time to clean things up and deploy the newest version, I can reproduce it on the hosted version but locally (with the code in the branch that I have linked) it works. |
There is a trick to do circles with one arc instead of two arcs by making a tiny step like 0.01 pixel. It makes even shorter paths than circle. |
Is your feature request related to a problem? Please describe.
There is already logic that merges curves into arcs, but it only works with curves and not a mix of curves and arcs.
Describe the solution you'd like
rx == ry
to curves before running the curve merge logicAdditional context
Also, 2 half circles could be converted to a circle element.
This could use the same logic, but might be better placed in a separate plugin.
The text was updated successfully, but these errors were encountered: