[SPIP Zone] [GIS] Zoom impossible si centrer_auto=oui

Salut,

depuis , le zoom in ne fonctionne plus lorsqu’il y a centrer_auto=oui. Sans le centrer_auto, ça fonctionne.

Voir en bas de la page

Le code:

[(#INCLURE{fond=modeles/carte_gis,
objet=article,
id_objet=#ID_ARTICLE,
zoom_wheel=non,
centrer_auto=oui,
hauteur=260px,
zoom=9,
zoom_molette=non})]

jeanmarie

Hop,

Le 25/09/2019 à 17:41, Jean Marie Grall a écrit :

Salut,

depuis https://zone.spip.net/trac/spip-zone/changeset/116107/spip-zone/_plugins_/gis/trunk , le zoom in ne fonctionne plus lorsqu'il y a centrer_auto=oui.
Sans le centrer_auto, ça fonctionne.

Voir en bas de la page https://www.brestculture.fr/leyla-mccalla.html

Le code:

[(#INCLURE{fond=modeles/carte_gis,
objet=article,
id_objet=#ID_ARTICLE,
zoom_wheel=non,
centrer_auto=oui,
hauteur=260px,
zoom=9,
zoom_molette=non})]

Ton plugin est à jour ? J'ai souvenir d'avoir corrigé ce bug il y a 3 semaines cf https://contrib.spip.net/GIS-4#comment502183-502163

++
b_b

Yep, up-to-date...
Et j'ai testé également avec la version de ton commit (4.47.5)

Le bug, avant ton commit, était différent : la carte s'affichait en zoom max. Là, elle s'affiche avec le bon zoom (passé en paramètre avec zoom=9) mais on ne peut pas zommer plus. Par contre, on peut dézoomer.

Le 25/09/2019 à 18:52, Bruno Bergot a écrit :

Hop,

Le 25/09/2019 à 17:41, Jean Marie Grall a écrit :

Salut,

depuis https://zone.spip.net/trac/spip-zone/changeset/116107/spip-zone/_plugins_/gis/trunk , le zoom in ne fonctionne plus lorsqu'il y a centrer_auto=oui.
Sans le centrer_auto, ça fonctionne.

Voir en bas de la page https://www.brestculture.fr/leyla-mccalla.html

Le code:

[(#INCLURE{fond=modeles/carte_gis,
objet=article,
id_objet=#ID_ARTICLE,
zoom_wheel=non,
centrer_auto=oui,
hauteur=260px,
zoom=9,
zoom_molette=non})]

Ton plugin est à jour ? J'ai souvenir d'avoir corrigé ce bug il y a 3 semaines cf https://contrib.spip.net/GIS-4#comment502183-502163

++
b_b

Testé à l'instant, je peux zommer et dézommer à volonté, je ne reproduis pas donc...

Une url à partager ?

++
b_b

Le 26/09/2019 à 09:11, Jean Marie Grall a écrit :

Yep, up-to-date...
Et j'ai testé également avec la version de ton commit (4.47.5)

Le bug, avant ton commit, était différent : la carte s'affichait en zoom max. Là, elle s'affiche avec le bon zoom (passé en paramètre avec zoom=9) mais on ne peut pas zommer plus. Par contre, on peut dézoomer.

Le 26/09/2019 à 09:21, Bruno Bergot a écrit :

Testé à l'instant, je peux zommer et dézommer à volonté, je ne reproduis pas donc...

Une url à partager ?

Hop Leyla Mccalla - Brest Culture (en bas de page)

Alors que là, pas de problème : Céoù Céki - Brest Culture

En relisant le code je pense que le bug c’est que avec un seul point je force le maxzoom avant d’appeler map.fitBounds pour eviter un zoom infini, mais il faudrait ensuite le rétablir. Sinon on reste bloqué à ce zoom maxi et on peut pas zoomer plus…

--
Cédric
Le 26 sept. 2019 à 1:08 AM +0200, Bruno Bergot <bruno@eliaz.fr>, a écrit :

Hop,

Le 25/09/2019 à 17:41, Jean Marie Grall a écrit :
> Salut,
>
> depuis
> https://zone.spip.net/trac/spip-zone/changeset/116107/spip-zone/_plugins_/gis/trunk
> , le zoom in ne fonctionne plus lorsqu'il y a centrer_auto=oui.
> Sans le centrer_auto, ça fonctionne.
>
> Voir en bas de la page https://www.brestculture.fr/leyla-mccalla.html
>
> Le code:
>
> [(#INCLURE{fond=modeles/carte_gis,
> objet=article,
> id_objet=#ID_ARTICLE,
> zoom_wheel=non,
> centrer_auto=oui,
> hauteur=260px,
> zoom=9,
> zoom_molette=non})]
>

Ton plugin est à jour ? J'ai souvenir d'avoir corrigé ce bug il y a 3
semaines cf https://contrib.spip.net/GIS-4#comment502183-502163

++
b_b
----
spip-zone@rezo.net - https://listes.rezo.net/mailman/listinfo/spip-zone

J'allais le dire, merci de m'avoir devancé :wink:

Je te laisse corriger ?

++
b_b

Le 26/09/2019 à 09:43, Cerdic a écrit :

En relisant le code je pense que le bug c’est que avec un seul point je force le maxzoom avant d’appeler map.fitBounds pour eviter un zoom infini, mais il faudrait ensuite le rétablir. Sinon on reste bloqué à ce zoom maxi et on peut pas zoomer plus…

--
Cédric

Allez Jean Marie, paye ton commmit :wink:

Un simple delete map.options.maxZoom; après map.fitBounds(bounds, options); à la ligne 220 et zou

++
b_b

Le 26/09/2019 à 09:45, Bruno Bergot a écrit :

J'allais le dire, merci de m'avoir devancé :wink:

Je te laisse corriger ?

++
b_b

Euh, ok : Connexion · GitLab

Mais j'ai un peu l'impression de toucher à la déco de ta chambre, ça me gêne :slight_smile:

Le 26/09/2019 à 09:54, Bruno Bergot a écrit :

Allez Jean Marie, paye ton commmit :wink:

Un simple delete map.options.maxZoom; après map.fitBounds(bounds, options); à la ligne 220 et zou

++
b_b

Le 26/09/2019 à 09:45, Bruno Bergot a écrit :

J'allais le dire, merci de m'avoir devancé :wink:

Je te laisse corriger ?

++
b_b

Range *ma* chambre !

Merci mignon :wink:

++
b_b

Le 26/09/2019 à 10:07, Jean Marie Grall a écrit :

Euh, ok : Connexion · GitLab

Mais j'ai un peu l'impression de toucher à la déco de ta chambre, ça me gêne :slight_smile:

peut-être pas delete, parce que si il y en avait un il faut le remettre, non ?

--
Cédric
Le 26 sept. 2019 à 9:54 AM +0200, Bruno Bergot <bruno@eliaz.fr>, a écrit :

Allez Jean Marie, paye ton commmit :wink:

Un simple delete map.options.maxZoom; après map.fitBounds(bounds,
options); à la ligne 220 et zou

++
b_b

Le 26/09/2019 à 09:45, Bruno Bergot a écrit :
> J'allais le dire, merci de m'avoir devancé :wink:
>
> Je te laisse corriger ?
>
> ++
> b_b

Ha oui bien vu, je te laisse voir ça, je m'absente quelques jours...

++
b_b

Le 26/09/2019 à 11:23, Cerdic a écrit :

peut-être pas delete, parce que si il y en avait un il faut le remettre, non ?

Voilà c'est fait par :

https://zone.spip.net/trac/spip-zone/changeset/118178

Plus un énième bugfix sur la modification de autocenterandzoom ici :

https://zone.spip.net/trac/spip-zone/changeset/118179

++
b_b

Le 26/09/2019 à 11:41, Bruno Bergot a écrit :

Ha oui bien vu, je te laisse voir ça, je m'absente quelques jours...

Le 26/09/2019 à 11:23, Cerdic a écrit :

peut-être pas delete, parce que si il y en avait un il faut le remettre, non ?