Skip to content
This repository has been archived by the owner on Nov 21, 2017. It is now read-only.

Update to asciidoctor.js 1.5.2 and opal 0.6.3 #23

Merged
merged 1 commit into from
Feb 6, 2015

Conversation

jmini
Copy link
Contributor

@jmini jmini commented Feb 4, 2015

See issue #22

@@ -1,5 +1,6 @@
/* Asciidoctor default stylesheet | MIT License | http://asciidoctor.org */
@import "https://fonts.googleapis.com/css?family=Open+Sans:300,300italic,400,400italic,600,600italic|Noto+Serif:400,400italic,700,700italic|Droid+Sans+Mono:400";
/* Remove the comments around the @import statement below when using this as a custom stylesheet */
/*@import "https://fonts.googleapis.com/css?family=Open+Sans:300,300italic,400,400italic,600,600italic%7CNoto+Serif:400,400italic,700,700italic%7CDroid+Sans+Mono:400";*/
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This @import line should be uncommented when used in the Firefox add-on. The reason is the Firefox addon renders the document to embeddable HTML, which does not include the fonts. This is kind of a pain atm, but suffice to say, it's a necessary manual change.

@mojavelinux
Copy link
Member

Uncomment the @import line for the fonts and this will be good to go!

@ggrossetie
Copy link
Member

Thanks again @jmini 👍

@jmini
Copy link
Contributor Author

jmini commented Feb 5, 2015

If I understand the GitHub workflow correctly, I have forced pushed an amended commit on my master branch. This new commit contains the fix asked by @mojavelinux

@mojavelinux
Copy link
Member

mojavelinux commented Feb 5, 2015 via email

ggrossetie added a commit that referenced this pull request Feb 6, 2015
Update to asciidoctor.js 1.5.2 and opal 0.6.3
@ggrossetie ggrossetie merged commit c0ee323 into asciidoctor:master Feb 6, 2015
@ggrossetie
Copy link
Member

Perfect 🍻

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

Successfully merging this pull request may close these issues.

3 participants