Development #9559
utiliser le nouveau data-dynamic-display pour le champs prefill
0%
Description
utiliser le js de b92bec84599d583501f2d3c2189c5fdb4ad8c6b7 pour éviter à avoir à taper sur ce bouton "appliquer" qui est so 2010...
Fichiers
Révisions associées
Historique
Mis à jour par Thomas Noël il y a plus de 8 ans
- Fichier 0001-Prefill-widget-use-new-dynamic-display-js-possibilit.patch 0001-Prefill-widget-use-new-dynamic-display-js-possibilit.patch ajouté
- Statut changé de Nouveau à En cours
- Patch proposed changé de Non à Oui
J'ai voulu essayer le patch ci-joint, mais ça ne marche pas au niveau de l'affichage initial (par exemple si c'est le type "texte" pour prefill, le champ input à côté ne s'affiche pas par défaut). Pour le reste, ça à l'air ok, mais je ne pas sûr non plus du "prefill$type".
(C'était un exercice du soir pour me décontrater.)
(je coche "patch proposed", sous-entendu : je propose un patch mais il est pas bon)
Mis à jour par Frédéric Péters il y a plus de 8 ans
Le js était fait pour des <input type=radio/>, pour gérer correctement le champ maitre en <select> il doit y avoir quelques adaptations, je regarderai.
Mis à jour par Frédéric Péters il y a plus de 8 ans
Ça devrait passer avec :
--- a/wcs/qommon/static/js/qommon.js +++ b/wcs/qommon/static/js/qommon.js @@ -9,12 +9,13 @@ $(function() { error: function(error) { windows.console && console.log('bouh', error); } }); }); - $('[data-dynamic-display-parent]').change(function() { + $('[data-dynamic-display-parent]').on('change keyup', function() { var sel1 = '[data-dynamic-display-child-of="' + $(this).attr('name') + '"]'; var sel2 = '[data-dynamic-display-value="' + $(this).val() + '"]'; $(sel1).hide(); $(sel1 + sel2).show(); }); $('[data-dynamic-display-child-of]').hide(); + $('select[data-dynamic-display-parent]').trigger('change'); $('[data-dynamic-display-parent]:checked').trigger('change'); });
Mis à jour par Frédéric Péters il y a plus de 8 ans
Cela ajouté, sur le patch en lui-même, il ne faut pas remplir tous les champs avec value=value.get('value')
parce que ça fait qu'en changeant de type de préremplissage on garde une valeur qui n'a aucun rapport.
Le tests/test_admin_pages.py::test_form_edit_field_advanced échoue.
Dans la même série, il y a la sélection de data source qui a aussi un bouton "apply", qui pourrait être adaptée.
Mis à jour par Frédéric Péters il y a environ 8 ans
- Fichier 0001-admin-use-new-dynamic-display-js-possibility-in-pref.patch 0001-admin-use-new-dynamic-display-js-possibility-in-pref.patch ajouté
Je viens de reprendre le patch avec ces commentaires.
Mis à jour par Frédéric Péters il y a environ 8 ans
- Statut changé de En cours à Résolu (à déployer)
commit 766b7e8397909ffa39cc7db743ea6a84d2ffdf82 Author: Thomas NOEL <tnoel@entrouvert.com> Date: Fri Jan 8 21:00:56 2016 +0100 admin: use new dynamic-display js possibility in prefill widgets (#9559)
Mis à jour par Thomas Noël il y a environ 8 ans
- Statut changé de Résolu (à déployer) à Solution déployée
admin: use new dynamic-display js possibility in prefill widgets (#9559)