fix(protocol-designer, components): 7.0 candidate a bug fixes

[jerader]

closes RQA-1080, RQA-1125, RQA-1102, RQA-1109, RAUT-589, RAUT-590, RQA-1110, RQA-1122, RQA-1100, RQA-1141, RQA-1148, RQA-1151

Overview

Addresses a handful of bugs

Test Plan

sandbox: https://sandbox.designer.opentrons.com/pd_bug-fixes-7.0/

On the sandbox build, test that each bug got addressed;

1. add a magnetic module for OT-2 protocol, adding labware on top of it, make sure the correct labware is available and make sure the help link goes to the correct place (RAUT-589, RAUT-590)
2. when editing the pipettes, make sure the arrow icon moves up and down when the dropdown is expanded or not (RQA-1080)
3. make sure you can duplicate labware successfully on a flex and Ot-2 protocol (RQA-1125)
4. add a labware to the deck and give it a verrry long nickname, make sure it truncates after ~3 lines (RQA-1102)
5. add a liquid with a verrry long nickname, make sure it truncates properly both on the liquid tab and when you add the liquid to the labware (RQA-1109, RQA-1110)
6. protocol name should truncate on design tab, but the subtitle to the right of it should not (RQA-1122)
7. removal of the beta tag. See Linda's comment in RQA-1100
8. touch_tip option should be disabled in advanced settings in the transfer step when you try to touch tip with a labware that doesn't support it. Hovering over it should show a tooltip explaining why its disabled (RQA-1148)
9. the moveLabware step in the timeline should have the correct information shown (should display labware, new location, and if gripper is used or manual intervention is required) RQA-1151
10. when you export a protocol with an unused pipette, the modal copy should be updated (RQA-1141)

Changelog

• various components in protocol-designer directory - adding truncated strings, labware to recommended labware list, fixing a url, duplicate labware bug

Review requests

see test plan

Risk assessment

low

[codecov]

Codecov Report

Merging #13179 (f4b6ab4) into edge (df3d742) will decrease coverage by 0.34%.
The diff coverage is 5.55%.

@@            Coverage Diff             @@
##             edge   #13179      +/-   ##
==========================================
- Coverage   72.56%   72.23%   -0.34%     
==========================================
  Files        2393     2357      -36     
  Lines       66055    64000    -2055     
  Branches     7348     7370      +22     
==========================================
- Hits        47935    46229    -1706     
+ Misses      16360    16006     -354     
- Partials     1760     1765       +5     

Flag	Coverage Δ
app	71.36% <ø> (ø)
components	63.08% <ø> (+0.03%)	⬆️
labware-library	49.17% <ø> (ø)
protocol-designer	47.13% <5.55%> (-0.24%)	⬇️
react-api-client	65.13% <ø> (ø)
step-generation	87.18% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Changed	Coverage Δ
components/src/forms/Select.tsx	78.57% <ø> (+5.23%)	⬆️
...c/components/BatchEditForm/BatchEditMoveLiquid.tsx	23.52% <0.00%> (-8.48%)	⬇️
...mponents/DeckSetup/LabwareOverlays/LabwareName.tsx	33.33% <0.00%> (-4.17%)	⬇️
...-designer/src/components/IngredientsList/index.tsx	0.00% <0.00%> (ø)
...esigner/src/components/KnowledgeBaseLink/index.tsx	100.00% <ø> (ø)
...ts/LabwareSelectionModal/LabwareSelectionModal.tsx	40.65% <ø> (+0.44%)	⬆️
...l-designer/src/components/LiquidsSidebar/index.tsx	0.00% <0.00%> (ø)
...omponents/StepEditForm/fields/CheckboxRowField.tsx	77.77% <0.00%> (-9.73%)	⬇️
...onents/StepEditForm/forms/MoveLiquidForm/index.tsx	0.00% <0.00%> (ø)
...ocol-designer/src/components/StepEditForm/utils.ts	34.92% <0.00%> (-1.75%)	⬇️
... and 9 more

... and 42 files with indirect coverage changes

[jerader]

this doesn't necessarily fix anything here but it is needed for multi tiprack support. This change also makes duplicate labware ids match the correct shape of a uuid + defURI

[koji]

maybe using Template literals instead of +?

Suggested change

	duplicateLabwareId: uuid() + ':' + templateLabwareDefURI,
	duplicateLabwareId: `${uuid()}:${templateLabwareDefURI}`,

[koji]

In sidebar, description/notes would be needed to truncate the text

[koji]

STARTING DECK STATE is supposed to display like☝️ when a protocol name is long?

[jerader]

STARTING DECK STATE is supposed to display like☝️ when a protocol name is long?

@koji, good catch! i think this is okay as is. the bug was that the Starting deck state text would get truncated. The protocol name is truncated using css's --truncate so i don't think there's much room for customization.

[koji]

the changes look good to me