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

resolves #121. Admonition icon customization by theme. #332

Merged
merged 4 commits into from
Oct 13, 2015

Conversation

jessedoyle
Copy link
Contributor

This patch allows for admonition icon customization via the pdf theme.

Here's an example of the theme syntax:

admonition:
  icon:
    tip:
      key: fa-arrows
      color: 0099FF
      size: 14

Supported icon keys currently are: key, color and size. If any of these are not specified, they will remain as the Asciidoctor defaults.

I haven't tested the patch to ensure that I haven't broken something, so it's probably best to double-check that everything works okay!

@mojavelinux
Copy link
Member

Excellent work @jessedoyle! I'm eager to try it out!

mojavelinux added a commit to mojavelinux/asciidoctor-pdf that referenced this pull request Oct 12, 2015
- change how admonition icon data is stored in theme
- properly evaluate and convert icon data values
- rename :key to :name in icon hash
- rename :color to :stroke_color in icon hash
- simplify and consolidate logic
mojavelinux and others added 2 commits October 12, 2015 17:40
- change how admonition icon data is stored in theme
- properly evaluate and convert icon data values
- rename :key to :name in icon hash
- rename :color to :stroke_color in icon hash
- simplify and consolidate logic
optimize custom admonition implementation. asciidoctor#332
@mojavelinux
Copy link
Member

This is fantastic! Such a great addition!

mojavelinux added a commit that referenced this pull request Oct 13, 2015
resolves #121. Admonition icon customization by theme.
@mojavelinux mojavelinux merged commit 8bb88aa into asciidoctor:master Oct 13, 2015
@jessedoyle jessedoyle deleted the custom-admonition branch October 13, 2015 04:44
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.

2 participants