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

turn katex on #202

Merged
merged 1 commit into from
Nov 25, 2021
Merged

turn katex on #202

merged 1 commit into from
Nov 25, 2021

Conversation

fmichonneau
Copy link
Contributor

@jhlegarreta following up from carpentries/carpentries-theme#22 (comment), to enable KaTeX, math: true must be added to the yaml header for the episode page. Thanks!

@netlify
Copy link

netlify bot commented Nov 24, 2021

✔️ Deploy Preview for carpentries-dmri ready!

🔨 Explore the source changes: 38f1195

🔍 Inspect the deploy log: https://app.netlify.com/sites/carpentries-dmri/deploys/619e4d57b0029a0007f85c42

😎 Browse the preview: https://deploy-preview-202--carpentries-dmri.netlify.app

@jhlegarreta
Copy link
Contributor

Thanks @fmichonneau . Not sure if it works as expected looking at how $mm^2/s$ is rendered:
https://deploy-preview-202--carpentries-dmri.netlify.app/diffusion_tensor_imaging/index.html

Is the ${math_expr}$ syntax the appropriate one?

@fmichonneau
Copy link
Contributor Author

What do you see? For me it looks like this:
Screenshot from 2021-11-25 10-39-00

@jhlegarreta
Copy link
Contributor

Ah, it must have been the browser cache: I do see the appropriate formatting now.

Merging. Thanks @fmichonneau 💯.

@jhlegarreta jhlegarreta merged commit 111d6de into main Nov 25, 2021
@jhlegarreta jhlegarreta deleted the add-katex branch November 25, 2021 13:46
@jhlegarreta jhlegarreta added the type:template and tools Issue about template and tools label Nov 25, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type:template and tools Issue about template and tools
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants