[Front] Mettre à jour les stats publiques #2467
Conversation
hmeneuvrier
force-pushed
the
feature/1550-front-maj-stats-publiques
branch
3 times, most recently
from
eb95def
to
88188ca
Compare
hmeneuvrier
changed the title
There was a problem hiding this comment.
Quelque remarques dans le code, d'autres ci-dessous :
- Je n'ai pas trouvé le second point du ticket "Taux de signalements en cours d'instruction", ce n'est plus d'actualité ?
- Je comprends pas pourquoi quand on change 2024/Total sur le graphique en barre on ne garde pas le même ordre en légende -ou bien que ce soit classé par ordre décroissant - (remarque qui sort un peu du scope de la PR)
- Le point "Ajouter un diagramme circulaire plein "DÉSORDRES PAR CATÉGORIE"" c'est transformé en diagramme donut ?
- En bas de page il faudrait que la partie bleuté aille jusqu'au séparateur bleu foncé + les dernières légende des donuts sont coupés chez moi
| captionTerritoryCountPlural: '', | ||
| minValue: 0, | ||
| maxValue: 6000, | ||
| minColor: '#FFFFFF', |
There was a problem hiding this comment.
Il faudrait partir d'une teinte de gris oub leu pale car en blanc sur le fond blanc ca fait vraiment bizarre
There was a problem hiding this comment.
j'ai mis #bfc4d5 au lieu de #FFFFFF pour le démarrage (c'est la couleur de la gironde)
ok pour toi @mathildepoulpux ?
ça a été fait par Emiliien entre-temps. Il n'a juste pas exactement le même nom
ça doit être une histoire d'order dans la PR, je regarderai si on peut faire mieux
bien vu ! J'avais mal lu !
mrlgblrbglrbglmrbglmrbglm ...css... glgblgbmbrl t'as raison je regarderai |
assets/vue/components/common/external/chartjs/HistoChartDoughnut.vue
Outdated
Show resolved
Hide resolved
oups, pas du tout, bien vu ! |
hmeneuvrier
force-pushed
the
feature/1550-front-maj-stats-publiques
branch
from
fea9cec
to
24199a5
Compare
| position: 'bottom', | ||
| align: 'start', | ||
| labels: { | ||
| padding: 20 |
There was a problem hiding this comment.
Les composants chartjs étant gérés en canva, c'est assez difficile je trouve d'avoir un super rendu (retour à la ligne automatique, espacement etc.)
Je n'ai pas réussi @numew à trouver une astuce pour éviter que le bas de la dernière phrase soit coupé.
(de même que mes retours à la ligne en fonction du nombre de caractères ne sont pas ouf en responsive...)
Je pense que la solution serait d'utiliser une html legend, https://www.chartjs.org/docs/latest/samples/legend/html.html, mais c'est un peu une usine à gaz
A voir si on trouve ça indispensable d'avoir une légende nickel ou pas... @mathildepoulpux tu en penses quoi ?
There was a problem hiding this comment.
Oui c'est important de pouvoir lire tout le texte 😁
There was a problem hiding this comment.
Ok je comprends.
Etant en vacances dans quelques heures, soit je passe la main, soit j'attend mon retour de vacances (et le retour d'Emilien) pour me pencher sur cette histoire de légende.
There was a problem hiding this comment.
J'ai magouillé en réduisant l'interlignement pour que ca passe
There was a problem hiding this comment.
Yes c'est bon pour cette MEP !
Je pense que je ferai un ticket "accessibilité" pour les stats pcq au-delà de corriger le problème de marge sur la légende (malheureusement une interligne trop petite c'est pas très accessible), il faut qu'on prévoit un affichage alternatif sous forme de tableau ou de description pour respecter le RGAA
|
Ticket
#1550
Description
Mise à jour des statistiques publiques http://localhost:8080/statistiques
Taux de signalements refusésqui correspond au % de signalements refusés / nb signalements enregistrés (sans les archivés et importés)Le prototype de la nouvelle page dispo sur Adobe Xd.
Changements apportés
Pré-requis
Tests