search

jdev-org / donneesplages

Configuration Mviewer pour le projet "Données Plages" du GIP LITORRAL & GIP ATEGERI
GNU General Public License v3.0
0 stars 0 forks source link

Erreur sur la couche plage cluster nouvelle version du mviewer (develop) #34

Open eprouteau opened 3 years ago

eprouteau commented 3 years ago

Bonjour tout le monde,

J'ai tenté de mettre à jour le core de mviewer pour l'appli plage. Tout fonctionne sauf le cluster de la couche, ce qui bloque également les tooltip et l'interrogation :

Voici l'URL de l'appli :

https://www.pigma.org/public/visualiseur/donneesplages/?config=apps/donneesplages/donneesplages.xml

Bonne journée,

Emeric,

Gaetanbrl commented 3 years ago

@eprouteau sur ton lien les clusters semblent bien fonctionner, tu es revenu sur une version antérieur du Mviewer pour retrouver les clusters ?

eprouteau commented 3 years ago

Les cluster fonctionne, mais il n'y a plus de tooltip ni de sélection possible.

Gaetanbrl commented 3 years ago

Je viens de voir que c'est très probablement lié à une issue en cours :

https://github.com/geobretagne/mviewer/issues/397

C'est une régression suite à la contribution sur la surbrillance qui avait été faite.

Gaetanbrl commented 3 years ago

@eprouteau pour la tooltip je vois dans ton appli que tu as désactivé la tooltip :

tooltip="false

Auparavant cette option était mal prise en compte. Nous avons corrigé ce comportement et ce type d'option fonctionne à présent.

Je viens de tester et avec tooltip="true" je vois bien les tooltip :

image

eprouteau commented 3 years ago

Ok je teste ca

eprouteau commented 3 years ago

Je viens de tester (nickel pour le tooltip j'avais en plus observer ce comportement sur ma visionneuse perso ;) ), par contre au clic sur le cluster ca ne zoom pas.

https://www.pigma.org/public/visualiseur/donneesplages/?config=apps/donneesplages/donneesplages.xml

Gaetanbrl commented 3 years ago

par contre au clic sur le cluster ca ne zoom pas.

Oui je pense que c'est lié à cette fonction de surbrillance non désactivable qui a été intégrée... geobretagne/mviewer#397

eprouteau commented 3 years ago

Dans un premier temps il vaut donc mieux que je passe sur une version antérieur afin de ne pas être bloqué.

Gaetanbrl commented 3 years ago

je pense que ce serait mieux en attendant un correctif... bientôt j'espère car il y a d'autres cas reproduits.

eprouteau commented 3 years ago

Ok ca marche

Gaetanbrl commented 3 years ago

@eprouteau peux-tu tester un fix que je propose sur la brancheissue-34 stp sans revenir sur une version antérieur ?

La branche issue-34 reprend les correctifs de la branche issue-33 comme ca si c'est bon je fais un merge depuis la branche issue-34 seulement vers la branche master (à récupérer ensuite).