GDGToulouse / site-devfest-toulouse-2017

DevFest Toulouse website for 2017 forked from https://github.com/gdg-x/hoverboard
https://devfesttoulouse.fr/
Other
0 stars 7 forks source link

Changer le lien de la carte #33

Closed EmmanuelVinas closed 7 years ago

EmmanuelVinas commented 7 years ago

C'est mieux d'avoir la direction dans maps : https://www.google.com/maps/dir//4+Impasse+Emile+Zola,+31700+Blagnac,+France

Actuellement il y a: https://www.google.com/maps/dir/43.644436,1.373449/@43.644436,1.373449,18z Mais maps a pas l'air de comprendre le lien.

davinkevin commented 7 years ago

Côté web, le résultat est identique pour moi (il fait la conversion entre coordonnées Lat/lng et adresse)

EmmanuelVinas commented 7 years ago

image

Ok mais en fait c'est le to qui déconne. Ca peut etre ca aussi : https://www.google.com/maps/dir//43.644436,1.373449

davinkevin commented 7 years ago

@JulienDelRio a dit sur le slack :

Pour la map, il faut mettre le lien vers le CGR sur Maps : https://goo.gl/maps/ihbCaXeLYqz

Vous voulez donc que cela remplace le lien présent sur la flèche de direction (en rouge ci dessous) :

capture d ecran 2017-05-08 a 12 46 23

Entre ton message @EmmanuelVinas et celui de @JulienDelRio, je ne sais donc pas quoi faire 😉

Pour info, la location fait parti du template avec une URL généré à partir des coordonnées : https://github.com/GDGToulouse/site-devfest-toulouse-2017/blob/5c19cd030ea64e1d2331235c45e6787b8aa175ab/src/elements/map-block.html#L157-L157

L'on peut le changer, mais cela va nécessiter de la hard-coder...

Donc, votre dernier mot, vous voulez quoi @EmmanuelVinas et @JulienDelRio ? 😄

Je propose l'url suivante

https://www.google.fr/maps/dir//Cinéma+CGR+Blagnac,+ZAC+du+Grand+Noble,+2+Allée+Emile+Zola,+31700+Blagnac/@43.6442066,1.3722505,17.76z

qui génère le résultat suivant : image

davinkevin commented 7 years ago

La PR est prête, donc il n'y a plus qu'à 😄

JulienDelRio commented 7 years ago

Et il a confusion.

Moi je parle de rendre cliquable l'ensemble de la carte. @EmmanuelVinas parle juste du bouton itinéraire.

davinkevin commented 7 years ago

Ok, donc je merge cette PR pour @EmmanuelVinas et j'ouvre une nouvelle issue pour ta demande