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

Layout manager problem on moving nodes appears on 2.4.1 version #836

Closed
MehdyKarimpour opened this issue Jul 24, 2018 · 3 comments
Closed

Layout manager problem on moving nodes appears on 2.4.1 version #836

MehdyKarimpour opened this issue Jul 24, 2018 · 3 comments
Labels
bug Something isn't working
Milestone
B26

Comments

@MehdyKarimpour
Copy link

MehdyKarimpour commented Jul 24, 2018 •
edited
Loading

When moving nodes, modeler creates additional sequence flows, sometimes behind the activity that could not be removed by the user.

Expected Behavior

When you are moving nodes the layout manager should handle best path to connect sequence flows to the related nodes.

Actual Behavior

Like version 2.3.1. It is quite okay.

Steps to reproduce the Behavior

  1. Open modeler.
  2. Add nodes to have the model have mentioned in the below image.
  3. Move activity to the near of End.

image
@nikku nikku added the bug Something isn't working label Jul 24, 2018
@nikku
Copy link
Member

nikku commented Jul 24, 2018

We are aware of this issue and are going to ship a fix soon.

@MehdyKarimpour MehdyKarimpour changed the title Moving nodes and sequence flows problem appears on 2.4.1 version Layout manager problem on moving nodes appears on 2.4.1 version Jul 24, 2018
@MehdyKarimpour
Copy link
Author

MehdyKarimpour commented Jul 24, 2018 •
edited
Loading

Perfect, we have roll backed to 2.3.1 version temporarily

@falko falko mentioned this issue Jul 24, 2018
Closed
@nikku nikku added the in progress Currently worked on label Jul 24, 2018
nikku added a commit that referenced this issue Jul 24, 2018
@nikku
fix(modeling): disallow inserting on incoming/outgoing connection
aa31d8f 
Closes #836
@nikku nikku mentioned this issue Jul 24, 2018
Merged
@nikku nikku added needs review Review pending and removed in progress Currently worked on labels Jul 24, 2018
philippfromme pushed a commit that referenced this issue Jul 25, 2018
@nikku @philippfromme
fix(modeling): disallow inserting on incoming/outgoing connection
249ea6a 
Closes #836
@ghost ghost removed the needs review Review pending label Jul 25, 2018
@nikku nikku added this to the B26 milestone Jul 25, 2018
@MehdyKarimpour
Copy link
Author

Thank you for immediate reply.

@nikku nikku mentioned this issue Jan 7, 2019
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
B26
Development

No branches or pull requests
2 participants
@nikku @MehdyKarimpour