Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Improve arrows #3

Closed
nikolay opened this issue Nov 28, 2014 · 7 comments
Closed

Improve arrows #3

nikolay opened this issue Nov 28, 2014 · 7 comments
Labels
Contributor needed Type: Enhancement New feature or request

Comments

@nikolay
Copy link

nikolay commented Nov 28, 2014

The arrows need to be monochromatic and should be more symmetrical and standard-looking.
@knsv
Copy link
Collaborator

knsv commented Nov 28, 2014

Different arrows are part of the plan. Was thinking about other symbols then arrow heads like circles and maybe a cross. This could into that work.

@nikolay
Copy link
Author

nikolay commented Dec 1, 2014

I think by default all arrows should be "normal" as in here: http://www.graphviz.org/doc/info/arrows.html

@knsv knsv added Type: Enhancement New feature or request Contributor needed labels Dec 1, 2014
@knsv
Copy link
Collaborator

knsv commented Dec 2, 2014

The parsing supports --o as well but there are no rendering for that yet so when the arrows are overlooked, the arrow ending in a dot should also be included and maybe a --x as well.

@Travis-S
Copy link

Travis-S commented Dec 2, 2014

Along those lines, is there support for double-headed arrows? This would be useful for indicating bidirectional relationships, and would also avoid cluttering up the graph with two arrows.

@knsv
Copy link
Collaborator

knsv commented Dec 3, 2014

Yes I can see the point with that. The obvious syntax will not work though <-> will be interpreted as a tag by the browser.

bjowes added a commit that referenced this issue Jan 14, 2015
@bjowes
Merge pull request #3 from knsv/master
6a1550f 
Merge from master
@knsv
Copy link
Collaborator

knsv commented Feb 28, 2015

I will close this issue and open a new issue for double headed arrows. I hope that is ok.

@knsv knsv closed this as completed Feb 28, 2015
@Travis-S
Copy link

Yeah. A specific issue is easier to keep track of.

IOrlandoni pushed a commit that referenced this issue Nov 27, 2019
@jgreywolf
Merge pull request #3 from mermaid-js/develop
4ff5c3b 
Develop
knsv pushed a commit that referenced this issue Dec 11, 2019
@GDFaber
Merge pull request #3 from mermaid-js/develop
813b2fc 
sync develop branch
@tmatthias29 tmatthias29 mentioned this issue Sep 1, 2020
Closed
knsv pushed a commit that referenced this issue Aug 4, 2021
@dependabot
Bump andymckay/labeler from 1.0.2 to 1.0.3 (#3)
7229b6a 
Bumps [andymckay/labeler](https://github.com/andymckay/labeler) from 1.0.2 to 1.0.3.
- [Release notes](https://github.com/andymckay/labeler/releases)
- [Commits](andymckay/labeler@1.0.2...1.0.3)

---
updated-dependencies:
- dependency-name: andymckay/labeler
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@dnlmc dnlmc mentioned this issue Mar 1, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Contributor needed Type: Enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@nikolay @knsv @Travis-S