9
103
Fork
You've already forked web
30

fix: masquer les canaux sous-terrains #1556

Merged
etienneJr merged 2 commits from hide-underground-canals into master 2026年02月15日 21:21:20 +01:00
Member
Copy link

@etienneJr voilà le fix lié aux changements de #1251.

Je n'ai pas pu réutiliser ['>=', ['coalesce', ['get', 'layer'], 0], 0] dans ce cas de figure, je ne suis pas sûre de pourquoi... Par contre, cette technique-ci a l'air de bien fonctionner aussi. Je cache par contre uniquement les waterway qui ont la classe canal — c'est bien ce qu'on voulait ?

Closes #1251

@etienneJr voilà le fix lié aux changements de #1251. Je n'ai pas pu réutiliser `['>=', ['coalesce', ['get', 'layer'], 0], 0]` dans ce cas de figure, je ne suis pas sûre de pourquoi... Par contre, cette technique-ci a l'air de bien fonctionner aussi. Je cache par contre uniquement les `waterway` qui ont la classe `canal` — c'est bien ce qu'on voulait ? Closes #1251

@hgcl wrote in #1556 (comment):

Je n'ai pas pu réutiliser ['>=', ['coalesce', ['get', 'layer'], 0], 0] dans ce cas de figure, je ne suis pas sûre de pourquoi...

Je crois savoir pourquoi. Car j'ai eu le même pb hier sur les itinéraires de rando. Je me suis trompé quand j'ai modifié les tuiles. J'ai utilisé la fonction Attribute() qui enregistre les valeurs en texte, au lieu d'utiliser AttributeNumeric() qui enregistre les valeurs en nombre. Tu pourrais utiliser ['to-number, ['get', 'layer']]` pour transformer le texte en nombre, mais c'est la peine, je m'occupe de modifier le script des tuiles ce soir, ça sera bon demain soir.

@hgcl wrote in https://codeberg.org/cartes/web/pulls/1556#issue-3437692: > Je n'ai pas pu réutiliser `['>=', ['coalesce', ['get', 'layer'], 0], 0]` dans ce cas de figure, je ne suis pas sûre de pourquoi... Je crois savoir pourquoi. Car j'ai eu le même pb hier sur les itinéraires de rando. Je me suis trompé quand j'ai modifié les tuiles. J'ai utilisé la fonction `Attribute()` qui enregistre les valeurs en texte, au lieu d'utiliser `AttributeNumeric()` qui enregistre les valeurs en nombre. Tu pourrais utiliser `['to-number`, ['get', 'layer']]` pour transformer le texte en nombre, mais c'est la peine, je m'occupe de modifier le script des tuiles ce soir, ça sera bon demain soir.

@hgcl wrote in #1556 (comment):

Je cache par contre uniquement les waterway qui ont la classe canal — c'est bien ce qu'on voulait ?

As tu essayé de ne pas les masquer totalement, mais de les mettre quasi transparent ?
via un truc du genre (à tester) : 'line-opacity': ['case', ['==', 'class', 'canal'], 0.2, ['==', 'layer', -1], 0.2, 1]

@hgcl wrote in https://codeberg.org/cartes/web/pulls/1556#issue-3437692: > Je cache par contre uniquement les `waterway` qui ont la classe `canal` — c'est bien ce qu'on voulait ? As tu essayé de ne pas les masquer totalement, mais de les mettre quasi transparent ? via un truc du genre (à tester) : `'line-opacity': ['case', ['==', 'class', 'canal'], 0.2, ['==', 'layer', -1], 0.2, 1]`
Author
Member
Copy link

Bonne idée de les laisser visible avec moins d'opacité. Je crois que ça marche bien avec ce nouveau changement.

Bonne idée de les laisser visible avec moins d'opacité. Je crois que ça marche bien avec ce nouveau changement.

C'est parfait, merci ! je fusionne.
(et désolé pour le délai de review)

C'est parfait, merci ! je fusionne. (et désolé pour le délai de review)
etienneJr deleted branch hide-underground-canals 2026年02月15日 21:21:23 +01:00
Sign in to join this conversation.
No reviewers
Labels
Clear labels
🎨 Design
À propos de l'interface et de l'UX
💡 Idée
Nouvelle feature à intégrer
question
Further information is requested
🌍 OSM
À propos de l'affichage et l'édition des éléments OSM
🌱 Amélioration
Amélioration/fix de l'existant
🐞 Bug
Something isn't working
📊 SEO
📋 Catégories
À propos des catégories de recherche et de l'annuaire
📖 Wiki
📝Avis
A propos de la fonction avis et de bluesky
📢 Oyez
📷 Photos
A propos des photos (sources, affichage)
🔍Recherche
A propos de la barre de recherche et de photon
🔥 Actuel
🔧 tech
🖌 Styles
Styles de la carte
🧝i18n
🧪 R&D
🚍 Transports
🚙 Voiture
🚨Prioritaire
Ticket à traiter rapidement
🚲Vélo
Cartes
climat
Communication
demo
Cette branche doit être déployée comme une démo
dependencies
Pull requests that update a dependency file
javascript
Pull requests that update javascript code
Pas urgent
Plein-air
Tuiles
Verso
Vieux
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
cartes/web!1556
Reference in a new issue
cartes/web
No description provided.
Delete branch "hide-underground-canals"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?