Skip to content

Commit bb2bf60

Browse files
authored
fix(aria-allowed-attr): allow aria-required on role=slider (#4035)
1 parent 25859dd commit bb2bf60

2 files changed

Lines changed: 4 additions & 0 deletions

File tree

lib/standards/aria-roles.js

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -631,11 +631,14 @@ const ariaRoles = {
631631
// Note: since the slider role has implicit
632632
// aria-orientation, aria-valuemax, aria-valuemin values it
633633
// is not required to be added by the user
634+
// Note: aria-required is not in the 1.1 or 1.2 specs but is
635+
// consistently supported in ATs
634636
allowedAttrs: [
635637
'aria-valuemax',
636638
'aria-valuemin',
637639
'aria-orientation',
638640
'aria-readonly',
641+
'aria-required',
639642
'aria-valuetext'
640643
],
641644
superclassRole: ['input', 'range'],

test/integration/rules/aria-allowed-attr/passes.html

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1509,6 +1509,7 @@
15091509
aria-owns="value"
15101510
aria-relevant="value"
15111511
aria-readonly="value"
1512+
aria-required="value"
15121513
>
15131514
ok
15141515
</div>

0 commit comments

Comments
 (0)