[Front] 306 Mettre à jour le wizard et le rapport vis à vis des maquettes#307
[Front] 306 Mettre à jour le wizard et le rapport vis à vis des maquettes#307AntoineQuesnel wants to merge 16 commits into
Conversation
update: stepper buttons
|
📦 Web extension build artifacts: https://github.com/dataforgoodfr/14_BalanceTesHaters/actions/runs/25865370834/artifacts/6996532152 |
sarod
left a comment
sarod
left a comment
There was a problem hiding this comment.
👍
Quelques commentaires mais rien de bloquant.
A toi de voir ce que tu veux corriger dans cette PR ou tracer ailleur
| data-slot="alert-description" | ||
| className={cn( | ||
| "text-sm text-balance text-muted-foreground md:text-pretty [&_a]:underline [&_a]:underline-offset-3 [&_a]:hover:text-foreground [&_p:not(:last-child)]:mb-4", | ||
| "text-sm text-balance md:text-pretty [&_a]:underline [&_a]:underline-offset-3 [&_a]:hover:text-foreground [&_p:not(:last-child)]:mb-4", |
There was a problem hiding this comment.
Est-ce que la suppresion de text-muted-foreground ne change pas l'apparence des autres alert utilisé dansla Popup et le side Panel?
Peut-être que ClosableAlert devrait surcharger la couleur par le className mais laisser text-muted-foreground par defaut?
| contenus. | ||
| </li> | ||
| </ul> | ||
| Option : demander un constat par commissaire de justice |
There was a problem hiding this comment.
Dans figma l'icône https://lucide.dev/icons/arrow-big-right est utilisé devant option
| {(field) => ( | ||
| <RadioGroup | ||
| defaultValue={ReportOrganizationType.BY_PUBLICATION} | ||
| defaultValue={ReportOrganizationType.BY_AUTHOR} |
There was a problem hiding this comment.
Je pense que BY_AUTHOR devrait être disabled avec un label "en cours de developpement" jusqu'à ce qu'on implémente cette organisation
| isLoading={isLoadingPosts} | ||
| /> | ||
| <CategoryDistribution /> | ||
| <NumberHatefulCommentsKpiCard |
There was a problem hiding this comment.
Dans le figma du rapport seul le nombre de commentaires mavlveillant est affiché le nombre total est masqué.
2 participants
Je voulais partir à la base sur le regroupement par auteur mais j'ai préféré mettre à jour les composants du stepper et du rapport avant de faire ça.
J'anticipe ton commentaire sur le dark mode Sebastien, je ne pense pas que certaines couleurs que j'ai mises soient satisfaisantes mais j'ai l'impression qu'on en a de plus en plus, pas toujours cohérentes.
Je souhaite discuter avec Olga du design system pour voir comment elle a penser ça et voir comment adapter au darkmode.