Development #42765
carto, passer les paramètres de la couche dans la cellule plutôt qu'au niveau du geojson
Statut:
Fermé
Priorité:
Normal
Assigné à:
-
Version cible:
-
Début:
12 mai 2020
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Non
Description
Aujourd'hui on ajoute au geojson une propriété layer avec les infos de la couche (slug, couleurs, icône, etc.); maintenant qu'on a un geojson par couche, on pourrait plutôt passer cette info au niveau de la cellule, plutôt qu'avoir à la dupliquer pour tous les marqueurs.
Fichiers
Révisions associées
tests: update test_download_geojson after #42765
Historique
Mis à jour par Frédéric Péters il y a presque 4 ans
- Fichier 0001-maps-give-common-layer-properties-to-map-cell-42765.patch 0001-maps-give-common-layer-properties-to-map-cell-42765.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
À noter le passage :
- $(cell).trigger('combo:map-feature-prepare', {'feature': feature, 'layer': layer}); + $(cell).trigger('combo:map-feature-prepare', + {'feature': feature, 'layer': layer, 'geojson_layer': geojson_layer});
C'est pour Toodego, qui exploitait de sa manière les infos passées dans properties.layer, et qui n'y aurait plus accès sans ça.
Le reste est plutôt basique, et simplifie/nettoie à la suite de #42761, en utilisant le filtre |as_json.
Mis à jour par Christophe Siraut il y a presque 4 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Frédéric Péters il y a presque 4 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 179f77f4bc0f81d64fed59e9020c505b9f67dcf3 Author: Frédéric Péters <fpeters@entrouvert.com> Date: Tue May 12 13:37:41 2020 +0200 maps: give common layer properties to map cell (#42765)
Mis à jour par Frédéric Péters il y a presque 4 ans
- Statut changé de Résolu (à déployer) à Solution déployée
maps: give common layer properties to map cell (#42765)