-
Notifications
You must be signed in to change notification settings - Fork 158
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
fix(language-selector): added clear label attribute #4914
fix(language-selector): added clear label attribute #4914
Conversation
Deploy preview created for package Built with commit: f8e0fa1a30e8fd7759649a762d7ca718c461dcfc |
Deploy preview created for package Built with commit: f8e0fa1a30e8fd7759649a762d7ca718c461dcfc |
Deploy preview created for package Built with commit: f8e0fa1a30e8fd7759649a762d7ca718c461dcfc |
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.
Hi @IgnacioBecerra thank you for trying to fix this - Would you want to try fixing this directly in carbon-web-components
unless this needs to be included in v1.15.0
?
@asudoh Yeah, I think that's a good idea. I don't think we need to get this into the release right away, and besides this is a change that should be coming from ComboBox itself. |
There was a button label attribute for ComboBox, so that addition can be reflected on our side. |
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 👍 - Thanks @IgnacioBecerra for the update!
…stem#4914) ### Related Ticket(s) carbon-design-system#4889 ### Description The component now has the role of `edit combo` to ensure the screen readers convey its purpose to the users. The clear button also lacked a label to convey functionality upon click/enter. ### Changelog **New** - added clear button label property <!-- React and Web Component deploy previews are enabled by default. --> <!-- To enable additional available deploy previews, apply the following --> <!-- labels for the corresponding package: --> <!-- *** "package: services": Services --> <!-- *** "package: utilities": Utilities --> <!-- *** "package: styles": Carbon Expressive --> <!-- *** "RTL": React / Web Components (RTL) --> <!-- *** "feature flag": React / Web Components (experimental) -->
Related Ticket(s)
#4889
Description
The component now has the role of
edit combo
to ensure the screen readers convey its purpose to the users. The clear button also lacked a label to convey functionality upon click/enter.Changelog
New