josdejong/jsoneditor

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

Offen

#1.088 geöffnet am 17.08.2020

Auf GitHub ansehenContributor Guide
 (10 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (2.078 Forks)batch import
featurehelp wanted

Repository-Metriken

Stars
 (12.263 Sterne)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

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!

Contributor Guide

Research-Richtung
Untersuchen Sie die Implementierung des onCreateMenu Callbacks und die Rendering Logik des Kontextmenüs im Quellcode. Ändern Sie die Typdefinition und den Rendering Code, um die Rückgabe von undefined oder eines leeren Arrays vom Callback zu behandeln, und fügen Sie dann einen Testfall hinzu.
Tech Stack
javascript
Domain
frontend
Issue Type
Funktion
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
JavaScriptGit
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70