Angular-google-maps: Bereinigung von Ereignisbehandlungen

Erstellt am 20. Juni 2016  ·  4Kommentare  ·  Quelle: SebastianM/angular-google-maps

Hallo @SebastianM @ brian-Sänger, bitte korrigieren Sie mich, wenn ich falsch

  1. Ich bin nicht sicher, ob wir Listener für Markerereignisse manuell entfernen müssen, aber es sieht so aus, als könnten wir davon profitieren, wenn wir im Marker-Manager google.maps.event.clearInstanceListeners(marker); aufrufen: https://github.com/SebastianM/angular2-google-maps/ blob / master / src / core / services / manager / marker-manager.ts # L25
  2. In der Marker-Direktive abonnieren wir Ereignisse mit _addEventListeners , abonnieren sie jedoch nicht explizit bei destroy: https://github.com/SebastianM/angular2-google-maps/blob/master/ src / core / directives / google-map-marker.ts # L125 ähnlich wie hier: https://github.com/SebastianM/angular2-google-maps/blob/master/src/core/directives/google- map.ts # L224

Es mag auch andere geben, aber diese sind es, auf die ich sofort aufmerksam werde. Was denken Sie? Gerne in einer PR ansprechen, wenn Sie möchten. Vielen Dank!

urgent PRs welcome! stale bug
Quelle
picture alexweber
👍1

Hilfreichster Kommentar

@alexweber ya, das ist ein Punkt, der schon eine Weile auf meiner TODO-Liste steht. Wir sollten dies in der nächsten Version ansprechen. Ich werde etwas weiter darüber nachdenken und oben kommentieren, wenn ich einen guten Weg gefunden habe, dies in Zukunft zu handhaben. Danke Leute!

picture SebastianM am 21. Juni 2016
👍2

Alle 4 Kommentare

Ja, ich bin damit einverstanden, dass der Google Marker dann einen veralteten Verweis auf den SebM Marker enthält

brian-singer picture brian-singer am 20. Juni 2016

@alexweber ya, das ist ein Punkt, der schon eine Weile auf meiner TODO-Liste steht. Wir sollten dies in der nächsten Version ansprechen. Ich werde etwas weiter darüber nachdenken und oben kommentieren, wenn ich einen guten Weg gefunden habe, dies in Zukunft zu handhaben. Danke Leute!

SebastianM picture SebastianM am 21. Juni 2016
👍2

Ich habe Punkt 2 im obigen Commit angesprochen, immer noch unklar, ob wir google.maps.event.clearInstanceListeners(marker); also lassen Sie es vorerst

alexweber picture alexweber am 6. Juli 2016
👍1

Dieses Problem wurde automatisch als veraltet markiert, da es in letzter Zeit keine Aktivitäten gab. Es wird geschlossen, wenn keine weitere Aktivität stattfindet. Vielen Dank für Ihre Beiträge.

stale[bot] picture stale[bot] am 14. Nov. 2018
War diese Seite hilfreich?
0 / 5 - 0 Bewertungen

Verwandte Themen

n1t3w0lf picture n1t3w0lf  ·  3Kommentare
marcelinobadin picture marcelinobadin  ·  3Kommentare
dineshkumar20 picture dineshkumar20  ·  3Kommentare
PeterSisovsky picture PeterSisovsky  ·  3Kommentare
muhammad-shahzad-anjum picture muhammad-shahzad-anjum  ·  4Kommentare