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

fix(protocol-designer): cleanup and add clear functionality to LiquidToolbox #16843

Open
wants to merge 3 commits into
base: edge
Choose a base branch
from

Conversation

ncdiehl11
Copy link
Collaborator

@ncdiehl11 ncdiehl11 commented Nov 14, 2024

Overview

This PR updates style according to latest designs and implements missing functionality in LiquidToolbox. Here, I add the ability to clear a specific liquid from its loaded wells from its liquid card, and update some other minor style issues. I also add an InfoScreen if there are no loaded liquids on the selected labware and no wells are selected.

Lastly, I add prevention of highlighting labware column and row labels when clicking and dragging

Closes RQA-3341

Test Plan and Hands on Testing

Screen.Recording.2024-11-14.at.5.47.43.PM.mov
  • on protocol starting deck tab, select a labware without liquid, and select 'add liquid'
  • verify that no liquids infoscreen shows in toolbox
  • select some wells and verify that the add liquid box appears
  • add some liquid to the selected wells
  • select 'delete' from the new liquid card and verify that liquids are correctly removed from its wells

Click and drag fix

before fix:

Screen.Recording.2024-11-14.at.5.58.53.PM.mov

after fix:

Screen.Recording.2024-11-14.at.5.56.03.PM.mov

Changelog

  • refine InfoScreen component and add to LiquidToolbox when liquids are empty and no wells are selected
  • use Tag component in dropdown
  • add delete liquid functionality to liquid card

Review requests

see test plan

Risk assessmen

low

… toolbox

This PR updates style according to latest designs and implements missing functionality in
LiquidToolbox. Here, I add the ability to clear a specific liquid from its loaded wells from its
liquid card, and update some other minor style issues. I also add an InfoScreen if there are no
loaded liquids on the selected labware and no wells are selected.

Closes RQA-3341
@ncdiehl11 ncdiehl11 marked this pull request as ready for review November 14, 2024 22:48
@ncdiehl11 ncdiehl11 requested review from a team as code owners November 14, 2024 22:48
@ncdiehl11 ncdiehl11 removed request for a team November 14, 2024 22:49
@ncdiehl11 ncdiehl11 self-assigned this Nov 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant