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

Search field needs accessibility tweaks in toolkit.js #488

Open
Detzner opened this issue Jul 19, 2023 · 1 comment
Open

Search field needs accessibility tweaks in toolkit.js #488

Detzner opened this issue Jul 19, 2023 · 1 comment
Assignees

Comments

@Detzner
Copy link

Detzner commented Jul 19, 2023

AInspector is calling these errors in the search form:

• Add label to INPUT[SEARCH]: NO LABEL control.
 
• Add text content to the BUTTON element.
 
• IMAGE is not a defined ARIA role, change the ROLE attribute value to an appropriate widget, landmark, section or live region role.

To make the search field accessible, we’d need to change toolkit.js, v 2.15, in beautified version:

line 5778 change
\nlabel {\n display: none;\n} to
\n position:absolute; left: -9999px;\n}
(this adds the label)

in line 5966 after
• delete role=“image” from the svg tag (this fixes ' IMAGE is not a defined ARIA role,')
• add Submit search (this adds text to the magnifying glass button)

@Detzner Detzner self-assigned this Jul 19, 2023
@github-actions
Copy link

@trubach trubach transferred this issue from web-illinois/illinois_framework_theme Jul 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant