-
Notifications
You must be signed in to change notification settings - Fork 10
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
front end module accessibility errors #203
Comments
hey @atn38 thanks for the report. Can you check @francoischalifour? |
Thanks @atn38—this is now fixed in the Autocomplete repo in this commit. |
Thanks Francois. I don't need to do anything to get the update, correct?
In that case the accessibility checker I use still throws up errors:
https://wave.webaim.org/report#/https://accessibility--eager-sammet-b7ed61.netlify.app/.
See the search box on upper right corner.
If I'm not mistaken it is asking for the <label> and <button> elements in
the module to have text content, plus an "aria-label" attribute.
…On Mon, Feb 22, 2021 at 6:03 AM François Chalifour ***@***.***> wrote:
Thanks @atn38 <https://github.com/atn38>—this is now fixed in the
Autocomplete repo in this commit
<algolia/autocomplete@45944e4>
.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#203 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AKAZD5UF6MTA3L6ZIE5DOBTTAJBXVANCNFSM4XQE3D5A>
.
|
Hullo, @francoischalifour, pinging again since I don't think this is closed yet. Please see my previous comment. |
@atn38 The fix was released in Autocomplete v1.0.0-alpha.44, and the Netlify plugin upgraded to this version. Are you still experiencing issues after upgrading? |
Hi folks, just FYI, when I run my website which has the Algolia front-end bundle through a web accessibility checker, it flags two things in the front end that I can't change but perhaps would be easy enough changes for you:
The text was updated successfully, but these errors were encountered: