feat(protocol-designer): introduce touch tip speed field to moveLiquid (

#17545) This PR introduces aspirate and dispense touch tip speed to `MoveLiquidsToolbox`, and wires up all the way down through step generation. Closes AUTH-909, Closes AUTH-910
Opentrons · Feb 21, 2025 · f0b21f7 · f0b21f7
1 parent 477ed2f
commit f0b21f7
Show file tree

Hide file tree

Showing 33 changed files with 489 additions and 3,356 deletions.
diff --git a/app/src/organisms/ODD/QuickTransferFlow/types.ts b/app/src/organisms/ODD/QuickTransferFlow/types.ts
@@ -51,6 +51,7 @@ export interface QuickTransferSummaryState {
     positionFromBottom: number
   }
   touchTipAspirate?: number
+  touchTipAspirateSpeed?: number
   airGapAspirate?: number
   tipPositionDispense: number
   mixOnDispense?: {
@@ -62,6 +63,7 @@ export interface QuickTransferSummaryState {
     positionFromBottom: number
   }
   touchTipDispense?: number
+  touchTipDispenseSpeed?: number
   disposalVolume?: number
   blowOut?: BlowOutLocation
   airGapDispense?: number

diff --git a/app/src/organisms/ODD/QuickTransferFlow/utils/generateQuickTransferArgs.ts b/app/src/organisms/ODD/QuickTransferFlow/utils/generateQuickTransferArgs.ts
@@ -376,9 +376,13 @@ export function generateQuickTransferArgs(
     aspirateAirGapVolume: quickTransferState.airGapAspirate ?? null,
     dispenseAirGapVolume: quickTransferState.airGapDispense ?? null,
     touchTipAfterAspirate: quickTransferState.touchTipAspirate != null,
+    touchTipAfterAspirateSpeed:
+      quickTransferState.touchTipAspirateSpeed ?? null,
     touchTipAfterAspirateOffsetMmFromTop,
     touchTipAfterDispense: quickTransferState.touchTipDispense != null,
     touchTipAfterDispenseOffsetMmFromTop,
+    touchTipAfterDispenseSpeed:
+      quickTransferState.touchTipDispenseSpeed ?? null,
     dropTipLocation,
     aspirateXOffset: 0,
     aspirateYOffset: 0,

diff --git a/components/src/atoms/ListButton/index.tsx b/components/src/atoms/ListButton/index.tsx
@@ -9,7 +9,7 @@ import type { StyleProps } from '../../primitives'
 
 export * from './ListButtonChildren/index'
 
-export type ListButtonType = 'noActive' | 'connected' | 'notConnected'
+type ListButtonType = 'noActive' | 'connected' | 'notConnected' | 'onColor'
 
 interface ListButtonProps extends StyleProps {
   type: ListButtonType
@@ -35,6 +35,10 @@ const LISTBUTTON_PROPS_BY_TYPE: Record<
     backgroundColor: COLORS.yellow30,
     hoverBackgroundColor: COLORS.yellow35,
   },
+  onColor: {
+    backgroundColor: COLORS.white,
+    hoverBackgroundColor: COLORS.grey10,
+  },
 }
 
 /*
@@ -78,7 +82,10 @@ export function ListButton(props: ListButtonProps): JSX.Element {
   return (
     <Flex
       data-testid={testId ?? `ListButton_${type}`}
-      onClick={onClick}
+      onClick={(e: MouseEvent) => {
+        onClick?.()
+        e.stopPropagation()
+      }}
       css={LIST_BUTTON_STYLE}
       tabIndex={0}
       {...styleProps}