-
Notifications
You must be signed in to change notification settings - Fork 13.8k
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: Adds a helper text option to the Select component #21269
feat: Adds a helper text option to the Select component #21269
Conversation
Codecov Report
@@ Coverage Diff @@
## master #21269 +/- ##
==========================================
+ Coverage 66.43% 66.45% +0.01%
==========================================
Files 1784 1784
Lines 68185 68243 +58
Branches 7265 7265
==========================================
+ Hits 45298 45348 +50
- Misses 21018 21026 +8
Partials 1869 1869
Flags with carried forward coverage won't be shown. Click here to find out more.
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
@@ -162,6 +170,16 @@ const StyledLoadingText = styled.div` | |||
`} | |||
`; | |||
|
|||
const StyledHelperText = styled.div` |
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 identical to the AsyncSelect one. Should we put in a common file and import it from there?
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.
Yes. I left it duplicated on purpose in this PR because @cccs-RyanK is already working on the common file in #21094
Co-authored-by: Geido <60598000+geido@users.noreply.github.com>
Co-authored-by: Geido <60598000+geido@users.noreply.github.com>
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!
SUMMARY
Adds a helper text option to the Select component to aid users when dealing with more complex flows.
@kasiazjc @codyml @cccs-RyanK
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
Screen.Recording.2022-08-31.at.10.53.55.AM.mov
TESTING INSTRUCTIONS
Check the video for instructions.
ADDITIONAL INFORMATION