-
Notifications
You must be signed in to change notification settings - Fork 178
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
feat(components, app): update Chip component for unification #14708
Conversation
update Chip component for unification. the details are below close AUTH-214
add chip component for desktop close AUTH-214
app/src/organisms/Devices/ProtocolRun/ProtocolRunRunTimeParameters.tsx
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nice! i left a few comments
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm!
@ncdiehl11, could you take a look, when you are available? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks good. I can now use the unified Chip
from components
in my RTP PR.
* feat(components, app): update Chip component for unification
Overview
update Chip component for unification.
https://github.com/Opentrons/opentrons/assets/474225/0def2528-15b6-4425-971c-82303fcba5fe
a few tests are commented out because of jsdom's limitations (media-query). I will keep investigating this issue.
close AUTH-214
Test Plan
Changelog
Review requests
Risk assessment