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

Proposal: add mermaid support #47

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

aigjermo
Copy link

@aigjermo aigjermo commented Nov 2, 2018

Solves #34.

This is just the bare minimum to get it working, I haven't added any config options to opt out or anything like that.

@eastwood
Copy link

This is a decent PR. I actually just came here to implement this and was happy to see that someone beat me to it. previm supports mermaid by default and without config options as well. This would be nice to see in master

@titanism
Copy link

titanism commented Nov 3, 2023

Can we please have this merged and released to npm @shime ?

@titanism
Copy link

titanism commented Nov 3, 2023

Actually this PR is broken. It does not work.

The reason why is because 0e8b97f#diff-2f062de8635f61fbec7693ba8f7bdc5dd196f7e694240826c4b304a1e83baff3 is referencing a file path. You cannot do this. The actual file must be included, similar to Zepto.

@shime if you give me PR/GitHub/npm access we will fix this properly.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants