Changeset 46090 for trunk/src/wp-includes/comment-template.php

Timestamp:

09/11/2019 01:50:23 AM (6 years ago)

Author:

SergeyBiryukov

Message:

Accessibility: Comments: In comment_form(), remove aria-describedby attribute from the email field if there's no associated description in the comment_notes_before argument.

The attribute is meant to communicate to user agents and assistive technologies that the element has a description. If the referenced description is missing, it will be flagged as failure by any automated accessibility checker tool and, most importantly, is potentially confusing for assistive technologies users.

Props afercia, joedolson, dinhtungdu, donmhico, SergeyBiryukov.
Fixes #47975.

File:

• trunk/src/wp-includes/comment-template.php (modified) (2 diffs)

trunk/src/wp-includes/comment-template.php

@@ -2429 +2429 @@
     $args = array_merge( $defaults, $args );
 
+    // Remove aria-describedby from the email field if there's no associated description.
+    if ( false === strpos( $args['comment_notes_before'], 'id="email-notes"' ) ) {
+        $args['fields']['email'] = str_replace(
+            ' aria-describedby="email-notes"',
+            '',
+            $args['fields']['email']
+        );
+    }
+
     /**
      * Fires before the comment form.
@@ -2510 +2519 @@
             endif;
 
-            // Prepare an array of all fields, including the textarea
+            // Prepare an array of all fields, including the textarea.
             $comment_fields = array( 'comment' => $args['comment_field'] ) + (array) $args['fields'];