CovidTrackerFr / vitemadose-front

Interface utilisateur de l'outil ViteMaDose
https://vitemadose.covidtracker.fr
Other
80 stars 51 forks source link

Map on search results #238

Open fcamblor opened 3 years ago

fcamblor commented 3 years ago

Cette Pull Request est

Description

Il s'agit d'une ré-application des commits des PR #228 & #236 de @Joxit, visant à intégrer une carte à la liste de résultats de VMD, et qui ont été revert lors de #237. image

J'aimerais discuter d'au moins 3 points que je juge importants avant d'intégrer cette PR :

  1. Le fond de carte d'OSM a été remplacé par celui de JAWG (https://www.jawg.io/fr/) qui nous fournit gracieusement une API key. @Joxit travaillant pour JAWG (qui est un éditeur FR cocorico), ça peut envoyer le message qu'il s'agit d'un placement produit. Étant donné que la demande n'est pas partie de VMD, j'aimerais être à l'aise avec les autres membres de l'équipe vis à vis de cette problématique.

  2. La forme utilisée pour la carte : actuellement il s'agit d'un onglet (qui est incompatible avec #233), j'aimerais qu'on explore d'autres options (par exemple, un affichage en popup)

  3. Le bien fondé de la fonctionnalité : certains sont contre tout affichage de carte, d'autres sont pour l'affichage d'une carte par centre, et enfin la proposition ici est d'afficher tous les centres de la liste sur la carte (ce qui peut faire beaucoup dans les zones denses en lieux de vaccination comme Paris). Factuellement, l'implémentation jawg rajoute un overhead d'environ 30% à nos assets totaux en prod (cf ce commentaire)

fcamblor commented 3 years ago

@Joxit @loicortola Hi here !

Loïc, par rapport à ton message sur #237, continuons la discussion ici :)

Pour information, nous avons supprimé l'onglet chronodoses hier soir, et nous sommes en train de transformer l'écran de résultats pour y intégrer une notion temporelle (affichage du nombre de créneaux par jours), ce qui risque d'être l'info importante pour les utilisateurs dans les jours/semaines à venir.

Tout ceci est visible dans la PR #233 (état actuel déployé ici : https://dev.vitemado.se/disponibilite-generale/) => j'ai le sentiment qu'afficher la carte sous forme d'onglet ne soit pas top niveau ergonomique, d'où ma proposition de peut-être déporter ça dans une popup.

Autre point de vigilance : nous avons également rajouté un "infinite scroll" à la page pour ne pas charger tous les centres dans le DOM (note importante : nous avons tous les centres en mémoire, l'infinite scroll ne va pas "chercher sur le back" les données pour afficher les fenêtres glissantes de résultats).

Voila pour l'état des lieux.

Sur le mode de fonctionnement, pas de soucis pour faire une Visio si vous préférez (les commentaires sur l'issue me vont bien aussi :))

J'invoque @francoisbouchet dans la discussion car il est généralement de bon conseil quand il s'agit d'ergonomie (je vous préviens, il va vous falloir vous armer de quelques arguments pour réussir à le convaincre d'afficher un composant map car il a un avis assez tranché sur la question ... pas contre JAWG mais plus sur la complexité qu'une map représente en nombre d'intéractions utilisateur ... c'est notre monsieur KISS de l'équipe, et ça nous a plutôt bien réussi jusqu'à maintenant ;-) )

fcamblor commented 3 years ago

@Joxit @loicortola Je réalise que maintenant, mais le fait que je sois le créateur de la PR qui se base sur une PR du repo ne doit pas vous permettre de pusher dessus.

@Joxit possible de re-créer une nouvelle PR depuis ton repository ? (tu peux reprendre les commits tels quels si tu ne veux pas t'embêter (je pourrai m'occuper de la résolution des conflits))

(je fermerai cette PR une fois que ce sera fait, pour pas oublier :-) )

loicortola commented 3 years ago

Hello :) merci @fcamblor, on se rejoint là pour la suite ! Enchanté @francoisBouchet, super de pouvoir être challengés sur l'usage. Tout dépend de la manière dont tu cherches à utiliser la carte. Je suis plutôt contre la manière "clickodrôme" ou la personne navigue sur la map pour trouver le bon endroit.

Par contre, affiner une liste de résultats après une recherche géographique (géocodage) avec un "qu'est-ce qui est vraiment le plus pratique pour moi" fonctionne très bien. Une adresse évoque peu de choses pour les gens. Par contre, voir qu'il y a leur station de métro qui passe à côté ou voir que sur leur route habituelle il y en a un donne une lecture visuellement intéressante.

Il y a aussi un débat sur l'expérience Mobile vs Desktop qui fait sens ^^

francoisBouchet commented 3 years ago

Hello hello,

pour ce qui est de mon point de vue ; je tiens à rappeler que l'on est à "1 clic" de propulser l'utilisateur/l'utilisatrice vers sa destination : la plateforme de réservation. donc rajouter un module, qui nécessite une dizaine d'interactions a minima (zoomer/dézoomer, swiper, cliquer...) pour finalement parvenir au même but me paraît effectivement pas très efficace.

notre page résultats est une page de proposition de créneaux, et la géolocalisation se pose après avoir suscité un intérêt sur une date (jour+horaire) de vaccination. C'est une information complémentaire au choix d'une date , qui ne nécessite rien de plus que d'afficher l'adresse du lieu de vax sur une map.

L'utilisation évoquée par Loïc est une "recherche de centre/lieu de vaccination"... ça nous replace un step en arrière car la date reste à choisir (on pourrait afficher la liste des dates dispos dans la popup map&fiche-centre) ou en tout cas n'est pas garantie (on pourrait directement renvoyer vers la page centre de la plateforme)...

perso je pense que si notre trafic apporté est aussi quali pour les plateformes de santé (cf. les taux de conversion annoncés par Stanislas Niox-C sur Docto en comparaison de ceux de santé.fr qui n'affichait que "des centres"), c'est parce que l'on apportait une bonne garantie de disponibilité sur les créneaux trouvés (un peu malmenée avec les chronodoses, mais ceci est une autre histoire).

francoisBouchet commented 3 years ago

je m'étais mockupé les écrans ci-joint semaine dernière suite à la discussion avec la team au sujet de la map (et qui rejoignent le point de vue exprimé par Loïc).

liste de centre - taper un item centre la map sur ce dernier (problème de la hauteur dispo sur les smartphones...) MAP_phone 00 liste centres

fiche centre qu'on pourrait afficher en tapant sur l'adresse sur la card résultat actuelle... ou sur voir les créneaux de l'écran liste... MAP_phone 01 fiche centre

loicortola commented 3 years ago

Hello @francoisBouchet c'est top comme ça. Est-ce que tout le monde est OK si on l'implémente de cette manière ? Pour s'assurer des bons détails :

Merci de tes réponses, je passe la balle à @Joxit :D