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

Impossible de fermer la page de contribution #509

Closed
DaFrenchFrog opened this issue Oct 19, 2022 · 6 comments · Fixed by #514
Closed

Impossible de fermer la page de contribution #509

DaFrenchFrog opened this issue Oct 19, 2022 · 6 comments · Fixed by #514
Labels
bug Something isn't working

Comments

@DaFrenchFrog
Copy link
Collaborator

DaFrenchFrog commented Oct 19, 2022

Description du bug

Lorsque j'édite une fiche catalogue le clic sur la croix pour fermer n'a aucun effet.

Comportement attendu

Il faudrait que la page se referme

Pour reproduire

sur catalogue.data.gouv.fr :

  • Ouvrir une fiche catalogue
  • Cliquer sur "Modifier"
  • La page de modification apparait
  • Cliquer sur Fermer (tout va bien ça fonctionne)
  • Refaire la procédure et cette fois-ci cliquer sur un champ
  • Cliquer sur "Fermer"
  • ça ne ferme plus...

Analyse technique du problème

Le problème semble venir d'ici ici

Grossomodo l'idée de l'implem actuelle était basée sur deux boutons : un ouvrait la modale et l'autre faisait la redirection.

Le choix d'afficher l'un ou l'autre bouton dépendait de si on considérait que le formulaire a été touché Dans le vocabulaire tech cela veut dire que un utilisateur a effectué une action.

L'action minimale pour "toucher" un formuaire est de mettre son curseur dans un champs et de cliquer en dehors du formulaire.

Pourquoi deux bouttons ?

De ce que je me souvient c'était lié a une restriction spécifique du DSFR.

Il faut que le bouton ouvrant la modale ait l'attribut aria-controls, le problème est que le boutton fermant le formulaire sans demander de confirmation ne DOIT PAS avoir cet attribut.

D'où vient le problème ?

De ce que j'ai remarqué, en mettant la souri dans un champs du formulaire et en cliquant directement sur la croix, le formulaire n'est pas encore considéré comme "touché" mais ce n'est pas pour autant que l'action de fermer le formulaire est executée.

A compléter ...

Piste de solultion ?

TO DO

Contexte supplémentaire

@DaFrenchFrog DaFrenchFrog added the bug Something isn't working label Oct 19, 2022
@johanricher johanricher changed the title Imposible de fermer la page de contribution Impossible de fermer la page de contribution Oct 19, 2022
@johanricher johanricher added this to Backlog in Outil de catalogage de données via automation Oct 19, 2022
@johanricher
Copy link
Member

johanricher commented Oct 20, 2022

J'arrive à reproduire sur Firefox et Chromium, quand au moins une lettre a été ajoutée dans un des champs.

Le comportement attendu (si je comprends bien) : lors d'une création ou d'une modification de fiche sur la page /contribuer, cliquer sur la croix renvoie directement sur la page d'accueil.

Mode avocat du bug ON : Si on fait ça, est-ce que des utilisateurs ne risquent pas de perdre leur travail en cours s'ils font une mauvaise manipulation (cliquer sur la croix par erreur) ?

@DaFrenchFrog
Copy link
Collaborator Author

OK moi ça me le fait même sans saisir de lettre. Il me suffit juste de cliquer sur le champ.

Je me demande si le blocage ne vient pas du fait qu'on doit afficher la modale de sauvegarde justement. Je crois qu'elle était implémentée du coup le problème ce serait que la modale de sauvegarde ne s'affiche plus ce qui résoudrait le sujet de la perte du travail.

@johanricher
Copy link
Member

johanricher commented Oct 20, 2022

Le problème ce serait que la modale de sauvegarde ne s'affiche plus

Oui en fait c'est peut-être ça le "bug".

Le code est là : https://github.com/etalab/catalogage-donnees/blob/master/client/src/lib/components/ModalExitFormConfirmation/ModalExitFormConfirmation.svelte

@Volubyl on cherche à savoir si cette modale devrait s'afficher et si c'est le cas pourquoi elle ne s'affiche pas. Une fois qu'on aura répondu à ça on pourra décider ce qu'il faut faire pour résoudre ce problème dans une prochaine itération.

@Volubyl
Copy link
Collaborator

Volubyl commented Oct 20, 2022

J'arrive à reproduire. Je regarde pour comprendre d'où ça vient !

@Volubyl Volubyl moved this from Backlog to Tâches en cours in Outil de catalogage de données Oct 20, 2022
@johanricher
Copy link
Member

Pour être plus explicite au cas où : le problème n'est pas que le bouton fermer ne renvoie pas l'utilisateur à la page d'accueil mais que la modale ModalExitFormConfirmation ne s'affiche pas.

@DaFrenchFrog
Copy link
Collaborator Author

J'ajouterai qu'il ne faut pas que cela renvoie vers l'accueil mais plutôt vers la page précédente (mais je crois que c'est déjà ce qui est prévu).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Development

Successfully merging a pull request may close this issue.

3 participants