-
-
Notifications
You must be signed in to change notification settings - Fork 1k
Merge and optimize icon paths #1395
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
base: main
Are you sure you want to change the base?
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
ef91d43 to
889d591
Compare
889d591 to
50b5d04
Compare
|
@codecalm please review when you get a chance. |
|
@codecalm please review, and merge when you get a chance. |
|
@codecalm hello, please let me know what i can do to get this merged. happy to make changed to the PR if needed. |
|
This PR is stale because it has been open 45 days with no activity. Remove stale label or comment or this will be closed in 5 days. |
|
@codecalm would appreciate any kind of update or response on how I can unblock this. Thank you. |
Merge multiple SVG paths into single paths for outline icons
Fixes issue reported here: #1392
Summary
icons/outline/directorydattributesoptimize.mjs) to automatically merge paths for all future iconsChanges
1. Batch Processing of Existing Icons
Merged paths for all existing multi-path SVG files using a custom script. For example:
Before:
After:
2. Build Pipeline Enhancement
Modified
.build/optimize.mjsto automatically merge paths during the optimization process, ensuring:Benefits
Testing
optimize.mjs)Files Changed
iconswith merged paths.build/optimize.mjs- Added path merging logic to the build pipeline