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

Label of former child element deleted when deleting Sub-Process #1050

Closed
falko opened this Issue Dec 3, 2018 · 4 comments

Comments

Projects
None yet
2 participants
@falko
Copy link
Member

falko commented Dec 3, 2018

Describe the Bug

The Label of a former child element gets deleted when deleting a Sub-Process.

2018-12-03 label of former child element deleted when deleting sub-process

Steps to Reproduce

Steps to reproduce the behavior:

  1. Open the attached BPMN file
    2018-12-03 Label of former child element deleted when deleting Sub-Process.bpmn.txt

  2. Move the End Event "Dokument in Akte abgelegt" out of the Event Sub-Process.

  3. Delete the Event Sub-Process.

Expected Behavior

The End Event "Dokument in Akte abgelegt" keeps its Label, because it is no longer a child of the Sub-Process.

Environment

Please complete the following information:

  • OS: Ubuntu 16.04.5 LTS
  • Camunda Modeler Version: 2.2.0
@nikku

This comment has been minimized.

Copy link
Member

nikku commented Dec 4, 2018

There seems to be a bug in the way labels move on the diagram.

The following screencast shows a related issue:

foo

@falko

This comment has been minimized.

Copy link
Member Author

falko commented Dec 4, 2018

If you say that it is about how labels are moved, could #1011 be related as well?

@nikku

This comment has been minimized.

Copy link
Member

nikku commented Dec 4, 2018

Unfortunately not. We simply do not support embedded labels.

@nikku

This comment has been minimized.

Copy link
Member

nikku commented Dec 4, 2018

Gets addressed via bpmn-io/bpmn-js#920.

@nikku nikku added this to the M26 milestone Dec 4, 2018

@nikku nikku closed this in a75eb7d Dec 4, 2018

barmac added a commit that referenced this issue Jan 17, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment