Development #42761
carto, (un peu) refactoring js
Début:
12 mai 2020
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Non
Description
Il y a à la fois un geojson_layers avec des infos sur les couches (genre url) et un map.geojson_layers qui est lui le mapping slug→leayer leafet.
Proposition d'unifier ça.
Fichiers
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a presque 4 ans
- Fichier 0001-maps-factor-all-geojson-layer-tracking-into-a-single.patch 0001-maps-factor-all-geojson-layer-tracking-into-a-single.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
En passant par des méthodes ajoutées à l'objet Map, ce qui fait à mon sens un code plus clair avec des moments d'actions précis.
Petit changement fonctionnel, le paramétrage "centrer/zoomer pour contenir les marqueurs" s'appliquait à chaque couche de marqueurs chargée, ne s'applique plus qu'une fois tout chargé.
Mis à jour par Frédéric Péters il y a presque 4 ans
- Fichier 0001-maps-factor-all-geojson-layer-tracking-into-a-single.patch 0001-maps-factor-all-geojson-layer-tracking-into-a-single.patch ajouté
Adaptation nécessaire à trigger_markers_if_ready.
Mis à jour par Frédéric Péters il y a presque 4 ans
- Statut changé de Solution proposée à Résolu (à déployer)
KO couché des relecteurs,
commit 0da0669e802ef7d3f3d2d7fdeb26df0aaf315fdc Author: Frédéric Péters <fpeters@entrouvert.com> Date: Tue May 12 11:43:02 2020 +0200 maps: factor all geojson layer tracking into a single object (#42761)
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: factor all geojson layer tracking into a single object (#42761)