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

Fix syntax hightlighting #55

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

ire4ever1190
Copy link

@ire4ever1190 ire4ever1190 commented Dec 2, 2022

Github API uses pygment so we need pygment compatible CSS for syntax highlighting which this PR adds.

Example of syntax highlighting for taskman

image

The colours are from Nims normal syntax highlighting colours

This removes unneeded static files (hightlight.js and friends)

@FedericoCeratto
Copy link
Owner

Thanks!

@ire4ever1190 ire4ever1190 changed the title Fix syntax hightlighting and static files Fix syntax hightlighting Mar 19, 2023
@ire4ever1190
Copy link
Author

ire4ever1190 commented Mar 19, 2023

Latest changes now merged in 👍

Just need to do a few fixes for light theme

@FedericoCeratto
Copy link
Owner

@ire4ever1190 thanks. Can you please rebase on master now?

Stops using symlinks. Wondered why I kept getting 404 during development

Add Nim to highlight
This might not work for everything, but can easily add more colours in future
Make its look more inline with github rendered readme
Fix issues with colours not being readable on light theme
@ire4ever1190
Copy link
Author

Alright, finished rebasing 👍

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

Successfully merging this pull request may close these issues.

None yet

2 participants