Angular-google-maps: Обработка событий очистки

Созданный на 20 июн. 2016 · 4Комментарии · Источник: SebastianM/angular-google-maps

Привет, @SebastianM @

Я не уверен, нужно ли нам вручную удалять прослушиватели событий маркеров, но похоже, что нам может быть полезно вызвать google.maps.event.clearInstanceListeners(marker); в диспетчере маркеров: https://github.com/SebastianM/angular2-google-maps/ blob / master / src / core / services / manager / marker-manager.ts # L25
В директиве маркера мы подписываемся на события с помощью _addEventListeners но явно не отменяем подписку на них при уничтожении: https://github.com/SebastianM/angular2-google-maps/blob/master/ src / core / directives / google-map-marker.ts # L125 аналогично тому, что мы делаем здесь: https://github.com/SebastianM/angular2-google-maps/blob/master/src/core/directives/google- map.ts # L224

Могут быть и другие, но это те, которые сразу привлекают мое внимание, что вы думаете? С радостью обращусь в PR, если хотите. Спасибо!

urgent PRs welcome! stale bug

Источник

alexweber

👍1

Самый полезный комментарий

@alexweber, да , это уже какое-то время в моем списке TODO. Мы должны решить это в следующем выпуске. Я подумаю об этом немного дальше и прокомментирую выше, когда найду хороший способ справиться с этим в будущем. Спасибо, парни!

SebastianM 21 июн. 2016

👍2

Все 4 Комментарий

Да, я согласен, что в Google Marker есть устаревшая ссылка на SebM Marker.

brian-singer 20 июн. 2016

SebastianM 21 июн. 2016

👍2

Я рассмотрел пункт 2 в коммите выше, но до сих пор неясно, нужно ли нам использовать google.maps.event.clearInstanceListeners(marker); поэтому пока оставим его.

alexweber 6 июл. 2016

👍1

Эта проблема была автоматически помечена как устаревшая, поскольку в последнее время не было активности. Он будет закрыт, если больше не будет активности. Спасибо за ваш вклад.