feat(viewer): add process conditional statements using isLegacyViewport

lunit-io · Dec 1, 2022 · 7cc19d5 · 7cc19d5
1 parent c097c3e
commit 7cc19d5
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 3 deletions.
diff --git a/libs/insight-viewer/src/hooks/useImageDisplay/index.ts b/libs/insight-viewer/src/hooks/useImageDisplay/index.ts
@@ -61,11 +61,15 @@ const useImageDisplay: UseImageDisplay = ({ element, image, viewportRef, onViewp
       resetViewportRef.current = viewportRef.current._initialViewport
     }
 
-    // Updates the viewport prop of Viewer.
-    if (onViewportChange) {
+    /**
+     * Updates the viewport prop of Viewer.
+     * TODO: Need to delete this logic when removing legacy useViewport
+     */
+    if (onViewportChange && viewportRef.current['isLegacyViewport']) {
       onViewportChange({
         ...formatViewerViewport(viewport),
         _viewportOptions: viewportRef.current._viewportOptions ?? DEFAULT_VIEWPORT_OPTIONS,
+        isLegacyViewport: true,
       })
     }
   }, [image, element, viewportRef, onViewportChange])

diff --git a/libs/insight-viewer/src/hooks/useViewportUpdate.ts b/libs/insight-viewer/src/hooks/useViewportUpdate.ts
@@ -49,7 +49,7 @@ export default function useViewportUpdate({ element, image, viewport: newViewpor
 
     setViewport(<HTMLDivElement>element, formatCornerstoneViewport(viewport, elementUpdatedViewport))
 
-    if (onViewportChange) {
+    if (onViewportChange && newViewportProp.isLegacyViewport) {
       // When resetting, update Viewer's viewport prop
       if (willReset) {
         onViewportChange({