bpmn-io/bpmn-js

A bug occurs when deleting a box ：A DOM element reference is required, But there is no problem using shift to select

Open

#2 221 ouverte le 19 août 2024

Voir sur GitHub
 (11 commentaires) (0 réactions) (0 assignés)JavaScript (1 249 forks)batch import
backlogbuggood first issuespring 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

I don't know if this is a feature: shift+left mouse button to select the element dragged into the canvas, and then use the shortcut key del to delete, a bug will appear: console.ts:40 unhandled error in event listener Error: A DOM element reference is required, But there is no problem using shift to select

Steps to Reproduce

Shift+left mouse button selects the elements dragged into the canvas, and then uses the shortcut key del to delete shift+left mouse button to select the elements dragged into the canvas, and then uses the shortcut key del to delete

The following cases occurred in：https://demo.bpmn.io/s/start

https://github.com/user-attachments/assets/3b263e7e-f539-4618-9e6a-990f23684291

Expected Behavior

Delete the node selected by the box

Environment

  • Browser: [Chrome 127.0.6533.120]
  • OS: [e.g. Windows 11]

Guide contributeur

Direction de recherche
Examinez la gestion de la sélection et les écouteurs d'événements clavier dans le code source de bpmn js. L'erreur 'A DOM element reference is required' suggère que lors de la suppression après une sélection par shift+clic, une référence à un élément DOM est manquante. Étudiez le comportement de BoxSelection et le gestionnaire de la touche de suppression. Vérifiez si les éléments sélectionnés sont correctement référencés lorsque la suppression est déclenchée. Les fichiers pertinents peuvent inclure selection/Selection.js, features/box selection/ et keyboard/KeyboardBindings.js.
Stack technique
javascripthtmlcss
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.
2
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
Basic JavaScript knowledgeFamiliarity with DOM events
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75