Feature handle markers

[elias75015]

Pull request checklist

Verifiez que votre Pull Request remplit les conditions suivantes :

• Des tests ont été ajoutés pour les changements (corrections de bugs ou features)
• De la documentation a été mise à jour ou ajoutée si nécessaire (corrections de bugs ou features)
• Un build (npm run build) a été lancé localement et s'est correctement déroulé
• Les exemples impactés par les modifications (npm run samples) ont été testés et validés localement
• Les tests (npm run test) sont passés localement

Type de Pull request

Ajout de fonctions de manipulation des markers et des overlays en 2D

Quel type de changement cette Pull Request introduit-elle :

• Bugfix
• Feature
• Mise à jour du style du code (syntaxe, renommage de fonctions)
• Refactoring (lisibilité/performance du code, sans changements fonctionnels)
• Changement sur le processus de build
• Contenu de la documentation
• Autres (décrire ci-après) :

Quel est le comportement actuel (avant PR) :

Actuellement, le seul moyen d'ajouter des markers avec le SDK et de rajouter le paramètre markerOptions au chargement de la carte (Gp.Map.Load)

Il n'y a ensuite aucun moyen, via l'interface du SDK, de remplacer ou de supprimer ces markers ainsi ajoutés.
Numéro du ticket : #23

Quel est le nouveau comportement :

• Ajout d'un tableau map._markers pour stocker côté SDK les markersOptions courantes de la carte
• Ajout d'un ensemble de fonctions à OlMapLayers pour manipuler les markers :
  • _removeMarkers() : fonction privée pour supprimer tous les markers de la carte
  • getMarkers(): fonction publique qui renvoie le tableau _markers et donc les options des markers sur la carte
  • setMarkerOptions() : fonction publique pour remplacer les markers présents sur la carte.
• Mise à jour de la documentation (jsdoc) en conséquence

Cette PR introduit-elle des breaking changes ?

• Oui
• Non

Autres informations

Demande initiale : #23

[pjjmunier]

OK pour moi. Ca fonctionne bien en 2D.
Par contre tu ne testes pas ta nouvelle fonctionnalité dans l'exemple (je dis ça parce que tu as cocher la case..)

Je me suis aperçu que lors d'un switch 2D->3D les markers disparaissent. Et que si on fait 2 switch 2D->3D->2D les markers ne réapparaissent pas.
C'était peut être voulu que ça ne fonctionne qu'en 2D, mais il faudrait peut-être voir pour assurer la persistence des markers lors de switchs 2D/3D/2D. (avec le conteneur de markers que tu as mis en place ça devrait pouvoir ce faire)
Et même, de même que pour le geojson, voir avec itowns si on ne pourrait pas afficher les markers en 3D...

[elias75015]

Ajout de 3 commits :

• sauvegarde des markers en 3D (0625650)
• ajout d'un exemple 3D pour tester la conservation des markers (15e24a4)
• mise à jour de la doc (11b5812)

@pjjmunier @lowzonenose : Si vous voulez refaire une passe avec ces derniers ajouts. Sinon je m'autovalide demain soir. 😆