Zaid-Ajaj / Feliz

A fresh retake of the React API in Fable and a collection of high-quality components to build React applications in F#, optimized for happiness
https://zaid-ajaj.github.io/Feliz/
MIT License
534 stars 78 forks source link

React memory leak warning when using Feliz.RoughViz #381

Open Numpsy opened 3 years ago

Numpsy commented 3 years ago

Hi,

I'm a total beginner with React and Feliz and such so I might just be missing something simple, but in case - I recently tried creating a simple app using the SAFE.React template and added some simple charts to it using Feliz.RoughViz, and then when I was looking at something else in the Firefox dev tools I noticed that I was getting this warning when navigating away from the page containing the charts:

image

I tried doing the same thing with the Felix navigation bar example and got the same warning (there is a commit containing the change at https://github.com/Numpsy/navigation-bar-with-feliz/commit/292769aedd7ceca7a5bc168a633402d4eee5c724 )

Any ideas?

Thanks.

Zaid-Ajaj commented 3 years ago

Not sure why this is happening, I will take a look!