Docs: Apply Styling to Graph#26154
Docs: Apply Styling to Graph#26154alexfornuto merged 3 commits intopaul.gottschling/14485-diagramfrom
Conversation
ptgott
left a comment
ptgott
left a comment
There was a problem hiding this comment.
Thanks for the edits! I think removing the "Public Network" label/box makes sense, as does the bidirectional arrow.
I'm not sure I like the yellow-on-purple coloring, though, and I would rather have a consistent stylesheet that we could apply to all Mermaid graphs so docs authors wouldn't need to worry about styling.
What would you think about reverting the coloring changes but keeping the rest?
Agreed. We could always change the yellow color.
Yes! If you like the coloring, we could add some classes to our main page style and call them in the mermaid charts. My other PR has them defined in the chart, but we could make them site-wide. In the meantime though, we can play around with the colors per guide, then standardize them once we have a master stylesheet.
Sure, unless you wanna think about any of the suggestions ^. |
* apply styling to graph * remove unused image
* Refresh Kubernetes Access Getting Started diagram Closes #14485 - Use a Mermaid diagram instead of an image (see #25883) - Update terms used in the diagram - Ensure that the name of the Teleport cluster is consistent between the diagram and the body text * Docs: Apply Styling to Graph (#26154) * apply styling to graph * remove unused image --------- Co-authored-by: Alex Fornuto <alex.fornuto@goteleport.com> Co-authored-by: Steven Martin <steven@goteleport.com>
* apply styling to graph * remove unused image
* apply styling to graph * remove unused image
* apply styling to graph * remove unused image
* Refresh Kubernetes Access Getting Started diagram Closes #14485 - Use a Mermaid diagram instead of an image (see #25883) - Update terms used in the diagram - Ensure that the name of the Teleport cluster is consistent between the diagram and the body text * Docs: Apply Styling to Graph (#26154) * apply styling to graph * remove unused image --------- Co-authored-by: Alex Fornuto <alex.fornuto@goteleport.com>
* Refresh Kubernetes Access Getting Started diagram Closes #14485 - Use a Mermaid diagram instead of an image (see #25883) - Update terms used in the diagram - Ensure that the name of the Teleport cluster is consistent between the diagram and the body text * Docs: Apply Styling to Graph (#26154) * apply styling to graph * remove unused image --------- Co-authored-by: Alex Fornuto <alex.fornuto@goteleport.com>
* Refresh Kubernetes Access Getting Started diagram Closes #14485 - Use a Mermaid diagram instead of an image (see #25883) - Update terms used in the diagram - Ensure that the name of the Teleport cluster is consistent between the diagram and the body text * Docs: Apply Styling to Graph (#26154) * apply styling to graph * remove unused image --------- Co-authored-by: Alex Fornuto <alex.fornuto@goteleport.com>
2 participants
Suggested edits to new mermaid graph in #25887.
Before:
After:
Also removes the now-unused image.