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

v4: Disabled link cleanup (backport) #34924

Merged
merged 1 commit into from
Sep 9, 2021

Commits on Sep 8, 2021

  1. v4: Disabled link cleanup

    manual backport of #34886
    
    per https://www.w3.org/TR/html-aria/#docconformance
    
    > It is NOT RECOMMENDED to use `aria-disabled="true"` on an `a` element with an `href` attribute.
    >
    >NOTE
    >If a link needs to be "disabled", remove the `href` attribute.
    
    This PR removes the unnecessary `href="#"`, `tabindex="-1"`, and `aria-disabled="true"` from disabled links in both docs pages and examples. `aria-disabled="true"` *is* kept for disabled link-based buttons (that have `role="button"`) as there it's appropriate to use (you *want* to convey to assistive technologies that this thing you're claiming is a button is also disabled at the moment)
    
    Further, the PR extends the "Link functionality caveat" to show the "proper" way (removing `href` and adding `.disabled` class only) to disable a link, but then explains what to do if that's not possible (and then keeps an example with all the traditional `href="#" tabindex="-1" aria-disabled="true"`, but explains clearly that it's not ideal). Same sort of explanation is also added to the pointer event utilities page
    patrickhlauke committed Sep 8, 2021
    Configuration menu
    Copy the full SHA
    8e91d34 View commit details
    Browse the repository at this point in the history