-
Notifications
You must be signed in to change notification settings - Fork 201
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 markdown support for diagrams using js-sequence style blocks #228
Comments
Isn't this something that could be done (or would have to be done) as a post processing? Like the js-sequence library would, in the browser, find HTML like |
@srawlins I don't know enough javascript to know without some research, but that doesn't sound unreasonable. In that case all markdown would need to do is generate that when it sees the appropriate markdown and leave it to the js-sequence library? |
Yeah, I think Markdown provides good enough support for this out-of-the-box:
So the js-sequence library could work on that output. |
Mermaid might be the new way to do this. https://github.blog/2022-02-14-include-diagrams-markdown-files-mermaid/ Supported on GitHub out of the box. |
I created a interop wrapper package for mermaid.js and have submitted a PR to illustrate the changes needed to the markdown example code to support mermaid diagrams. Here is an example illustrating a number of Mermaid diagram types See PR #429 for more details. Mermaid was generally impressive, and in conjunction with Markdown seems to make a fast combo. You can see how fast real time editing in the example above is, even with an example of every diagram type re-rendering with every key stroke. |
Allowing inline diagram support from comment text to markdown (and indirectly, dartdoc), would be a nice thing to have for doc writers, so they don't have to have opaque image links in the source code when a simple diagram would suffice.
https://support.typora.io/Draw-Diagrams-With-Markdown
https://github.com/bramp/js-sequence-diagrams
The text was updated successfully, but these errors were encountered: