-
Notifications
You must be signed in to change notification settings - Fork 3.2k
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
fix(docs): render Mermaid diagrams in docs #12464
Conversation
- `docs/retries.md` has a Mermaid diagram that renders out-of-the-box on GH but not on the docs site itself - add a `pymdownx.superfences` config to render out Mermaid diagrams in `mkdocs` per https://squidfunk.github.io/mkdocs-material/reference/diagrams/#configuration follow-up to 310bb5a Signed-off-by: Anton Gilgur <agilgur5@gmail.com>
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.
I am curious - is this reactive? Can you try resizing your browser?
I think you mean responsive. Technically speaking, yes, it's a percentage of the screen: That's a variant of responsive design (it used to be called adaptive design back in the day (~2014), but that now means something different). In full responsive design, you may want to render a different image for different screen sizes. GitHub has the fancy controls and mouse gestures that let you zoom and scroll the diagram which this does not. Those are nice, but I think this is good enough. Users can click the image to see it full screen if they want to and then zoom from there etc |
Great thanks! Yes I meant responsive. |
Rendering properly on RTD now 🙂 : https://argo-workflows.readthedocs.io/en/latest/retries/#retry-policies |
Includes fixes from argoproj#12464, argoproj#12466, argoproj#12465 - all authored by @agilgur5 Includes files list from argoproj#12414 description Signed-off-by: jmeridth <jmeridth@gmail.com> Co-authored-by: Anton Gilgur <agilgur5@gmail.com>
Includes fixes from argoproj#12464, argoproj#12466, argoproj#12465 - all authored by @agilgur5 Includes files list from argoproj#12414 description Signed-off-by: jmeridth <jmeridth@gmail.com> Co-authored-by: Anton Gilgur <agilgur5@gmail.com>
Includes fixes from argoproj#12464, argoproj#12466, argoproj#12465 - all authored by @agilgur5 Includes files list from argoproj#12414 description Signed-off-by: jmeridth <jmeridth@gmail.com> Co-authored-by: Anton Gilgur <agilgur5@gmail.com>
Signed-off-by: Isitha Subasinghe <isubasinghe@student.unimelb.edu.au>
Motivation
docs/retries.md
has a Mermaid diagram that renders out-of-the-box on GH but not on the docs site itselfmkdocs
follow-up to #11005, #11390 (comment), and Nov 21st Contributor Meeting
Modifications
pymdownx.superfences
config to render out Mermaid diagrams inmkdocs
per https://squidfunk.github.io/mkdocs-material/reference/diagrams/#configurationVerification
Ran
make docs
followed byopen site/retries/index.html
to view in the browser.Screenshots:
Before:
After: