Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Tableaux de bord – Revue UX #1806

Closed
10 of 13 tasks
AdelineCelier opened this issue Oct 29, 2024 · 5 comments · Fixed by #1882
Closed
10 of 13 tasks

Tableaux de bord – Revue UX #1806

AdelineCelier opened this issue Oct 29, 2024 · 5 comments · Fixed by #1882
Labels
bug Something isn't working dev feat. enhancement Amélioration/évolution d'une fonctionnalité

Comments

@AdelineCelier
Copy link
Collaborator

AdelineCelier commented Oct 29, 2024

Côté side window

Bugs

Points manquants

  • il manque une pop-up de confirmation quand on quitte le tableau de bord sans l'avoir enregistré. En wording pour cette pop-up :
    • titre : "Enregistrer les modifications"
    • texte : "Vous êtes d'abandonner l'édition du tableau de bord. Voulez-vous enregistrer les modifications avant de quitter ?"
    • boutons : "Quitter sans enregistrer" / "Retourner à l'édition" 👍🏻
  • faire persister les filtres sélectionnés par l'utilisateur a minima en local : il doit pouvoir les retrouver en rouvrant un tableau, et après avoir fermé puis rouvert Env sur son poste. 👍🏻
  • Sur le fonctionnement du scroll : passer les colonnes 2 et 3 sur le même système de "hauteur max" que la 1. ERRATUM: on part sur le systeme d'ancre

Améliorations suppl. identifiées

Côté carto

Point manquant

  • rajouter le bouton "œil" dans le menu briefs sur la carto pour pouvoir afficher/masquer les briefs, comme pour les autres objets de la carte. 👍🏻

Bug (pas prio)

Tableaux Metabase

  • les tableaux Metabase sur les zones reg et les zones AMP n'affichent pas de résultats dès qu'il y a un filtre sélectionné (plutôt à voir avec Morgan). Globalement, ils ne me semblent pas très intuitifs à utiliser.
    => mails envoyés à Morgan sur le sujet le 06/12 et le 09/12/24

Perfs

  • par moments, il semble que l'app ralentisse. Comportements que j'ai pu observer (désolée c'est un peu flou, mais ptet qu'il faut garder en tête le sujet des perfs) :
    • latences quand je tape dans le champ "Commentaires" : ce que je tape met du temps à apparaître 👍🏻
    • latences quand je clique sur l'icône du brief pour figer sa pop-up, et petite latence aussi quand je clique sur "éditer le brief" pour ouvrir la side window 👍🏻
    • en fait j'ai l'impression qu'il y a des latences quand un tableau de bord est ouvert et que je clique sur n'importe quel item de la carte (un signalement, un sémaphore...) : la pop-up met 1 ou 2 secondes à s'afficher et la carte freeze pendant ce temps 👍🏻
@AdelineCelier AdelineCelier changed the title Tableaux de bord – Retours revue design Tableaux de bord – Revue UX Oct 31, 2024
@AdelineCelier AdelineCelier added bug Something isn't working dev feat. enhancement Amélioration/évolution d'une fonctionnalité labels Oct 31, 2024
@AdelineCelier
Copy link
Collaborator Author

AdelineCelier commented Nov 7, 2024

@claire2212 comme convenu, je rajoute ici qques points que j'ai repérés pendant la revue d'aujourd'hui.

Sur la carto, quand on affiche les zones du tableau de bord :

  • quand on clique à un endroit où plusieurs zones se superposent, le curseur qui pointe l'endroit cliqué s'affiche en-dessous des couches, et n'est donc pas très visible. 👍🏻
  • j'ai l'impression que les zones AMP s'affichent au-dessus des zones reg, ce qui fait que si une zone reg est plus petite qu'une AMP, on ne peut pas la sélectionner 👍🏻
  • quand on clique sur une zone depuis la carto ou depuis la liste du tableau de bord, son contour ne se met pas en surbrillance, donc pas facile de repérer laquelle c'est sur la carte. 👍🏻

@AdelineCelier
Copy link
Collaborator Author

@maximeperraultdev donc au final pour les scrolls on ne part pas sur les hauteurs max mais sur le système d'ancres.

Les écrans ont été mis à jour dans le XD, cf. écrans 15 à 17
Les règles de fonctionnement des ancres :

  • il y a un seul grand scroll pour chacune des 3 colonnes
  • aussitôt qu'un des titres de section disparaît (en début ou en fin de colonne), une ancre apparaît pour renvoyer directement vers le titre
  • côté UI
    • il y a une zone-tampon de 24 px entre la barre de filtres et le contenu des colonnes
    • les ancres ont un contour en Gainsboro de 1 px et une ombre portée pour bien se détacher du fond
    • elles sont décalées de 8 px vers la droite par rapport à la colonne pour donner un effet "marque-page" et permettre de lire le plus de contenu possible.

@maximeperraultdev
Copy link
Collaborator

Merci @AdelineCelier. On part la dessus, j'ai modifié l'issue pour dire qu'on part sur cette solution. On a également réglé le problème de perf si tu as du temps de regarder ce serait chouette.

@AdelineCelier
Copy link
Collaborator Author

AdelineCelier commented Dec 9, 2024

Ok tout m'a l'air quasi bon sur ce ticket, modulo les deux points suivants. Je vais créer un nouveau ticket pour le 1er point, et pour le 2è je ne sais pas trop ce que vous voulez faire : est-ce qu'on crée un ticket dédié Env pour ce souci de rendering côté side window ?

1er point

quand je centre la carte sur un signalement relié à une mission qui n'est pas actuellement affichée sur la carte, ça relie le signalement à un cercle vide. On pourrait afficher l'icône de mission ?

Ticket #1806 créé et mis dans le backlog.

2è point

Quand je navigue d'une reg à l'autre, il faut que je clique 2 fois pour afficher le résumé reg : au 1er clic, seule la fenêtre apparaît, mais sans rien, au 2è clic son contenu s'affiche (cf. vidéo)

En restestant, il y a toujours une latence de qques secondes au 1er affichage du résumé d'une zone, mais j'ai l'impression que c'est un pb de rendering sur la side window, car quand je bouge un peu ma fenêtre, le résumé s'affiche tout de suite (pb identifié côté Fish, cf. ticket 3807).

@AdelineCelier AdelineCelier moved this from To Review in INT to Review in INT in progress in Kanban MonitorEnv Dec 9, 2024
@claire2212
Copy link
Collaborator

@Adeline, est-ce que tu peux faire un ticket pour le 2eme point? comme ça on met celui-ci en done

@claire2212 claire2212 moved this from Review in INT in progress to Done in Kanban MonitorEnv Jan 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working dev feat. enhancement Amélioration/évolution d'une fonctionnalité
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

3 participants