removeEventListener never works

[giannoudak]

Hello,

i am developing a library for a site, in which a have a scroll listener and check if a specific div is visible.
when i reach that point i immediately remove the scroll listener and do an ad call

the site where my library runs has the

http://cdnjs.cloudflare.com/ajax/libs/zone.js/0.8.18/zone.min.js

on it.

However although i remove the listener it continues to fire again and again. As a result a have multiple ad calls. I tried to block the zone.js script using ublock extention and everything works ok. So it seems that its a zone.js issue

heres is my code snippet

var addScrollListener = function(){


  if (isDebug()){
    window.addEventListener('scroll', onScroll);
  }

}

var onScroll = function(event){

  if (isAdVisible()){
    
    console.log(event);

    window.removeEventListener('scroll', onScroll);
    loadAdTad();
  }
}

var bindEvents = function(){
  addScrollListener();

}

bindEvents();

am i doing something wrong?

[JiaLiPassion]

@giannoudak , I tried your sample code but I can't reproduce, the onScroll only triggered once.
could you post a reproduce plunker or repo?

[bddean]

@JiaLiPassion I had a similar issue reproducible here: https://plnkr.co/edit/wU6Vc5sFCHlqWE7qB3B0?p=preview

ZoneJS's removeEventListener doesn't seem to work for handlers that have been added by the native version of addEventListener (so before zone.js loads). This is a problem for third party scripts/libraries that load before zone.js does.

[JiaLiPassion]

@bddean, thank you for the reproduce plunker, I will fix this one.

[bddean]

Thanks!