Hacker News Reader

Parent Full thread

gherkinnn

The event listeners get cleaned up by the GC. At least in modern browsers.

While events aren’t explicitly mentioned, MDN offers an insightful read:

https://developer.mozilla.org/en-US/docs/Web/JavaScript/Memo...

Alternately, one can use event delegation [0]. As does React under the hood [1].

0 - https://javascript.info/event-delegation

1 - https://reactjs.org/blog/2020/08/10/react-v17-rc.html#change...

View on HN

I think there's a reason why events are not mentioned in your linked document... The reference is maintained in memory because the browser has no idea if you will be using it again. After creating the listener, you may very well append it to another element later for all the browser knows.

Also the browser doesn't make a distinction to the type of event (such as `click` ), and say "Oh well you can't click on an element not in the DOM, I think I can clean this up"

So no it will not clean it up and it will leak memory, see for yourself

Here's what I came up with for switching to event delegation: https://github.com/1Marc/todomvc-vanillajs-2022/blob/event-d...

Reply on news.ycombinator.com