josdejong/jsoneditor

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

Aperta

#1088 aperta il 17 ago 2020

Vedi su GitHubGuida contributor
 (10 commenti) (0 reazioni) (0 assegnatari)JavaScript (2078 fork)batch import
featurehelp wanted

Metriche repository

Star
 (12.263 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

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!

Guida contributor

Direzione di ricerca
Esamina l'implementazione della callback onCreateMenu e la logica di rendering del menu contestuale nel codice sorgente. Modifica la definizione del tipo e il codice di rendering per gestire il ritorno di undefined o di un array vuoto dalla callback, quindi aggiungi un caso di test.
Tech stack
javascript
Dominio
frontend
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
JavaScriptGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70