-
-
Notifications
You must be signed in to change notification settings - Fork 47.5k
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: Adds aria-* to TreeSelect #41978
fix: Adds aria-* to TreeSelect #41978
Conversation
Codecov ReportPatch and project coverage have no change.
Additional details and impacted files@@ Coverage Diff @@
## master #41978 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 616 616
Lines 10529 10531 +2
Branches 2882 2884 +2
=========================================
+ Hits 10529 10531 +2
... and 2 files with indirect coverage changes Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report in Codecov by Sentry. |
Could we support aria-*? |
I have done this ~~~ |
[中文版模板 / Chinese template]
🤔 This is a ...
🔗 Related issue link
💡 Background and solution
📝 Changelog
☑️ Self-Check before Merge
🚀 Summary
🤖 Generated by Copilot at 8b2bbea
This pull request enhances the accessibility of the
TreeSelect
component by adding thearia-label
prop, which allows users to specify custom labels for the tree nodes. It also adds a test case to verify the functionality of the new prop.🔍 Walkthrough
🤖 Generated by Copilot at 8b2bbea
aria-label
prop toTreeSelect
component to support accessibility labels (link, link, link, link)aria-label
prop inTreeSelectProps
interface incomponents/tree-select/index.tsx
(link)aria-label
prop toariaLabel
variable inInternalTreeSelect
function incomponents/tree-select/index.tsx
(link)ariaLabel
variable torc-tree-select
component inInternalTreeSelect
function incomponents/tree-select/index.tsx
(link)aria-label
prop incomponents/tree-select/__tests__/index.test.tsx
(link)