-
-
Notifications
You must be signed in to change notification settings - Fork 67
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
Accessibility issues #145
Comments
Hi, regarding the checkbox, are the |
Well done, I hadn't seen it.
|
Thanks for the pointers, @nicozerr and @edouardroger <3 Would anyone mind sending a PR for the same? |
I'll try to create a PR ASAP |
I'm sorry but using Docker, WSL 2 and all these stuffs is too complicated for me. I'm always stuck somewhere, I don't understand this mess. I'll try again when I'll have many hours/days to figure it out. |
Hello,
Some accessibility problems have been detected, according to international WCAG, french RGAA... Here are the accessibility issues to be fixed:
<iframe>
tag, replaceframeborder
,scrolling
,width
andheight
HTML attributes by their equivalents in CSS (border
,overflow
,width
andheight
).It will allow some users to customize the appareance by overriding CSS.
<iframe>
tag, replacearia-roledescription=presentation
(that provides no interesting information) byrole=group title="mCaptcha: Mandatory checkbox for security check"
(or something like that).<main>
tag by a<div>
tag.No need to define a main content area while this module is always implemented via an iframe in a specific area of a website)
The wrapping link will have the accessible name "mcaptcha" instead of "logo mcaptcha mCaptcha".
The text was updated successfully, but these errors were encountered: