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

Tableau des fiches | modification de l'UI #393

Closed
4 of 6 tasks
tglatt opened this issue Nov 12, 2024 · 11 comments
Closed
4 of 6 tasks

Tableau des fiches | modification de l'UI #393

tglatt opened this issue Nov 12, 2024 · 11 comments
Assignees
Milestone

Comments

@tglatt
Copy link
Collaborator

tglatt commented Nov 12, 2024

  • supprimer le titre Liste des fiches
  • réduire les espaces horizontaux (dans le but d’avoir plus de lignes affichées dans la liste)
  • réduire la colonne numéro de fiche
  • ON ne fait que deux lignes maximum, après on tronque avec ....
  • On voir le nom de complet de l'ON dans un tooltip
  • ajouter un tooltip sur icône cloche => notifié a l’AC

Sur petit écran:
Capture d’écran 2024-11-12 à 18 24 15

@tglatt tglatt added this to the Soft Launch milestone Nov 12, 2024
@tglatt tglatt added this to Sèves Nov 12, 2024
@tglatt tglatt moved this to ToDo in Sèves Nov 12, 2024
@Anto59290 Anto59290 self-assigned this Nov 12, 2024
@Anto59290
Copy link
Collaborator

J'ai commencé les premiers points ici : #396

Par contre pour les deux derniers ce n'est clairement pas évident. On est déjà obligé de tricher pour les liens dans le tableau car ce n'est pas prévu dans le DSFR. Les tooltips sont fait pour être utilisés sur des liens (https://www.systeme-de-design.gouv.fr/composants-et-modeles/composants/infobulle/). Si on transforme nos liens en liens DSFR (fr-link) ça va casser tout le design et on devra donc écraser les règles du DSFR pour retourner à des règles textes.
Idem pour le composant bagde il ne propose pas d'intégration avec le composant tooltip, intégrer le composant tooltip me semble aller à l'encontre du DSFR sur ces deux composants.

L'autre solution serait de coder un tooltip custom. Qu'est ce que vous en pensez @tglatt @CoralieCotonPro

@tglatt
Copy link
Collaborator Author

tglatt commented Nov 13, 2024

@Anto59290 , peut-on utiliser un fr-link et modifier le class fr-link uniquement pour le tableau?

@Anto59290
Copy link
Collaborator

C'est ce que je proposais en écrivant on devra donc écraser les règles du DSFR pour retourner à des règles textes(désolé si ce n'était pas clair). C'est possible mais je ne garanti pas le rendu du composant tooltop qui n'a pas du tout été prévu pour ce genre de situation

@tglatt
Copy link
Collaborator Author

tglatt commented Nov 13, 2024

si ca te va, tu tentes et on voit le résultat

@CoralieCotonPro
Copy link
Collaborator

CoralieCotonPro commented Nov 13, 2024

Au SRAL d'angers ils ont dit que c'était primordial d'avoir la totalité du nom qui s'affiche. en effet les ON ont un nom latin + un petit nom "français" mais les noms latin sont a cours de refonte (BDD EPPO global database mise a jour au fil de l'eau) donc ils ont non seulement besoin d'avoir toute la ligne visible mais EN PLUS dans les champs combo search/déroulant il faut que ça cherche dans tous les termes même ceux entre parenthèses. sur ce dernier point je ne suis pas sure du comportement j'ai fait le test et ça marche pas a tous les coups . exemple : enroulement chlorotique de l’abricotier vs candidatus phytoplasma prunorum

@Anto59290
Copy link
Collaborator

@tglatt je propose déjà de review/merger la PR existante et de voir les deux derniers points ensuite afin que l'on puisse déjà bénéficier des premiers changement en recette et pour la suite sans attendre d'avantage de prise de décisions

@tglatt
Copy link
Collaborator Author

tglatt commented Nov 13, 2024

Parfait @Anto59290

@tglatt tglatt closed this as completed Nov 14, 2024
@tglatt
Copy link
Collaborator Author

tglatt commented Nov 14, 2024

@Anto59290 , possible de gérer Détection et la date sur une ligne et non deux?

Image

@tglatt tglatt reopened this Nov 14, 2024
@Anto59290
Copy link
Collaborator

D'après moi si on veut faire ça il ne faut plus juste faire du bricolage, mais faire du vrai responsive car en fonction de la taille de l'écran et des infos affichées ça pourrait reporter le problème sur d'autres colonnes. Je peux tester quelque chose, mais je pense que ça va juste deplacer le problème ailleurs.

@tglatt
Copy link
Collaborator Author

tglatt commented Nov 14, 2024

@Anto59290 , comme vu, tu bricolesun truc pour avoir Détection et la date de création sur une ligne et on reprendra proprement le fonctionnement en mode responsive plus tard

@tglatt tglatt moved this from Recette to En dev in Sèves Nov 14, 2024
@Anto59290
Copy link
Collaborator

J'ai bien bricolé dans #414 ;)

@tglatt tglatt closed this as completed Nov 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Terminé
Development

No branches or pull requests

3 participants