Development #13349
essayer un peu d'empêcher le double clic sur les boutons
0%
Description
Sans que ça n'ait porté à conséquence on s'est trouvé avec deux POST vers la même demande, le premier interrompu par le client (erreur 499).
89.91.205.26 - - [28/Sep/2016:09:36:13 +0200] "POST /inscriptions-creche/contrat-creche-admission-partie-2/45/ HTTP/1.1" 499 0 "https://demarches.vincennes.fr/inscriptions-creche/contrat-creche-admission-partie-2/45/" "Mozilla/5.0 (Windows NT 6.1; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/50.0.2661.94 Safari/537.36" 89.91.205.26 - - [28/Sep/2016:09:36:17 +0200] "POST /inscriptions-creche/contrat-creche-admission-partie-2/45/ HTTP/1.1" 303 32 "https://demarches.vincennes.fr/inscriptions-creche/contrat-creche-admission-partie-2/45/" "Mozilla/5.0 (Windows NT 6.1; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/50.0.2661.94 Safari/537.36"
Il y a hypothèse du double clic, c'est peut-être plutôt un second clic d'impatience (vu qu'il y a quatre secondes entre).
Fichiers
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a plus de 7 ans
- Fichier 0001-misc-disable-buttons-for-5-seconds-after-a-click-133.patch 0001-misc-disable-buttons-for-5-seconds-after-a-click-133.patch ajouté
- Statut changé de Nouveau à En cours
- Patch proposed changé de Non à Oui
Patch qui désactive le bouton pendant 5 secondes après un clic; à tester en vrai.
Mis à jour par Frédéric Péters il y a plus de 7 ans
- Patch proposed changé de Oui à Non
Testé; ça casse tout.
Mis à jour par Benjamin Dauvergne il y a plus de 6 ans
Là1 ils disent de le faire dans le onSubmit pas le onClick.
[1]: https://stackoverflow.com/questions/5691054/disable-submit-button-on-form-submit
Aussi je mettrai plutôt 30s puisque c'est le timeout standard de nos web-service (et un truc qui rame c'est forcément un truc qui appelle un web-service).
Mis à jour par Frédéric Péters il y a environ 6 ans
- Fichier 0001-misc-disable-form-buttons-on-submit-13349.patch 0001-misc-disable-form-buttons-on-submit-13349.patch ajouté
- Fichier 0001-scss-dim-buttons-during-submit-13349.patch 0001-scss-dim-buttons-during-submit-13349.patch ajouté
- Patch proposed changé de Non à Oui
Patch w.c.s. + publik-base-theme; c'est juste du code de support dans w.c.s. pour poser une classe et côté publik-base-theme ça tape juste un pointer-events: none qui évite le clic.
C'est plutôt bidon mais ça ferait je pense office.
Mis à jour par Benjamin Dauvergne il y a environ 6 ans
Je veux bien un commentaire sur le opacity: 1
de manière générale je pense qu'on pourrait (je dis on en sachant très bien que c'est toujours la même 1 personne et demi qui s'occupe de CSS chez nous mais bon) avoir plus de commentaires dans les CSS, ça reste toujours difficile de comprendre la raison d'être d'une règle ou d'une propriété. Typiquement je pense que le commentaire ici devrait pointer que ça sert à la règle "disabled-during-submit" qui est plus loin.
Mis à jour par Frédéric Péters il y a environ 6 ans
avoir plus de commentaires
Avec scss on peut taper des commentaires en // et ils sont squeezés de la sortie css, donc je pense aussi qu'on devrait commencer à se permettre ça.
Sur le patch en lui-même, lors d'un retour à la page précédente, ça reste inactif, il faudrait je trouve que ça redevienne actif.
Mis à jour par Frédéric Péters il y a environ 6 ans
- Fichier 0001-misc-disable-form-buttons-on-submit-13349.patch 0001-misc-disable-form-buttons-on-submit-13349.patch ajouté
- Fichier 0001-scss-dim-buttons-during-submit-13349.patch 0001-scss-dim-buttons-during-submit-13349.patch ajouté
Voilà le patch modifié pour retirer la classe disabled-during-submit à l'affichage de la page. Côté publik-base-theme j'ai retiré la déclaration explicite d'opacity qui n'était pas nécessaire et j'ai ajouté un commentaire en début de bloc pour dire quand il entrait en jeu.
Mis à jour par Frédéric Péters il y a environ 6 ans
- Fichier 0001-scss-dim-buttons-during-submit-13349.patch 0001-scss-dim-buttons-during-submit-13349.patch ajouté
- Fichier 0001-misc-disable-form-buttons-on-submit-13349.patch 0001-misc-disable-form-buttons-on-submit-13349.patch ajouté
Et un patch pour gadjo et la modif du patch w.c.s. pour que ça aie aussi lieu en backoffice.
Mis à jour par Thomas Noël il y a environ 6 ans
Moi aussi je joue, question : pourquoi retirer la classe après 30 secondes ? ie pourquoi ne pas la laisser ad vitam ? Il n'y a selon moi jamais aucune raison de re-cliquer sur le bouton.
Mis à jour par Frédéric Péters il y a environ 6 ans
- Fichier 0001-misc-disable-form-buttons-on-submit-13349.patch 0001-misc-disable-form-buttons-on-submit-13349.patch ajouté
Bien incapable de me souvenir de ce qui a initialement pu motiver ça.
Mis à jour par Frédéric Péters il y a environ 6 ans
- Statut changé de En cours à Résolu (à déployer)
commit 59a6416acef65a277fdd95bf843e6e512a4980f7 Author: Frédéric Péters <fpeters@entrouvert.com> Date: Sat Feb 10 20:12:56 2018 +0100 misc: disable form (buttons) on submit (#13349)
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
misc: disable form (buttons) on submit (#13349)