Drupal
/
drupal
mirror of https://git.drupalcode.org/project/drupal.git
1
0
Fork 0
Code Issues Projects Releases Wiki Activity

Issue #3029614 by cdykstra, cilefen, andrewmacpherson, jrockowitz: Fix aria-required in states.js

8.7.x
Lauri Eskola 2019-02-12 12:42:10 +02:00
parent 76d6e0d62c
commit 1e0af070cb
No known key found for this signature in database
GPG Key ID: 37E6EF00B7EEF188
2 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
core/misc/states.es6.js
View File

@ -695,7 +695,7 @@
if (e.value) {
const label = `label${e.target.id ? `[for=${e.target.id}]` : ''}`;
const $label = $(e.target)
.attr({ required: 'required', 'aria-required': 'aria-required' })
.attr({ required: 'required', 'aria-required': 'true' })
.closest('.js-form-item, .js-form-wrapper')
.find(label);
// Avoids duplicate required markers on initialization.

2
core/misc/states.js
View File

@ -352,7 +352,7 @@
if (e.trigger) {
if (e.value) {
var label = 'label' + (e.target.id ? '[for=' + e.target.id + ']' : '');
var $label = $(e.target).attr({ required: 'required', 'aria-required': 'aria-required' }).closest('.js-form-item, .js-form-wrapper').find(label);
var $label = $(e.target).attr({ required: 'required', 'aria-required': 'true' }).closest('.js-form-item, .js-form-wrapper').find(label);
if (!$label.hasClass('js-form-required').length) {
$label.addClass('js-form-required form-required');