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

📑 Use glossarium to support glossaries in typst #1286

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

agoose77
Copy link
Contributor

@agoose77 agoose77 commented Jun 11, 2024

This PR adds a glossary handler to the myst-to-typst package. I would like to have used the existing definitionList handling logic, but it's not possible to naively label the keys. Glossarium works around this by embedding each entry in a figure.

Right now this generates #gls("term") syntax instead of @term. This should be pretty simple to do, we just need to normalise the identifier.

This PR encodes the assumption that the identifier of a term starts with term. Can we assume this? There is a fallback in any case.

Fixes #944

@agoose77 agoose77 requested a review from rowanc1 June 11, 2024 13:16
@agoose77
Copy link
Contributor Author

@rowanc1 can you remember what we concluded here about next steps?

@rowanc1
Copy link
Collaborator

rowanc1 commented Jun 14, 2024

Can we hold off on merging this for about a month? We are going to do significant work on Typst exports for SciPy in July, I would like to tackle it then.

If you want to merge in before, I think putting it behind a feature flag would be the addition I would want to see!

@agoose77 agoose77 added the blocked Waiting for another task to be compled label Jun 14, 2024
@agoose77
Copy link
Contributor Author

agoose77 commented Aug 9, 2024

@rowanc1 we spoke about reducing the scope of this. Now that SciPy is over, can we just merge as-is?

@rowanc1
Copy link
Collaborator

rowanc1 commented Aug 9, 2024

We are starting to build out the SciPy Typst themes now. @fwkoch merged the first of that earlier this week. Hoping will come back to this over the next few weeks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
blocked Waiting for another task to be compled
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Glossary handling not implemented for Typst
2 participants