Project

General

Profile

Development #33026

faire qu'annuler supprime le brouillon courant

Added by Frédéric Péters 3 months ago. Updated 3 months ago.

Status:
Solution déployée
Priority:
Normal
Assignee:
-
Target version:
-
Start date:
12 May 2019
Due date:
% Done:

0%

Patch proposed:
Yes
Planning:
No

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.

0001-forms-add-an-explicit-discard-button-instead-of-canc.patch View (7.06 KB) Frédéric Péters, 03 Jun 2019 04:41 PM

Associated revisions

Revision 98c8808b (diff)
Added by Frédéric Péters 3 months ago

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

History

#1 Updated by Thomas Noël 3 months ago

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 Updated by Thomas Noël 3 months ago

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

#3 Updated by Frédéric Péters 3 months ago

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 Updated by Thomas Noël 3 months ago

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 Updated by Frédéric Péters 3 months ago

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 Updated by Thomas Noël 3 months ago

  • Status changed from Solution proposée to 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 Updated by Frédéric Péters 3 months ago

  • Status changed from Solution validée to 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 Updated by Frédéric Péters 3 months ago

  • Status changed from Résolu (à déployer) to Solution déployée

Also available in: Atom PDF