adresse.data.gouv.fr icon indicating copy to clipboard operation
adresse.data.gouv.fr copied to clipboard

Published 20 hours ago • verified publisher icon BaseAdresseNationale

Amélioration de l'affichage des métrics

Open Maxine-Le-Pennec opened this issue 1 year ago • 10 comments

Dans le cadre de l'amélioration de l'affichage des données métriques grâce à des composants réutilisables, deux éléments ont été modifiés ou créés.

  • Des cartes indiquant des données numériques Déjà utilisées dans la page commune, mais ayant besoin d'être adaptée pour s'afficher également sur l'explorateur. Capture d’écran 2022-08-29 à 10 43 45

  • Une barre de progression Cette barre servira dans un premier temps à indiquer le niveau d'avancement des certifications d'adresses sur l'explorateur. Elle est accompagnée d'informations telles que le nombre d'adresses précises étant certifiées/non-certifiées.

Capture d’écran 2022-08-29 à 10 42 35 Capture d’écran 2022-08-29 à 10 42 58

Maxine-Le-Pennec avatar Aug 24 '22 15:08 Maxine-Le-Pennec

Les métriques sous la barre de progression ne semblent pas très pertinentes dans le cas où toutes les adresses sont certifiées ou non.

Nous pourrions juste afficher x adresses non certifiées pour 0% de certification ou x adresses certifiées pour 100%

tmerlier avatar Aug 25 '22 10:08 tmerlier

L'affichage mobile subit un peu le manque de contrainte. Capture d’écran 2022-08-25 à 14 10 15

tmerlier avatar Aug 25 '22 12:08 tmerlier

Les métriques sous la barre de progression ne semblent pas très pertinentes dans le cas où toutes les adresses sont certifiées ou non.

Nous pourrions juste afficher x adresses non certifiées pour 0% de certification ou x adresses certifiées pour 100%

Je pense que ça va être un peu redondant avec l'icône de certification survolable à côté de la barre de progression. Autant ne rien mettre non ?

J'ai testé et je trouve que ça fait un peu trop de répétition : Capture d’écran 2022-08-26 à 10 10 08

Maxine-Le-Pennec avatar Aug 26 '22 08:08 Maxine-Le-Pennec

Non, car on a le nombre d'adresses de la commune.

tmerlier avatar Aug 30 '22 08:08 tmerlier

Non, car on a le nombre d'adresses de la commune.

Résolu via le commit e7bec1b

Maxine-Le-Pennec avatar Aug 30 '22 15:08 Maxine-Le-Pennec

On manque un peu de padding Capture d’écran 2022-08-31 à 11 51 44

tmerlier avatar Aug 31 '22 09:08 tmerlier

On manque un peu de padding Capture d’écran 2022-08-31 à 11 51 44

Résolu via le commit https://github.com/BaseAdresseNationale/adresse.data.gouv.fr/pull/1277#issuecomment-1232722576

Maxine-Le-Pennec avatar Aug 31 '22 13:08 Maxine-Le-Pennec

Je ne vois pas beaucoup de différence avec l'original : PR A gauche le résultat en local sur cette branche et à droite l'original. Ets-ce que ça vaut le coup? Je trouve qu'on comprend moins bien le terme "répertorié"

mmortier avatar Dec 14 '22 12:12 mmortier

Je ne vois pas beaucoup de différence avec l'original : PR A gauche le résultat en local sur cette branche et à droite l'original. Ets-ce que ça vaut le coup? Je trouve qu'on comprend moins bien le terme "répertorié"

Il ne s'agit là que d'une partie des modifications. @Maxine-Le-Pennec a décrit l'intégralité des changements en description de la PR. La majeure partie des évolutions graphiques se trouve surtout sur la carte BAN.

Attention d'ailleurs à ne pas seulement regarder le rendu, mais aussi le code. Cette PR propose de partager des composants entre ces différentes pages.

tmerlier avatar Dec 14 '22 13:12 tmerlier

Ok là sur la carte BAN il y a du changement. Mais comme il y a eu des modifications qui ont été au final supprimées c'était un peu dur à suivre..

mmortier avatar Dec 14 '22 13:12 mmortier