-
-
Notifications
You must be signed in to change notification settings - Fork 850
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 component: add a suffix to .form-control-input #2063
Select component: add a suffix to .form-control-input #2063
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
I believe @lindsaym-fa's comment from the previous PR applies here.
|
Hi, @lindsaym-fa We have a suffix in the dropdown component which we use as a button for the user to change their dropdown preferences: We are looking to have the same suffix feature in the select component |
@claviska, @lindsaym-fa The input component has prefix and suffix: However the select component only has a prefix: We required the select component to also have a suffix part. From our perspective, this is more consistent. |
Thank you, @newdigate. 🙏 We appreciate the clean PR, but we're hesitant to bring this feature into the library. Importantly, |
hi @lindsaym-fa here is a non-interactive use-case: we have a select component and we want to indicate the selected user is out-of-office: |
It's a compelling use case, for sure. Like you said, as long as the docs are updated to reflect it. |
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.
Thanks for the great use case! We'll get the docs updated and get this one in for the next release.
We added a suffix (part, slot) to the select component
Having the
suffix
slot/part allows us to make the suffix in theselect
component consistent with the suffix in oursl-input
components.I think this feature will be useful for other projects too.
Many thank you for such an amazing, awe-inspiring component library! I hope this change is helpful for you :)