-
Notifications
You must be signed in to change notification settings - Fork 1
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
Comments
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. L'autre solution serait de coder un tooltip custom. Qu'est ce que vous en pensez @tglatt @CoralieCotonPro |
@Anto59290 , peut-on utiliser un fr-link et modifier le class |
C'est ce que je proposais en écrivant |
si ca te va, tu tentes et on voit le résultat |
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 |
@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 |
Parfait @Anto59290 |
@Anto59290 , possible de gérer |
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. |
@Anto59290 , comme vu, tu |
J'ai bien bricolé dans #414 ;) |
Liste des fiches
...
.notifié a l’AC
Sur petit écran:
The text was updated successfully, but these errors were encountered: