Préavis – Ajouter et éditer un préavis pour une navire de moins de 12 mètres

[ivangabriele]

• Formulaire UI
• Endpoint API d'enregistrement (mocké)
• Toute modification reset le champ isSent à false
• Si quantité = 0, on affiche le tag Préavis 0

Ecran :

[AdelineCelier]

Remarque du pôle outre-mer : il peut arriver qu'un préavis soit long à remplir et que l'utilisateur soit interrompu en cours de route et doive par ex. saisir un rapport de mission. Il ne faut pas qu'il perde les infos saisies entre-temps.

Solution pour la V1

• Faire en sorte que si l'on change de vue sans fermer le formulaire, celui-ci soit toujours ouvert (et les infos saisies dedans toujours là) quand on revient sur la vue préavis.
• Si l'utilisateur ferme le formulaire avant d'avoir cliqué sur "créer le préavis", afficher une pop-up demandant confirmation.

Solution à plus long terme
A voir s'il faut une solution plus robuste, type possibilité d'enregistrer un brouillon, ou mettre en place un enregistrement auto, ou une possibilité de "ranger" temporairement le préavis, comme on le fait pour les signalements dans Env. Etudier la solution la plus proportionnée par rapport au besoin.

[AdelineCelier]

Revue du formulaire de préavis -12 m

Bugs

• J'étais en train de remplir un formulaire quand soudain il a disparu. => cela ne s'est pas reproduit
• /!\ La sélection dans les différents menus déroulants est capricieuse...j'ai l'impression qu'il y a toujours ce problème de temps de chargement, qui fait que quand on scrolle ou quand on recherche, les résultats s'affichent mais mettent du temps à charger, temps pendant lequel on ne peut pas cliquer dessus
• Quand je cherche l'espèce "cabillaud" (ou "COD"), je ne la trouve pas, c'est normal ? 👍
• Quand j'ai rouvert le préavis manuel que j'avais créé, le champ "zone de pêche" s'est affiché comme inactif pendant un bon moment (bien 10 à 20 secondes je dirais). Cela m'est arrivé 2 fois.

• Les calculs ne se mettent pas à jour quand je supprime ou remplace certains items du formulaire du préavis en cours de création. 👍
• J'ai créé un préavis en saisissant des infos, puis j'ai modifié ces infos, les calculs ne se sont pas mis à jour (cf. ci-dessus). J'ai quand même cliqué sur enregistrer, et cela fait plusieurs minutes maintenant que le formulaire charge et ne se crée pas. En revenant plus tard sur la liste, je me rends compte que le préavis a été créé 3 fois dans la liste. Mais quand j'essaye d'ouvrir un des 3, cela continue de charger et cela n'affiche pas le formulaire. (Variante du même problème observé cet après-midi après la 2è mise en inté d'Ivan : j'ai saisi les infos du préavis et je cliquais sur "créer le préavis", mais rien ne se passait. Au final je quitte le préavis puis je me rends compte qu'il a été créé 8 fois).
  => Pb identifié : ça le fait quand on sélectionne le thon rouge en espèce, parce qu'elle n'existe pas en base (pb BF1, 2, 3) 👍

Revue UX

• Le tag "préavis zéro" ne doit s'afficher qu'à partir du moment où des espèces ont été renseignées et que leurs quantités sont égales à 0. Là il s'affiche dès qu'on ouvre le formulaire 👍
• Il manque la pop-up de confirmation quand on quitte le formulaire alors qu'il n'est pas enregistré (en cliquant sur "fermer" ou la croix ou à côté du formulaire)
• Ajouter la mention "(UTC)" dans les champs heure, en Marianne Regular Slate gray (cf. maquettes)
• S'il n'y a pas de changement dans le formulaire, possible de griser le bouton "enregistrer" ? 👍
• Quand on clique sur "enregistrer", je me demande s'il ne faut pas fermer aussi le formulaire (sous-entendu "enregistrer et fermer")
• Afin de permettre de changer de vue sans perdre le formulaire en cours de saisie (cf. mon dernier commentaire ci-dessous), il faut que le "voile" sombre qui recouvre la vue liste ne recouvre pas la barre de menu de gauche 👍

Revue UI

• Le champ des quantités est trop grand, il doit avoir une taille fixe de 96 px (notamment pour prévoir le cas où il faudra afficher sur la même ligne un sélecteur pour la zone de capture)
• Je ne suis pas sûre que les flèches pour ajuster la quantité soient nécessaires, est-ce qu'on peut les retirer ?
• La champ du trigramme est trop long, il doit faire 120 px 👍
• Il n'y a pas assez d'espace entre le texte et les icônes des champs date et heure : mettre 16 px
• L'icône dans le bouton "ajouter un préavis" de la vue liste a une drôle de tête, on dirait qu'elle n'est pas totalement opaque ^^
  

[ivangabriele]

@louptheron :

Les calculs ne se mettent pas à jour quand je supprime ou remplace certains items du formulaire du préavis en cours de création.

J'ai créé un préavis en saisissant des infos, puis j'ai modifié ces infos, les calculs ne se sont pas mis à jour (cf. ci-dessus). J'ai quand même cliqué sur enregistrer, et cela fait plusieurs minutes maintenant que le formulaire charge et ne se crée pas. En revenant plus tard sur la liste, je me rends compte que le préavis a été créé 3 fois dans la liste. Mais quand j'essaye d'ouvrir un des 3, cela continue de charger et cela n'affiche pas le formulaire. (Variante du même problème observé cet après-midi après la 2è mise en inté d'Ivan : j'ai saisi les infos du préavis et je cliquais sur "créer le préavis", mais rien ne se passait. Au final je quitte le préavis puis je me rends compte qu'il a été créé 8 fois).
=> Pb identifié : ça le fait quand on sélectionne le thon rouge en espèce, parce qu'elle n'existe pas en base (pb BF1, 2, 3)

Ce n'est possiblement plus un bug mais à double-checker. Pour le BF1, BF2, BF3, il fallait ajouter les espèces dans la DB (via le CSV open data).

Je ne suis pas sûre que les flèches pour ajuster la quantité soient nécessaires, est-ce qu'on peut les retirer ?

Il me semble que Maxime a ouvert une PR qui pourrait répondre à cette demande car il utilise un text input.

• Pour les largeurs (et certaines hauteurs) :

Note

Faire attention à l'histoire de compatibilité Firefox < 126 quand ouvert en mode side window pour les calculs de largeur. C'est super chiant à tester :(.

[louptheron]

Ajouter la mention "(UTC)" dans les champs heure, en Marianne Regular Slate gray (cf. maquettes)

Il faut changer le composant monitor-ui, en attendant j'ai ajouté (UTC) à la fin de la ligne.

---

Pour info, j'ai modifié la recherche dans les espèces pour qu'elle soit aussi simple que dans le CR de contrôle.

[louptheron]

/!\ La sélection dans les différents menus déroulants est capricieuse...j'ai l'impression qu'il y a toujours ce problème de temps de chargement, qui fait que quand on scrolle ou quand on recherche, les résultats s'affichent mais mettent du temps à charger, temps pendant lequel on ne peut pas cliquer dessus

Je n'ai pas réussi à reproduire le souci rencontré en production sur monitor-ui.
J'ai essayé une PR sur monitor-ui, pour être ISO avec ce qu'il y a dans le formulaire des préavis :

• Rajouter le cacheKey sur monitor-ui
• Rajouter un searchable
• Mettre 10 <Select/>

J'ai testé à chaque fois sur Storybook (dans la VM Windows DSI) le scrolling, et il n'y a pas le souci.
cf. MTES-MCT/monitor-ui#1272

@ivangabriele Tu aurais une idée ?

[AdelineCelier]

@louptheron j'ai vérifié les points cochés à date, et cela me semble good. Juste une petite remarque sur la pop-up de confirmation de fermeture du formulaire : le bouton "quitter sans enregistrer" est tronqué, possible de l'afficher en entier (quitte à élargir un peu la pop-up pour respecter les marges) ?

[AdelineCelier]

C'est good