tl-its-umich-edu / canvas-app-explorer

A Web application that presents a list of Canvas external (LTI) tools with details. When integrated within Canvas, the user can search for specific LTI tool(s), and add or remove those tools from Canvas courses.
Apache License 2.0
4 stars 6 forks source link

Add `aria-hidden=false` and `role='button'` to focusable icon in `ToolCard` #260

Closed ssciolla closed 2 years ago

ssciolla commented 2 years ago

From the accessibility team:

Issue: The control that invokes the tooltip is an that admits focus but has an aria-hidden=“true” attribute. Normally and font icons that are decorative should have that attribute, but not when they are informational.

To Fix: Remove the aria-hidden attribute and might as well give it a role=“button” since that is what it is, really.