Opentrons / opentrons

Software for writing protocols and running them on the Opentrons Flex and Opentrons OT-2
https://opentrons.com
Apache License 2.0
405 stars 175 forks source link

bug: PD - touch tip minus button should go down to 0 instead of min #7508

Closed reachojhaneha closed 3 years ago

reachojhaneha commented 3 years ago

Bug

Tip position modal currently disables the - button when approaching the minimum. This is apparent in Touch Tip -> Tip Position (as opposed to Aspirate/Dispense Tip Position)

Desired behavior

IanLondon commented 3 years ago

I think this one is desired behavior, because in prod (old Tip Position modal) we didn't allow out-of-range values, which was annoying bc you couldn't type freely. Now we do allow values even when out of range. The pattern of allowing any number to be entered through typing is continue in allowing any height to be set with the arrow keys.

reachojhaneha commented 3 years ago

I just checked with Morgan about this and we concluded that we should preserve the "minus button" lowering the value to minimum value and not below minimum. Because the minus button does not lower the value to "0". For example if the minimum value for tip position is 12.5, minus button if clicked multiple times lowers the value till 12 and not 0.

IanLondon commented 3 years ago

Could we rename + relabel this as a feature and not a bug? Was not specified in the ticket #7143 when this was done

anthonywilleee commented 3 years ago

@howisthisnamenottakenyet - also curious your thoughts on this one..should we preserve not being able to go past min / max when hitting minus button