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

feat: display counters on each line map #219

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Delapouite
Copy link
Contributor

Bonsoir

Cette PR ajoute l'affichage des compteurs sur la page de chaque voie lyonnaise où ils s'appliquent.

Exemple avec les deux compteurs ajoutés recemment sur la voie 3:

image

@benoitdemaegdt
Copy link
Owner

Hello @Delapouite 👋

Merci pour cette nouvelle PR ! 👍

On avait discuté avec @ThibautChd le fait de mettre les compteurs directement sur la map.
Je trouve ta solution élégante, dans le sens ou elle ne va afficher que les compteurs sur lesquels la VL passe effectivement.

Par contre, je trouve ces points bleus trop lourds, et trop visibles sur la map, alors que c'est une info qui me semble secondaire par rapport au tracés et aux avancements des tronçons.

@ThibautChd je suis bien preneur de ton avis, sachant que ça n'apparait pas sur la carte principale (celle avec toutes les VLs), mais que sur les petites cartes, dans les pages de chaque VL)

Capture d’écran 2023-12-06 à 08 30 09
Capture d’écran 2023-12-06 à 08 30 17
Capture d’écran 2023-12-06 à 08 30 27

@benoitdemaegdt
Copy link
Owner

(au passage, ça m'a fait voir qu'on a un ptit bug : il n'y a plus le bouton légende sur cette carte, alors qu'il serait bien utile 🙈 )

@benoitdemaegdt
Copy link
Owner

Discuté avec Thibaut : on ne va pas merger ça tel quel.
L'idée serait d'avoir quelque chose de moins visible. Peut-être une gestion des layers, qui permette de rendre visible / cacher une layer.
Je propose de laisser en ouverte cette PR, le temps qu'on développe la brique sous-jacente 👍

@Delapouite
Copy link
Contributor Author

Merci pour le retour, Rien d'urgent de mon coté.

@PhilippeWhite
Copy link

Bonjour,
Faire figurer les compteurs sur la carte de la ligne pourrait se réfléchir dans une logique de mettre en valeur la dynamique de fréquentation et à la lier avec la mise en place progressive des aménagements.
On pourrait avoir une rubrique "fréquentation" sur la page de la VL, avec les différents compteurs et la vision annuelle de la fréquentation, en indiquant les années d'amenagement au niveau du compteur.

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

Successfully merging this pull request may close these issues.

3 participants