Opentrons / opentrons

Software for writing protocols and running them on the Opentrons Flex and Opentrons OT-2

https://opentrons.com

Apache License 2.0

429 stars 179 forks source link

fix(protocol-designer): only show tooltip if confirm button disabled #16975

Closed ncdiehl11 closed 4 days ago

ncdiehl11 commented 4 days ago

Overview

On the WizardBody component used for creating a new protocol file, we optionally pass a tooltip to be shown when hovering the confirm button if it is disabled. This PR fixes a bug where the tooltip showed on hover even if the button is enabled.

Test Plan and Hands on Testing

create a new protocol file and land on select pipettes step
verify that before you add a pipette, the confirm button is disabled, and hovering produces the correct tooltip
add the pipette to enable the confirm button. verify that hovering the enabled button does not produce tooltip

Changelog

add check for disabled to show tooltip

Review requests

see test plan

Risk assessment

low