Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore(release): merge chore_release-pd-8.4.0 into edge #17452

Closed
wants to merge 21 commits into from
Closed
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
21 commits
Select commit Hold shift + click to select a range
65e125e
fix(protocol-designer): modify tiprack status check for deselecting …
koji Jan 29, 2025
42dc7bf
fix(protocol-designer): filter trashbins from move labware new locati…
ncdiehl11 Jan 29, 2025
890ab23
fix(step-generation): properly update robotState when dispense is aft…
jerader Jan 31, 2025
624d307
fix(protocol-designer): modify the list button width (#17389)
koji Jan 31, 2025
38e60c3
fix(protocol-designer): fix magnetic module errors (#17395)
ncdiehl11 Jan 31, 2025
7eb69e9
fix(protocol-designer): fix StepSummary wrapping (#17401)
ncdiehl11 Feb 3, 2025
2525a63
fix(protocol-designer): delete labware if staging area deleted (#17397)
ncdiehl11 Feb 3, 2025
dc48e10
fix(protocol-designer): fix staging area logic (#17403)
koji Feb 3, 2025
e37b61f
fix(protocol-designer, components) fix deck view size issue (#17394)
koji Feb 4, 2025
e2bc455
fix(protocol-designer): fix empty selector button size issue (#17418)
koji Feb 4, 2025
645267a
fix(components): Fix large button outline issue (#17415)
koji Feb 4, 2025
6c89f59
fix(protocol-designer): fix step overflow menu position (#17439)
koji Feb 6, 2025
cea40c2
fix(protocol-designer): fix presaved absorbance reader step form type…
ncdiehl11 Feb 6, 2025
9c70e65
fix(protocol-designer): various DQA fixes (#17440)
ncdiehl11 Feb 6, 2025
78848e5
fix(components, protocol-designer): fix design bugs from PD 8.4.0 alp…
ncdiehl11 Feb 6, 2025
a486e17
fix(components, protocol-designer): fix lineclamp for dropdown menu (…
ncdiehl11 Feb 6, 2025
52fe227
fix(components): add focus-visible style to list button (#17450)
koji Feb 6, 2025
09e127d
fix(protocol-designer): fix temperature step form (#17447)
ncdiehl11 Feb 6, 2025
058b72b
Merge branch 'chore_release-pd-8.4.0' into mergeback_release-pd-840_i…
ncdiehl11 Feb 6, 2025
5b25f4b
fix tests post-conflict resolution
ncdiehl11 Feb 6, 2025
899d36c
fix pd test fixture
ncdiehl11 Feb 6, 2025
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@ describe('ListButton', () => {
it('should render correct style - noActive', () => {
render(props)
const listButton = screen.getByTestId('ListButton_noActive')
expect(listButton).toHaveStyle(`backgroundColor: ${COLORS.grey30}`)
expect(listButton).toHaveStyle(`backgroundColor: ${COLORS.grey20}`)
})
it('should render correct style - connected', () => {
props.type = 'connected'
Expand Down
10 changes: 8 additions & 2 deletions components/src/atoms/ListButton/index.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -23,8 +23,8 @@ const LISTBUTTON_PROPS_BY_TYPE: Record<
{ backgroundColor: string; hoverBackgroundColor: string }
> = {
noActive: {
backgroundColor: COLORS.grey30,
hoverBackgroundColor: COLORS.grey35,
backgroundColor: COLORS.grey20,
hoverBackgroundColor: COLORS.grey30,
},
connected: {
backgroundColor: COLORS.green30,
Expand Down Expand Up @@ -60,13 +60,19 @@ export function ListButton(props: ListButtonProps): JSX.Element {
? COLORS.grey20
: listButtonProps.hoverBackgroundColor};
}

&:focus-visible {
outline: 2px solid ${COLORS.blue50};
outline-offset: 0.25rem;
}
`

return (
<Flex
data-testid={`ListButton_${type}`}
onClick={onClick}
css={LIST_BUTTON_STYLE}
tabIndex={0}
{...styleProps}
>
{children}
Expand Down
5 changes: 2 additions & 3 deletions components/src/atoms/buttons/EmptySelectorButton.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,6 @@ import {
ALIGN_CENTER,
CURSOR_DEFAULT,
CURSOR_POINTER,
FLEX_MAX_CONTENT,
Icon,
JUSTIFY_CENTER,
JUSTIFY_START,
Expand Down Expand Up @@ -62,8 +61,8 @@ interface ButtonProps {

const StyledButton = styled.button<ButtonProps>`
border: none;
width: ${FLEX_MAX_CONTENT};
height: ${FLEX_MAX_CONTENT};
width: 100%;
height: 100%;
cursor: ${CURSOR_POINTER};
background-color: ${COLORS.blue30};
border-radius: ${BORDERS.borderRadius8};
Expand Down
4 changes: 3 additions & 1 deletion components/src/atoms/buttons/LargeButton.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -206,7 +206,9 @@ export function LargeButton(props: LargeButtonProps): JSX.Element {

border: ${buttonType === 'stroke'
? `2px solid ${COLORS.blue55}`
: `${computedBorderStyle()}`};
: buttonType === 'primary'
? `4px solid ${COLORS.blue55}`
: computedBorderStyle()};
}

&:focus-visible {
Expand Down
21 changes: 15 additions & 6 deletions components/src/molecules/DropdownMenu/index.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -27,6 +27,7 @@
import { DeckInfoLabel } from '../DeckInfoLabel'

import type { FocusEventHandler } from 'react'
import type { FlattenSimpleInterpolation } from 'styled-components'

export interface DropdownOption {
name: string
Expand Down Expand Up @@ -268,7 +269,7 @@
>
<StyledText
desktopStyle="captionRegular"
css={LINE_CLAMP_TEXT_STYLE}
css={LINE_CLAMP_TEXT_STYLE(1)}
>
{currentOption.name}
</StyledText>
Expand Down Expand Up @@ -327,12 +328,15 @@
flexDirection={DIRECTION_COLUMN}
gridGap={option.subtext != null ? SPACING.spacing4 : '0'}
>
<StyledText desktopStyle="captionRegular">
<StyledText
desktopStyle="captionRegular"
css={LINE_CLAMP_TEXT_STYLE(3)}
>
{option.name}
</StyledText>
<StyledText
desktopStyle="captionRegular"
color={COLORS.black70}
color={COLORS.grey60}
>
{option.subtext}
</StyledText>
Expand Down Expand Up @@ -363,12 +367,17 @@
)
}

const LINE_CLAMP_TEXT_STYLE = css`
export const LINE_CLAMP_TEXT_STYLE = (
lineClamp?: number,
title?: boolean
): FlattenSimpleInterpolation => css`
display: -webkit-box;
-webkit-box-orient: vertical;
overflow: ${OVERFLOW_HIDDEN};
text-overflow: ellipsis;
word-wrap: break-word;
-webkit-line-clamp: 1;
word-break: break-all;
-webkit-line-clamp: ${lineClamp ?? 1};
word-break: ${title === true
? 'normal'

Check warning on line 381 in components/src/molecules/DropdownMenu/index.tsx

View check run for this annotation

Codecov / codecov/patch

components/src/molecules/DropdownMenu/index.tsx#L381

Added line #L381 was not covered by tests
: 'break-all'}; // normal for tile and break-all for a non word case like aaaaaaaa
`
69 changes: 38 additions & 31 deletions protocol-designer/fixtures/protocol/8/doItAllV4MigratedToV8.json
Original file line number Diff line number Diff line change
Expand Up @@ -6,16 +6,16 @@
"author": "Fixture",
"description": "Test all v4 commands",
"created": 1585930833548,
"lastModified": 1738682299723,
"lastModified": 1738870198854,
"category": null,
"subcategory": null,
"tags": []
},
"designerApplication": {
"name": "opentrons/protocol-designer",
"version": "8.5.0",
"version": "8.4.0-alpha.0",
"data": {
"_internalAppBuildDate": "Wed, 29 Jan 2025 13:23:47 GMT",
"_internalAppBuildDate": "Thu, 06 Feb 2025 19:02:29 GMT",
"pipetteTiprackAssignments": {
"0b3f2210-75c7-11ea-b42f-4b64e50f43e5": [
"opentrons/opentrons_96_tiprack_300ul/1"
Expand All @@ -28,10 +28,10 @@
"ingredients": {
"0": {
"displayName": "Water",
"displayColor": "#b925ff",
"description": null,
"liquidGroupId": "0",
"pythonName": "liquid_1",
"liquidGroupId": "0"
"displayColor": "#b925ff"
}
},
"ingredLocations": {
Expand Down Expand Up @@ -133,7 +133,7 @@
"0b3f2210-75c7-11ea-b42f-4b64e50f43e5": "left"
},
"trashBinLocationUpdate": {
"8c280499-46f1-4a7c-9096-553e87fe9d1e:trashBin": "cutout12"
"9492f840-1c61-4cc7-9e0e-5f5d7382dc5d:trashBin": "cutout12"
},
"wasteChuteLocationUpdate": {},
"stagingAreaLocationUpdate": {},
Expand Down Expand Up @@ -194,7 +194,7 @@
"aspirate_y_position": 0,
"blowout_checkbox": false,
"blowout_flowRate": null,
"blowout_location": "8c280499-46f1-4a7c-9096-553e87fe9d1e:trashBin",
"blowout_location": "9492f840-1c61-4cc7-9e0e-5f5d7382dc5d:trashBin",
"blowout_z_offset": 0,
"changeTip": "always",
"dispense_airGap_checkbox": false,
Expand All @@ -219,7 +219,7 @@
"dispense_y_position": 0,
"disposalVolume_checkbox": true,
"disposalVolume_volume": "20",
"dropTip_location": "8c280499-46f1-4a7c-9096-553e87fe9d1e:trashBin",
"dropTip_location": "9492f840-1c61-4cc7-9e0e-5f5d7382dc5d:trashBin",
"nozzles": null,
"path": "single",
"pipette": "0b3f2210-75c7-11ea-b42f-4b64e50f43e5",
Expand Down Expand Up @@ -2667,7 +2667,7 @@
"commandSchemaId": "opentronsCommandSchemaV8",
"commands": [
{
"key": "f18c57de-1196-4656-9149-6f8f519ded50",
"key": "8ad8de05-e089-4d95-8ec3-5f3f153a9f68",
"commandType": "loadPipette",
"params": {
"pipetteName": "p300_single_gen2",
Expand All @@ -2676,7 +2676,7 @@
}
},
{
"key": "f659bbae-c163-4770-ba70-94d779944e3e",
"key": "c9408e25-2234-4213-9842-b3c11f825557",
"commandType": "loadModule",
"params": {
"model": "magneticModuleV2",
Expand All @@ -2687,7 +2687,7 @@
}
},
{
"key": "211b1d23-4554-4fcb-9053-9b7620f3370a",
"key": "6845e4df-214f-4f14-a6db-586046200267",
"commandType": "loadModule",
"params": {
"model": "temperatureModuleV2",
Expand All @@ -2698,7 +2698,7 @@
}
},
{
"key": "02c8120e-ca2c-4fa2-a48c-0bf29c82d552",
"key": "71a3b441-44ea-46d2-8177-961875945428",
"commandType": "loadLabware",
"params": {
"displayName": "Opentrons 96 Tip Rack 300 µL",
Expand All @@ -2712,7 +2712,7 @@
}
},
{
"key": "692d6fc7-be48-40ae-8b96-46a6ae939af3",
"key": "e8faad5f-c68f-4bc6-bad3-71c2187c207a",
"commandType": "loadLabware",
"params": {
"displayName": "NEST 96 Well Plate 100 µL PCR Full Skirt",
Expand All @@ -2726,7 +2726,7 @@
}
},
{
"key": "e96ea68c-e1e6-43f2-a4b2-213372f41623",
"key": "e89bc408-8749-4fcb-9f6a-71fa34960457",
"commandType": "loadLabware",
"params": {
"displayName": "Opentrons 24 Well Aluminum Block with Generic 2 mL Screwcap",
Expand All @@ -2741,7 +2741,7 @@
},
{
"commandType": "loadLiquid",
"key": "8e9bdf50-5ef7-451f-9d0e-95015241fc6e",
"key": "e5fce912-b6f6-4313-8240-71126db4b74d",
"params": {
"liquidId": "0",
"labwareId": "1e610d40-75c7-11ea-b42f-4b64e50f43e5:opentrons/nest_96_wellplate_100ul_pcr_full_skirt/1",
Expand All @@ -2767,31 +2767,31 @@
},
{
"commandType": "magneticModule/engage",
"key": "77b1d364-2ab1-4a4e-8fae-a9ecf4b88323",
"key": "6908e704-a7f0-4c36-9373-a6d12d16b91d",
"params": {
"moduleId": "0b419310-75c7-11ea-b42f-4b64e50f43e5:magneticModuleType",
"height": 6
}
},
{
"commandType": "temperatureModule/setTargetTemperature",
"key": "11956c88-bcad-4580-9036-869cc86a7b6b",
"key": "910fd1ca-19ac-4ea5-9737-76d4d6f86de7",
"params": {
"moduleId": "0b4319b0-75c7-11ea-b42f-4b64e50f43e5:temperatureModuleType",
"celsius": 25
}
},
{
"commandType": "waitForDuration",
"key": "c9520683-6f9d-44ae-b158-3f5de7b1a4b5",
"key": "e082e2b2-6a25-4ede-9fcd-f02231311c3a",
"params": {
"seconds": 62,
"message": ""
}
},
{
"commandType": "pickUpTip",
"key": "56cecd58-ca52-433d-aa71-98d59558d388",
"key": "495834d9-24d3-4c61-a82b-0f78147c31e8",
"params": {
"pipetteId": "0b3f2210-75c7-11ea-b42f-4b64e50f43e5",
"labwareId": "0b44c760-75c7-11ea-b42f-4b64e50f43e5:opentrons/opentrons_96_tiprack_300ul/1",
Expand All @@ -2800,7 +2800,7 @@
},
{
"commandType": "aspirate",
"key": "e5f246a4-3ebf-4f73-99c9-2a69ad19f4bc",
"key": "4ac4c27d-0904-458e-8afa-b5e9679600b5",
"params": {
"pipetteId": "0b3f2210-75c7-11ea-b42f-4b64e50f43e5",
"volume": 30,
Expand All @@ -2819,7 +2819,7 @@
},
{
"commandType": "dispense",
"key": "fb1f0bb6-8f12-4662-be15-7e8d49368112",
"key": "eb5b5bb0-61c2-436b-b009-1626e7797857",
"params": {
"pipetteId": "0b3f2210-75c7-11ea-b42f-4b64e50f43e5",
"volume": 30,
Expand All @@ -2838,7 +2838,7 @@
},
{
"commandType": "moveToAddressableAreaForDropTip",
"key": "321568ae-8d86-4c60-9ff3-2af0e73b6ab1",
"key": "002343a7-59fb-4aae-999b-7dbe30dc5cd7",
"params": {
"pipetteId": "0b3f2210-75c7-11ea-b42f-4b64e50f43e5",
"addressableAreaName": "fixedTrash",
Expand All @@ -2852,14 +2852,14 @@
},
{
"commandType": "dropTipInPlace",
"key": "d432d161-9da6-462d-a757-196de673d3d4",
"key": "1bd89a6e-2228-4918-800c-adc2d24a39c2",
"params": {
"pipetteId": "0b3f2210-75c7-11ea-b42f-4b64e50f43e5"
}
},
{
"commandType": "pickUpTip",
"key": "1fd1fdbf-592a-43f4-99ad-b1e5c3b1feab",
"key": "b8c2cc90-9a27-4c1a-9744-28481ea3ba8c",
"params": {
"pipetteId": "0b3f2210-75c7-11ea-b42f-4b64e50f43e5",
"labwareId": "0b44c760-75c7-11ea-b42f-4b64e50f43e5:opentrons/opentrons_96_tiprack_300ul/1",
Expand All @@ -2868,7 +2868,7 @@
},
{
"commandType": "aspirate",
"key": "8af4e21b-c9da-4dc9-b033-3f2ae823fa4c",
"key": "697eb74c-71db-4ba7-a1bc-39268cbf42e3",
"params": {
"pipetteId": "0b3f2210-75c7-11ea-b42f-4b64e50f43e5",
"volume": 30,
Expand All @@ -2887,7 +2887,7 @@
},
{
"commandType": "dispense",
"key": "6b4526b1-4738-4600-9920-53881f0735c8",
"key": "dcc41db6-9022-451b-b8e6-431392c330ba",
"params": {
"pipetteId": "0b3f2210-75c7-11ea-b42f-4b64e50f43e5",
"volume": 30,
Expand All @@ -2906,7 +2906,7 @@
},
{
"commandType": "moveToAddressableAreaForDropTip",
"key": "0510a4d6-796d-4917-8a2f-f26a19078091",
"key": "a05cebee-5028-4036-8bad-167d82d241a0",
"params": {
"pipetteId": "0b3f2210-75c7-11ea-b42f-4b64e50f43e5",
"addressableAreaName": "fixedTrash",
Expand All @@ -2920,32 +2920,39 @@
},
{
"commandType": "dropTipInPlace",
"key": "69bac7a6-d8a0-4674-9b7f-5b8b0174d765",
"key": "b68f05c5-ed53-4256-8b26-50b1d60a13a6",
"params": {
"pipetteId": "0b3f2210-75c7-11ea-b42f-4b64e50f43e5"
}
},
{
"commandType": "temperatureModule/waitForTemperature",
"key": "d58f0651-b95c-4997-a06d-017fab7c27e4",
"key": "5a707661-6c2b-4246-961e-723e87cb98f5",
"params": {
"moduleId": "0b4319b0-75c7-11ea-b42f-4b64e50f43e5:temperatureModuleType",
"celsius": 25
}
},
{
"commandType": "magneticModule/disengage",
"key": "92a4f6f7-521f-42d7-aa0a-c8553fc11333",
"key": "cf07905e-f248-4483-b181-ed32cd84726e",
"params": {
"moduleId": "0b419310-75c7-11ea-b42f-4b64e50f43e5:magneticModuleType"
}
},
{
"commandType": "waitForResume",
"key": "df776ec4-c8ef-416b-be42-1b2ea7ef51bc",
"key": "7f60d95b-be21-4e77-b6e2-28e3f3f603f1",
"params": {
"message": "Wait until user intervention"
}
},
{
"commandType": "temperatureModule/deactivate",
"key": "35b74171-cc80-45a9-ba40-a2f2081d5f53",
"params": {
"moduleId": "0b4319b0-75c7-11ea-b42f-4b64e50f43e5:temperatureModuleType"
}
}
],
"commandAnnotationSchemaId": "opentronsCommandAnnotationSchemaV1",
Expand Down
Loading
Loading