Development #19030
carto : permettre au thème de préciser que les contrôles de zoom doivent être ailleurs
0%
Description
Leaflet pose par défaut les boutons de zoom en haut à gauche mais dans certaines intégrations graphiques (lire: gnm) on les voudrait ailleurs.
Fichiers
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a plus de 6 ans
- Fichier 0001-maps-allow-zoom-control-position-to-be-overridded-by.patch 0001-maps-allow-zoom-control-position-to-be-overridded-by.patch ajouté
- Statut changé de Nouveau à En cours
- Patch proposed changé de Non à Oui
Mis à jour par Frédéric Péters il y a plus de 6 ans
- Fichier 0001-maps-allow-map-controls-position-to-be-overridded-by.patch 0001-maps-allow-map-controls-position-to-be-overridded-by.patch ajouté
Mieux avec un nom plus générique, qui pourra couvrir l'ajout de contrôles supplémentaires. (comme c'est déjà le cas dans w.c.s.).
Mis à jour par Thomas Noël il y a plus de 6 ans
Si j'ai compris l'idée, ça se base sur body, donc dans le theme.html on aura un <body {% if map_controls_position %}map-controls-position="{{ map_controls_position}}"{% endif %}> et un "map_controls_position" défini quand nécessaire dans le themes.json ?
Et donc ma question existentielle : pourquoi body et pas quelque chose dans « maps/map_cell.html », genre {% block map_attrs %}{% endblock %} que le thème pourrait surcharger tranquillou ?
Mis à jour par Frédéric Péters il y a plus de 6 ans
Et donc ma question existentielle : pourquoi body et pas quelque chose dans « maps/map_cell.html », genre {% block map_attrs }{ endblock %} que le thème pourrait surcharger tranquillou ?
Principalement parce que w.c.s. : il n'y a pas que la cellule carte qui affiche une carte. Et la surcharge dans le thème, ça n'oblige pas à dupliquer theme.html, juste à taper un {% block bodyargs }data-xxx{ endblock %}.
Mis à jour par Thomas Noël il y a plus de 6 ans
Ack donc ainsi, même si ça éparpille un peu les trucs à mon goût (j'aurais vraiment vu ça dans $map_widget.data('map-controls-position');) ; ça sera pas compliqué de reprendre l'affaire quand un map.html commun entre wcs et combo sera dessiné
Mis à jour par Frédéric Péters il y a plus de 6 ans
- Statut changé de En cours à Résolu (à déployer)
Merci,
commit c58b6083ef90362af94a781ca290594a5572530b
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Wed Sep 27 17:52:06 2017 +0200
maps: allow map controls position to be overridded by the theme (#19030)
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
maps: allow map controls position to be overridded by the theme (#19030)