-
Notifications
You must be signed in to change notification settings - Fork 16
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
Should now be able to use Material Design Icon font. #59
Conversation
Hey @maneex - thanks so much for the contribution! This looks great! I'm going to push a commit or two to your branch to touch up a few minor things, but ultimately this is awesome - thanks again! |
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.
Awesome, thanks so much for the contribution! There's a few non-related items that I'm hoping to merge in, then cut a new release!
The next release should be version 3.1.0
. Thanks again!
658f307
to
c631024
Compare
* Add attribution for the author, thanks @maneex * Add a CHANGELOG entry for Material Design Icon use. * Bump the version to 3.1.0 - this is a non-breaking change. * Update documentation in the README to include the `mdi` namespace. * Add a test that explicitly references an MDI icon.
c631024
to
42147c2
Compare
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.
Sorry for the force push - just bringing the branch up to date with master!
Should now be able to use Material Design Icon font. Co-authored-by: Perceval <[email protected]>
Should now be able to use Material Design Icon font. Co-authored-by: maneex <[email protected]>
Thank you for this @maneex |
This PR should allow to properly use Material Design Icon font (https://materialdesignicons.com/) with Prawn-icon.
Resolves: #58