Développement #111064
pouvoir passer des attributs HTML sur le conteneur des widgets des formulaires
0%
Description
Il y a dans combo une copie/adaptation de la prise en charge des attributs data-dynamic-display-*; c'est actuellement uniquement ok pour l'édition du rendu de la cellule fiche mais ça serait bien de pouvoir étendre ça aux formulaires django (rendus via |with_template).
Pour faire ça il faut que le rendu puisse inclure des attributs sur le div contenant le widget.
Related issues
Associated revisions
History
Updated by Frédéric Péters about 1 month ago
- Status changed from Nouveau to En cours
🤖 Une pull request concernant ce ticket a été ouverte :
- URL : https://git.entrouvert.org/entrouvert/gadjo/pulls/94
- Titre : WIP: templates: add support for widget container attributes (#111064)
- Modifications : https://git.entrouvert.org/entrouvert/gadjo/pulls/94/files
Updated by Valentin Deniaud about 1 month ago
Frédéric Péters a écrit :
ça serait bien de pouvoir étendre ça aux formulaires django (rendus via |with_template).
Pour info ça marche déjà dans chrono depuis https://dev.entrouvert.org/issues/100200, donc je ne sais pas trop ce qui nécessite un patch dans gadjo
Updated by Frédéric Péters 29 days ago
Pour info ça marche déjà dans chrono depuis https://dev.entrouvert.org/issues/100200, donc je ne sais pas trop ce qui nécessite un patch dans gadjo
Ça permet de fonctionner avec le js qui est dans w.c.s. et dans combo. Je vois ta remarque sur chrono et il m'a fallu un temps pour voir que le js qui est dedans n'est pas le même, il ajoute des .parents('.widget') pour retrouver le conteneur à afficher/cacher, vs le fonctionnement du js de w.c.s. et combo qui attend que les attributs data-dynamic-display-child-of soient sur le conteneur.
Et je préférerais ne pas aller adapter le code dans ces modules (je peux par contre m'engager à aller adapter chrono pour utiliser le même code, qu'idéalement j'arriverais à déplacer dans gadjo).
Updated by Frédéric Péters 24 days ago
- Related to Développement #111066: champs dynamiques aussi dans les popups de gadjo added
Updated by Valentin Deniaud 22 days ago
- Status changed from Solution proposée to Solution validée
🤖 Pull request approuvée :
Updated by Frédéric Péters 20 days ago
- Status changed from Solution validée to Résolu (à déployer)
🤖 Pull request fusionnée :
- URL : https://git.entrouvert.org/entrouvert/gadjo/pulls/94
- Titre : templates: add support for widget container attributes (#111064)
- Modifications : https://git.entrouvert.org/entrouvert/gadjo/pulls/94/files
Updated by Transition automatique 16 days ago
- Status changed from Résolu (à déployer) to Solution déployée
templates: add support for widget container attributes (#111064)