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

Map on search results #238

Open
wants to merge 3 commits into
base: dev
Choose a base branch
from
Open

Map on search results #238

wants to merge 3 commits into from

Conversation

fcamblor
Copy link
Collaborator

Cette Pull Request est

  • Un correctif
  • Une nouvelle fonctionnalité

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 Nouveau mode de fonctionnement en prévision de l'éligibilité 18+ : introduction du calendrier des créneaux #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 fcamblor mentioned this pull request May 27, 2021
3 tasks
@fcamblor fcamblor force-pushed the map-on-search-results branch from ab96172 to fc1077d Compare May 27, 2021 21:49
@fcamblor
Copy link
Collaborator Author

fcamblor commented May 31, 2021

@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
Copy link
Collaborator Author

fcamblor commented May 31, 2021

@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
Copy link

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
Copy link

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
Copy link

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
Copy link

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 :

  • Sur le premier écran, on affichera le rectangle dans lequel s'inscrivent les centres affichés dans la liste, c'est bon pour toi ?
  • Comment est-ce que tu vois la carte ? Devrait-elle être navigable ? On peut aussi la bloquer pour qu'elle ne perturbe pas l'expérience (ou le swipe down pour fermer la modale)

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants