Projet

Général

Profil

Bug #24166

Tirer le bon formulaire pour les popups

Ajouté par Josué Kouka il y a presque 6 ans. Mis à jour il y a plus de 5 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
30 mai 2018
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

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

Révision 73eb8c17 (diff)
Ajouté par Frédéric Péters il y a presque 6 ans

make edit/delete links target specific forms (#24166)

Historique

#4

Mis à jour par Josué Kouka il y a presque 6 ans

#5

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).

#6

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.

#7

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)

#8

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 ?

#10

Mis à jour par Serghei Mihai il y a presque 6 ans

Ack, go.

#11

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)

#12

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

Formats disponibles : Atom PDF