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

Liste de fiches (revue design) #176

Open
2 tasks done
CoralieCotonPro opened this issue Jul 17, 2024 · 10 comments
Open
2 tasks done

Liste de fiches (revue design) #176

CoralieCotonPro opened this issue Jul 17, 2024 · 10 comments

Comments

@CoralieCotonPro
Copy link
Collaborator

CoralieCotonPro commented Jul 17, 2024

  • pas de filet autour du tableau
  • toute la ligne est cliquable
  • dans la création d'une fiche on peut chercher dans le déroulant nuisible, peut on appliquer le meme champ aux filtres ?

Tasks

  1. sv - fiche détection
    alanzirek
  2. sv - fiche détection
    Anto59290
@CoralieCotonPro CoralieCotonPro added the bug Something isn't working label Jul 17, 2024
@Anto59290
Copy link
Collaborator

pas de filet autour du tableau

Le filet c'est la bordure pour toi ? Je pense qu'on peut la retirer mais elle est de manière native dans le composant

dans la création d'une fiche on peut chercher dans le déroulant nuisible, peut on appliquer le meme champ aux filtres ?

Qu'est ce que tu en pense @alanzirek je crois que l'idée serait d'appliquer choiceJS à tous les champs des filtres. J'ai l'impression que c'est quand même un peu de travail, je ne sais pas si ça ne va pas poser problème au niveau des tests E2E existants.

@alanzirek
Copy link
Collaborator

alanzirek commented Jul 18, 2024

@Anto59290 Je pense qu'il faut l'ajouter uniquement pour le champs organisme nuisible car la liste est assez longue et il y a déjà la possibilité de le faire lors de la création d'une fiche détection.
Je ne vois pas l'utilité de l'appliquer aux autres champs dans le formulaire de recherche (région, état).

@CoralieCotonPro
Copy link
Collaborator Author

CoralieCotonPro commented Jul 18, 2024 via email

@CoralieCotonPro
Copy link
Collaborator Author

CoralieCotonPro commented Jul 18, 2024 via email

@alanzirek alanzirek added this to the MVP SV milestone Jul 18, 2024
@alanzirek
Copy link
Collaborator

juste pour le mvp soit on met se composant a chaque fois qu'on a un
organisme nuisible soit on le met nul part ?

@CoralieCotonPro techniquement le mettre nul part est bien plus arrangeant mais on avait validé avec le métier la nécessité de pouvoir rechercher un organisme nuisible car difficilement envisageable pour un agent d'aller chercher dans la longue liste du champ le bon organisme nuisible.
Donc je pense qu'on peut mettre à minima se composant externe en attendant celui du DSFR pour les organismes nuisibles.
J'ai créé l'issue dans ce sens -> #182

pour la bordure du tableau je crois que ce qui est chelou aussi c'est que ça flotte a mort a l'intérieur, quand on aura toutes les colonnes ça me chquera peut etre moins, peut etre qu'on peut juste mettre un gris plus clair ?

@CoralieCotonPro Je comprends pas ce que tu veux dire par flottement à l'intérieur du tableau ?

@CoralieCotonPro
Copy link
Collaborator Author

@alanzirek ok pour organisme nuisible !
pour le tableau, quand je dis que ça flotte c'est que les éléments ne sont pas calés car ils ont un trop grand écart les uns entre les autres, quand on aura toutes les colonnes (type de fiche, date de création, visibilité, liaisons) ça sera plus dense.
d'ailleurs je me rend compte qu'on avait dit qu'on virait la région car la structure créatrice embarque le nom de la région (draaf Centre Val de Loire par exemple). Par contre je vois sur la recette que c'est possible de faire DAAF Réunion en Pays de la Loire, ce n'est peut etre pas souhaitable ?

@alanzirek
Copy link
Collaborator

alanzirek commented Jul 23, 2024

Par contre je vois sur la recette que c'est possible de faire DAAF Réunion en Pays de la Loire, ce n'est peut etre pas souhaitable ?

@CoralieCotonPro Il ne me semble pas que c'est un prérequis côté métier.

@CoralieCotonPro
Copy link
Collaborator Author

@alanzirek que la DAAF réunion puisse créer une détection à nantes ???

@alanzirek alanzirek changed the title Liste de fiches Liste de fiches (revue design) Jul 24, 2024
@alanzirek alanzirek removed the bug Something isn't working label Jul 24, 2024
@alanzirek
Copy link
Collaborator

alanzirek commented Jul 25, 2024

@alanzirek que la DAAF réunion puisse créer une détection à nantes ???

@CoralieCotonPro oui, pour moi il n'y a pas de vérification automatique à faire entre la structure du créateur de la fiche et les lieux.
D'autant plus que si on va dans ce sens, il fraudrait vérifier pour la métropole : pouvoir ajouter des lieux uniquement si la commune est dans la structure du créateur.
Sachant aussi que certaines structures (MUS...) se sont pas rattachées à un département ou région.

@alanzirek
Copy link
Collaborator

  • toute la ligne est cliquable

J'ai ajouté le ticket #206

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Validation
Development

No branches or pull requests

3 participants