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

Update syntax highlighting #254

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

Update syntax highlighting #254

wants to merge 1 commit into from

Conversation

dj2
Copy link
Contributor

@dj2 dj2 commented Sep 14, 2024

This Cl switches to the rouge highlighter which matches the Vulkan spec. The rouge-extended-css.rb file is copied in from the Vulkan-Docs repo.

The code examples are updated to use source,glsl instead of source,c++. This means when mousing over the code blocks in the HTML version the syntax will correctly be identified as GLSL.

This Cl switches to the `rouge` highlighter which matches the Vulkan
spec. The `rouge-extended-css.rb` file is copied in from the Vulkan-Docs
repo.

The code examples are updated to use `source,glsl` instead of
`source,c++`. This means when mousing over the code blocks in the HTML
version the syntax will correctly be identified as GLSL.
@gnl21
Copy link
Contributor

gnl21 commented Sep 30, 2024

Consistency with Vulkan is good and having the code identified correctly in the HTML is also good. Thanks for doing this.

When I build with this branch, though, the highlighting doesn't look as good as in the Vulkan spec and not quite as good as without this change (too many bold typefaces and bright colours). I'll try to track down the difference and we can merge this. Do you get consistency with the Vulkan spec when you build from this branch?

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