-
Notifications
You must be signed in to change notification settings - Fork 0
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 issue #1
Comments
For some reason, the code I pasted above did not stick. The label that is needed is to show that the form should not pull from autocomplete - so it is needing an aria label for the input box. |
@luukverhoeven could you please put this on your calendar to check how much time is needed to fix it. |
@vincentcornelis Can you check this? Thanks |
What Moodle version are you receiving this accessibility issue for? Correct me if i'm wrong, but this issue would be regarding the following element?: <input ... name="efquicklistsearch">...</input> Default display of this element has the text-input as white on white with a dark-gray border, which when focussed gives the additional blue outline, which should provide a contrast of 3.02 for the gray, (which, i realise, would still not reach the aforementioned 3.1 contrast. However, we don't apply additional styling, this is the case for all default Moodle form elements), and a contrast of 3.36 for the blue. Moodle 4.2:Moodle 3.11:I am however unable to discover a place where the contrast is the 1.49:1 you mentioned. Like to hear back from you! |
@gemguardian @emmarichardson Do you have an update on this? |
@luukverhoeven as it seems we cannot replicate this issue, or received other remarks on it. I tend to close this issue as 'won't fix'. @emmarichardson if you have time to reply we dan take a nother look at it. |
Sorry for lack of response. I ended up using css to get around the contrast issue. Using Foundation as theme. I still receive the complaint about the lack of label for aria but have yet to determine if that is an actual issue because it is expecting a label for autofill which is not appropriate. |
@emmarichardson thank you for your response, since we use the stying of Moodle Boost form, and don't 'override' the styling, this sounds as something you could make a Moodle tracker of so that moodle pty ltd is aware of this accessibility issue. as for the lack of label aria, I checked the wcag 1.2 guidelines and I do think we can assist in this - but we only deploy it for the latest master/main branche. @luukverhoeven I would suggest the Aria Seach because it is a search form https://developer.mozilla.org/en-US/docs/Web/Accessibility/ARIA/Roles/searchbox_role or - following the fact we use form - we could use the aria form : https://developer.mozilla.org/en-US/docs/Web/Accessibility/ARIA/Roles/form_role Could you please let me know what you think and the time invested to fix for the main /master branche? |
@gemguardian We estimate that it will take 1 or 2 hours to develop and test this. |
@luukverhoeven This is approved but for the latest main branche version, we don't backport this issue. |
@gemguardian Can you check if the following branch (https://github.com/Lesterhuis-Training-en-Consultancy/moodle-block_quickcourselist/tree/86byan3tr-accessibility-fix) |
@luukverhoeven I can confirm that we see a aria-label "Search for courses" in M4.4 |
@luukverhoeven could yuo please this to the master and release it on Moodle.org - Could you please check Moodle.org branches as well, I doubt that this is the same line as we use Github.
|
@gemguardian Merged and published on Moodle.org |
I am receiving an error on an accessibility scan that the form is missing a label - the error is pointing to this in the html.
The text was updated successfully, but these errors were encountered: