-
-
Notifications
You must be signed in to change notification settings - Fork 6.6k
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
Arrow heads missing (cli, 7.0.3) #544
Comments
I tested the web version and it seemed OK. So this problem is probably a cli specific issue. |
I am facing same, arrow heads missing seems it only happens on flowchart.
|
Confirmed broken with multiple of my diagrams. |
Could anybody confirm: does version 7.0.0 have this issue or not? |
@tylerlong |
I upgraded the library from 7.0.0 to 7.0.3. I didn't change any major logic. One possible reason might be I upgraded phantomjs to latest version. I will definitely take a look. Maybe we will replace phantomjs with headless Chrome. For now, please stick to 7.0.0. Because 7.0.3 adds no new feature (major change is webpack support added) |
Looking at the generated SVG, it seems that the reference of the arrow head definition is not correct. |
I am sorry I have been busy with my daily job. Now I am back to the mermaid development. I just did a quick test in the live editor: https://mermaidjs.github.io/mermaid-live-editor/#/edit/Z3JhcGggVEQKICAgIEEgLS0+IEIKICAgIEIgLS4tPiBDCg== There is no problem. So I think this is a CLI only issue and it is most likely caused by PhantomJS. PhantomJS is discontinued: http://carstenwindler.de/software-testing/phantomjs-is-discontinued/ Here are my plans:
|
tylerlong, headless chromium is also an option now. Just confirming I still have the issue in CLI 7.0.4 |
I faced the same issue on both the Demo page and Online Editor Arrow heads of the last 2 lines are missing And when I inspect it, arrow heads aren't even in SVG tree Note: I'm using Firefox 55 on Linux |
It is by design: https://mermaidjs.github.io/sequenceDiagram.html
@ALL, I confirm that flowchart has this arrow missing issue, while sequence diagram works as expected. I have found the root cause and I will fix it soon. |
@tylerlong I'm sorry and thank you for telling me the documentation. I missed it. |
My solution is as follows:
By the way,there not exist double sided arrow markup in flowchar,is it right? |
Guys, sorry for the delay. Please check this project: https://github.com/mermaidjs/mermaid.cli I replaced PhantomJS with puppeteer. I want to move CLI out of mermaid main project in the near future. It's new so there might be bugs. Its code is deadly simple so everyone here can contribute: https://github.com/mermaidjs/mermaid.cli/blob/master/index.js I will keep this issue open till the end of this week. Please reply if you still have issues. |
Arrow heads are missing, when using the cli (version 7.0.3).
Last version which worked: 7.0.0
Test case:
Which results in the following image:
Cli arguments used:
The text was updated successfully, but these errors were encountered: