Skip to content

Conversation

@Cyliaelle
Copy link

@Cyliaelle Cyliaelle commented Nov 23, 2025

Fixes #28

Fix de la gestion du collapse/expand sur les évènements dans la visualisation des commentaires du CEC.

J'ai aussi extrait la partie affichage des évènements dans un composant dédié et rajouté un bouton toggle pour collapse/expand tous les évènements d'un commentaire.

{5CECD03D-CF7D-4083-B3D2-CC8BD11A4618}

Par défaut, les évènements sont tous repliés.

Note : en rajoutant l'icone, j'ai remarqué que le typage des iconNames de AppIcon est incorrect parce que le record est typé <string,string>. On pourrait retirer l'annotation de type et rajouter as const pour corriger la vérif de type, mais je l'ai pas fait parce que ca rajoute pas mal d'erreurs de type qu'il faudrait corriger à droite à gauche.

@hellochloe1989
Copy link

hellochloe1989 commented Nov 23, 2025

C'est bon pour moi, merci ! 😁
Eventuellement on peut éviter d'importer EventList dans FormFields en le référençant sous son nom complet <ViewerCommonEventList> au lieu de <EventList> dans le template, comme c'est le cas pour <ViewerCommonScoreJauge> par exemple.

@Cyliaelle Cyliaelle force-pushed the feat/comment-accordion branch from 679e04b to 80aba3e Compare November 23, 2025 20:17
@hellochloe1989 hellochloe1989 added the amélioration Nouvelle fonctionnalité, optimisation, amélioration du CI… label Nov 23, 2025
@hellochloe1989
Copy link

hellochloe1989 commented Nov 26, 2025

Ils sont beaux ces boutons, et fonctionnels aussi 😁
Eventuellement je ne sais pas s'il faudrait mieux qu'ils soient dans l'ordre "déplier" / "replier" (actuellement c'est l'inverse), mais je ne sais pas ce qui est le plus naturel pour les gens.
En tout cas c'est bon ici, merci !

Copy link

@chitochi chitochi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Top ! 🎉

Concernant l’ordre des deux boutons, effectivement je trouve ça plus intuitif « Tout déplier » puis « Tout replier ».
Déjà parce qu’il y a un « re- », mais aussi parce que par défaut tout est plié, et du coup la première action qu’on peut vouloir faire c’est « Tout déplier ».

@Cyliaelle Cyliaelle force-pushed the feat/comment-accordion branch from c65415d to b32b6f4 Compare December 13, 2025 16:58
@Cyliaelle Cyliaelle force-pushed the feat/comment-accordion branch from 58254dd to 9c74db5 Compare December 13, 2025 17:01
@hellochloe1989
Copy link

J'ai testé après le rebase, ca fonctionne et les boutons sont dans le bon ordre 👍

@Cyliaelle Cyliaelle merged commit b983cec into Fransgenre:main Dec 13, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

amélioration Nouvelle fonctionnalité, optimisation, amélioration du CI…

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Accordéons dans les listes d'évènements

3 participants