-
Notifications
You must be signed in to change notification settings - Fork 40
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
Select/dropdown - improve error handling and focus state #390
Conversation
Leaving this in draft for the moment, currently having trouble validating whether the |
Folding some additional tweaks into this PR:
(Took a couple of wrong turns and broke everything along the way, please disregard the commit/revert spam in the middle!) |
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.
Most excellent @mkernohanbc 🚀 . There are a couple tweaks needed in CSS, but the new error slot stuff looks awesome.
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.
Great stuff ✅
This PR contains minor tweaks to the Select component, to address the issues documented in #388 (based on work done for #389):
<Button>
element when theisInvalid
prop is passedFieldError
andValidationResult
. Creates an error message slot underneath the Select<Button>