betagouv / seves

Gestion mutualisée de tous les événements sanitaires
MIT License
1 stars 0 forks source link

Recette 26/09/2024 #275

Open CoralieCotonPro opened 1 week ago

CoralieCotonPro commented 1 week ago

Page index/liste de fiches Recherche

Menu principal

Fiche en mode consultation

Fiche en mode modification

Fiche en mode création

Anto59290 commented 1 week ago

(déjà parlé mais je sais plus ce qu'on avait dit) Le déroulant organisme n’est pas ok (noms sur 3 lignes / bouton press to select inutile..)

Pour le passer en moins de 3 lignes il faut faire passer le form au global sur 2 lignes (comme sur la maquette), par exemple mettre état et les boutons sur la ligne du dessous. Est ce qu'on est ok avec ça ?

bloc de suivi / fil de suivi : la typo du bouton « + élement de suivi » parait d’une taille différente de celle du bouton de nouveau message

Je ne comprends pas, pour moi il font 40px de hauteur tout les deux

Si possible l’animation de swipe ne devrait jouer que lors des changements d’onglets, si pas possible alors la supprimer totalement

De mémoire on avait déjà regardé et ce n'est pas possible de retirer l'animation (en tout cas pas prévu, peut être possible en bricolant le DSFR)

bloc de suivi / documents : possible de fermer le tooltip de commentaire en re cliquant sur l’icone i ?

Ca ne semble pas prévu par le composant je crois

base adresse sera intégrée MVP ? Il me semble qu'on avait dit oui sinon trop de problemes par la suite

A statuer, je pense que ça serait bien aussi, mais si on se lance la dedans je pense qu'il y a beaucoup à revoir dans la gestion des addresses (régions, départements, etc.)

@alanzirek qu'est ce que tu en penses ?

Je vais essayer de traiter une partie des points où il y a concensus et j'éditerais l'issue pour mettre en évidence ce que j'ai fait / pas fait.

CoralieCotonPro commented 1 week ago
alanzirek commented 1 week ago

Si possible l’animation de swipe ne devrait jouer que lors des changements d’onglets, si pas possible alors la supprimer totalement

Concernant l'animation, j'ai posé la question et voici la réponse. Visiblement par compliqué à désactiver.

image

A statuer, je pense que ça serait bien aussi, mais si on se lance la dedans je pense qu'il y a beaucoup à revoir dans la gestion des addresses (régions, départements, etc.) @alanzirek qu'est ce que tu en penses ?

@Anto59290 Oui, dispo pour en parler. Mise à part supprimer les modèles Département et Régions et mettre à la place la valeur brute via CharField, tu vois autre chose à revoir ?

Anto59290 commented 1 week ago

pour la base adresse : tu penses qu'il faut revoir l'ux de saisie des adresses ou c'est plutot un probleme de reprise d'historique ? parlons en je pense que ça serait compliqué de pas l'avoir

Ok cela sera géré dans une PR séparée.

Anto59290 commented 1 week ago

bloc de suivi / documents : bold sur les typo de titres des cartes

@CoralieCotonPro sur toutes les infos ? Car sur la maquette je vois du bold uniquement sur le type de doc ("Traça fournisseur")

CoralieCotonPro commented 1 week ago

@Anto59290 "traçabilité fournisseur" est un titre ça pourrait etre un autre nom quoiqu'il en soit c'eest biern si il est en bold pour qu'il ressorte plus que les autres infos de la carte. donc bld sur tous les titres peut importe le type de doc

Capture d’écran 2024-10-01 à 10 46 52
Anto59290 commented 1 week ago

si je modifie un prélèvement en cliquant sur « détecté » alors le mot « détecté » s’affiche sur la carte

Dans la maquette on utilise parfois "Postif" parfois "Détecté", j'ai collé à la maquette mais ça demande peut être une uniformisation

Anto59290 commented 1 week ago

bloc de suivi/contact : lorsque j’ajoute une struture elle se positionne en dessous au lieu de a la ligne

Je n'ai pas compris / reproduit ce point la

CoralieCotonPro commented 1 week ago

Effectivement le terme a changé suite à une indication métier il faut bien parler de "détecté/non détecté" j'ai trouvé 2 occurances de "positif" dans la maquette que j'ai corrigé j'espère qu'il n'y en a pas d'autres

CoralieCotonPro commented 1 week ago
Capture d’écran 2024-10-01 à 17 07 15

je me demandais pourquoi la deuxieme carte ne se metttait pas a suivre dans la grille ?

alanzirek commented 1 week ago

@CoralieCotonPro Il me semble que l'on a opté de séparer les agents des structures pour une meilleure visibilité (1ère ligne pour les agents et 2ème ligne pour les structures). Tu confirmes ?

CoralieCotonPro commented 1 week ago

Je confirmes qu’on avait dit qu’on pouvait filtrer si besoin de les différencier 😃 comme sur la maquette Si la Solution de faire deux bloc est priorisée avant les filtres je n’y vois pas d’inconvénient mais il faudrait ajouter des titres du coup