Project

General

Profile

Development #42761

carto, (un peu) refactoring js

Added by Frédéric Péters 7 months ago. Updated 6 months ago.

Status:
Solution déployée
Priority:
Normal
Target version:
-
Start date:
12 May 2020
Due date:
% Done:

0%

Patch proposed:
Yes
Planning:
No

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.

0001-maps-factor-all-geojson-layer-tracking-into-a-single.patch View (6.41 KB) Frédéric Péters, 12 May 2020 11:47 AM

0001-maps-factor-all-geojson-layer-tracking-into-a-single.patch View (6.95 KB) Frédéric Péters, 12 May 2020 03:45 PM

Associated revisions

Revision 0da0669e (diff)
Added by Frédéric Péters 6 months ago

maps: factor all geojson layer tracking into a single object (#42761)

History

#1 Updated by Frédéric Péters 7 months ago

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é.

#2 Updated by Frédéric Péters 7 months ago

Adaptation nécessaire à trigger_markers_if_ready.

#3 Updated by Frédéric Péters 6 months ago

  • Status changed from Solution proposée to 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)

#4 Updated by Frédéric Péters 6 months ago

  • Status changed from Résolu (à déployer) to Solution déployée

Also available in: Atom PDF