josdejong/jsoneditor

Attach custom classnames to the DOM root of a node instead of nested inside

Open

#1.088 aberto em 17 de ago. de 2020

Ver no GitHub
 (10 comments) (0 reactions) (0 assignees)JavaScript (2.034 forks)batch import
featurehelp wanted

Métricas do repositório

Stars
 (10.781 stars)
Métricas de merge de PR
 (Mesclagem média 15h 36m) (3 fundiu PRs em 30d)

Description

Another small suggestion for a neater design! Would be fantastic for the onCreateMenu callback to be able to return undefined and have the rendering of the context menu icon skipped or set to display:none. This way the tree view can more easily convey at a glance which of the elements may have context actions for those of us that are replacing the menu with our own actions.

JSONEditorOptions.onCreateMenu?: ((menuItems: MenuItem[], node: MenuItemNode) => MenuItem[]) | undefined
=>
JSONEditorOptions.onCreateMenu?: ((menuItems: MenuItem[], node: MenuItemNode) => MenuItem[] | undefined) | undefined

Alternatively, an isEmpty check of the MenuItem array would do the same job.

Cheers!

Guia do colaborador

Direção de pesquisa
Examine a implementação do callback onCreateMenu e a lógica de renderização do menu de contexto no código fonte. Modifique a definição de tipo e o código de renderização para lidar com o retorno de undefined ou de uma array vazia do callback e adicione um caso de teste.
Pilha de tecnologia
javascript
Domain
frontend
Tipo Issue
Funcionalidade
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
JavaScriptGit
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
70