josdejong/jsoneditor

Implement a way to trigger executing onEditable again

Open

#1.261 geöffnet am 5. März 2021

Auf GitHub ansehen
 (2 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (2.034 Forks)batch import
featurehelp wanted

Repository-Metriken

Stars
 (10.781 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 15h 36m) (3 gemergte PRs in 30 T)

Beschreibung

We have a case where we dynamically switch the editor from read-only to editable. However, in code or text mode this isn't applied properly because the onEditable callback is only called on creation.

https://github.com/josdejong/jsoneditor/blob/1e79ead1ec96a9acfdb8b35097595a3da5a9bacc/src/js/textmode.js#L255-L256

This is not clear from the docs:

In modes text and code, the callback is invoked as editable(node) where node is an empty object (no field, value, or path). In that case the function can return false to make the text or code editor completely read-only.

If it's not possible to handle this dynamically, having to call .refresh() to update the mode also comes a long way. Currently we recreate the editor when we want to switch from read-only to editable.

Contributor Guide

Research-Richtung
Untersuchen Sie, wie der onEditable Callback in den Modi Text/Code aufgerufen wird, und entwerfen Sie eine Möglichkeit, ihn neu zu bewerten, wenn der Editor von schreibgeschützt auf bearbeitbar umschaltet, möglicherweise durch Hinzufügen einer Methode oder Auslösen einer Neubewertung bei Moduswechsel.
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.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-2 Tage
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.
Meist klar
Voraussetzungen
JavaScriptjsoneditor API
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
60