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 aberto em 19 de ago. de 2024

Ver no GitHub
 (11 comments) (0 reactions) (0 assignees)JavaScript (1.249 forks)batch import
backlogbuggood first issuespring cleaning

Métricas do repositório

Stars
 (7.739 stars)
Métricas de merge de PR
 (Mesclagem média 17h 6m) (4 fundiu PRs em 30d)

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]

Guia do colaborador

Direção de pesquisa
Investigue o tratamento de seleção e os ouvintes de eventos de teclado no código fonte do bpmn js. O erro 'A DOM element reference is required' sugere que, durante a exclusão após uma seleção com shift+click, falta uma referência a um elemento DOM. Analise o comportamento do BoxSelection e o manipulador da tecla de exclusão. Verifique se os elementos selecionados são referenciados corretamente quando a exclusão é acionada. Os arquivos relevantes podem incluir selection/Selection.js, features/box selection/ e keyboard/KeyboardBindings.js.
Pilha de tecnologia
javascripthtmlcss
Domain
frontend
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
Basic JavaScript knowledgeFamiliarity with DOM events
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
75