fix(Slice): Refactor code and set active view when mouse scrolling

Author: Laurent Chauvin

src/components/vtk/VtkSliceViewSlicingManipulator.vue

@@ -11,7 +11,8 @@ import vtkMouseRangeManipulator, {
 } from '@kitware/vtk.js/Interaction/Manipulators/MouseRangeManipulator';
 import vtkInteractorStyleManipulator from '@kitware/vtk.js/Interaction/Style/InteractorStyleManipulator';
 import { syncRef } from '@vueuse/core';
-import { inject, toRefs, computed } from 'vue';
+import { inject, toRefs, unref, watch, computed } from 'vue';
+import { useViewStore } from '@/src/store/views';
 
 interface Props {
   viewId: string;
@@ -58,6 +59,13 @@ const scroll = useMouseRangeManipulatorListener(
   sliceConfig.slice.value
 );
 
+watch(scroll, () => {
+  const viewStore = useViewStore();
+  if (unref(viewId) !== viewStore.activeViewID) {
+    viewStore.setActiveViewID(unref(viewId));
+  }
+});
+
 syncRef(scroll, sliceConfig.slice, { immediate: true });
 </script>
 

src/composables/actions.ts

@@ -39,20 +39,12 @@ const showKeyboardShortcuts = () => {
   keyboardStore.settingsOpen = !keyboardStore.settingsOpen;
 };
 
-const nextSlice = () => () => {
+const changeSlice = (offset: number) => () => {
   const { currentImageID } = useCurrentImage();
   const { activeViewID } = useViewStore();
 
   const { slice: currentSlice } = useSliceConfig(activeViewID, currentImageID);
-  currentSlice.value += 1;
-};
-
-const previousSlice = () => () => {
-  const { currentImageID } = useCurrentImage();
-  const { activeViewID } = useViewStore();
-
-  const { slice: currentSlice } = useSliceConfig(activeViewID, currentImageID);
-  currentSlice.value -= 1;
+  currentSlice.value += offset;
 };
 
 export const ACTION_TO_FUNC = {
@@ -67,8 +59,8 @@ export const ACTION_TO_FUNC = {
   polygon: setTool(Tools.Polygon),
   select: setTool(Tools.Select),
 
-  nextSlice: nextSlice(),
-  previousSlice: previousSlice(),
+  nextSlice: changeSlice(1),
+  previousSlice: changeSlice(-1),
 
   decrementLabel: applyLabelOffset(-1),
   incrementLabel: applyLabelOffset(1),