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

Icônes de carte personnalisés #776

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Icônes de carte personnalisés #776

wants to merge 3 commits into from

Conversation

laem
Copy link
Collaborator

@laem laem commented Jan 20, 2025

image

@laem laem added the 🎨 Design À propos de l'interface et de l'UX label Jan 21, 2025
@etienneJr
Copy link
Contributor

etienneJr commented Jan 26, 2025

Salut @laem , j'ai voulu pousser un commit (avec un fallback vers les sprites sprite si les sprites cartes ne sont pas dispos) directement dans cette PR mais je n'ai pas réussi, sûrement à cause d'une question de droits d'accès à cette branche. Est-ce que je dois d'abord copier la branche dans mon fork, puis pousser dans cette branche forkée ? ça fera le lien avec cette PR en cours ? ou faut que je PR dans cette branche ?
En plus j'avais voulu rebaser et j'avais un conflit sur bun.lockb que je ne savais pas comment gérer.

Et à propos des sprites, tu préfères rester sur spritezero ? ou bien tu veux que j'essaye la technique que j'ai proposée ici qui permettrait d'utiliser directement les svg ?

@etienneJr
Copy link
Contributor

@laem J'ai réussi à faire ce à quoi je pensais :

  • en parcourant la liste des catégories, on charge au départ tous les svg dans la carte (je l'ai mis dans useAddMap mais je me doute que ça n'est pas le bon endroit ?)
  • dans le style, pour les Other POI, on ajoute un fallback :
    • si le svg est dispo, on l'utilise
    • sinon on utilise le sprite png du style d'origine

Tu peux voir le code correspondant ici. Est-ce que ça te conviendrait ?
Et ça donne ça (tiens ya un restaurant pour lequel ça n'a pas marché, à investiguer) :
image

@etienneJr
Copy link
Contributor

etienneJr commented Jan 29, 2025

tiens ya un restaurant pour lequel ça n'a pas marché, à investiguer

J'ai identifié que ça ne marche pas quand dans le pmtiles le poi a la propriété level=0, mais je ne vois pas vraiment le lien 🤔​ Si je modifie d'autres propriétés du layer Other POI, elles ne sont pas prises en compte non plus, donc ça donne l'impression qu'il utilise un autre layer pour afficher les objets qui ont level=0. Est-ce que ça pourrait avoir un lien avec indoorequal ?

[EDIT] Le problème vient bien d'indoorequal qui affiche les poi contenant level=0 dans ses layers indoor-poi-rank1 et indoor-poi-rank2, au lieu qu'ils soient affichés dans le layer Other POI du style france. Désactiver ces 2 layers corrige bien le pb, mais je me doute que ça n'est pas une solution...

voilà ce que ça donne :

  • l'icone orange s'affiche bien sur le restaurant "Au marché des Lices"
  • les icones svg qui seraient adaptées mais n'apparaissent pas sont dues à une différence d'id, par exemple bar pour la subclass VS beer pour l'icone. Il y aura une harmonisation à faire.
    image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🎨 Design À propos de l'interface et de l'UX
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants