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

Example of syntax theming #136

Open
wants to merge 1 commit into
base: source
Choose a base branch
from
Open

Example of syntax theming #136

wants to merge 1 commit into from

Conversation

khwilson
Copy link
Contributor

This is how you'd do syntax theming if you'd like to choose a good one. :-)

cc #135 @aeturrell

@aeturrell
Copy link
Contributor

Nice, thanks @khwilson! Not sure if I'm doing it right, but when I bundle exec jekyll serve with this commit, the auto-regeneration that jekyll does seems to delete, eg, the github.css file from assets/css/ and so there is no change in syntax highlighting. I wonder if it's because there are already 'just the docs' css files in there that jekyll tries to protect. Did you manage to run this with the highlighting? And, if so, how? Worth saying that I've no idea what I'm doing with ruby or jekyll (despite using the latter for my own website...).

@khwilson
Copy link
Contributor Author

It definitely works for me locally with bundle exec jekyll build.

Perhaps try running bundle install first? The versions of things can really mess things up in ruby land.

@aeturrell
Copy link
Contributor

aeturrell commented Jun 6, 2021

Okay, spent a while fiddling with this and the problem I was having is nothing to do with the changes you've made. When I build the site, I get pages with giant symbols on. Odd because I'm sure I've built the site before. Tried locally, but also using codespaces with the 2.6.4 version of ruby. Anyway, I think this is my issue and we can ignore.

In general, fully support merging this in with eg the github or pastie themes!

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