9
103
Fork
You've already forked web
30

Add "ping pong" (without dash) in category dictionnary #1096

Merged
etienneJr merged 5 commits from waterced/web:master into master 2025年09月23日 14:00:02 +02:00

Pour facilité la recherche, suite à https://forum.openstreetmap.fr/t/cartes-app-thematique-sport/25684/17

Au passage, j'ai vu #1092 qui fait peut-être ça plus proprement.

Pour facilité la recherche, suite à https://forum.openstreetmap.fr/t/cartes-app-thematique-sport/25684/17 Au passage, j'ai vu https://codeberg.org/cartes/web/pulls/1092 qui fait peut-être ça plus proprement.

Je suis super surpris que "ping pong" ne soit pas trouvé alors que la catégorie s'appelle "ping-pong". Ça veut que le script (don't j'ai oublié son nom) qui calcule le rapprochement, et dont on trouve qu'il est trop laxiste (voir #945), n'est pas capable d'échapper un "-" ?!?

Je suis super surpris que "ping pong" ne soit pas trouvé alors que la catégorie s'appelle "ping-pong". Ça veut que le script (don't j'ai oublié son nom) qui calcule le rapprochement, et dont on trouve qu'il est trop laxiste (voir #945), n'est pas capable d'échapper un "-" ?!?

oui oui j'avais bien observé la même chose que sur tes captures.

Mais par exemple chercher fast food fonctionne aussi bien que fast-food alors qu'il n'y a pas fast food dans le dictionnaire. C'est exactement le même nombre de lettres, c'est incompréhensible pour moi que ça ne marche pas pour ping pong alors que ça marche pour fast food.

de la même façon sage femme fonctionne aussi bien que sage-femme, pique nique fonctionne aussi bien que pique-nique, canoe kayak fonctionne aussi bien que canoe-kayak. Franchement c'est quoi le problème de Fuse.js avec le ping pong ?!?

oui oui j'avais bien observé la même chose que sur tes captures. Mais par exemple chercher `fast food` fonctionne aussi bien que `fast-food` alors qu'il n'y a pas `fast food` dans le dictionnaire. C'est exactement le même nombre de lettres, c'est incompréhensible pour moi que ça ne marche pas pour `ping pong` alors que ça marche pour `fast food`. de la même façon `sage femme` fonctionne aussi bien que `sage-femme`, `pique nique` fonctionne aussi bien que `pique-nique`, `canoe kayak` fonctionne aussi bien que `canoe-kayak`. Franchement c'est quoi le problème de Fuse.js avec le `ping pong` ?!?
Member
Copy link

Ça marche très bien chez moi pour ping, comme pour pong

Problèmede zoom trop éloigné lors de la recherche ?

Ça marche très bien chez moi pour ping, comme pour pong Problèmede zoom trop éloigné lors de la recherche ?

Non non, là c'est pas la question du zoom trop éloigné.
ping marche, et pong marche aussi, mais bizarrement ping pong ne marche pas. Si t'es un as du clavier et que t'écris d'une traite ping pong alors il ne te propose pas la catégorie.

Non non, là c'est pas la question du zoom trop éloigné. `ping` marche, et `pong` marche aussi, mais bizarrement `ping pong` ne marche pas. Si t'es un as du clavier et que t'écris d'une traite `ping pong` alors il ne te propose pas la catégorie.

ah zut, j'ai oublié de merger plus tôt, et maintenant que je voulais le faire, ça fait un conflit car moreCategories.yaml a été renommé en categories.yaml. @waterced tu peux mettre à jour ? J'ai essayé mais n’ai pas su faire, et surement pas les droits dans ton fork. Pour les prochaines fois, ça serait ptet mieux que @laem te donne les droits pour contribuer directement dans le repo ?

ah zut, j'ai oublié de merger plus tôt, et maintenant que je voulais le faire, ça fait un conflit car `moreCategories.yaml` a été renommé en `categories.yaml`. @waterced tu peux mettre à jour ? J'ai essayé mais n’ai pas su faire, et surement pas les droits dans ton fork. Pour les prochaines fois, ça serait ptet mieux que @laem te donne les droits pour contribuer directement dans le repo ?
Author
Member
Copy link

ça doit être bon !

ça doit être bon !

parfait merci ! j'ai fusionné.

parfait merci ! j'ai fusionné.
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
3 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!1096
Reference in a new issue
cartes/web
No description provided.
Delete branch "waterced/web:master"

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?