masquer les chemins sous-terrains (stations de métro)

publié le 11 décembre 2025, mis à jour 11 décembre 2025

etienneJr

https://cartes.app/?style=base#18.25/48.829686/2.377587

hgcl
Bonjour, j'adorerais contribuer à ce chouette projet. Cette issue est-elle libre et adaptée comme première contribution ? Merci !
etienneJr
Salut,

j'adorerais contribuer à ce chouette projet.

Super nouvelle 😍

Cette issue est-elle libre

Oui, et il y a aussi tous les autres tickets que je me suis auto-assigné pour les retrouver facilement par filtrage, mais sur lesquels je n'ai pas commencé à bosser.

adaptée comme première contribution ?

Oui à condition que tu aies déjà une petite idée de comment est codée une feuille de style maplibre. Sinon ça risque de faire une marche un peu grande à passer. La doc est ici https://maplibre.org/maplibre-style-spec/layers/

Je pense que la modif est à faire dans ce layer: https://codeberg.org/cartes/web/src/commit/2e9692ad58a0cee086aaf5345b6f6dfb59d58dac/app/styles/roadLayers.ts#L756

Ça me fait penser qu'il faut que je documente plus ces feuilles de style de carte maintenant que ça gère en même temps les 3 styles base, comaps et outdoors. Ca devient un peu le bazar, ce qui ne facilite pas l'arrivée de nouveaux contributeurs comme toi...

Au delà du fond de carte maplibre, c'est quoi qui te motiverait comme fonctions sur lequelles contribuer ?
hgcl
Super ! Merci pour l'aiguillage 😊

Au delà du fond de carte maplibre, c'est quoi qui te motiverait comme fonctions sur lequelles contribuer ?

Je profite de cette occasion pour apprendre à programmer au sein d'une équipe. Je n'ai que participé à de minuscules projets (solo ou à deux) jusqu'à maintenant. Donc finalement, je suis ouverte à tout et suis motivée à voir de la diversité.

Par rapport au ticket spécifiquement. Est-ce qu'on veut filtrer les stations de métro spécifiquement ou tout chemin sous-terrain ? Si l'on veut tous les filtrer, j'imagine que quelque chose comme ça ferait l'affaire (dans id: Path) ?
```
filter: [
				'all',
				['==', ['geometry-type'], 'LineString'],
				[
					'in',
					['get', 'subclass'],
					[
						'literal',
						['path', 'pedestrian', 'footway', 'platform', 'bridleway'],
					], 
				],
				['!=', ['get', 'brunnel'], 'tunnel'],
				['>', ['coalesce', ['get', 'layer'], 0], 0],   // <--- ajouter cette ligne
			],
```
etienneJr

Donc finalement, je suis ouverte à tout et suis motivée à voir de la diversité.

Youpi ! Est-ce que tu as pu démarrer le serveur en local en suivant les infos du CONTRIBUTING.md ?

Est-ce qu'on veut filtrer les stations de métro spécifiquement ou tout chemin sous-terrain ?

Hum, non, on n'a pas cette info dans les tuiles, ni même directement dans les tags de l'élément dans OSM. On va être obligés de masquer tout chemin sous-terrain. Il y a peut-être des endroits où ça posera problème (je n'en ai pas en tête comme ça), mais tant pis on n'a pas le choix.

j'imagine que quelque chose comme ça ferait l'affaire

Super idée le coalesce pour gérer le cas où la propriété layer est absente, je n'y aurais même pas pensé ! 😅 Je vais d'ailleurs pouvoir l'utiliser pour simplifier d'autres endroits où j'ai fait des tests compliqués avec des ['has', 'layer']; En revanche il faut mettre un >= pour ne pas masquer les chemins au niveau de la rue. Et en testant, je remarque qu'il va falloir mettre le même filtre sur le layer des escaliers, ici : https://codeberg.org/cartes/web/src/commit/2e9692ad58a0cee086aaf5345b6f6dfb59d58dac/app/styles/roadLayers.ts#L816

Je te laisse faire ?
hgcl

Youpi ! Est-ce que tu as pu démarrer le serveur en local en suivant les infos du CONTRIBUTING.md ?

Oui, c'est tout bon. J'ai réussi à le démarrer sans souci !

Je te laisse faire ?

Je m'en occupe ! 😊
hgcl
@etienneJr il me semble avoir un souci de permission quand j'essaie de push une nouvelle branche pour créer une PR. Est-ce qu'il me faut des permissions supplémentaires ou est-ce que j'ai peut-être mal fait le setup ?
etienneJr
En effet, il faut une permission pour avoir le droit de pousser une branche dans ce repo... J'ai demandé à @laem de t'ajouter dans l'équipe liée au repo pour que tu aies les droits associés (je n'ai moi même pas les droits pour t'y ajouter). En attendant, tu peux bosser dans un fork (ça implique de changer l'origin de ton dossier local), pousser la branche dans ton fork, et faire une PR. Ça t'irait ?
etienneJr
@hgcl n'hésite pas à te signaler, comme tu l'as fait sur celui là, sur d'autres tickets qui te motiveraient, pour que je puisse t'aiguiller à nouveau si je peux.
hgcl
Super @etienneJr ! J'ai donc fait la PR ci-dessus.

J'irai aussi jeter un coup d'œil aux autres tickets, merci encore pour ton aide !
etienneJr
@etienneJr wrote in https://codeberg.org/cartes/web/issues/1283#issuecomment-8722224:

J'ai demandé à @laem de t'ajouter dans l'équipe liée au repo pour que tu aies les droits associés

C'est bon pour vous 3 @hgcl @pmiossec et @waterced, plus besoin de fork, vous pouvez bosser directement dans ce repo et pousser vos branches locales pour faire les PR.
pmiossec
@etienneJr merci pour cette confiance rapidement accordée mais je vais continuer avec mon fork, c'est un workflow auquel je suis habitué... 😉
etienneJr
@pmiossec wrote in https://codeberg.org/cartes/web/issues/1283#issuecomment-8793084:

merci pour cette confiance rapidement accordée mais je vais continuer avec mon fork, c'est un workflow auquel je suis habitué... :wink:

bah en fait, c'est plus pour nous que pour toi 😅 car on n'a pas les droits pour bosser dans ton fork. Pour pouvoir tester tes PR, voire pousser des commits dedans, on va être bloqués si c'est dans ton fork. Ca sera beaucoup plus facile de bosser à plusieurs sur une PR si tu bosses dans ce repo. Bon après, je sais bien que c'est assez rare d'être plusieurs à bosser sur la même PR, tu peux continuer comme ça et puis on verra si on se retrouve bloqués à un moment.

✏️ Participer à la discussion