Projet

Général

Profil

Development #33026

faire qu'annuler supprime le brouillon courant

Ajouté par Frédéric Péters il y a presque 5 ans. Mis à jour il y a presque 5 ans.

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

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

C'est une demande à GL (#32978), à réfléchir j'en arriverais à dire que c'est un comportement raisonnable.

Reste la personne qui voudrait arrêter la saisie pour la reprendre plus tard (mais une personne cliquerait sur "annuler" avec cette idée ?). Dans #32978 il y a suggestion de demander confirmation mais ça je je ne suis pas bien convaincu.


Fichiers

Révisions associées

Révision 98c8808b (diff)
Ajouté par Frédéric Péters il y a presque 5 ans

forms: add an explicit discard button (instead of "cancel") (#33026)

Historique

#1

Mis à jour par Thomas Noël il y a presque 5 ans

Je dirais oui aussi, sauf dans le cas où c'est un brouillon repris... (mais comment faire la différence, techniquement, mmh) Et peut-être enfoncer le clou en nommant le bouton "Abandonner".

#2

Mis à jour par Thomas Noël il y a presque 5 ans

Peut-être, en fait, passer par une popup à cet instant, qui demande si on veut abandonner définitivement ou conserver un brouillon.

#3

Mis à jour par Frédéric Péters il y a presque 5 ans

Voilà sans le côté boite de confirmation, pour un brouillon qui a été restauré, il y a un bouton "annuler" et il y a le bouton de suppression explicite qui est présent dans le pavé code de suivi.

#4

Mis à jour par Thomas Noël il y a presque 5 ans

J'ai un peu de mal avec cette magie finale lors du clic sur cancel (if form.get_submit() == 'cancel': ...)

Je pensais que l'idée serait plutôt :
  • en cas de nouveau formulaire, donc abandonnable : pas de bouton 'cancel', mais on affiche un submit de type 'removedraft' qui porte le nom "abandonner"
  • en cas de reprise d'un brouillon, on laisse le 'cancel' actuel

Peut-être que je rate un truc ?

#5

Mis à jour par Frédéric Péters il y a presque 5 ans

Peut-être que je rate un truc ?

Je ne suis pas tout à fait sûr de ce que tu vois comme "magie", je tente sur "des conditions au moment du submit plutôt qu'avoir simplement au début taper un nom différent au bouton" : J'ai trouvé plutôt pratique de rester sur un chemin commun cancel/discard et à ce moment de prendre la décision cancel/discard, question intégration graphique ou suivi du cancelurl. (plutôt que venir toucher à l'action removedraft).

#6

Mis à jour par Thomas Noël il y a presque 5 ans

  • Statut changé de Solution proposée à Solution validée

Frédéric Péters a écrit :

Peut-être que je rate un truc ?

Je ne suis pas tout à fait sûr de ce que tu vois comme "magie"

Au lieu d'un bouton "cancel" unique et qui agit d'une façon ou d'une autre, je pensais laisser le bouton "cancel" et son comportemant actuel pour les reprises de brouillon, mais dans les autres cas avoir un nouveau bouton pour l'abandon (soit créer "discard", soit recyclage de "removedraft"). Mais c'est sans doute que je rate un peu "les conditions au moment du submit". Mon idée derrière et que si un jour on va vers un popup sur un clic sur "annuler", il s'agirait juste de présenter les deux boutons "cancel" et "discard" côte à côte, et hop.

Bref.

Allons vers ton approche qui me semble de toute façon très bien fonctionner.

(Pour ma part, je trouve que depuis qu'on a retiré le bouton "Sauvegarder en tant que brouillon" on a une façon trop secrète de créer des brouillons, plutôt cachée aux usagers, qui ne découvrent la possibilité que par hasard ... mais c'est vraiment une autre histoire qui n'est pas le lieu de ce ticket)

#7

Mis à jour par Frédéric Péters il y a presque 5 ans

  • Statut changé de Solution validée à Résolu (à déployer)

Merci, et ça continuera bien sûr à évoluer,

commit 98c8808bd4be8af5456ea4cd1d617822ac5e12ac
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Mon Jun 3 15:10:22 2019 +0200

    forms: add an explicit discard button (instead of "cancel") (#33026)
#8

Mis à jour par Frédéric Péters il y a presque 5 ans

  • Statut changé de Résolu (à déployer) à Solution déployée

Formats disponibles : Atom PDF