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 geöffnet am 19. Aug. 2024

Auf GitHub ansehen
 (11 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (1.249 Forks)batch import
backlogbuggood first issuespring cleaning

Repository-Metriken

Stars
 (7.739 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 17h 6m) (4 gemergte PRs in 30 T)

Beschreibung

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]

Contributor Guide

Research-Richtung
Untersuchen Sie die Auswahlbehandlung und Tastaturereignis Listener im bpmn js Quellcode. Der Fehler 'A DOM element reference is required' deutet darauf hin, dass beim Löschen nach einer Shift+Klick Auswahl eine DOM Elementreferenz fehlt. Untersuchen Sie das BoxSelection Verhalten und den Löschtasten Handler. Prüfen Sie, ob die ausgewählten Elemente beim Auslösen des Löschvorgangs korrekt referenziert werden. Relevante Dateien könnten selection/Selection.js, features/box selection/ und keyboard/KeyboardBindings.js umfassen.
Tech Stack
javascripthtmlcss
Domain
frontend
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
Basic JavaScript knowledgeFamiliarity with DOM events
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
75