Changement de la couleur pour les valeurs 'moyennes' (graphes d'avis de la démarche) pour un meilleur contraste avec la valeur 'mauvais'

[lucaa]

Une nuance d'orange est utilisée pour qualifier les valeurs 'moyennes' à 2 endroits de l'observatoire:

• pour marquer des valeurs "moyennes" des indicateurs sur l'observatoire - voir #1086
• pour marquer les réponses "moyen" des avis des utilisateurs sur les démarches, dans la synthèse des avis - voir #1087 .

Pour un meilleur contraste, ces 2 couleurs doivent passer à $warning-625 au lieu de $warning-425.

[Edit] Après discussions autour de cette tâche, son scope a été restreint à la page des graphes de démarche, et la tâche a été fermée avec ce scope limité. Le titre de la tâche a également évolué pour refléter cela.

[lucaa]

2 questions:

• pour @ugodtn ou @kamalcon : pour l'observatoire, cette couleur est utilisée comme foreground dans 2 tyes de badges: sur du fond blanc, et sur fond $warning-950 , voir https://www.figma.com/file/kG2znZqEzySDPUxiWbrEme/JDMA?node-id=1193%3A65814 . Est-ce que le $warning-950 reste le même ou il faudra le changer aussi?
• pour @aluca ou @ClemDee : ce jeu de couleurs est spécifique à ces pages et à leur styles ou ce sont des couleurs reprises d'une autre configuration de la plateforme, et changer un de ces oranges impacterait aussi tous les autres endroits qui utilisent la même config? S'il y a impact, quels sont les endroits impactés?

[clemdee]

pour @aluca ou @ClemDee : ce jeu de couleurs est spécifique à ces pages et à leur styles ou ce sont des couleurs reprises d'une autre configuration de la plateforme, et changer un de ces oranges impacterait aussi tous les autres endroits qui utilisent la même config? S'il y a impact, quels sont les endroits impactés?

Dans le DSFRTheme, il y a la variable @brand-warning qui vaut @warning-425. Cette variable @brand-warning est utilisée pour :

• la couleur "moyenne" des graphes de la synthèse des avis
• la couleur des alertes de type "warning"

Et de son coté la variable @warning-425 est utilisée directement pour :

• la couleur des valeurs "moyennes" des indicateurs de l'observatoire

Du coup on peut modifier la couleur de l'observatoire pour du @warning-625 sans problème. Pour ce qui est de la couleur des graphes des avis, modifier globalement la couleur @brand-warning modifierai aussi la couleur des alertes de type "warning" (qui elle est correcte), du coup pour moi le mieux à faire est de directement indiquer la valeur @warning-625 dans la DemarchesSheet.

[clemdee]

Du coup j'ai seulement modifié la couleur du graphe, qui est maintenant découplée du @brand-warning et vaut @warning-625 comme convenu.

[lucaa]

Je ne suis pas sure que la solution ci-dessus marche, @ClemDee (mais sans tester).

Les couleurs pour les graphes sont données à chartjs pour dessiner les graphes sur des canvas, IIRC, comme ça: https://github.com/DISIC/observatoire/blob/master/wikis/xwiki/src/main/resources/Demarches/Code/DemarcheAvisStatisticsTab.xml#L502 . Si ce code ne fait plus rien, il faut l'enlever. Si le code du CSS ne fait plus rien, il faut l'enlever (enlever du CSS inutile à ce niveau là fait plus ou moins partie de #1150 ou un nouveau ticket lié, en tout cas).

[clemdee]

Les couleurs pour les graphes sont données à chartjs pour dessiner les graphes sur des canvas, IIRC, comme ça: https://github.com/DISIC/observatoire/blob/master/wikis/xwiki/src/main/resources/Demarches/Code/DemarcheAvisStatisticsTab.xml#L502 .

En effet ça venait bien de là, l'autre endroit était du code qui n'était plus utilisé, que j'ai donc supprimé.

[kamalcon]

On a décidé de changer encore les couleurs des badges sur le tableau

On utilise les couleurs DSFR $xxx-950 sur $xxx-425 [sauf fond orange qui sort du DSFR (#FFDECB au lieu de $warning-950 qui n’est pas assez contrasté avec le rouge $error-950)]

[Spec dans figma]

⚠️Attention, la couleur #FFDECB s’applique seulement sur les badges du tableau. Ce changement ne doit pas être répercuté partout.⚠️

[lucaa]

Évitons de rouvrir de tickets des versions fermées et installées en production, ça met un peu de chaos dans le suivi des modifications de la production (car la moitié de la tâche est en fait faite et livrée alors que l'autre moitié - ce nouveau commentaire - ne l'est ni l'un ni l'autre. La tâche, dans sa totalité, est alors à la fois faite et pas faite, donc difficile à tracer).

C'est vrai que cette tâche parlait initialement de tous les oranges utilisés, donc y compris de l'orange de l'observatoire, on aurait du mieux requalifier la tâche quand on a décidé son scope pour la 4.10 pour qu'elle puisse être correctement fermée dans 4.10. Je le fais maintenant et je re-ferme la tâche pour marquer sa résolution dans 4.10.

Pour les oranges de l'observatoire j'ai créé #1173 qui va traiter cet aspect.