about summary refs log tree commit diff
path: root/app/javascript
diff options
context:
space:
mode:
authoralpaca-tc <alpaca-tc@alpaca.tc>2017-06-19 18:29:57 +0900
committerEugen Rochko <eugen@zeonfederated.com>2017-06-19 11:29:57 +0200
commitcf6fe4f8cbafeb2f4e3a92bac4f3909cfc327edf (patch)
treeedff310deff7accca16afb187b23ce6b92fb4ed8 /app/javascript
parent436744328726d5ba32a90dbdaf60e57272c788db (diff)
Unobserve status on unmount (#3851)
Diffstat (limited to 'app/javascript')
-rw-r--r--app/javascript/mastodon/components/status.js10
-rw-r--r--app/javascript/mastodon/features/ui/util/intersection_observer_wrapper.js7
2 files changed, 17 insertions, 0 deletions
diff --git a/app/javascript/mastodon/components/status.js b/app/javascript/mastodon/components/status.js
index 7ca898c39..3b8f88bf7 100644
--- a/app/javascript/mastodon/components/status.js
+++ b/app/javascript/mastodon/components/status.js
@@ -88,6 +88,16 @@ class Status extends ImmutablePureComponent {
     );
   }
 
+  componentWillUnmount () {
+    if (!this.props.intersectionObserverWrapper) {
+      // TODO: enable IntersectionObserver optimization for notification statuses.
+      // These are managed in notifications/index.js rather than status_list.js
+      return;
+    }
+
+    this.props.intersectionObserverWrapper.unobserve(this.props.id, this.node);
+  }
+
   handleIntersection = (entry) => {
     // Edge 15 doesn't support isIntersecting, but we can infer it
     // https://developer.microsoft.com/en-us/microsoft-edge/platform/issues/12156111/
diff --git a/app/javascript/mastodon/features/ui/util/intersection_observer_wrapper.js b/app/javascript/mastodon/features/ui/util/intersection_observer_wrapper.js
index 0e959f9ae..b15f8b9ef 100644
--- a/app/javascript/mastodon/features/ui/util/intersection_observer_wrapper.js
+++ b/app/javascript/mastodon/features/ui/util/intersection_observer_wrapper.js
@@ -37,6 +37,13 @@ class IntersectionObserverWrapper {
     }
   }
 
+  unobserve (id, node) {
+    if (this.observer) {
+      delete this.callbacks[id];
+      this.observer.unobserve(node);
+    }
+  }
+
   disconnect () {
     if (this.observer) {
       this.observer.disconnect();