Skip to content

fix(vis): fix map marker count & publisher data in tooltip

Enzo Doyen requested to merge map into main

Pour faire suite à ton e-mail @ruizfabo

La PR contient les changements suivants :

  1. Les marqueurs de même type et de même langue n'apparaissent plus en double s'ils ont les mêmes coordonnés (ce n'est plus nécessaire puisque normalement, toutes les pièces de même langue avec les mêmes coordonnées s'affichent maintenant toutes regroupées en liste dans un tooltip sous un seul marqueur). Cela réduit le nombre de marqueurs sur la carte et améliore en conséquence la lisibilité et les performances (notamment pour Paris où il y avait énormément de marqueurs au même endroit).
  2. Il y avait un bug pour les tooltips listant plusieurs pièces où settingText et settingTime étaient identiques pour toutes les pièces (les valeurs étaient copiées de la première pièce). Cela devrait être corrigé maintenant.
  3. Autres petites modifications mineures relatives à TypeScript + meilleure détection des erreurs.

Concernant le fait que certains tooltips ne contiennent pas les infos des pièces, il me semble que c'est un problème de données (mais j'aimerais confirmation). En fait, pour récupérer les données des pièces (nom de la pièce, date, etc.), l'implémentation actuelle se base sur publisher_data, car c'est le seul type de données autre que location_data et setting_data qui contient placeId (à ma connaissance). La jointure est donc basée là-dessus. Elle se fait initialement dans vis.ts (fonction getPublisherMapData()) pour créer un objet PublisherMapData qui rassemble les informations dont on a besoin, celui-ci étant ensuite utilisé dans map.ts (fonction extendData()).

Mais il semblerait que certains éditeurs n'aient pas de placeId. Pour reprendre l'exemple donné (Hannover) :

  1. On trouve Hannover dans location_data :
{
   "placeId": 46,
   "lang": "ger",
   "name": "Hannover",
   "wikidataId": "Q1715",
   "OSMRId": "59418",
   "OSMLatLon": "52.3744779, 9.7385532",
   "nature": "real",
   "comments": null
}
  1. Mais "placeId": 46 ne donne aucun résultat dans publisher_data

N'hésite pas à me dire si j'ai juste, car tu connais mieux les données que moi.

Je me demande aussi s'il ne faudrait pas améliorer quelque peu l'affichage des tooltips, car actuellement, quand on a toutes les données (playName, playDate, settingTime, settingText, etc.), l'affichage est un peu fouillis, d'autant plus quand on a une liste de plusieurs pièces. Soit il faudrait réfléchir à une autre manière d'organiser les données dans les tooltips, ou éventuellement mettre un style particulier pour chaque type de données (couleurs ?).

Merge request reports