fix: document event listener potentially not unregistered (#4509)

Co-authored-by: Hugo Newman <hugo.newman@aize.io> Co-authored-by: Christopher J. Tannum <christopher.tannum@cognite.com>
cognitedata · May 21, 2024 · e7a707d · e7a707d
1 parent 5480f6b
commit e7a707d
Showing 1 changed file with 2 additions and 0 deletions.
diff --git a/viewer/packages/camera-manager/src/DefaultCameraManager.ts b/viewer/packages/camera-manager/src/DefaultCameraManager.ts
@@ -350,6 +350,7 @@ export class DefaultCameraManager implements CameraManager {
         }
         this.keyboardNavigationEnabled = keyboardNavigationEnabled;
         this._domElement.removeEventListener('pointerdown', stopTween);
+        document.removeEventListener('keydown', stopTween);
       })
       .start(TWEEN.now());
     tween.update(TWEEN.now());
@@ -411,6 +412,7 @@ export class DefaultCameraManager implements CameraManager {
         this.setComboControlsOptions({ lookAtViewTarget: false });
         this.keyboardNavigationEnabled = keyboardNavigationEnabled;
         this._domElement.removeEventListener('pointerdown', stopTween);
+        document.removeEventListener('keydown', stopTween);
       })
       .start(TWEEN.now());
     tween.update(TWEEN.now());