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

Simplify appending expanded sub-processes #2004

Closed
nikku opened this issue Nov 6, 2020 · 4 comments
Closed

Simplify appending expanded sub-processes #2004

nikku opened this issue Nov 6, 2020 · 4 comments
Labels
BPMN enhancement New feature or request ux

Comments

@nikku
Copy link
Member

nikku commented Nov 6, 2020

Is your feature request related to a problem? Please describe.

When building a BPMN diagram, sometimes during the design phase I'd like to append an expanded sub-process directly from the current element. I'd even argue that, in the automation context, this is the more prominent action than appending a collapsed sub-process.

Right now though, this interaction is poorly supported. I.e. it takes an incredible amount of effort to get this done, even on an empty canvas:

capture QwPa8Y_optimized

Describe the solution you'd like

It should be easy to append an expanded sub-process right from the currently selected flow element.

  • Update the append menu to allow appending an expanded sub-process or
  • Update the replace menu to allow replacing a task to an expanded sub-process

Describe alternatives you've considered

As mentioned above, there is a few options.

Additional context

This is related to #1244, i.e. the fact that replacing a task with a sub-process (after it has been placed) results in unnecessary noise on the diagram. ➡️ It may be better to support appending the sub-process right away.

@nikku nikku added enhancement New feature or request ux labels Nov 6, 2020
@pinussilvestrus pinussilvestrus added the backlog Queued in backlog label Nov 6, 2020 — with bpmn-io-tasks
@pinussilvestrus pinussilvestrus added the spring cleaning Could be cleaned up one day label Nov 24, 2020
@philippfromme philippfromme self-assigned this Apr 14, 2021
@philippfromme
Copy link
Contributor

I've investigated this issue and there is one major roadblock making this a bit more effort to implement. The Modeling#appendShape API doesn't support multiple elements at the moment the way the Create feature does. Thus, appending a subprocess with a start event isn't as easy as adding the respective context pad entry.

@philippfromme philippfromme removed their assignment Apr 15, 2021
@philippfromme philippfromme removed the spring cleaning Could be cleaned up one day label Apr 15, 2021
@korthout
Copy link
Member

I would really love this feature. Currently, there isn't even an option to expand a collapsed sub-process for C8 models. So I have to delete the element and drag it in from the side.
Screen Shot 2022-05-16 at 17 06 18Screen Shot 2022-05-16 at 17 07 07

@nikku
Copy link
Member Author

nikku commented May 16, 2022

Will be solved via create/append anything:

capture qpon4Z_optimized

@nikku
Copy link
Member Author

nikku commented Dec 7, 2023

Fully supported with create/append anything in bpmn-js@13+:

capture byijla_optimized

@nikku nikku closed this as not planned Won't fix, can't repro, duplicate, stale Dec 7, 2023
@bpmn-io-tasks bpmn-io-tasks bot removed the backlog Queued in backlog label Dec 7, 2023
@nikku nikku closed this as completed Dec 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BPMN enhancement New feature or request ux
Projects
None yet
Development

No branches or pull requests

4 participants