Angular-google-maps: Tratamento de eventos de limpeza

Criado em 20 jun. 2016  ·  4Comentários  ·  Fonte: SebastianM/angular-google-maps

Olá @SebastianM @ brian-singer, Por favor, corrija-me se eu estiver errado, mas parece que poderíamos restringir o tratamento de eventos em alguns lugares:

  1. Não tenho certeza se precisamos remover manualmente os ouvintes de eventos do marcador, mas parece que podemos nos beneficiar chamando google.maps.event.clearInstanceListeners(marker); no gerenciador de marcadores: https://github.com/SebastianM/angular2-google-maps/ blob / master / src / core / services /agers / marker-manager.ts # L25
  2. Na diretiva do marcador, estamos inscrevendo eventos com _addEventListeners mas não cancele explicitamente a inscrição deles no destroy: https://github.com/SebastianM/angular2-google-maps/blob/master/ src / core / directives / google-map-marker.ts # L125 semelhante ao que fazemos aqui: https://github.com/SebastianM/angular2-google-maps/blob/master/src/core/directives/google- map.ts # L224

Pode haver outros também, mas esses são os que chamam minha atenção imediatamente, quais são os seus pensamentos? Fico feliz em falar em um PR, se quiser. Obrigado!

urgent PRs welcome! stale bug

Comentários muito úteis

@alexweber ya, este é um ponto que está na minha lista TODO há um tempo. Devemos resolver isso no próximo lançamento. Vou pensar um pouco mais sobre isso e comentarei acima, quando encontrar uma boa maneira de lidar com isso no futuro. Obrigado rapazes!

Todos 4 comentários

Sim, concordo que o Google Marker tem uma referência desatualizada para o SebM Marker

@alexweber ya, este é um ponto que está na minha lista TODO há um tempo. Devemos resolver isso no próximo lançamento. Vou pensar um pouco mais sobre isso e comentarei acima, quando encontrar uma boa maneira de lidar com isso no futuro. Obrigado rapazes!

Abordei o item 2 no commit acima, ainda não está claro se precisamos usar google.maps.event.clearInstanceListeners(marker); , deixando por enquanto

Este problema foi automaticamente marcado como obsoleto porque não teve atividades recentes. Ele será fechado se nenhuma outra atividade ocorrer. Obrigado por suas contribuições.

Esta página foi útil?
0 / 5 - 0 avaliações

Questões relacionadas

PeterSisovsky picture PeterSisovsky  ·  3Comentários

marcelinobadin picture marcelinobadin  ·  3Comentários

maneesht picture maneesht  ·  3Comentários

n1t3w0lf picture n1t3w0lf  ·  3Comentários

matishw picture matishw  ·  3Comentários