Skip to content

Commit 70a3a1c

Browse files
committed
Issue #3029614 by cdykstra, cilefen, andrewmacpherson, jrockowitz: Fix aria-required in states.js
(cherry picked from commit 1e0af07)
1 parent 0e8a2c2 commit 70a3a1c

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

misc/states.es6.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -695,7 +695,7 @@
695695
if (e.value) {
696696
const label = `label${e.target.id ? `[for=${e.target.id}]` : ''}`;
697697
const $label = $(e.target)
698-
.attr({ required: 'required', 'aria-required': 'aria-required' })
698+
.attr({ required: 'required', 'aria-required': 'true' })
699699
.closest('.js-form-item, .js-form-wrapper')
700700
.find(label);
701701
// Avoids duplicate required markers on initialization.

misc/states.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -352,7 +352,7 @@
352352
if (e.trigger) {
353353
if (e.value) {
354354
var label = 'label' + (e.target.id ? '[for=' + e.target.id + ']' : '');
355-
var $label = $(e.target).attr({ required: 'required', 'aria-required': 'aria-required' }).closest('.js-form-item, .js-form-wrapper').find(label);
355+
var $label = $(e.target).attr({ required: 'required', 'aria-required': 'true' }).closest('.js-form-item, .js-form-wrapper').find(label);
356356

357357
if (!$label.hasClass('js-form-required').length) {
358358
$label.addClass('js-form-required form-required');

0 commit comments

Comments
 (0)