Projet

Général

Profil

Bug #16708

cartes : le zoom à 19 (fourmi) ne fonctionne pas

Ajouté par Frédéric Péters il y a presque 7 ans. Mis à jour il y a plus de 5 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
-
Version cible:
-
Début:
06 juin 2017
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

Cf https://github.com/Leaflet/Leaflet/issues/5289

Le tileLayer a maxZoom = 18 par défaut, si la carte a un maxzoom à 19, bim, ce niveau est gris.

--- a/combo/apps/maps/static/js/combo.map.js
+++ b/combo/apps/maps/static/js/combo.map.js
@@ -10,7 +10,11 @@ $(function() {
         }
         var init_state = $map_widget.data('init-state');
         var max_zoom = parseInt($map_widget.data('max-zoom'));
-        if (!isNaN(max_zoom)) map_options.maxZoom = max_zoom;
+        if (!isNaN(max_zoom)) {
+            map_options.maxZoom = max_zoom;
+        } else {
+            map_options.maxZoom = 19;
+        }
         var min_zoom = parseInt($map_widget.data('min-zoom'));
         if (!isNaN(min_zoom)) map_options.minZoom = min_zoom;
         var latlng = [$map_widget.data('init-lat'), $map_widget.data('init-lng')];
@@ -31,7 +35,8 @@ $(function() {

         L.tileLayer(map_tile_url,
             {
-                attribution: map_attribution
+                attribution: map_attribution,
+                maxZoom: map_options.maxZoom
             }).addTo(map);
         if (geojson_url) {
             $.getJSON(geojson_url, function(data) {

Mais. carton.entrouvert.org fournit des tuiles jusque 18 maximum; je propose donc de plutôt faire de 18 le niveau max.


Fichiers

Révisions associées

Révision 44670d04 (diff)
Ajouté par Frédéric Péters il y a presque 7 ans

maps: let leaflet zoom up to level 19 (#16708)

Historique

#1

Mis à jour par Frédéric Péters il y a presque 7 ans

#2

Mis à jour par Thomas Noël il y a presque 7 ans

Le niveau 19 existe et fonctionne par ailleurs, dommage de s'en priver. http://b.tile.openstreetmap.fr/osmfr/19/265915/181526.png

Ok pour mettre à 18 par défaut, mais laisser la possibilité du 19 "Ant", et ajouter un niveau 18 "Human".

#3

Mis à jour par Frédéric Péters il y a presque 7 ans

Bof. Plutôt alors corriger pour permettre 19 et voir pour générer de telles tuiles sur carton.

#4

Mis à jour par Thomas Noël il y a presque 7 ans

Frédéric Péters a écrit :

Bof. Plutôt alors corriger pour permettre 19 et voir pour générer de telles tuiles sur carton.

Oui, de toute façon 19 me semble nécessaire pour la déclaration de trous dans la route. Je demande à Champs Libres.

#6

Mis à jour par Thomas Noël il y a presque 7 ans

Ack

#7

Mis à jour par Frédéric Péters il y a presque 7 ans

  • Statut changé de En cours à Résolu (à déployer)
commit 44670d04332ec78d8228fea2e0538c72fbbff140
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Tue Jun 6 17:56:23 2017 +0200

    maps: let leaflet zoom up to level 19 (#16708)
#8

Mis à jour par Frédéric Péters il y a plus de 5 ans

  • Statut changé de Résolu (à déployer) à Solution déployée

Formats disponibles : Atom PDF