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

Feature : affichage du tracé d'une rando au passage de la souris sur la card #694

Conversation

JordaneNS
Copy link
Collaborator

Détails de la PR :

  • affichage du tracé de la rando au hover de la card

Screenshots

image

@camillemonchicourt
Copy link
Member

Cool, le chargement reste assez réactif ?
En lien avec #368

@JordaneNS
Copy link
Collaborator Author

Je mets en démo pour que tu puisse tester, je t'envoie le lien dès que c'est déployé. Mais oui c'est très réactif ! Il n'y a pas de "call api" supplémentaire

@camillemonchicourt
Copy link
Member

OK, alors là je suis surpris.
J'étais persuadé qu'on ne disposait pas des tracés linéaires des randos au niveau du client, pour avoir un chargement le plus léger possible.
Si on récupère les tracés précis de toutes les randos dans tous les cas au niveau du navigateur, alors ça nous aide pour cette fonctionnalité, mais c'est à analyser sur la lourdeur du chargement des contenus de la page Recherche.

@JordaneNS
Copy link
Collaborator Author

JordaneNS commented Jun 14, 2022

image

@camillemonchicourt En gros à la place de hoveredPoint il y avait le selectableMarker qui permettait au clic sur le marker de tracer le chemin sur la map. On a remplacé ce selectableMarker par le hoveredPoint pour que le tracé se dessine automatiquement au survol. Donc aucun call api rajouté, les call sont déjà présents à l'arrivée sur la page et à la pagination :)

(https://demo.geotrekrando.natural-solutions.eu)

@camillemonchicourt
Copy link
Member

OK merci pour les précisions.
En effet, ça fonctionne très bien.

@camillemonchicourt camillemonchicourt merged commit 8136e17 into GeotrekCE:develop Jun 14, 2022
@camillemonchicourt
Copy link
Member

Salut @JordaneNS, je ne sais pas si c'est lié à cette évolution.
Mais j'ai l'impression qu'il y a eu un effet de bord.
Auparavant, quand on cliquait sur une itinéraire sur la carte de recherche, son tracé s'affichait.

Capture d’écran de 2022-06-24 18-01-10

Cela ne semble plus être le cas depuis qu'on a intégré cette PR : https://rando.ecrins-parcnational.fr/search?practices=4
Capture d’écran de 2022-06-24 18-02-10

Si il s'agit bien d'un effet de bord, à voir si tu peux le corriger ?
Et si c'est possible, en passant, ajouter l'affichage du tracé au survol d'une rando sur la carte, pas uniquement au clic sur la rando sur la carte ?

Merci.

@JordaneNS
Copy link
Collaborator Author

JordaneNS commented Jun 27, 2022 via email

@JordaneNS
Copy link
Collaborator Author

JordaneNS commented Jun 27, 2022 via email

@camillemonchicourt
Copy link
Member

OK en plus tu l'avais indiqué.
C'est moi qui n'ai pas capté.
Merci de l'avoir remis au clic sur la carte et de l'avoir ajouté au survol sur la carte, c'est super comme ça.

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.

None yet

3 participants