Bug #64883
Duplication de page : pop-up peu compréhensible
0%
Description
Je vois qu'une nouvelle ergonomie vient d'arriver pour dupliquer une page, alors je viens râler un peu. Elle n'est pas compréhensible, on ne comprend pas ce qu'il se passe, on ne comprend pas ce qu'on nous demande. Je ne sais pas ce que cette fonctionnalité doit permettre de faire plus simplement, ça manque d'explication. Ne passez pas ça en prod svp.
Fichiers
Révisions associées
Historique
Mis à jour par Thomas Noël il y a environ 2 ans
Tu as l'URL où c'est visible ? (ça ressemble surtout à un bogue de css)
Mis à jour par Thomas Noël il y a environ 2 ans
- Tracker changé de Development à Bug
Je m'autoréponds, c'est visible partout et c'est pas un bogue de CSS, c'est un bogue tout court. On voulait afficher une demande de confirmation et il y a confusion dans le code certainement.
Mis à jour par Frédéric Péters il y a environ 2 ans
- Statut changé de Nouveau à Information nécessaire
- Assigné à mis à Marie Kuntz
Mis à jour par Thomas Noël il y a environ 2 ans
- Statut changé de Information nécessaire à Nouveau
- Assigné à changé de Marie Kuntz à Thomas Noël
Ok le bogue arrive quand on veut dupliquer une page qui n'a aucun champ.
201 def duplicate(self): 202 if self.field.type == 'page': 203 page_fields = self.get_page_fields() 204 if page_fields: 205 # display popup if there are fields in the page, to offer to 206 # duplicate them too. 207 return self.duplicate_page(page_fields) 208 field_pos = self.objectdef.fields.index(self.field) 209 fields = self.objectdef.fields 210 new_field = copy.deepcopy(self.field) ... 215 return self.redirect_field_anchor(new_field)
si page_fields est vide, on déroule la copie classique (lignes 208 à 215) mais le résultat s'affiche dans la popup, car rel=popup.
Je regarde si une popup peut ne pas s'afficher... j'ai des doutes...
Mis à jour par Frédéric Péters il y a environ 2 ans
Honnêtement je serais ok pour tout le temps afficher la popup et même pour y laisser la case à cocher "également copier les champs", et attendre le ticket qui viendra nous dire que c'est un peu bête d'avoir cette case alors qu'il n'y a pas de champs.
Mis à jour par Lauréline Guérin il y a environ 2 ans
- Assigné à
Thomas Noëlsupprimé - Priorité changé de Normal à Bas
Revenir sur #58892 et proposer une autre solution ?
Mis à jour par Lauréline Guérin il y a environ 2 ans
- Assigné à mis à Thomas Noël
- Priorité changé de Bas à Normal
Mis à jour par Frédéric Péters il y a environ 2 ans
et attendre le ticket qui viendra nous dire que c'est un peu bête d'avoir cette case alors qu'il n'y a pas de champs.
Ah bah on a donc déjà eu ce ticket. Top.
Mis à jour par Lauréline Guérin il y a environ 2 ans
Pas tout à fait, à l'époque on avait une popup sans case à cocher si pas de champ dans la page :)
Mis à jour par Thomas Noël il y a environ 2 ans
- Fichier 0001-admin-always-show-a-popup-when-duplicating-a-page-64.patch 0001-admin-always-show-a-popup-when-duplicating-a-page-64.patch ajouté
Dans l'écriture du pache je n'ai pas lu ces échanges et voilà : la popup quand même mais sans la case à cocher quand y'a pas de champs et je trouve que c'est bien suffisant comme effort d'UX de ma part.
Mis à jour par Frédéric Péters il y a environ 2 ans
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Frédéric Péters il y a environ 2 ans
- Statut changé de Solution proposée à Résolu (à déployer)
commit 53b1ba16f326e1bcb77390957cc0a54d244a6eb2
Author: Thomas Noël <tnoel@entrouvert.com>
Date: Thu May 5 14:36:54 2022 +0200
admin: always show a popup when duplicating a page (#64883)
Mis à jour par Transition automatique il y a environ 2 ans
- Statut changé de Résolu (à déployer) à Solution déployée
admin: always show a popup when duplicating a page (#64883)