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

Be able to limit append anything to elements that can be appended #30

Open
philippfromme opened this issue Apr 17, 2024 · 6 comments
Open
Assignees
Labels
backlog Queued in backlog enhancement New feature or request

Comments

@philippfromme
Copy link
Contributor

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

Currently both the create and append anything popup menu list all elements that can be created. The append popup menu does not limit the list to elements that can be appended which can will often hide the most relevant elements in the list.

Example: Event-based gateway

There are only a few elements that can be appended but they're hard to find. It doesn't really make sense to offer any other options here.

image

Describe the solution you'd like

Change append anything to only list elements that can be appended. This could also be configurable so users can decide themselves.

Describe alternatives you've considered

Require users to add their own provider that filters the list.

Additional context

Related to camunda/improved-canvas#56

@philippfromme philippfromme added the enhancement New feature or request label Apr 17, 2024
@philippfromme philippfromme changed the title Be able to limit append menu entries to what can acutally be appended Be able to limit append anything to elements that can acutally be appended Apr 17, 2024
@philippfromme philippfromme self-assigned this Apr 17, 2024
@philippfromme philippfromme changed the title Be able to limit append anything to elements that can acutally be appended Be able to limit append anything to elements that can actually be appended Apr 17, 2024
@philippfromme philippfromme changed the title Be able to limit append anything to elements that can actually be appended Be able to limit append anything to elements that can be appended Apr 17, 2024
@nikku
Copy link
Member

nikku commented Apr 22, 2024

There are only a few elements that can be appended but they're hard to find. It doesn't really make sense to offer any other options here.

@philippfromme Let's discuss this.

Append anything allows you to add any element. We've received feedback / requests in the past to remove the palette entirely (which is usually far away from you). Append anything is the (only) powerful tool you usually need.

I'm all in for offering common elements in a domain specific manner. You should be able to append/create any relevant element (in the flow) though.

TLDR: If we introduce this, then we break existing user flow, make users search for a particular element in different places, require them to understand BPMN inside out, and in the end make them not use the tool.

@philippfromme
Copy link
Contributor Author

Based on the discussion we'll close this issue for now.

@nikku
Copy link
Member

nikku commented Apr 22, 2024

One thing brought up in the discussion: With the re-introduction of quick append we don't need common anymore.

@philippfromme
Copy link
Contributor Author

@nikku The common group was added by @camunda/improved-canvas and has been removed in the meantime.

@philippfromme
Copy link
Contributor Author

Reopening based on user feedback.

@philippfromme philippfromme reopened this Apr 30, 2024
@philippfromme
Copy link
Contributor Author

philippfromme commented Apr 30, 2024

From a user test: "From an event-based gateway I should not be able to append a task", which is exactly the issue I described.

@philippfromme philippfromme added the backlog Queued in backlog label Apr 30, 2024 — with bpmn-io-tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backlog Queued in backlog enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants