Bug #19154
CompositeWidget n'applique pas les extra_class_css de ses sous-widgets
0%
Description
parce que if hasattr(self, 'extra_css_class') and widget.extra_css_class: mais c'est pas dans self qu'il faut chercher.
Fichiers
Révisions associées
Historique
Mis à jour par Thomas Noël il y a plus de 6 ans
- Fichier 0001-forms-fix-widgets-extra_css_class-use-in-composite-w.patch 0001-forms-fix-widgets-extra_css_class-use-in-composite-w.patch ajouté
- Statut changé de Nouveau à En cours
- Patch proposed changé de Non à Oui
Mis à jour par Benjamin Dauvergne il y a plus de 6 ans
Ack (peut-être un test pour valider le rendu, faut trouver une page avec un CompositeWidget ou alors juste faire un render sur un form créé dans le test)
Mis à jour par Thomas Noël il y a plus de 6 ans
- Fichier 0001-forms-fix-widgets-extra_css_class-use-in-composite-w.patch 0001-forms-fix-widgets-extra_css_class-use-in-composite-w.patch ajouté
Voilà, avec un test qui vérifie un peu le comportement par défaut de CompositeWidget, et pour la partie "extra_css_class" qui concerne ce patch, j'utilise un FileWithPreviewWidget dans un composite, le FileWith.. étant un widget avec extra_css_class par défaut.
Mis à jour par Frédéric Péters il y a plus de 6 ans
Petite hésitation sur l'idée qu'on pourrait avoir un "tableau de longueur libre" sur lequel on a mis dans l'admin un extra_css_class et qu'au niveau du rendu on s'attende à avoir cette classe appliquée sur chacune des cellules. (mais ma lecture devait être mauvaise parce que ça ne le fait de toute façon pas); ack sur le patch.
Mis à jour par Thomas Noël il y a plus de 6 ans
- Statut changé de En cours à Résolu (à déployer)
commit 80d5b7a1942b7cd471fb6ca7fd9c99210cb13e8d
Author: Thomas NOEL <tnoel@entrouvert.com>
Date: Mon Oct 2 22:45:27 2017 +0200
forms: fix widgets extra_css_class use in composite widget (#19154)
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Statut changé de Résolu (à déployer) à Solution déployée
forms: fix widgets extra_css_class use in composite widget (#19154)