fix: mark exclusion zones data as dirty to allow changes to be applied

src/component/panels/filtersPanel/Filters/ExclusionZonesOptionsPanel.tsx

@@ -73,6 +73,7 @@ export default function ExclusionZonesOptionsPanel(
       setValue(
         'zones',
         getValues('zones').filter((_, i) => i !== index),
+        { shouldDirty: true },
       );
     },
     [getValues, setValue],

src/component/panels/filtersPanel/Filters/FiltersSectionsPanel.tsx

@@ -27,6 +27,15 @@ export const nonRemovableFilters = new Set<BaseFilterEntry['name']>([
   'digitalFilter',
   'digitalFilter2D',
 ]);
+const readOnlyFilters = new Set<BaseFilterEntry['name']>([
+  'digitalFilter',
+  'shiftX',
+  'exclusionZones',
+  'fft',
+  'digitalFilter2D',
+  'fftDimension1',
+  'fftDimension2',
+]);
 
 const IconButton = styled(Button)`
   font-size: 16px;
@@ -121,9 +130,14 @@ function FilterElements(props: FilterElementsProps) {
     });
   }
 
+  const isEditable =
+    !hideFilterRestoreButton &&
+    activeFilterID !== id &&
+    !readOnlyFilters.has(name);
+
   return (
     <>
-      {!hideFilterRestoreButton && activeFilterID !== id && (
+      {isEditable && (
         <IconButton
           intent="success"
           tooltipProps={{
@@ -244,11 +258,20 @@ function FiltersInner(props: FiltersInnerProps) {
         return null;
       });
     }
+  }, [filters, selectedTool]);
 
+  useEffect(() => {
     if (Filters?.[selectedTool]) {
       openSection(selectedTool);
+      return;
     }
-  }, [filters, selectedTool]);
+
+    const filter = filters.find((filter) => filter.id === activeFilterID);
+
+    if (filter) {
+      openSection(filter.name);
+    }
+  }, [activeFilterID, filters, selectedTool]);
 
   const filtersList = [...filters];
 

src/component/reducer/actions/FiltersActions.ts

@@ -446,9 +446,13 @@ function rollbackSpectrumByFilter(
     const filterIndex = spectrum.filters.findIndex((f) => f[searchBy] === key);
     if (filterIndex === -1 || reset) {
       if (draft.tempData) {
-        const activeFilterIndex = toolData.activeFilterID
-          ? spectrum.filters.findIndex((f) => f.id === toolData.activeFilterID)
-          : spectrum.filters.length - 1;
+        const activeFilterIndex =
+          !reset && toolData.activeFilterID
+            ? spectrum.filters.findIndex(
+                (f) => f.id === toolData.activeFilterID,
+              )
+            : spectrum.filters.length;
+
         const filters = spectrum.filters.slice(0, activeFilterIndex);
         reapplyFilters(spectrum, filters);
       }
@@ -793,6 +797,8 @@ function handleShiftSpectrumAlongXAxis(
     return;
   }
 
+  const activeFilterIndex = getActiveFilterIndex(draft);
+
   //apply filter into the spectrum
   const options = action.payload;
 
@@ -808,9 +814,14 @@ function handleShiftSpectrumAlongXAxis(
       applyFilter: false,
     });
 
-    Filters1DManager.applyFilters(draft.data[index] as Spectrum1D, [
-      { name: 'shiftX', value: { shift } },
-    ]);
+    Filters1DManager.applyFilters(
+      draft.data[index] as Spectrum1D,
+      [{ name: 'shiftX', value: { shift } }],
+      {
+        filterIndex: activeFilterIndex,
+        forceReapply: activeFilterIndex !== -1,
+      },
+    );
 
     updateView(draft, shiftX.domainUpdateRules);
   }
@@ -824,9 +835,14 @@ function handleShiftSpectrumAlongXAxis(
         searchBy: 'name',
         applyFilter: false,
       });
-      Filters2DManager.applyFilters(draft.data[index] as Spectrum2D, [
-        { name: 'shift2DX', value: { shift: shiftX } },
-      ]);
+      Filters2DManager.applyFilters(
+        draft.data[index] as Spectrum2D,
+        [{ name: 'shift2DX', value: { shift: shiftX } }],
+        {
+          filterIndex: activeFilterIndex,
+          forceReapply: activeFilterIndex !== -1,
+        },
+      );
       updateView(draft, shift2DX.domainUpdateRules);
     }
 
@@ -836,9 +852,14 @@ function handleShiftSpectrumAlongXAxis(
         searchBy: 'name',
         applyFilter: false,
       });
-      Filters2DManager.applyFilters(draft.data[index] as Spectrum2D, [
-        { name: 'shift2DY', value: { shift: shiftY } },
-      ]);
+      Filters2DManager.applyFilters(
+        draft.data[index] as Spectrum2D,
+        [{ name: 'shift2DY', value: { shift: shiftY } }],
+        {
+          filterIndex: activeFilterIndex,
+          forceReapply: activeFilterIndex !== -1,
+        },
+      );
 
       updateView(draft, shift2DY.domainUpdateRules);
     }

src/component/reducer/actions/ToolsActions.ts

@@ -148,10 +148,11 @@ function handleResetSelectedTool(draft: Draft<State>) {
   resetSelectedTool(draft);
 }
 function resetSelectedTool(draft: Draft<State>) {
-  if (
-    draft.toolOptions.selectedTool &&
-    Tools[draft.toolOptions.selectedTool].isFilter
-  ) {
+  const {
+    selectedTool,
+    data: { activeFilterID },
+  } = draft.toolOptions;
+  if ((selectedTool && Tools[selectedTool].isFilter) || activeFilterID) {
     resetTool(draft, { reset: true, toolId: draft.toolOptions.selectedTool });
   }
 }