10
105
Fork
You've already forked web
30

WIP: feat: Données OSM dans un tag <details> #1301

Draft
pmiossec wants to merge 1 commit from pmiossec/cartes-web:osm_details into master
pull from: pmiossec/cartes-web:osm_details
merge into: cartes:master
cartes:master
cartes:abostats
cartes:SynchroM-master
cartes:gps
cartes:embeddings
cartes:lysios-patch-5
cartes:lysios-patch-4
cartes:lysios-patch-3
cartes:lysios-patch-2
cartes:lysios-patch-1
cartes:abo-serveur2
cartes:clim
cartes:zone-en-favori
cartes:categories_toujours_visibles
cartes:rechercher_ici_avant_resultats
cartes:test-2365
cartes:panoramax-contribution
cartes:add-categories-validations
cartes:gel-bug-prod
cartes:osm-note-2
cartes:broken
cartes:wikidata_booking
cartes:fuse7.3
cartes:lysios-patrimoine-3
cartes:gps2
cartes:atproto-fix-2
cartes:styles-secondaires-mais-maison
cartes:atproto-scopes
cartes:itineraire-printemps-2026
cartes:modal-sheet-5.6
cartes:nlnet-cost-climate-perf
cartes:pb-icones-religieux
cartes:couvoiture
cartes:etiennejr-patch-2
cartes:meteo-fainéante
cartes:logo-static-url
cartes:poc-page-vitrine
cartes:remove-compass
cartes:test-reproduce-1795
cartes:display-cycleway-lanes-2
cartes:poc-image
cartes:add-media-condition
cartes:mapterhorn
cartes:test-indoor-equal
cartes:outdoors-VS-indoorEqual
cartes:hiking-colors
cartes:fix-blog-typing
cartes:poc-next-international
cartes:fix-map-style-modal-not-closing
cartes:i18n
cartes:hide-odd-zoom-levels
cartes:filter_categories
cartes:resume_wikipedia
cartes:deplacement_bouttons
cartes:fix-1095
cartes:mobile_permettre_selection
cartes:position-carte-initiale
cartes:dégradé-icônes
cartes:better_wikipedia
cartes:affichage_donnees_wikidata
cartes:mobile-permettre-selection
cartes:avis-details
cartes:perf-seo
cartes:modal-sheet-5
cartes:plans-novembre
cartes:display-cycleway-lanes
cartes:fix-1227
cartes:fix-1234
cartes:etiennejr-patch-1
cartes:update-1240
cartes:fix-1236
cartes:fix-1131
cartes:photon-search-street
cartes:fix-1221
cartes:article-avis
cartes:net-haut-zoom
cartes:baies-moins-visibles
cartes:belles-fiches-lieu
cartes:fix-1157
cartes:bug-page-vie-privée
cartes:use-watername
cartes:fix-1176
cartes:carte-du-onde
cartes:fix-1166
cartes:page-news
cartes:tchou
cartes:add-icon-alias
cartes:vitesse
cartes:photon-opensearch
cartes:private-in-italic
cartes:bâtiments-3D
cartes:merge-categories
cartes:handle-private-highway
cartes:improve-findcategory
cartes:injection-pmtiles
cartes:fix-1128
cartes:check-icons
cartes:update-readme
cartes:category-rugby
cartes:bug-position-carte-vs-lieu
cartes:limites-vitesse
cartes:improve-find-category-from-tags-v2
cartes:integrer
cartes:document-osmfeature
cartes:translate-document
cartes:velo-fin-aout-25
cartes:velo-ete-2025
cartes:moins-de-maptiler
cartes:planet-pmtiles-tilemaker
cartes:fix-1048
cartes:panoramax-leger
cartes:plume
cartes:horaires-i18n
cartes:planet
cartes:maj-deps-juillet-2025
cartes:improve-find-category-from-tags
cartes:bug-encart
cartes:interactions-encart-mobile
cartes:clic-on-station
cartes:fix-1028
cartes:fix-1014
cartes:edit
cartes:add-big-rivers
cartes:add-shortbread-tiles
cartes:handle-deleted-osm-elements-v2
cartes:codeberg
cartes:improve-useEffect
cartes:immoloc
cartes:update-dashboard
cartes:analyze
cartes:terradraw
cartes:sombre2
cartes:calendar
cartes:poi-sprites-using-svg
cartes:style-sombre
cartes:sprites
cartes:annuaire
cartes:motis-gtfs-attributes
cartes:next-styled-components
cartes:dev
cartes:cool2
cartes:cool
cartes:panoramax-angle
cartes:env-static-urls
cartes:borders-only
cartes:panoramax2
cartes:intelligence
cartes:debug-iles
cartes:nouveau-serveur
cartes:france-protomaps
cartes:turbo
cartes:fouss/cartes-#338
cartes:blog2
cartes:legislatives
cartes:presentation
cartes:pas-de-geoloc-si-hash-different
cartes:protomaps
cartes:vaul
cartes:carte-trains
cartes:citiz
cartes:maj-paquets
cartes:icones
cartes:ui2
cartes:bblog
cartes:voyagevoyage
cartes:test-image-origine
cartes:bugimagefin
cartes:searchParams
cartes:voyage
cartes:avion
cartes:perf
cartes:studio
cartes:piscine
cartes:journée
cartes:année
cartes:journée-transition
cartes:score-dès-acceuil
cartes:fix-space-problem
cartes:rebase-5août
cartes:correspondance
cartes:règles-séparées
cartes:indeps-acre
cartes:demo
cartes:indeps
cartes:indeps-cachés
cartes:indication-temps-plein
cartes:indeps-refactor-simulations
cartes:indeps-vue-comparaison
cartes:indeps-inversion
cartes:barème-continu
cartes:indeps-démo
cartes:textes-branches-sécu
cartes:engine-enum
cartes:2019
cartes:transport
cartes:publicodes
cartes:as
cartes:infrance-nouvelle-landing
cartes:better-translation
cartes:co2
cartes:simplifATMP
cartes:controle-SMIC
cartes:webi18n
cartes:bug-avantages-en-nature
cartes:revert-352-infrance-navigation-v2
cartes:laem-patch-1
cartes:robots.txt
cartes:improve-perf
cartes:iframe-bugfix
cartes:quickfixes
cartes:lib
cartes:irsimple
cartes:hiringinfrance.com
cartes:dégradé
cartes:refstate
cartes:johan
cartes:zrr
cartes:logo-interactif
cartes:revert-216-improve-tracking
cartes:redesign2
cartes:zrr2
cartes:fixtrad
cartes:correction-salaire-brut
cartes:petit-state
cartes:alien-consolidate
cartes:translate-ui
cartes:npm
cartes:alien-brut-net
cartes:translatable
cartes:différent
cartes:ir18
cartes:fix-typo
cartes:add-tracking
cartes:fix-collect-missing
cartes:défauts-partout
cartes:fix-double-unfold-three
cartes:fix-double-unfold
cartes:versement-transport
cartes:blocker-questions
  • Titre qui inscite à contribuer
  • Deplacement plus haut pour que cela soit plus visible
  • Replié par défaut pour réduire la surcharge d'information par défaut dans le panneau de résultat

Avant: image

Après:

image
&
image

* Titre qui inscite à contribuer * Deplacement plus haut pour que cela soit plus visible * Replié par défaut pour réduire la surcharge d'information par défaut dans le panneau de résultat Avant: ![image](/attachments/c849d3fb-841d-4a77-933d-fe83c6ffa65b) Après: ![image](/attachments/569ec9c3-b220-4cde-aafb-2f0ecae66f70) & ![image](/attachments/6f6bd68d-6a76-40f6-be42-6a15d1225408)

Salut,
Merci pour la PR. J'aime beaucoup l'idée de réduire la surcharge d'information dans ce panneau. Et aussi de faire un menu qui incite à contribuer. Mais j'ai quand même 2 réflexions 😁​ :

  • la date de dernière mise à jour me parait une info primaire qui doit être directement visible. (certes on peut lui reprocher que dans un changeset toutes les infos n'ont pas forcément été mise à jour, mais c'est un autre débat, et en attendant mieux c'est quand même une bonne info moyenne). Peut-être la garder visible mais à un autre endroit ?
  • Quitte à faire un menu qui incite à contribuer, j'aurais envie de mettre dedans toutes les possibilités de contribution : poster un avis, modifier les horaires, voire ajouter des liens vers d'autres services : poster une photo sur panoramax, etc

Qu'en penses tu ?

Salut, Merci pour la PR. J'aime beaucoup l'idée de réduire la surcharge d'information dans ce panneau. Et aussi de faire un menu qui incite à contribuer. Mais j'ai quand même 2 réflexions 😁​ : - la date de dernière mise à jour me parait une info primaire qui doit être directement visible. (certes on peut lui reprocher que dans un changeset toutes les infos n'ont pas forcément été mise à jour, mais c'est un autre débat, et en attendant mieux c'est quand même une bonne info moyenne). Peut-être la garder visible mais à un autre endroit ? - Quitte à faire un menu qui incite à contribuer, j'aurais envie de mettre dedans toutes les possibilités de contribution : poster un avis, modifier les horaires, voire ajouter des liens vers d'autres services : poster une photo sur panoramax, etc Qu'en penses tu ?
Owner
Copy link

À mon avis, déplacer cet élément plus haut n'est pas une bonne idée : la contribution sur openstreetmap.org est conçue pour les experts. L'interface id est trop compliquée.

Pour faire contribuer le grand public, c'est via notre système de contribution qu'il faut agir. Il n'est pas assez mis en avant, et pas assez développé : on pourrait afficher un des boutons du style "ajouter un numéro de téléphone" s'il n'existe pas, ou un bouton "mettre à jour" s'il existe.

À mon avis, déplacer cet élément plus haut n'est pas une bonne idée : la contribution sur openstreetmap.org est conçue pour les experts. L'interface *id* est trop compliquée. Pour faire contribuer le grand public, c'est via notre système de contribution qu'il faut agir. Il n'est pas assez mis en avant, et pas assez développé : on pourrait afficher un des boutons du style "ajouter un numéro de téléphone" s'il n'existe pas, ou un bouton "mettre à jour" s'il existe.
Owner
Copy link

Concernant le bandeau lui-même, je pense que c'est important de garder le logo OpenStreetMap visible. Le projet entier repose sur OSM, bien que ce ne soit pas la seule base de donnée nécessaire, de loin. Les données d'une fiche sont quand même à 90 % OSM.

C'est donc très important à mon avis de garder sans <details le bandeau contributeur et le logo OSM.

Concernant le bandeau lui-même, je pense que c'est important de garder le logo OpenStreetMap visible. Le projet entier repose sur OSM, bien que ce ne soit pas la seule base de donnée nécessaire, de loin. Les données d'une fiche sont quand même à 90 % OSM. C'est donc très important à mon avis de garder sans *<details* le bandeau contributeur *et* le logo OSM.
Owner
Copy link

Par contre pour aller dans le sens de cette PR, je serais d'avis de réduire la place allouée aux lieux similaires. On voit dans la capture de ce lieu
image qu'il y a trop de choix dans la liste. En réduisant également l'espace alloué aux avis (cf ton autre PR 👍), ça peut rationaliser la place attribuée aux éléments et remonter le bandeau OSM.

On peut proposer genre 4 lieux similaires et tout le reste au clic.

Par contre pour aller dans le sens de cette PR, je serais d'avis de réduire la place allouée aux lieux similaires. On voit dans la capture de [ce lieu](https://cartes.app/?allez=Gioco|n1262015777|2.3098|48.8512#15/48.85122/2.30985/15/40) ![image](/attachments/41b550b4-3bd2-4e50-9d9d-8de2aef6515e) qu'il y a trop de choix dans la liste. En réduisant également l'espace alloué aux avis (cf ton autre PR 👍), ça peut rationaliser la place attribuée aux éléments et remonter le bandeau OSM. On peut proposer genre 4 lieux similaires et tout le reste au clic.
243 KiB

@laem wrote in #1301 (comment):

On peut proposer genre 4 lieux similaires et tout le reste au clic.

Tu ne veux pas carrément tout masquer, pour ne lancer la requête que lors d'un clic sur un bouton "chercher les lieux similaires" ?

@laem wrote in https://codeberg.org/cartes/web/pulls/1301#issuecomment-8788326: > On peut proposer genre 4 lieux similaires et tout le reste au clic. Tu ne veux pas carrément tout masquer, pour ne lancer la requête que lors d'un clic sur un bouton "chercher les lieux similaires" ?
Owner
Copy link

Tu ne veux pas carrément tout masquer, pour ne lancer la requête que lors d'un clic sur un bouton "chercher les lieux similaires" ?

J'avais ajouté ce blog surtout pour le SEO, et accessoirement pour l'utilisateur : l'une des choses essentielles pour un SEO qui marche, c'est des liens entre les pages. Ne le charger qu'au clic enlèverait ça.

> Tu ne veux pas carrément tout masquer, pour ne lancer la requête que lors d'un clic sur un bouton "chercher les lieux similaires" ? J'avais ajouté ce blog surtout pour le SEO, et accessoirement pour l'utilisateur : l'une des choses essentielles pour un SEO qui marche, c'est des liens entre les pages. Ne le charger qu'au clic enlèverait ça.

@laem wrote in #1301 (comment):

J'avais ajouté ce blog surtout pour le SEO, et accessoirement pour l'utilisateur : l'une des choses essentielles pour un SEO qui marche, c'est des liens entre les pages. Ne le charger qu'au clic enlèverait ça.

ah oui en effet c'est important, je n'avais pas pensé à ça, donc ok on garde !

@laem wrote in #1301 (comment):

Pour faire contribuer le grand public, c'est via notre système de contribution qu'il faut agir.

Je vais m'écarter un peu du sujet, mais en y repensant, on a un pb d'incohérence intrinsèque : on veut pousser l'utilisateur à contribuer via cartes.app, mais pour pouvoir le faire, il doit quand même aller se créer un compte OSM ...

En passant, ça veut dire qu'on peut (doit?) différencier l'affichage selon si l'utilisateur est connecté à OSM ou non. (en réalité il faudrait même différencier selon si la personne est un gros contributeur OSM ou qq qui vient de se créer un compte !)

Au delà, ça pose la question de comment on fait pour convaincre l'utilisateur de cartes.app d'aller se créer un compte OSM d'une part (et se créer un compte bluesky d'autre part pour les avis). Il va falloir un petit blabla pour expliquer les bienfaits de contribuer à des bases ouvertes réutilisables par d'autres projets. Je pense qu'il y a toute une UX à construire autour de ça.

@laem wrote in https://codeberg.org/cartes/web/pulls/1301#issuecomment-8792619: > J'avais ajouté ce blog surtout pour le SEO, et accessoirement pour l'utilisateur : l'une des choses essentielles pour un SEO qui marche, c'est des liens entre les pages. Ne le charger qu'au clic enlèverait ça. ah oui en effet c'est important, je n'avais pas pensé à ça, donc ok on garde ! @laem wrote in https://codeberg.org/cartes/web/pulls/1301#issuecomment-8788260: > Pour faire contribuer le grand public, c'est via notre système de contribution qu'il faut agir. Je vais m'écarter un peu du sujet, mais en y repensant, on a un pb d'incohérence intrinsèque : on veut pousser l'utilisateur à contribuer via cartes.app, mais pour pouvoir le faire, il doit quand même aller se créer un compte OSM ... En passant, ça veut dire qu'on peut (doit?) différencier l'affichage selon si l'utilisateur est connecté à OSM ou non. (en réalité il faudrait même différencier selon si la personne est un gros contributeur OSM ou qq qui vient de se créer un compte !) Au delà, ça pose la question de comment on fait pour convaincre l'utilisateur de cartes.app d'aller se créer un compte OSM d'une part (et se créer un compte bluesky d'autre part pour les avis). Il va falloir un petit blabla pour expliquer les bienfaits de contribuer à des bases ouvertes réutilisables par d'autres projets. Je pense qu'il y a toute une UX à construire autour de ça.
Owner
Copy link

Oui clairement, à ce stade la contribution passe par la création d'un compte, et deux en cumulant les avis, c'est vrai.

Je ne pense pas que ce soit un problème d'expliquer à l'utilisateur qu'il doive se connecter, au contraire : on ne lui dit pas de se créer un nième compte dédié à cartes.app. Mais tu as raison, faut l'expliquer.

Oui clairement, à ce stade la contribution passe par la création d'un compte, et deux en cumulant les avis, c'est vrai. Je ne pense pas que ce soit un problème d'expliquer à l'utilisateur qu'il doive se connecter, au contraire : on ne lui dit pas de se créer un nième compte dédié à cartes.app. Mais tu as raison, faut l'expliquer.
Author
Member
Copy link

Je savais que cette proposition était limite. Je reconnais la valeur de OSM et je ne veut pas que cela ait un effet négatif.
Je ne suis pas sûr d'arriver à une meilleur proposition.
Je verrais plus tard après avoir intégrer tous les devs en cours sur le volet de recherche.

On peut proposer genre 4 lieux similaires et tout le reste au clic.

En attendant, j'ai fait une PR pour çà...

Je savais que cette proposition était limite. Je reconnais la valeur de OSM et je ne veut pas que cela ait un effet négatif. Je ne suis pas sûr d'arriver à une meilleur proposition. Je verrais plus tard après avoir intégrer tous les devs en cours sur le volet de recherche. >On peut proposer genre 4 lieux similaires et tout le reste au clic. En attendant, j'ai fait une PR pour çà...
pmiossec changed title from (削除) feat: Données OSM dans un tag <details> (削除ここまで) to WIP: feat: Données OSM dans un tag <details> 2025年12月09日 01:43:54 +01:00
Owner
Copy link

En fait perso je trouve que reléguer la mention d'OSM dans le petit (i) en bas à droite et dans une section <details ce n'est pas très honnête. C'est le minimum légal, mais j'aime pas ce que font Mappy ou Apple : invisibiliser au maximum OSM. D'où mon insistance pour que le logo apparaisse. C'est vrai que les liens vers la page osm.org me semble moins importants (même s'il faut les avoir) que de montrer clairement que ce sont des données OSM. Mais bon, pour l'instant malheureusement osm.org est indissociable d'OSM.

En fait perso je trouve que reléguer la mention d'OSM dans le petit (i) en bas à droite et dans une section <details ce n'est pas très honnête. C'est le minimum légal, mais j'aime pas ce que font Mappy ou Apple : invisibiliser au maximum OSM. D'où mon insistance pour que le logo apparaisse. C'est vrai que les liens vers la page osm.org me semble moins importants (même s'il faut les avoir) que de montrer clairement que ce sont des données OSM. Mais bon, pour l'instant malheureusement osm.org est indissociable d'OSM.
This pull request has changes conflicting with the target branch.
  • app/OsmFeature.tsx
  • components/OsmLinksAndMeta.tsx
View command line instructions

Manual merge helper

Use this merge commit message when completing the merge manually.

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u osm_details:pmiossec-osm_details
git switch pmiossec-osm_details
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!1301
Reference in a new issue
cartes/web
No description provided.
Delete branch "pmiossec/cartes-web:osm_details"

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?