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

Text annotations do not move with annotated target #4172

Open
nikku opened this issue Mar 4, 2024 · 9 comments
Open

Text annotations do not move with annotated target #4172

nikku opened this issue Mar 4, 2024 · 9 comments
Assignees
Labels
BPMN bug Something isn't working channel:support ready Ready to be worked on spring cleaning Could be cleaned up one day ux

Comments

@nikku
Copy link
Member

nikku commented Mar 4, 2024

Describe the bug

Using the space tool I'm used to text annotations moving with the annotated target. Since a couple of releases this does no longer seem to be the case. In fact the annotations stay where they are, and if I use them extensively, then it takes a long time to put them back to their to be expected place:

capture oIJzWK_optimized

Note how the behavior changes whether I#m within a participant making space or outside of it.

Steps to reproduce

  1. Open text-annotation-move.bpmn
  2. Make space within participant, between two tasks
  3. See that text annotation position is sticky

Expected behavior

Text annotations move with elements, same as labels.

Environment

  • OS: Any
  • Camunda Modeler Version: v5.19.0
  • Execution Platform: Any
  • Installed plug-ins: None

Additional context

Also raised in SUPPORT-22608

SUPPORT-25375
Depends on bpmn-io/bpmn-js#2226

@nikku nikku added bug Something isn't working ux BPMN ready Ready to be worked on labels Mar 4, 2024
@nikku
Copy link
Member Author

nikku commented Mar 4, 2024

Moving to ready. This is a fallout from a recent improvement, and a real time waster in larger diagrams.

@nikku nikku added the backlog Queued in backlog label Mar 22, 2024 — with bpmn-io-tasks
@nikku nikku removed the ready Ready to be worked on label Mar 22, 2024
@crobbins215
Copy link

Customer has raised this https://jira.camunda.com/browse/SUPPORT-22608 also.

@crobbins215
Copy link

Raised in another way as its confusing a customer - bpmn-io/bpmn-js#2215 (comment)

@nikku
Copy link
Member Author

nikku commented Sep 9, 2024

Raised via bpmn-io/bpmn-js#2226, too.

@nikku
Copy link
Member Author

nikku commented Sep 10, 2024

Related to #4492.

@nikku nikku added backlog Queued in backlog spring cleaning Could be cleaned up one day and removed ready Ready to be worked on labels Sep 10, 2024
@alex-axiobit
Copy link

This also happens when you move a pool

@barmac barmac marked this as a duplicate of #4797 Feb 4, 2025
@barmac barmac added ready Ready to be worked on and removed backlog Queued in backlog labels Feb 4, 2025
@jarekdanielak
Copy link
Contributor

jarekdanielak commented Feb 18, 2025

It started happening in v16.4.0.

Specifically with bpmn-io/bpmn-js@ca230c3

@nikku
Copy link
Member Author

nikku commented Feb 18, 2025

Probably worth to checkout bpmn-io/bpmn-js#2049 to understand the original intend (why we worked on bpmn-io/bpmn-js#2078 in the first place).

The space tool, starting from [email protected] / [email protected] is local per default, which in combination causes this issue.

@jarekdanielak
Copy link
Contributor

After a discussion with @nikku, we concluded that text annotations should move when using a local space tool if they are inside of the participant, but NOT when they are outside (unlike external labels which always move with the target).

A possible follow-up is to apply a similar behavior to groups.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BPMN bug Something isn't working channel:support ready Ready to be worked on spring cleaning Could be cleaned up one day ux
Projects
None yet
Development

No branches or pull requests

5 participants