Skip to content
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

Merged
merged 2 commits into from
Sep 1, 2022

Conversation

maneex
Copy link

@maneex maneex commented Aug 30, 2022

This PR should allow to properly use Material Design Icon font (https://materialdesignicons.com/) with Prawn-icon.

Resolves: #58

@jessedoyle
Copy link
Owner

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!

jessedoyle
jessedoyle previously approved these changes Sep 1, 2022
Copy link
Owner

@jessedoyle jessedoyle left a 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!

* 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.
Copy link
Owner

@jessedoyle jessedoyle left a 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!

@jessedoyle jessedoyle merged commit 7e9c640 into jessedoyle:master Sep 1, 2022
jessedoyle added a commit that referenced this pull request Sep 1, 2022
Should now be able to use Material Design Icon font.

Co-authored-by: Perceval <[email protected]>
jessedoyle added a commit that referenced this pull request Sep 1, 2022
Should now be able to use Material Design Icon font.

Co-authored-by: maneex <[email protected]>
@jessedoyle
Copy link
Owner

@maneex - I've published version 3.1.0 to RubyGems that includes this change.

Thanks again!

@markcraig
Copy link

Thank you for this @maneex

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support for Material Design Icons
3 participants