-
-
Notifications
You must be signed in to change notification settings - Fork 7k
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
Feature/support fa kit custom icons #5430
Feature/support fa kit custom icons #5430
Conversation
✅ Deploy Preview for mermaid-js ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #5430 +/- ##
=======================================
Coverage 5.74% 5.74%
=======================================
Files 277 276 -1
Lines 41898 41886 -12
Branches 489 514 +25
=======================================
Hits 2407 2407
+ Misses 39491 39479 -12
Flags with carried forward coverage won't be shown. Click here to find out more.
|
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.
Changes looks good.
Can you also move the replace function into utils, so that we can avoid duplicating the logic 3 times?
Will do 👍 |
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.
Nice work @jakobskrym 🚀
Co-authored-by: Sidharth Vinod <[email protected]>
@jakobskrym, Thank you for the contribution! |
@sidharthv96 any idea when this will make it out to production? Saw that it wasn't included in the latest release 🧐 |
Another ping regarding this @sidharthv96 - really want to start using it in our projects 🙏 |
📑 Summary
Extends the current fontawesome support to also handle the case of custom icons being served through the
fak
prefix.📏 Design Decisions
Logic already existed for the basic fontawesome prefixes so mainly extended that and added a section to the flowchart documentation about it.
📋 Tasks
Make sure you
MERMAID_RELEASE_VERSION
is used for all new features.develop
branch