josdejong/jsoneditor

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

Open

#1 088 ouverte le 17 août 2020

Voir sur GitHub
 (10 commentaires) (0 réactions) (0 assignés)JavaScript (2 034 forks)batch import
featurehelp wanted

Métriques du dépôt

Stars
 (10 781 stars)
Métriques de merge PR
 (Merge moyen 15h 36m) (3 PRs mergées en 30 j)

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!

Guide contributeur

Direction de recherche
Examinez l'implémentation du callback onCreateMenu et la logique de rendu du menu contextuel dans le code source. Modifiez la définition de type et le code de rendu pour gérer le retour de undefined ou d'un tableau vide de la part du callback, puis ajoutez un cas de test.
Stack technique
javascript
Domaine
frontend
Type d'issue
Fonctionnalité
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
JavaScriptGit
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70