Skip to content

Conversation

@borkdude
Copy link
Collaborator

@borkdude borkdude commented Nov 24, 2025

Allow setting {:nextjournal.clerk/markdown {:disable-inline-formulas true}} as ns metadata to disable markdown parsing of $-delimited inline formulas. Fixes #775.

@borkdude
Copy link
Collaborator Author

borkdude commented Dec 1, 2025

@mk This looks pretty good imo. I will bump markdown in bb and then the tests should work in bb too. Do we need to support markdown config in other places, e.g. via a system property? Or one system property with a clerk config to rule them all?

@mk
Copy link
Member

mk commented Dec 3, 2025

Do we need to support markdown config in other places, e.g. via a system property? Or one system property with a clerk config to rule them all?

I need to think more about this. Would leave this pr to only add the notebook metadata. We can start collecting use cases for a global clerk config in an issue and then think about how to solve it separately.

@borkdude
Copy link
Collaborator Author

borkdude commented Dec 4, 2025

We can start collecting use cases for a global clerk config in an issue and then think about how to solve it separately.

We probably don't even need an issue, we can wait for people to create one after they want more than just the notebook annotation.

Restricting to just notebook metadata for now sounds good.

@mk mk changed the title Fix #775: option to disable inline formulas in markdown add option to disable inline formulas in markdown Dec 8, 2025
@mk mk merged commit d843da4 into main Dec 8, 2025
14 checks passed
@mk mk deleted the markdown-options branch December 8, 2025 23:11
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.

Dollar signs in comments incorrectly trigger TeX rendering

3 participants