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

Improve color schemes for code highlighting #18

Closed
wants to merge 2 commits into from
Closed

Improve color schemes for code highlighting #18

wants to merge 2 commits into from

Conversation

jimporter
Copy link
Contributor

This improves the colors used for code highlighting so that they're more consistent with the themes. I tried to pick built-in highlight.js themes that roughly match each Bootswatch theme. I'll post screenshots tomorrow if you think it's necessary (but it'd be kind of tedious to capture all of them).

@d0ugal This should hopefully resolve some of the issues mentioned over in mkdocs/mkdocs#821.

Jim Porter added 2 commits June 28, 2016 23:53
We now use the following highlight.js themes:
- Amelia: Agate
- Cyborg: Dracula
- Slate: Darkula
We now use the following highlight.js themes:
- Cerulean: Color Brewer
- Cosmo: Purebasic
- Flatly: Docco
- Journal: Atelier Forest Light
- Readable: Mono Blue
- Simplex: Atelier Forest Light
- Spacelab: Color Brewer
- Yeti: Color Brewer
@d0ugal
Copy link
Member

d0ugal commented Jun 29, 2016

Thanks @jimporter. I'll try this out later tonight.

@jimporter
Copy link
Contributor Author

@d0ugal Have you had a chance to look at this?

@d0ugal
Copy link
Member

d0ugal commented Feb 12, 2017

No, sorry, I completely forgot. I'll try to take a look this week - please feel free to remind me in a few days if I don't reply here.

@jimporter
Copy link
Contributor Author

This is superseded by #29. (It seemed easier to just make a whole new branch instead of updating this one.)

@jimporter jimporter closed this Sep 23, 2017
@jimporter jimporter deleted the improved-code-colors branch September 23, 2017 21:25
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