You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Thanks for the report - what would fix it? I notice that the bootstrap docs now have role="button" which isn't in the markup produced by the plugin.. (also screen reader only text)
I think that besides the role="button", the links need a text content. As you said, a screen reader only text should be sufficient.
Furthermore, aria-hidden="true" must be placed in the glyphicon span, such as the Bootstrap docs shows.
Thanks @ricardomoro - I will add these when I next do some work on the plugin (could be while, I haven't actually used it myself for years now). Feel free to open a PR if you'd like to see it added sooner.
This error appears in the accessibility audit tool a11y:
I'm using Twitter Bootstrap Version 3 output.
The text was updated successfully, but these errors were encountered: