Skip to content

Commit bccb83f

Browse files
authored
fix(protocol-designer): fix temperature module stepform copy (#19004)
This PR updates the default title for a new temperature module form from "temperature module state" to "temperature". It also adds the header for "temperature module state" above the heat/cool field Closes RQA-4433
1 parent 7ce9ebc commit bccb83f

File tree

5 files changed

+15
-4
lines changed

5 files changed

+15
-4
lines changed

protocol-designer/cypress/support/ModuleSteps.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@ export enum ModLocators {
1919
export enum ModContent {
2020
ModState = 'Heat or cool',
2121
DeactivateTempDeck = 'Off',
22-
Temperature = 'Temperature module state',
22+
Temperature = 'Temperature',
2323
Save = 'Save',
2424
Temp4CVerification = `Build a pause step to wait until Temperature Module GEN2 reaches 4°C`,
2525
PlateReader = 'Absorbance Plate Reader Module GEN1',

protocol-designer/src/assets/localization/en/form.json

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -407,6 +407,9 @@
407407
"sterility": "sterility",
408408
"sterility&motion": "sterility & motion"
409409
},
410+
"temperature": {
411+
"state": "Temperature Module state"
412+
},
410413
"tipRack": "tip rack",
411414
"wellSelectionLabel": {
412415
"choose_wells": "Choose wells",

protocol-designer/src/pages/Designer/ProtocolSteps/StepForm/StepTools/TemperatureTools/index.tsx

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@ import {
77
DIRECTION_COLUMN,
88
Flex,
99
SPACING,
10+
StyledText,
1011
} from '@opentrons/components'
1112

1213
import {
@@ -44,7 +45,14 @@ export function TemperatureTools(props: StepFormProps): JSX.Element {
4445
}}
4546
/>
4647
<Box borderBottom={`1px solid ${COLORS.grey30}`} />
47-
<Flex padding={`0 ${SPACING.spacing16}`}>
48+
<Flex
49+
flexDirection={DIRECTION_COLUMN}
50+
gridGap={SPACING.spacing4}
51+
padding={`0 ${SPACING.spacing16}`}
52+
>
53+
<StyledText desktopStyle="bodyDefaultSemiBold">
54+
{t('form:step_edit_form.temperature.state')}
55+
</StyledText>
4856
<ToggleExpandStepFormField
4957
{...propsForFields.targetTemperature}
5058
toggleValue={propsForFields.setTemperature.value}

protocol-designer/src/step-forms/test/createPresavedStepForm.test.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -363,7 +363,7 @@ describe('createPresavedStepForm', () => {
363363
// Default fields
364364
setTemperature: null,
365365
targetTemperature: null,
366-
stepName: 'temperature module state',
366+
stepName: 'temperature',
367367
stepDetails: '',
368368
stepNumber: 0,
369369
})

protocol-designer/src/steplist/formLevel/createBlankForm.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,7 @@ const getStepType = (stepType: StepType): string => {
3131
return 'magnetic module state'
3232
}
3333
case 'temperature': {
34-
return 'temperature module state'
34+
return 'temperature'
3535
}
3636
default: {
3737
return stepType

0 commit comments

Comments
 (0)