bpmn-io/bpmn-js

The template element(hover) icon rendering error

Open

#2.206 aberto em 21 de jul. de 2024

Ver no GitHub
 (4 comments) (0 reactions) (0 assignees)JavaScript (1.249 forks)batch import
backlogbugelement templatesgood first issuemodeling

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

An element is backed by an element template with an attached icon. If I preview the element for append, then the custom icon is not shown.

image

Describe the solution

The preview picks up the custom icon, if @bpmn-io/element-template-icon-renderer is hooked up.

Environment

  • Browser: Chrome 126.0.6478.63
  • OS: macos 14.5
  • Library version: bpmn-js 17.9.1

Guia do colaborador

Direção de pesquisa
Investigue a lógica de renderização da pré visualização do menu popup em bpmn js. Veja como o ícone personalizado é buscado por meio da integração @bpmn io/element template icon renderer. Verifique os arquivos de origem relevantes (ex.: PopupMenu.js, ElementTemplates) para garantir que a pré visualização use corretamente o renderizador de ícones. Os comentários da issue podem fornecer dicas adicionais; ainda não há PRs vinculados.
Pilha de tecnologia
javascript
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.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-2 dias
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Recente
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
basic understanding of BPMNfamiliarity with element templates
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
60