Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Envoyer un formulaire sur le nouvel éditeur avec Ctrl+Entrée ne vide pas le stockage local #6309

Closed
Arnaud-D opened this issue May 20, 2022 · 0 comments · Fixed by #6310
Closed
Labels
C-Front Concerne l'interface du site S-BUG Corrige un problème

Comments

@Arnaud-D
Copy link
Contributor

Arnaud-D commented May 20, 2022

Description du bug

Quand on soumet le formulaire avec Ctrl+Entrée sur le nouvel éditeur, le stockage local n'est pas vidé alors qu'il l'est quand on appuie sur le bouton.

Pourquoi c'est gênant ?

Sur les forums, ça oblige à supprimer le contenu du formulaire si on intervient plusieurs fois dans le même topic.

Ça le fait aussi avec les tutos, ce qui cause des fourberies pour l'écriture collaborative. On peut avoir notre propre version au lieu de celle actuelle si celle actuelle a été écrite par quelqu'un d'autre ! Ça m'est déjà arrivé et c'est très pénible à corriger une fois qu'on a écraser les modifications de l'autre par accident.

Comment reproduire ?

Cela se manifeste sur le forum en envoyant le formulaire avec Ctrl+Entrée. Attendez 15 min et vous verrez que le formulaire est encore rempli avec ce que vous venez d'envoyer.

Ça arrive aussi sur les tutos, mais essentiellement invisible en dehors de l'écriture à plusieurs ou depuis plusieurs appareils.

Comportement attendu

Ctrl+Entrée et le bouton devrait avoir le même comportement.

Piste de résolution

Dans editor-new.js, on a un callback sur le Ctrl+Entrée qui appelle submit sur un formulaire. Cette méthode n'a pas le même comportement que le bouton.

Une possibilité serait d'utiliser requestSubmit, qui simule le comportement du bouton. Pas compatible avec Safari pour le moment, donc il faudrait être subtil. Faut se méfier aussi, parce que c'est plein de jQuery actuellement dans ce fichier, donc il y a quelques contournements à prendre pour effectivement appeler la méthode sur un objet normal et pas jQuery.

Alternativement, et je trouve ça plus robuste, on pourrait garder la méthode submit, mais faire en sorte d'émettre l'événement submit qui est écouté par l'éditeur, qui vide le local storage à ce moment-là.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-Front Concerne l'interface du site S-BUG Corrige un problème
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant