From 31d6f2ed7fda8b7947891ed8705e7a02b54bf321 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?H=C3=A5kon=20Flatval?= <70905152+haakonflatval-cognite@users.noreply.github.com> Date: Fri, 23 Jun 2023 16:59:34 +0200 Subject: [PATCH] fix: each icon set shows hover icon simultaneously (#3410) --- viewer/packages/360-images/src/Image360Facade.ts | 8 ++++++-- viewer/packages/api/src/api-helpers/Image360ApiHelper.ts | 4 ++-- 2 files changed, 8 insertions(+), 4 deletions(-) diff --git a/viewer/packages/360-images/src/Image360Facade.ts b/viewer/packages/360-images/src/Image360Facade.ts index f545679f32a..75ca66b4c1a 100644 --- a/viewer/packages/360-images/src/Image360Facade.ts +++ b/viewer/packages/360-images/src/Image360Facade.ts @@ -30,8 +30,12 @@ export class Image360Facade { 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 index c2126c5eac9..463362453db 100644 --- 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;