jessedoyle / prawn-icon

Easy icons for Prawn.
Other
28 stars 15 forks source link

Support for material icons #57

Closed markcraig closed 1 year ago

markcraig commented 1 year ago

We plan to use Antora Assembler, which relies on asciidoctor-pdf to generate PDFs, which in turn relies on prawn-icon for icon support.

We are increasingly using material icons in our UIs and therefore in our docs.

It would be great if prawn-icon could add support for material icons to make them easy to use when creating PDFs.

mojavelinux commented 1 year ago

If this proposal is accepted, I suggest using the icon prefix "mi". This follows the naming pattern used for foundation icons.

jessedoyle commented 1 year ago

Hey @markcraig! Thanks for using prawn-icon. I would absolutely support a PR that implements material icons into the library.

If this proposal is accepted, I suggest using the icon prefix "mi". This follows the naming pattern used for foundation icons.

Definitely agree!

Looks like Google uses the Apache License which should be fine for distribution in this library - so no concerns there!

I'd be happy to take a stab at this over the next month or two. If anyone wants to submit a PR, the primary AC would be:

Hope this helps - thanks!

mojavelinux commented 1 year ago

I appears that this was implemented, merged, and released. Is that correct? If so, I would recommend closing this issue to avoid confusion.

markcraig commented 1 year ago

Great news! Looking forward to being able to use these in asciidoctor-pdf going forward

jessedoyle commented 1 year ago

We did cut release 3.1.0 last week that includes support for Material Design Icons.

It's worthwhile to note that the Material Design Icons collection includes the base icon set from Google as well as a large number of community-maintained icons. The full set can be viewed here.

With that being said, I'm going to close this issue. Thanks!