josdejong/jsoneditor

Implement a way to trigger executing onEditable again

Open

#1 261 ouverte le 5 mars 2021

Voir sur GitHub
 (2 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

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.

Guide contributeur

Direction de recherche
Examinez comment le callback onEditable est invoqué en mode texte/code et concevez un moyen de le réévaluer lorsque l'éditeur passe de lecture seule à modifiable, éventuellement en ajoutant une méthode ou en déclenchant une réévaluation lors du changement de mode.
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.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-2 jours
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.
Plutôt claire
Prérequis
JavaScriptjsoneditor API
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60