Skip to content
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(ui5-combobox): support arial-labelledby and fix aria-expanded #1916

Merged
merged 1 commit into from
Jul 8, 2020

Conversation

vladitasev
Copy link
Contributor

@vladitasev vladitasev commented Jul 7, 2020

Bug:

  • aria-expanded is controlled by a non-existent property (open) which has been created

New implementation:

  • add support for aria-label and aria-labelledby

closes: #1912

@vladitasev vladitasev changed the title fix(ui5-combobox): support aria-label/arial-labelledby and fix aria-e… fix(ui5-combobox): support arial-labelledby and fix aria-expanded Jul 7, 2020
@vladitasev vladitasev merged commit 99a0e81 into master Jul 8, 2020
@vladitasev vladitasev deleted the combobox-acc-fix branch July 8, 2020 07:30
ilhan007 pushed a commit that referenced this pull request Jul 30, 2021
Part of #3107 

BREAKING CHANGE: The support for ```aria-labelledby``` have been deprecated in favour of  new attribute ```accessible-name-ref``` for the following components:
- ComboBox (Requested by SF #1916)
- DatePicker (Requested by SF #2107)
- Input (Requested #1866)
- List (Requested by SF #1886)
- Rating Indicator
- Select (Requested by SF #2107)
- StepInput (Implemented as part of the initial implementation #2804)
- TextArea (Requested by SF #2107)
- WizardStep (Implemented as part of the initial implementation #2400)
The ```aria-labelledby``` has been deprecated for the following component:
- Button (Requested by SF #1425)
The ```accessible-name``` has been deprecated for the following components:
- Link (Requested by SF #2356)
 Both ```aria-labelledby``` and ```accessible-name``` have been deprecated for the following components:
- Card (Requested by CBC #2127)
- CheckBox (Requested by SF #2265)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Accessibility issues with Combo Box
2 participants