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

[DRAFT] Wrong tail edge label (#905) #906

Merged

Conversation

lredor
Copy link
Contributor

@lredor lredor commented Dec 23, 2022

Proposition of fix. To be checked/completed because I have absolutely no control over the impacts.

Signed-off-by: Laurent Redor laurent.redor@obeo.fr

Signed-off-by: Laurent Redor <laurent.redor@obeo.fr>
@soerendomroes
Copy link
Contributor

@Eddykasp You recently worked on external ports (which seem to have something to do with this), can you please have a look and test this PR?

@soerendomroes soerendomroes added this to the Release 0.9.0 milestone Jan 9, 2023
@soerendomroes soerendomroes added the alg-layered Affects the ELK Layered algorithm. label Jan 9, 2023
@Eddykasp
Copy link
Contributor

@Eddykasp You recently worked on external ports (which seem to have something to do with this), can you please have a look and test this PR?

yes I'll test it

@Eddykasp
Copy link
Contributor

This looks good to me and I can't see any problems that would come with this change. Interestingly this example (without noLayout) also produces this issue although that does not appear to be a problem in ELK itself.

Copy link
Contributor

@Eddykasp Eddykasp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@soerendomroes soerendomroes merged commit b85ad1c into eclipse:master Mar 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
alg-layered Affects the ELK Layered algorithm.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants