-
Notifications
You must be signed in to change notification settings - Fork 76
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
Option to use highlightjs for syntax highlighting #450
Comments
Sure, I would be OK with switching to highlightjs and dropping prettify. Are you in a position to suggest the necessary changes in a PR? |
Not yet - I wanted to check this would be well-received first before moving forward with the actual changes. Are you sure it is OK to drop |
I don't think it is a problem if the syntax colouring slightly changes. highlightjs seems to offer a vast set of predefined designs. Configuring that will be more useful than switching between prettify and highlightjs. |
OK thanks; that definitely lowers the bar for changes I'd have to make/test. I'll aim to open a PR for this in the next week or so. |
Going straight for highlightjs seems fine to me too. I'm not sure if the fiddle integration interacts with the syntax highlighting at all - that might be one thing to double-check. |
Switching to highlightjs sounds good to me too. Thanks @harpocrates |
The API for
highlightjs
seems to be pretty much the same asprettyify
. I think both are quite widely used. However,highlightjs
is actively maintained and developed whileprettyify
is no longer maintainedhighlightjs
supports more languagesWould it be feasible to add an option for choosing the syntax highlighting library to use?
The text was updated successfully, but these errors were encountered: