angular-ui / bootstrap

PLEASE READ THE PROJECT STATUS BELOW. Native AngularJS (Angular) directives for Bootstrap. Smaller footprint (20kB gzipped), no 3rd party JS dependencies (jQuery, bootstrap JS) required. Please read the README.md file before submitting an issue!
http://angular-ui.github.io/bootstrap/
MIT License
14.3k stars 6.74k forks source link

aria-expanded on typeahead input field breaking validation #6543

Open pauldcollins opened 7 years ago

pauldcollins commented 7 years ago

Bug description:

Aria expanded false cannot be added to form inputs. It's reserved for certain roles, such as a button or a parent UL on a list. Please see the following links that list the roles it can be used on: https://www.w3.org/WAI/GL/wiki/Using_the_WAI-ARIA_aria-expanded_state_to_mark_expandable_and_collapsible_regions#Description https://www.w3.org/TR/wai-aria/states_and_properties#aria-expanded

Currently the typeahead is breaking accessibility validation when used. The solution is to remove the attribute from the input field, as it isn't required.

Link to minimally-working plunker that reproduces the issue:

https://plnkr.co/edit/?p=preview

Steps to reproduce the issue:

OPTIONAL: To check with validation tools

Version of Angular, UIBS, and Bootstrap

Angular: 1.5.7

UIBS: 2.5.0

Bootstrap: 3.3.7

moogleberry commented 7 years ago

aXe-core reports the same violation for the same reason. That attribute should not be there.

fanlearning commented 6 years ago

any progress on this issue?

fanlearning commented 6 years ago

Found this link may be useful: https://www.levelaccess.com/differences-aria-1-0-1-1-changes-rolecombobox/