handleScroll() null this object

Situation:

We recently incorporated this library, and our monitoring system (rollbar) began firing with issues like the this:

Our current hypothesis is that the scroll listener is fired after the object is already garbage collected, i.e. this is called after the object is removed: https://github.com/codecks-io/react-sticky-box/blob/master/src/index.js#L243

Expected:

(Assuming the listener is the issue) the listener would be removed before the object is cleaned up

Actual:

(Appears) the listener is not cleaned up.

Note:

Happy to open a PR for you to consider that adds the componentWillUnmount lifecycle method if that would be useful. The proposed change would be something like:

diff --git a/src/index.js b/src/index.js
index fb08721..5a095ec 100644
--- a/src/index.js
+++ b/src/index.js
@@ -57,6 +57,10 @@ export default class StickyBox extends React.Component {
     }
   }
 
+  componentWillUnmount(){
+    this.registerContainerRef(null);
+  }
+
   registerContainerRef = n => {
     if (!stickyProp) return;
     this.node = n;

[danielberndt]

v0.9.3 released. Please let me now if it improves the situation!

[adrienlamotte]

Same issue here ! It happens after the route change (so the StickyBox in not on the page anymore), and I try to resize the page.