bpmn-io/bpmn-js

Label double moves on batch movement

Open

#1 766 ouverte le 10 nov. 2022

Voir sur GitHub
 (2 commentaires) (0 réactions) (0 assignés)JavaScript (1 249 forks)batch import
backlogbuggood first issuemodelingspring cleaning

Métriques du dépôt

Stars
 (7 739 stars)
Métriques de merge PR
 (Merge moyen 17h 6m) (4 PRs mergées en 30 j)

Description

Describe the Bug

If I move a label along with another element that connects to the label target the label moves twice, causing quirky behavior:

capture LMEmCl_optimized

Steps to Reproduce

  1. Open label-double-move.bpmn.txt on https://demo.bpmn.io/
  2. Select ASD label + choose reciepe task
  3. Move both
  4. See label move twice

Expected Behavior

Two options:

  • Move label with connection
  • Move label with task

Not sure which one is better. Do not move twice.

Environment

  • Browser: Any
  • OS: Any
  • Library version: 8.x, 9.x, 10.x (tested)

Guide contributeur

Direction de recherche
Étudiez la logique de déplacement des étiquettes dans la base de code, probablement dans MoveModule ou LabelUtil. Implémentez un correctif qui garantit que l'étiquette ne se déplace qu'une seule fois, soit avec la connexion, soit avec la tâche. Consultez les commentaires de l'issue pour les retours des mainteneurs sur le comportement préféré.
Stack technique
javascript
Domaine
frontend
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
GitJavaScript
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75