Angular-google-maps: Manejo de eventos de limpieza

Creado en 20 jun. 2016 · 4Comentarios · Fuente: SebastianM/angular-google-maps

Hola @SebastianM @ brian-singer,

No estoy seguro de si necesitamos eliminar manualmente los oyentes de eventos de marcadores, pero parece que podríamos beneficiarnos de llamar a google.maps.event.clearInstanceListeners(marker); en el administrador de marcadores: https://github.com/SebastianM/angular2-google-maps/ blob / master / src / core / services / managers / marker-manager.ts # L25
En la directiva de marcador, nos suscribimos a eventos con _addEventListeners pero no cancelemos la suscripción explícitamente a ellos al destruir: https://github.com/SebastianM/angular2-google-maps/blob/master/ src / core / directives / google-map-marker.ts # L125 similar a lo que hacemos aquí: https://github.com/SebastianM/angular2-google-maps/blob/master/src/core/directives/google- map.ts # L224

Puede que haya otros también, pero estos son los que me llaman la atención de inmediato, ¿cuáles son tus pensamientos? Feliz de abordar en un PR si quieres. ¡Gracias!

urgent PRs welcome! stale bug

Fuente

alexweber

👍1

Comentario más útil

@alexweber ya, este es un punto que está en mi lista de TODO desde hace un tiempo. Deberíamos abordar esto en la próxima versión. Pensaré en esto un poco más y comentaré más arriba cuando haya encontrado una buena manera de manejar esto en el futuro. ¡Gracias chicos!

SebastianM en 21 jun. 2016

👍2

Todos 4 comentarios

Sí, acepto que el marcador de Google tiene una referencia obsoleta al marcador de SebM

brian-singer en 20 jun. 2016

SebastianM en 21 jun. 2016

👍2

He abordado el elemento 2 en la confirmación anterior, aún no está claro si necesitamos usar google.maps.event.clearInstanceListeners(marker); así que lo dejo por ahora.

alexweber en 6 jul. 2016

👍1

Este problema se ha marcado automáticamente como obsoleto porque no ha tenido actividad reciente. Se cerrará si no se produce más actividad. Gracias por sus aportaciones.