Adding aria-describedby="elementId" to tooltip plugin for better accessibility
#13480
Comments
|
@cvrebert I made an attempt at authoring a few tests and changes to the tooltip plugin for bootstrap. I wanted to see if I could open a pull request for this issue, I am trying to follow the CONTRIBUTING guidelines, but still my first PR here. My branch is on topic branch of the fork. Would it help to open a PR now? |
|
@bassettsj Go for it. |
bassettsj
added a commit
to bassettsj/bootstrap
that referenced
this issue
May 9, 2014
resolves issue twbs#13480 - set up test - linted the code styles - passed the tests - integrated feedback from @cvrebert
bassettsj
added a commit
to bassettsj/bootstrap
that referenced
this issue
May 16, 2014
Generates a unique id for tooltip and adds [aria-describedby] to the element it is called on. Resolves issue twbs#13480 - set up test - linted the code styles - passed the tests - integrated feedback from @cvrebert
|
Fixed by #13557. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Adding a bit of info to the item with
aria-describedbyto the tooltip/popover would help signal to Assistive Technology, like screen readers, that element getting targeted by the tooltip/popover that it is better described by this other DOM thing. This is illustrated on paypal/bootstrap-accessibility-plugin source line 32Changes that would need to happen:
aria-describedby="elementId"on the element on showWe already have
role="tooltip"👍The text was updated successfully, but these errors were encountered: