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

Add Ayu theme #315

Merged
merged 1 commit into from
Jun 12, 2017
Merged

Add Ayu theme #315

merged 1 commit into from
Jun 12, 2017

Conversation

Cldfire
Copy link
Contributor

@Cldfire Cldfire commented Jun 6, 2017

Closes #223.

I finally got this done! Here is a comparison between the new theme (left) and the Navy theme (right):

comparison

The only modification I had to make to mdBook itself (beyond the addition of the theme throughout the codebase, of course) was to introduce a variable so that I could change inline code's text color. If you don't want this it can certainly be removed (although I feel that it makes the theme more readable).

And finally, to be clear: if you don't feel like mdBook should ship with this theme, feel free to say so. I'm not trying to force it on anyone :)

Also adds the a new variable, `$inline-code-color`, to base.styl. The `Ayu` theme needed this to change the text color of inline code.
@azerupi
Copy link
Contributor

azerupi commented Jun 6, 2017

Looks very nice! Thank you 😊

I will take a look at this PR when I have more time, but from a quick glance through the changes everything looks fine to merge.

@azerupi azerupi merged commit 44efc65 into rust-lang:master Jun 12, 2017
@azerupi
Copy link
Contributor

azerupi commented Jun 12, 2017

Merged. Thanks a lot!

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