Closed (fixed)
Project:
Webform
Version:
8.x-5.x-dev
Component:
Accessibility
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
10 May 2021 at 21:52 UTC
Updated:
31 May 2021 at 16:39 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
jrockowitz commentedPlease review the attached patch. This issue should be addressed in 8.x-5.x.
What about checkboxes? Should they have an aria-required attribute?
Comment #3
paulocsNo. Checkbox does not support
aria-required.See the
Required States and Properties:Comment #4
paulocsWords from Wilco Fiers:
Comment #5
jrockowitz commented@paulocs Thank you for doing that research.
Comment #6
paulocsI confirm that patch #5 removes the attribute
aria-requiredfor inputs typeradio, but not for inputs typecheckbox.Comment #7
jrockowitz commentedThe checkbox's aria-require attribute is coming from Drupal core.
@see \Drupal\Core\Render\Element\RenderElement::setAttributes
Still, for conditionally a required checkbox the patch is applicable.
Comment #8
paulocsMakes sense! Also if the webform contains a
checkboxeselement, the aria-required is removed from the checkbox input.Moving to RTBC.
Comment #10
jrockowitz commentedComment #12
carlygerardThanks @jrockowitz and @paulocs!