We should use the aria-disabled attribute correctly on the breadcrumbs menu. This aids in accessibility, as it signifies to screen readers whether or not a link should be clickable
Created attachment 189033 [details] [review] Bug 41177: Add aria-disabled to breadcrumbs This patch adds aria-disabled and tabindex = -1 to a breadcrumb, if it is the current page. This aids in accesssibility, and is generally good practice. == TO TEST: == a) Visit any page on the OPAC, and inspect the breadcrumb pertinent to the current page *) Notice the lack of aria-disabled and tabindex on the inspected element == APPLY PATCH == b) repeat step a *) Notice how there is now an aria-disabled and tabindex = -1 *) Notice you cannot tab on to the final breadcrumb anymore c) Ensure other breadcrumbs are unaffected and still work as links == SIGN OFF ==