-
Notifications
You must be signed in to change notification settings - Fork 48
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
Showing glossary terms with an apostrophe won't work #833
Comments
Thanks for opening your first issue here! Engagement like this is essential for open source projects! 🤗 |
Yes, the reason that the reference breaks is because markdown-it (the underlying tokenizer) applies the smartquote transform, subtly changing the deflist identifier. #727 introduced this change. I've not bisected to ensure that this is the PR that breaks things, but it looks right. Pinging @rowanc1 (PR author) for ideas here. My guess is that we might want to lift the smart-quotes to the MyST level so that we preserve more of the original markup. |
Fixed in #1034. Thank you for the bug report! |
Description
Showing glossary terms that contain an apostrophe stopped working.
For example:
glossary.md
:When trying to reference it using:
Myst reports it as non-existing. This used to work before.
Asking around on Discord I was told to make an issue, as it "looks like a problem with converting to smart typography, which we did introduce a few months ago".
Proposed solution
Hehe.. let's not ask for the captain obvious here :)
Additional notes
The text was updated successfully, but these errors were encountered: