benoitdemaegdt / voieslyonnaises

🚲 Aménagements cyclables à Lyon
https://cyclopolis.fr
Other
22 stars 12 forks source link

feat: add transport-link component #390

Closed Delapouite closed 2 months ago

Delapouite commented 2 months ago

Bonsoir

Voici un début de PR qui permet de mieux faire ressortir les interconnexions des VLs avec le réseau TCL, en reprenant leur charte graphique.

Voici ce que ça rend avec un exemple sur la Voie Lyonnaise 10:

image

Pour le moment, malgré que ce nouveau composant s'appelle transport-link, il n'y a aucun lien, mais on peut imaginer un développement futur où chaque ligne du réseau de transport ait sa page.

Si cette proposition vous convient, je compléterai cette PR en balisant le reste des pages de Voies Lyonnaises où l'on évoque les métro et les trams. (voire funiculaire si besoin)

ThibautChd commented 2 months ago

Chouette petite PR Bruno ! J'aime bien :-)

Pour un rendu optimal on pourrait ne garder que le mot "métro", "tram" ou "bus" avant l'élément coloré. Il faudra réussir à gérer correctement l'affichage de TEOL, le BHNS et le LEL. image image image

Pas besoin de créer une page dédiée sur Cyclopolis : le lien peut simplement renvoyer vers la page dédiée sur le site du SYTRAL.

Delapouite commented 2 months ago

Merci pour les retours.

J'ai bien retiré la répétition du nom de la ligne, on passe donc de "métro A MA" à "métro MA". Aussi, lorsque l'on clique on est désormais redirigé vers la page correspondante coté tcl.fr.

Pour les ajouts de TEOL et compagnie, je propose de les attaquer dans une pull-request indépendante pour ne pas ralentir le merge de celle-ci. Ces nouvelles lignes ont des noms à coucher dehors et ça va etre compliqué de les intégrer dans des espaces de textes si réduits.