fix: each icon set shows hover icon simultaneously (#3410)

cognitedata · Jun 23, 2023 · 31d6f2e · 31d6f2e
1 parent e2ca933
commit 31d6f2e
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 4 deletions.
diff --git a/viewer/packages/360-images/src/Image360Facade.ts b/viewer/packages/360-images/src/Image360Facade.ts
@@ -30,8 +30,12 @@ export class Image360Facade<T> {
     this._image360Collections.forEach(collection => collection.setSelectedForAll(visible));
   }
 
-  set hoverIconVisibility(visible: boolean) {
-    this._image360Collections.forEach(collection => collection.setSelectedVisibility(visible));
+  setHoverIconVisibilityForEntity(entity: Image360Entity, visible: boolean): void {
+    this.getCollectionContainingEntity(entity).setSelectedVisibility(visible);
+  }
+
+  hideAllHoverIcons(): void {
+    this._image360Collections.forEach(collection => collection.setSelectedVisibility(false));
   }
 
   set allIconCullingScheme(scheme: IconCullingScheme) {

diff --git a/viewer/packages/api/src/api-helpers/Image360ApiHelper.ts b/viewer/packages/api/src/api-helpers/Image360ApiHelper.ts
@@ -481,11 +481,11 @@ export class Image360ApiHelper {
     }
 
     if (entity !== undefined) {
-      this._image360Facade.hoverIconVisibility = true;
+      this._image360Facade.setHoverIconVisibilityForEntity(entity, true);
       entity.icon.selected = true;
       this._debouncePreLoad(entity);
     } else {
-      this._image360Facade.hoverIconVisibility = false;
+      this._image360Facade.hideAllHoverIcons();
     }
 
     this._needsRedraw = true;