-
Notifications
You must be signed in to change notification settings - Fork 129
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Motorbike icon added #2937
Motorbike icon added #2937
Conversation
🦋 Changeset detectedLatest commit: 6a408a6 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
packages/icons/manifest.json
Outdated
@@ -1868,6 +1868,11 @@ | |||
"category": "Product and feature", | |||
"size": "24" | |||
}, | |||
{ | |||
"name": "motorbike", | |||
"category": "Product and feature", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The icon is in the "Miscellaneous" category in Figma. Let's match that here (and re-order the entry accordingly).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Icons must use currentColor
instead of hard-coded hex values as the fill or stroke color (see the Contributing Icons docs).
Size Change: +9 B (0%) Total Size: 702 kB
ℹ️ View Unchanged
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #2937 +/- ##
=======================================
Coverage 87.60% 87.60%
=======================================
Files 228 228
Lines 13058 13058
Branches 1760 1760
=======================================
Hits 11440 11440
Misses 1561 1561
Partials 57 57 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the contribution! 🙌🏻 🏍️
Purpose
Motorbike icon is needed in the ze-dashboard but it is not in the design system library.
Approach and changes
Definition of done