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

Flowchart: Link text beginning with lowercase 'o' causes flowchart to break #407

Closed
ssurve opened this Issue Nov 9, 2016 · 3 comments

Comments

Projects
None yet
2 participants
@ssurve

ssurve commented Nov 9, 2016

E.g.: <div class='mermaid'>graph LR\n\tstep0(step0)--next-->step1(step1)\n\tstep1--next-->step2(step2)\n\tstep2--next-->step3(step3)\n\tstep3--ok-->DONE((DONE))</div>
results in this output:
flowchart display error with lowercase o in link text

Uppercase 'O' works as expected:
code: <div class='mermaid'>graph LR\n\tstep0(step0)--next-->step1(step1)\n\tstep1--next-->step2(step2)\n\tstep2--next-->step3(step3)\n\tstep3--Ok-->DONE((DONE))</div>
output:
correct flowchart display with uppercase O in link text

Is this a parser rule? Not starting a link text with lowercase 'o'? If yes, are there any more such rules? Please point me to the relevant documentation.

Thanks.

@tylerlong

This comment has been minimized.

Collaborator

tylerlong commented Sep 16, 2017

I find a workaround:

https://mermaidjs.github.io/mermaid-live-editor/#/edit/Z3JhcGggTFIKc3RlcDAoc3RlcDApLS1uZXh0LS0-c3RlcDEoc3RlcDEpCnN0ZXAxLS1uZXh0LS0-c3RlcDIoc3RlcDIpCnN0ZXAyLS1uZXh0LS0-c3RlcDMoc3RlcDMpCnN0ZXAzLS0ib2siLS0-RE9ORSgoRE9ORSkp

Put the link text inside double quotes then it can have any characters.

I still think it is a bug. I will dive into it whenever I have some time.

@tylerlong

This comment has been minimized.

Collaborator

tylerlong commented Mar 17, 2018

--o is a special syntax for ARROW_CIRCLE. I don't have any good solution for this behavior. I recommend the workaround above.

@tylerlong tylerlong closed this Mar 17, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment