-
-
Notifications
You must be signed in to change notification settings - Fork 6.8k
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
circle and ellipse cannot change color by classDef #271
Comments
Thanks for raising this issue. Will take a look |
Found the issue and added those elements as well. The fix will be included in the next release. |
Thanks very much. |
FWIW, this was also an issue nodes with an asymmetric shape (BTW, there is a typo in the docs where the section header is "A node in an asymetric shape" instead of "A node with an asymmetric shape"). If the fix mentioned above does not address this node shape, I can open the observation as a separate issue: <script src="https://cdn.rawgit.com/knsv/mermaid/0.5.6/dist/mermaid.min.js"></script>
<link rel="stylesheet" href="https://cdn.rawgit.com/knsv/mermaid/0.5.6/dist/mermaid.css">
<script>mermaid.initialize({startOnLoad:true});</script>
<div class="mermaid">
graph TD;
A-->B(B)
A-->C>C]
B-->D((D))
C-->D
classDef eco fill:green;
class A,B,C,D eco
</div> |
It seems circles are not colored (see @manor 's example above). The same fix implemented for ellipse seems to work fine for circles as well.
|
When I use mermaidAPI and mermaid.css, graph definiton like below:
graph LR;
A((start))-->B(step1);
B-->C[step2];
C-->D{step3};
D-->E[end];
classDef green fill:#9f6,stroke:#333,stroke-width:1px;
class A,B,C,D,E green;
The text was updated successfully, but these errors were encountered: