Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

sequence diagram, arrowhead instead of crosshead #227

Closed
tylerlong opened this issue Oct 21, 2015 · 0 comments
Closed

sequence diagram, arrowhead instead of crosshead #227

tylerlong opened this issue Oct 21, 2015 · 0 comments

Comments

@tylerlong
Copy link
Collaborator 

sequenceDiagram
A->>B: How are you?

Generated line:

<line x1="75" y1="100" x2="275" y2="100" class="messageLine0" stroke-width="2" stroke="black" marker-end="url(#crosshead)" style="fill: none;"></line>

should be:

<line x1="75" y1="100" x2="275" y2="100" class="messageLine0" stroke-width="2" stroke="black" marker-end="url(#arrowhead)" style="fill: none;"></line>

You made a typo here:

https://github.com/knsv/mermaid/blob/b43e695da2048515e0198ca075be3cb0f1b5e564/src/diagrams/sequenceDiagram/sequenceRenderer.js#L239
@tylerlong tylerlong mentioned this issue Oct 21, 2015
Merged
mgenereu pushed a commit to mgenereu/mermaid that referenced this issue Jun 25, 2022
@sidharthv96
Merge pull request mermaid-js#227 from mermaid-js/dependabot/npm_and_…
74d47a3 
…yarn/develop/mocha-9.0.1

Bump mocha from 9.0.0 to 9.0.1
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@tylerlong