-
Notifications
You must be signed in to change notification settings - Fork 18
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
Add minimal glossary #185
Add minimal glossary #185
Conversation
for more information, see https://pre-commit.ci
Co-authored-by: Bradley Dice <[email protected]>
Co-authored-by: Corwin Kerr <[email protected]>
Co-authored-by: Corwin Kerr <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this meant to cover all uses of the glossary terms?
@cbkerr you need to ensure that the readthedocs environment has the hoverxref plugin. |
for more information, see https://pre-commit.ci
Yes, these terms shouldn't have multiple definitions. I tried to pick the ones that had most settled in the google doc. |
I more meant, that many of the terms do not use hoverxref in the rest of the documentation. |
Description
My goal is to define the terms for signac-core "good enough" to get started to keep this moving.
I think the new hover extension is worth it to allow people to see the definition when hovering.
Motivation and Context
#59
Work continues at https://docs.google.com/document/d/1_merhcK3ohas4IloE616yL7gypMRFcaQLh2oChExC7o
Checklist: