Bug #24166
Tirer le bon formulaire pour les popups
0%
Description
L'interet est d'éviter d'avoir plusieurs popup qui s'affiche quand l'un des templates parents contient déja un form.
Fichiers
Révisions associées
Historique
Mis à jour par Josué Kouka il y a presque 6 ans
- Fichier 0001-manager-inherit-from-gadjo-base.html-in-deletion-con.patch 0001-manager-inherit-from-gadjo-base.html-in-deletion-con.patch ajouté
- Sujet changé de Faire directement hériter tous les templates de validation de gadjo/base à Hériter de gadjo/base.html dans le template de confirmation de suppression de document
- Patch proposed changé de Non à Oui
Mis à jour par Frédéric Péters il y a presque 6 ans
Mais non, fargo en front doit utiliser publik-base-theme (ce qui passe par celui-ci définissant son propre fargo/base.html), ne peut pas soudainement basculer sur gadjo.
Je ne comprends pas trop la description et #24142 manque de captures d'écran. (ça parle de jauge et ça me fait penser backoffice et ça me semble très curieux).
Mis à jour par Josué Kouka il y a presque 6 ans
Frédéric Péters a écrit :
Mais non, fargo en front doit utiliser publik-base-theme (ce qui passe par celui-ci définissant son propre fargo/base.html), ne peut pas soudainement basculer sur gadjo.
Je parle juste de la pop de la validation de suppression.
Je ne comprends pas trop la description et #24142 manque de captures d'écran. (ça parle de jauge et ça me fait penser backoffice et ça me semble très curieux).
(Plus de vpn pour l'instant je collerai une capture d'écran des que je peux).
Le souci est que lorsqu'on supprime un document du porte doc, au lieu d'avoir juste la popup de confirmation, on a aussi une popup de suivi de code
(et non jauge) parce que la barre laterale à une cellule suivi de code
.
Mis à jour par Frédéric Péters il y a presque 6 ans
Je parle juste de la pop de la validation de suppression.
Et ça ne change rien. On peut pas avoir soudainement sur une page un style différent; même si souvent le rendu sera le plus souvent en popup.
(Plus de vpn pour l'instant je collerai une capture d'écran des que je peux).
Tu veux dire que tu corriges en truc sans l'avoir reproduit en local ?
~~
Si je comprends bien, le bug c'est "fargo crée une popup à partir d'un formulaire mais considère qu'il n'y aura jamais qu'un formulaire sur la page et du coup dans une situation où ce n'est pas le cas (exemple en front avec une barre latérale avec un code de suivi, ou sur un site avec un formulaire de recherche dans l'entête, etc.) un des autres formulaires de la page peut être tiré pour l'affichage en popup.
La correction à ça, c'est de rendre spécifique le sélecteur, plutôt que $('form') faire $('form#whatever'). (cela dit sans rien regarder au code)
Mis à jour par Josué Kouka il y a presque 6 ans
Frédéric Péters a écrit :
Je parle juste de la pop de la validation de suppression.
Et ça ne change rien. On peut pas avoir soudainement sur une page un style différent; même si souvent le rendu sera le plus souvent en popup.
(Plus de vpn pour l'instant je collerai une capture d'écran des que je peux).
Tu veux dire que tu corriges en truc sans l'avoir reproduit en local ?
Non je pensais à une capture pour illustrer ce qui se passait sur le site de la Moselle.
~~
Si je comprends bien, le bug c'est "fargo crée une popup à partir d'un formulaire mais considère qu'il n'y aura jamais qu'un formulaire sur la page et du coup dans une situation où ce n'est pas le cas (exemple en front avec une barre latérale avec un code de suivi, ou sur un site avec un formulaire de recherche dans l'entête, etc.) un des autres formulaires de la page peut être tiré pour l'affichage en popup.
La correction à ça, c'est de rendre spécifique le sélecteur, plutôt que $('form') faire $('form#whatever'). (cela dit sans rien regarder au code)
Ok, ça implique changer pas mal de templates non ?
Mis à jour par Frédéric Péters il y a presque 6 ans
- Fichier 0001-make-edit-delete-links-target-specific-forms-24166.patch 0001-make-edit-delete-links-target-specific-forms-24166.patch ajouté
- Assigné à changé de Josué Kouka à Frédéric Péters
Mis à jour par Frédéric Péters il y a presque 6 ans
- Sujet changé de Hériter de gadjo/base.html dans le template de confirmation de suppression de document à Tirer le bon formulaire pour les popups
- Statut changé de En cours à Résolu (à déployer)
commit 73eb8c173af283ab5a1f3f7d1664fd5e77e56fd6 Author: Frédéric Péters <fpeters@entrouvert.com> Date: Tue Jun 5 16:33:48 2018 +0200 make edit/delete links target specific forms (#24166)
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
make edit/delete links target specific forms (#24166)