-
-
Notifications
You must be signed in to change notification settings - Fork 6.5k
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
The need for mermaid.css should be mentioned explicitly in the intro docs... #273
Comments
I also spent some time before I found 4 things related to the styling because they were not documented at all or clearly:
|
Thanks, good feedback! Hard to know what is clear/unclear in the docs |
+1. But what's the version of other css to use from my local install? There are three of mermaid.forest, for example: ./mermaid/dist/mermaid.forest.css |
+1 |
Issue mermaid-js#273 says "the need for mermaid.css should be mentioned explicitly in the intro docs" - I ran into this problem as well, because I read `mermaid.min.js This bundle contains everything you need to run mermaid` very literally, but it doesn't contain everything and it needs a stylesheet as well. This edit tries to make that more clear.
The section introducing simple usage on a web page doesn't mention the need for
mermaid.css
, which means that if one is following the docs literally, the resulting network renders incorrectly... Maybe it is obvious that CSS will be necessary, but I, for one, had to generate this:before realizing that there was something missing :-)
It's a shame that there is no way to package things such that the CSS file is rendered unnecessary by default, but I don't know if there is a way to do this without clobbering user-provided customization. At any rate, it is a little jarring to have the nodes appear black and the edges disappear entirely in the absence of
mermaid.css
.The text was updated successfully, but these errors were encountered: