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

Prefer Prism for code block highlighting #81

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

Conversation

elken
Copy link
Contributor

@elken elken commented Jan 5, 2024

The built-in implementation of Prism makes it awkward to add plugins, so
we disable it in favour of including our own bundled version of
prism.

These plugins clean up whitespace, adds selectable matching parens and
adds previews for things like CSS colours.

The first line of the prism.js file has the settings used to generate
the files.

Before
image

After
image

I've gone through all the blog posts and everything still seems to match, there's a couple of minor Kotlin issues which seem to relate to PrismJS/prism#3693, mostly type highlighting

I've intentionally left the bracket colouring the default so this is up for discussion on how we want to do it 😄

The built-in implementation of Prism makes it awkward to add plugins, so
we disable it in favour of including our own bundled version of
prism.

These plugins clean up whitespace, adds selectable matching parens and
adds previews for things like CSS colours.

The first line of the prism.js file has the settings used to generate
the files.
Copy link

netlify bot commented Jan 5, 2024

Deploy Preview for juxt-website ready!

Name Link
🔨 Latest commit 132995d
🔍 Latest deploy log https://app.netlify.com/sites/juxt-website/deploys/659821c73c5e180008673b09
😎 Deploy Preview https://deploy-preview-81--juxt-website.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@elken
Copy link
Contributor Author

elken commented Jan 5, 2024

I should add some docs on how to use everything too actually

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.

None yet

1 participant