Skip to content

Commit a78b34f

Browse files
committed
fix(protocol-designer): various DQA fixes
This PR provides DQA fixes for 8.4.0 alpha.1. Closes RQA-3929, Closes RQA-3930, Closes RQA-3931, Closes RQA-3933, Closes QA-3934
1 parent 645267a commit a78b34f

File tree

7 files changed

+13
-12
lines changed

7 files changed

+13
-12
lines changed

components/src/atoms/ListButton/__tests__/ListButton.test.tsx

+1-1
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@ describe('ListButton', () => {
2525
it('should render correct style - noActive', () => {
2626
render(props)
2727
const listButton = screen.getByTestId('ListButton_noActive')
28-
expect(listButton).toHaveStyle(`backgroundColor: ${COLORS.grey30}`)
28+
expect(listButton).toHaveStyle(`backgroundColor: ${COLORS.grey20}`)
2929
})
3030
it('should render correct style - connected', () => {
3131
props.type = 'connected'

components/src/atoms/ListButton/index.tsx

+2-2
Original file line numberDiff line numberDiff line change
@@ -23,8 +23,8 @@ const LISTBUTTON_PROPS_BY_TYPE: Record<
2323
{ backgroundColor: string; hoverBackgroundColor: string }
2424
> = {
2525
noActive: {
26-
backgroundColor: COLORS.grey30,
27-
hoverBackgroundColor: COLORS.grey35,
26+
backgroundColor: COLORS.grey20,
27+
hoverBackgroundColor: COLORS.grey30,
2828
},
2929
connected: {
3030
backgroundColor: COLORS.green30,

protocol-designer/src/pages/Designer/ProtocolSteps/StepForm/StepTools/AbsorbanceReaderTools/Initialization.tsx

+6-5
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,7 @@ import {
2424
Tooltip,
2525
useHoverTooltip,
2626
} from '@opentrons/components'
27+
import { LINK_BUTTON_STYLE } from '../../../../../../atoms'
2728
import {
2829
ABSORBANCE_READER_MAX_WAVELENGTH_NM,
2930
ABSORBANCE_READER_MIN_WAVELENGTH_NM,
@@ -244,6 +245,7 @@ function IntializationEditor(props: InitializationEditorProps): JSX.Element {
244245
text={t('step_edit_form.absorbanceReader.add_wavelength.label')}
245246
textAlignment="left"
246247
disabled={numWavelengths === MAX_WAVELENGTHS}
248+
iconName="plus"
247249
/>
248250
</Flex>
249251
) : null}
@@ -337,13 +339,12 @@ function WavelengthItem(props: WavelengthItemProps): JSX.Element {
337339
onClick={() => {
338340
handleDeleteWavelength(index)
339341
}}
340-
alignSelf={ALIGN_FLEX_END}
341342
padding={SPACING.spacing4}
343+
css={LINK_BUTTON_STYLE}
344+
alignSelf={ALIGN_FLEX_END}
345+
textDecoration={TEXT_DECORATION_UNDERLINE}
342346
>
343-
<StyledText
344-
desktopStyle="bodyDefaultRegular"
345-
textDecoration={TEXT_DECORATION_UNDERLINE}
346-
>
347+
<StyledText desktopStyle="bodyDefaultRegular">
347348
{t('step_edit_form.absorbanceReader.delete')}
348349
</StyledText>
349350
</Btn>

protocol-designer/src/pages/Designer/ProtocolSteps/Timeline/AddStepOverflowButton.tsx

+1-1
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,7 @@ export function AddStepOverflowButton(
4545
<StyledText desktopStyle="bodyDefaultRegular">
4646
{i18n.format(
4747
t(`application:stepType.${stepType}`, stepType),
48-
'capitalize'
48+
'titleCase'
4949
)}
5050
</StyledText>
5151
</MenuButton>

protocol-designer/src/pages/Designer/ProtocolSteps/Timeline/__tests__/AddStepButton.test.tsx

+1-1
Original file line numberDiff line numberDiff line change
@@ -87,7 +87,7 @@ describe('AddStepButton', () => {
8787
screen.getByText('Mix')
8888
screen.getByText('Pause')
8989
screen.getByText('Thermocycler')
90-
screen.getByText('Heater-shaker')
90+
screen.getByText('Heater-Shaker')
9191
screen.getByText('Temperature')
9292
screen.getByText('Magnet')
9393
})

protocol-designer/src/pages/Designer/ProtocolSteps/__tests__/ProtocolSteps.test.tsx

+1-1
Original file line numberDiff line numberDiff line change
@@ -115,6 +115,6 @@ describe('ProtocolSteps', () => {
115115

116116
it('renders the current step name', () => {
117117
render()
118-
screen.getByText('Custom pause')
118+
screen.getByText('Custom Pause')
119119
})
120120
})

protocol-designer/src/pages/Designer/ProtocolSteps/index.tsx

+1-1
Original file line numberDiff line numberDiff line change
@@ -120,7 +120,7 @@ export function ProtocolSteps(): JSX.Element {
120120
>
121121
{currentStep != null && hoveredTerminalItem == null ? (
122122
<StyledText desktopStyle="headingSmallBold">
123-
{i18n.format(currentStep.stepName, 'capitalize')}
123+
{i18n.format(currentStep.stepName, 'titleCase')}
124124
</StyledText>
125125
) : null}
126126
{(hoveredTerminalItem != null || selectedTerminalItem != null) &&

0 commit comments

Comments
 (0)