Mieux gérer et tester les fonctions de type "directive lit"
Motivation
Depuis que l'on a enlevé la dépendance à lit dans les domaines, on a également créé la classe outils-composants.ts et son test associé. Dans cette classe, on a une fonction de type "directive lit" qui produit des templateResult (formaterNombreEnEntierHtml), et une fonction templateResultToString qui sert uniquement à faire un TU sur formaterNombreEnEntierHtml.
Par ailleurs on a aussi récemment créé une autre fonction de type 'directive lit', dans Lien.ts
Il serait intéressant de mieux tester ces fonctions ( sans templateResultToString qui fait des hypothèses sur l'implem interne de TemplateResult de lit), et aussi de mieux les nommer/ranger.
Utiliser plus fréquemment ce genre de fonction, et en créer de manière plus fréquente qu'aujourd'hui peut nous permettre de simplifier et homogénéiser le html dans nos composants. C'est un intermédiaire intéressant (permet de factoriser de la logique de génération de html, mais sans avoir besoin de créer un web component)
Pistes à explorer
- supprimer templateResultToString
- tester les fonctions de type directive lit via des test cypress
- convention de nommmage sur ces fonctions : par ex
renderNombreEnEntier
, outoHtmlNombreEnEntier
,renderLien
,... - pour le cas particulier de formaterNombreEnEntierHtml :
- voir si elle est vraiment utile ? la supprimer sinon
- la classe "chiffre" ne semble plus utilisée...
- rangement :
- dans le meme fichier que sont composant (comme pour lien() ), ou dans un fichier à coté pour la rendre plus visible ?
- dans des dossiers ou fichiers dédiés pour les directives transverses ?
Remarque initiale et issue liée
The following discussion from !664 (merged) should be addressed:
-
@benjam started a discussion: (+1 comment) FIX: a virer elle est plus utilisée