SocialGouv / dashlord-actions

GitHub actions for DashLord
Apache License 2.0
2 stars 13 forks source link

Remove ambiguity on Accessibility colonnes #288

Closed astranchet closed 9 months ago

astranchet commented 11 months ago

Les utilisateurs pensent que "100 %" en accessibilité est une bonne nouvelle. Hors, il s'agit uniquement de tests automatiques. Je propose d'utiliser un affichage comme celui des trackers. Par exemple :

Et renommer la colonne en "Tests automatiques d'accessibilité"

Capture d’écran 2023-12-05 à 09 31 40

revolunet commented 10 months ago

Alors actuellement je n'ai pas le "nombre d'erreurs" d'accessibilité détectées dans les données, seulement un score de 0 à 1 qui est convertit en % à l'affichage. (a propos de ce score)

Capture d’écran 2023-12-10 à 23 05 10

D'autres idées ?

astranchet commented 10 months ago

Aïe, en console, j'avais l'impression que le nombre d'erreur était proposé aussi : https://github.com/GoogleChrome/lighthouse-ci

Pour les couleurs, on peut toujours améliorer :

Pour le texte, je pense qu'il faut vraiment s'écarter du pourcentage. À défaut du nombre d'erreurs, on pourrait alors avoir :

revolunet commented 10 months ago

Nouvelle proposition

Capture d’écran 2023-12-14 à 11 11 11

Avec cet algo pour définir la lettre affichée :

 if (value >= 1) {
  return "✔";
} else if (value >= 0.85) {
  return "B";
} else if (value >= 0.7) {
  return "C";
} else {
  return "⚠️";
}
astranchet commented 10 months ago

Possible de mettre le B en orange plutôt que vert ? Sinon pourquoi pas oui. Le warning pourrait aussi être un F.

revolunet commented 9 months ago

Voici les règles pour l'affichage de ce score lighthouse :

Score lighthouse Couleur Texte
100% success
>= 85% warning B
>= 70% danger C
< 70% danger ⚠️
Capture d’écran 2024-01-10 à 00 33 33

good ?

astranchet commented 9 months ago

Parfait ! Merci beaucoup !

github-actions[bot] commented 9 months ago

:tada: This issue has been resolved in version 1.33.1 :tada:

The release is available on GitHub release

Your semantic-release bot :package::rocket: