Angular-google-maps: Gestion des événements de nettoyage

Créé le 20 juin 2016 · 4Commentaires · Source: SebastianM/angular-google-maps

Salut @SebastianM @ brian-singer, Veuillez me corriger si je me trompe, mais il semble que nous pourrions resserrer la gestion des événements à certains endroits:

Je ne sais pas si nous devons supprimer manuellement les écouteurs d'événements de marqueurs, mais il semble que nous pourrions bénéficier d'appeler google.maps.event.clearInstanceListeners(marker); dans le gestionnaire de marqueurs: https://github.com/SebastianM/angular2-google-maps/ blob / master / src / core / services / managers / marker-manager.ts # L25
Dans la directive de marqueur, nous nous abonnons à des événements avec _addEventListeners mais ne nous désabonnez pas explicitement lors de la destruction: https://github.com/SebastianM/angular2-google-maps/blob/master/ src / core / directives / google-map-marker.ts # L125 similaire à ce que nous faisons ici: https://github.com/SebastianM/angular2-google-maps/blob/master/src/core/directives/google- map.ts # L224

Il y en a peut-être d'autres aussi mais ce sont celles-là qui attirent tout de suite mon attention, qu'en pensez-vous? Heureux de vous adresser à un PR si vous le souhaitez. Merci!

urgent PRs welcome! stale bug

Source

alexweber

👍1

Commentaire le plus utile

@alexweber ya, c'est un point qui est sur ma liste TODO depuis un moment maintenant. Nous devrions aborder cela dans la prochaine version. J'y réfléchirai un peu plus loin et je commenterai ci-dessus lorsque j'aurai trouvé un bon moyen de gérer cela à l'avenir. Merci les gars!

SebastianM le 21 juin 2016

👍2

Tous les 4 commentaires

Oui, j'accepte que le marqueur google ait alors une référence périmée au marqueur SebM

brian-singer le 20 juin 2016

SebastianM le 21 juin 2016

👍2

J'ai abordé le point 2 dans le commit ci-dessus, je ne sais toujours pas si nous devons utiliser google.maps.event.clearInstanceListeners(marker); alors laissez-le pour le moment

alexweber le 6 juil. 2016

👍1

Ce problème a été automatiquement marqué comme obsolète car il n'a pas eu d'activité récente. Il sera fermé si aucune autre activité ne se produit. Merci pour vos contributions.