Angular-google-maps: Penanganan acara pembersihan

Dibuat pada 20 Jun 2016 · 4Komentar · Sumber: SebastianM/angular-google-maps

Hai @SebastianM @ brian-penyanyi, Tolong koreksi saya jika saya salah, tetapi sepertinya kami dapat memperketat penanganan acara di beberapa tempat:

Saya tidak yakin apakah kita perlu menghapus pendengar peristiwa penanda secara manual, tetapi sepertinya kita mungkin mendapat manfaat dari memanggil google.maps.event.clearInstanceListeners(marker); di pengelola penanda: https://github.com/SebastianM/angular2-google-maps/ blob / master / src / core / services / manager / marker-manager.ts # L25
Dalam petunjuk penanda, kami berlangganan acara dengan _addEventListeners tetapi tidak secara eksplisit membatalkan langganan acara tersebut saat dihancurkan: https://github.com/SebastianM/angular2-google-maps/blob/master/ src / core / directives / google-map-marker.ts # L125 mirip dengan yang kami lakukan di sini: https://github.com/SebastianM/angular2-google-maps/blob/master/src/core/directives/google- map.ts # L224

Mungkin ada orang lain juga tetapi ini adalah yang langsung menarik perhatian saya, apa pendapat Anda? Senang berbicara di PR jika Anda mau. Terima kasih!

urgent PRs welcome! stale bug

Sumber

alexweber

👍1

Komentar yang paling membantu

@alexweber ya, ini adalah poin yang ada di daftar TODO saya untuk sementara waktu sekarang. Kami harus membahas ini di rilis berikutnya. Saya akan memikirkan hal ini lebih jauh dan akan memberikan komentar di atas ketika saya telah menemukan cara yang baik untuk menangani hal ini di masa mendatang. Terima kasih teman-teman!

SebastianM pada 21 Jun 2016

👍2

Semua 4 komentar

Ya, saya setuju bahwa Google Marker memiliki referensi basi ke SebM Marker

brian-singer pada 20 Jun 2016

SebastianM pada 21 Jun 2016

👍2

Saya telah membahas item 2 dalam komit di atas, masih belum jelas apakah kita perlu menggunakan google.maps.event.clearInstanceListeners(marker); jadi tinggalkan dulu untuk saat ini

alexweber pada 6 Jul 2016

👍1

Masalah ini secara otomatis ditandai sebagai usang karena tidak ada aktivitas terbaru. Ini akan ditutup jika tidak ada aktivitas lebih lanjut. Terima kasih atas kontribusi Anda.