Merge pull request #2550 from NCEAS/bugfix-2549-remove-cesium-listeners

Remove cesium listeners when map is removed from DOM

src/js/views/maps/CesiumWidgetView.js

@@ -1662,6 +1662,12 @@ define([
           );
         }
       },
+
+      /** Remove nav and mouse listeners when the view is closed */
+      onClose() {
+        this.removeMouseListeners();
+        this.removeNavigationListeners();
+      },
     },
   );
 

src/js/views/maps/MapWidgetContainerView.js

@@ -49,6 +49,7 @@ define([
           model: this.model,
         });
         mapWidget.render();
+        this.mapWidget = mapWidget;
       },
 
       /** Renders legend overlay. */
@@ -58,6 +59,7 @@ define([
         });
         legendContainerView.render();
         this.$el.append(legendContainerView.el);
+        this.legendContainerView = legendContainerView;
       },
 
       /**
@@ -80,6 +82,7 @@ define([
           pointModel: interactions.get("mousePosition"),
         });
         scaleBar.render();
+        this.scaleBar = scaleBar;
         this.$el.append(scaleBar.el);
 
         // If the interaction model or relevant sub-models are ever completely
@@ -95,6 +98,20 @@ define([
           this.renderScaleBar,
         );
       },
+
+      /** Call the onClose method of each subview. */
+      onClose() {
+        const subViews = [
+          this.scaleBar,
+          this.legendContainerView,
+          this.mapWidget,
+        ];
+        subViews.forEach((subView) => {
+          if (subView && typeof subView.onClose === "function") {
+            subView.onClose();
+          }
+        });
+      },
     },
   );
 

src/js/views/search/CatalogSearchView.js

@@ -893,6 +893,9 @@ define([
 
           // Remove the JSON-LD from the page
           document.getElementById("jsonld")?.remove();
+
+          // Remove the map
+          this.mapView?.onClose();
         } catch (e) {
           console.error("Couldn't close search view. ", e);
         }