Sub Process moves position

[mschoe]

Describe the Bug

Sub Process moves position after expand & collapse operation

Steps to Reproduce

// given
element chain of star event --> task --> end event

// when
I perform the follwong operations:

1. transform task into expanded sub process
2. collapse the sub process
3. expand the sub process

// then
the position of the sub process has moved and is not aligned with the other elements anymore

Expected Behavior

sub process keeps position

Environment

Please complete the following information:

• OS: linux
• Camunda Modeler Version: [3.2.0 nightly-190628]

[barmac]

Related to bpmn-io/bpmn-js#1026, bpmn-io/bpmn-js#1052

[pinussilvestrus]

Could also be related to bpmn-io/bpmn-js#1103

[pinussilvestrus]

Root Cause

Regarding the vertical movement:

The misalignment comes from an offset generated by GridSnapping.

Cf.ToggleElementCollapseBehavior autoResize to grid. This introduces the bug for not-yet-snapped diagrams.

However, this does not also cause the horizontal movement when transforming the task to a subprocess. Further investigation needed.

[pinussilvestrus]

Regarding the horizontal movement of the SubProcess:

The movement resolves in newly calculated optimal bounds which takes the automatically created start event into account. That's why the start Event seems to be moved inside the SubProcess after re-expanding (as the screencast above reveals).

This behavior is also shown in previous version v3.x.x when expanding collapsed subProcesses which have children

The behavior described above with empty subProcesses only comes from the fact we're automatically creating a new startEvent when morphing a task to an expanded subProcess. This relates to bpmn-io/bpmn-js#1031

So this is not a regression . It is caused by the fact when calculating the new bounds in AutoResize we take the collapsed subProcess and the elements of the expanded subProcess into account. Due to the fact it indicates the children outside of the colapsed subProcess' bounds, it adds an offset to the left, which is visually visible by the horizontal movement.

To fix this, we will have to somehow save the bounds from the expanded subProcess. For example via Extension Element.

[pinussilvestrus]

After a discussion with @volkergersabeck we concluded to postpone this when we tackle an overall improvement-run regarding Modeling with SubProcesses in the next quarter. This will include several improvements like this one or #1244, #1343.

Roo-Cause-Analysis is described in following commits

• Sub Process moves position #1418 (comment), vertically movement + disaligned sequence flows
• Sub Process moves position #1418 (comment), horizontally movement

[nikku]

Closing this as we do not allow to collapse sub-processes anymore.