feat: inset brush end

src/component/1d/BrushTracker1D.tsx

@@ -87,7 +87,10 @@ export function BrushTracker1D({ children }) {
 
       if (!inset) return;
 
-      dispatch({ type: 'MOVE_INSET', payload: { insetKey: inset.id, shiftX, shiftY: 0 } });
+      dispatch({
+        type: 'MOVE_INSET',
+        payload: { insetKey: inset.id, shiftX, shiftY: 0 },
+      });
     }
   }
 
@@ -272,15 +275,20 @@ export function BrushTracker1D({ children }) {
       brushStartRef.current = null;
 
       const keyModifiers = getModifiersKey(brushData as unknown as MouseEvent);
-
       if (
-        brushData.mouseButton === 'main' &&
+        !inset ||
+        brushData.mouseButton !== 'main' ||
         keyModifiers !== primaryKeyIdentifier
-      ) {
-        dispatch({ type: 'BRUSH_END', payload: brushData });
-      }
+      )
+        return;
+
+      const { startX, endX } = brushData;
+      dispatch({
+        type: 'BRUSH_END_INSET',
+        payload: { insetKey: inset.id, startX, endX },
+      });
     },
-    [getModifiersKey, primaryKeyIdentifier, dispatch],
+    [getModifiersKey, inset, primaryKeyIdentifier, dispatch],
   );
 
   const handleOnDoubleClick = useCallback(() => {

src/component/1d/inset/InsetProvider.tsx

@@ -5,6 +5,13 @@ import type { Margin } from '../../reducer/Reducer.js';
 
 import type { Inset } from './SpectraInsets.js';
 
+export const insetMargin: Margin = {
+  top: 10,
+  right: 10,
+  bottom: 30,
+  left: 10,
+} as const;
+
 interface BaseInsetPagContextProps
   extends Pick<Inset, 'xDomain' | 'yDomain' | 'id' | 'spectrumKey'> {
   width: number;
@@ -49,12 +56,7 @@ export function InsetProvider(props: InsetProviderProps) {
     return {
       width,
       height,
-      margin: {
-        top: 10,
-        right: 10,
-        bottom: 30,
-        left: 10,
-      },
+      margin: insetMargin,
       id,
       xDomain,
       yDomain,

src/component/reducer/Reducer.ts

@@ -777,6 +777,9 @@ function innerSpectrumReducer(draft: Draft<State>, action: Action) {
       case 'MOVE_INSET': {
         return InsetActions.handleMoveInset(draft, action);
       }
+      case 'BRUSH_END_INSET': {
+        return InsetActions.handleInsetBrushEnd(draft, action);
+      }
       case 'SECRET_THROW_ERROR': {
         throw new Error('Error thrown in main reducer');
       }

src/component/reducer/actions/DomainActions.ts

@@ -357,7 +357,8 @@ function handleSetAxisDomain(draft: Draft<State>, action: SetAxisDomainAction) {
 }
 
 interface Domain {
-  xDomain: number[]; yDomain: number[]
+  xDomain: number[];
+  yDomain: number[];
 }
 
 function moveOverAxis(

src/component/reducer/actions/InsetActions.ts

@@ -2,7 +2,9 @@ import { v4 } from '@lukeed/uuid';
 import type { Draft } from 'immer';
 
 import { isSpectrum1D } from '../../../data/data1d/Spectrum1D/isSpectrum1D.js';
+import { insetMargin } from '../../1d/inset/InsetProvider.js';
 import type { Inset, InsetBounding } from '../../1d/inset/SpectraInsets.js';
+import { getInsetXScale } from '../../1d/utilities/scale.js';
 import type { State } from '../Reducer.js';
 import { getActiveSpectrum } from '../helper/getActiveSpectrum.js';
 import getRange from '../helper/getRange.js';
@@ -11,6 +13,11 @@ import type { ActionType } from '../types/ActionType.js';
 import { moveOverAxis } from './DomainActions.js';
 import type { MoveOptions } from './DomainActions.js';
 
+interface BrushInsetBoundary {
+  startX: number;
+  endX: number;
+}
+
 type AddInsetAction = ActionType<'ADD_INSET', { startX: number; endX: number }>;
 type DeleteInsetAction = ActionType<'DELETE_INSET', { insetKey: string }>;
 type ChangeInsetBoundingAction = ActionType<
@@ -21,12 +28,17 @@ type MoveInsetAction = ActionType<
   'MOVE_INSET',
   MoveOptions & { insetKey: string }
 >;
+type BrushEndInsetAction = ActionType<
+  'BRUSH_END_INSET',
+  BrushInsetBoundary & { insetKey: string }
+>;
 
 export type InsetsActions =
   | AddInsetAction
   | DeleteInsetAction
   | ChangeInsetBoundingAction
-  | MoveInsetAction;
+  | MoveInsetAction
+  | BrushEndInsetAction;
 
 function handleAddInset(draft: Draft<State>, action: AddInsetAction) {
   const activeSpectrum = getActiveSpectrum(draft);
@@ -88,9 +100,7 @@ function handleDeleteInset(draft: Draft<State>, action: DeleteInsetAction) {
   );
 }
 
-
 function getInset(draft: Draft<State>, insetKey: string) {
-
   const {
     view: {
       spectra: { activeTab },
@@ -99,10 +109,8 @@ function getInset(draft: Draft<State>, insetKey: string) {
 
   const insets = draft.insets[activeTab];
   return insets.find((inset) => inset.id === insetKey);
-
 }
 
-
 function handleChangeInsetBounding(
   draft: Draft<State>,
   action: ChangeInsetBoundingAction,
@@ -116,16 +124,13 @@ function handleChangeInsetBounding(
   inset.bounding = { ...inset.bounding, ...bounding };
 }
 
-
-
-
 function handleMoveInset(draft: Draft<State>, action: MoveInsetAction) {
   const { insetKey, shiftX, shiftY } = action.payload;
   const inset = getInset(draft, insetKey);
 
   if (!inset) return;
 
-  const originXDomain = draft.originDomain.xDomains[inset.spectrumKey]
+  const originXDomain = draft.originDomain.xDomains[inset.spectrumKey];
   const originYDomain = draft.originDomain.yDomains[inset.spectrumKey];
   const { xDomain, yDomain } = moveOverAxis(
     { shiftX, shiftY },
@@ -136,5 +141,36 @@ function handleMoveInset(draft: Draft<State>, action: MoveInsetAction) {
   inset.yDomain = yDomain;
 }
 
+function handleInsetBrushEnd(draft: Draft<State>, action: BrushEndInsetAction) {
+  const { insetKey, startX: inputStartX, endX: inputEndX } = action.payload;
+  const inset = getInset(draft, insetKey);
+
+  if (!inset) return;
+
+  const { mode } = draft;
+
+  const {
+    xDomain: currentXDomain,
+    bounding: { width },
+  } = inset;
+
+  const xScale = getInsetXScale({
+    width,
+    xDomain: currentXDomain,
+    mode,
+    margin: insetMargin,
+  });
+
+  const startX = xScale.invert(inputStartX);
+  const endX = xScale.invert(inputEndX);
+  const xDomain = startX > endX ? [endX, startX] : [startX, endX];
+  inset.xDomain = xDomain;
+}
 
-export { handleAddInset, handleDeleteInset, handleChangeInsetBounding, handleMoveInset };
+export {
+  handleAddInset,
+  handleDeleteInset,
+  handleChangeInsetBounding,
+  handleMoveInset,
+  handleInsetBrushEnd,
+};