-
Notifications
You must be signed in to change notification settings - Fork 20
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Feat/improve max characters #282
Feat/improve max characters #282
Conversation
OK, merci pour cette PR liée à GeotrekCE/Geotrek-admin#3844. Pour la rétrocompatibilité, je ne sais pas trop. On peut peut-être faire un breaking change et bien l'indiquer dans le changelog de Geotrek-admin pour que ceux qui utilisent ce paramètre global, le modifient pour spécifier les champs dont ils souhaitent limiter le nombre de caractères. |
dd5991e
to
e392e73
Compare
Limites :
|
Pour résoudre les 2 points concernant l'UI : peut-être est-il possible d'interroger le composant TinyMCE pour récupérer le nombre courant de caractères et l'afficher en-dessous du champ avec le format : nb_courant / nb_max. (et on oublie le clic simulé) Une méthode pour récupérer le nombre de caractères est documentée : https://www.tiny.cloud/docs/plugins/opensource/wordcount/#api Par contre il faudrait pouvoir réagir sur un event émis par le champ TinyMCE. Je ne sais pas si TinyMCE propose une interface pour l'observabilité. |
Oui je m'étais poser la question de faire comme ça necessite de :
ps : qu'est ce qu'on fait pour les input qui n'on pas de limite défini en conf ? |
Pas d'opinion tranchée de mon côté. Juste je me dis que la remarque de Camille sur rassembler l'info visuellement est pertinente.
Je me disais qu'on pouvait le laisser. Il indiquera le nombre de mots et puis voilà. |
Si ce n'est pas un soucis d'ajouter un peu de code maison, alors autant faire un truc un peu plus clair et lisible. PS : Le wordcount de base de CKeditor (https://ckeditor.com/docs/ckeditor5/latest/features/word-count.html) est bien plus sympa et clair que celui de TinyMCE : |
63b466e
to
dee05c0
Compare
J'ai enlever le click programmatique et rajouter la limite. (5/6 characters) |
b3dfe2d
to
f2c97a8
Compare
55afe61
to
18cc0d5
Compare
- Improve UI when the limit is exceeded ref (GeotrekCE/Geotrek-admin#3844)
8156717
to
dc32db6
Compare
dc32db6
to
77d0e84
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## master #282 +/- ##
==========================================
+ Coverage 89.95% 90.00% +0.04%
==========================================
Files 32 32
Lines 2390 2401 +11
==========================================
+ Hits 2150 2161 +11
Misses 240 240 ☔ View full report in Codecov by Sentry. |
Dernière update :
Ready to review ! |
Limite le nombre de caractère uniquement sur les champs spécifiés en configuration.
Le contrôle reste informatif et non bloquant (contrôle effectué côté front)
Voir GeotrekCE/Geotrek-admin#3844
Le paramètre de configuration a ce format :
Je pensais mettre uniquement le nom du modèle en config (
touristicevent
par exemple, mais celui-ci n'est pas forcément unique)Question : est-ce qu'on maintient une rétrocompatibilité sur l'ancien format de config ?
MAPENTITY_CONFIG['MAX_CHARACTERS'] = <INT>
? En l'état, si les gens ne change pas leur config, ça fait planter l'app ..