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.

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

Layout loops depending on already existing connections #898

Open
philippfromme opened this issue Nov 5, 2018 · 4 comments
Open

Layout loops depending on already existing connections #898

philippfromme opened this issue Nov 5, 2018 · 4 comments
Labels
backlog Queued in backlog enhancement New feature or request needs discussion Needs further discussion

Comments

@philippfromme
Copy link
Contributor

philippfromme commented Nov 5, 2018

Description

When laying out a loop the connection will always go from bottom to left regardless of already exiting connections. This can potentially lead to overlapping connections:

2018-11-05_09-38-46

Current Behavior

Creating a loop will always result in a connection going from bottom to left.

Desired Behavior

Creating a loop results in a connection that ideally doesn't overlap with already existing connections.

We should be a bit smarter in this case.

@philippfromme philippfromme added bug Something isn't working backlog Queued in backlog labels Nov 5, 2018
@nikku
Copy link
Member

nikku commented Dec 4, 2018

What exactly is broken about this? What is the expected behavior?

@nikku nikku added help wanted Extra attention is needed and removed backlog Queued in backlog bug Something isn't working labels Dec 4, 2018
@philippfromme
Copy link
Contributor Author

The connections overlap. Layouting the loop doesn't take existing connections into account, it will always layout from bottom to right.

@nikku
Copy link
Member

nikku commented Dec 5, 2018

So what is the expected behavior then?

@philippfromme philippfromme added enhancement New feature or request needs discussion Needs further discussion labels Dec 5, 2018
@philippfromme philippfromme changed the title Rendering breaks in case there's an incoming connection on the left when creating a loop Layout loops depending on already existing connections Dec 10, 2018
@philippfromme philippfromme added the backlog Queued in backlog label Dec 10, 2018
@nikku
Copy link
Member

nikku commented Sep 9, 2019

Is this something we'd like to persue in the near future? I'd otherwise close this issue.

@nikku nikku removed the help wanted Extra attention is needed label Feb 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backlog Queued in backlog enhancement New feature or request needs discussion Needs further discussion
Projects
None yet
Development

No branches or pull requests

2 participants