Conversation
|
The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the Core Committers: Use this line as a base for the props when committing in SVN: To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook. |
Test using WordPress PlaygroundThe changes in this pull request can previewed and tested using a WordPress Playground instance. WordPress Playground is an experimental project that creates a full WordPress instance entirely within the browser. Some things to be aware of
For more details about these limitations and more, check out the Limitations page in the WordPress Playground documentation. |
| <td><fieldset> | ||
| <legend class="screen-reader-text"><span><?php echo $blog_privacy_selector_title; ?></span></legend> | ||
| <?php if ( has_action( 'blog_privacy_selector' ) ) : ?> | ||
| <input id="blog-public" type="radio" name="blog_public" value="1" <?php checked( '1', get_option( 'blog_public' ) ); ?> /> |
There was a problem hiding this comment.
Could we also add aria-describedby="public-desc" to the radio buttons here and id="public-desc" to the <p class="description"> here as well? This will ensure the setting is fully accessible even when blog_privacy_selector is active, matching wp-admin/install.php
There was a problem hiding this comment.
File wp-admin/install.php contains aria-describedby attribute within input id="blog-norobots". Should I add aria-describedby to input.blog-public, input.blog-norobots or probably both?
There was a problem hiding this comment.
Should I add
aria-describedbytoinput.blog-public,input.blog-norobotsor probably both?
Sorry, I missed this reply.
| ?> | ||
| <?php else : ?> | ||
| <label for="blog_public"><input name="blog_public" type="checkbox" id="blog_public" value="0" <?php checked( '0', get_option( 'blog_public' ) ); ?> /> | ||
| <label for="blog_public"><input name="blog_public" type="checkbox" id="blog_public" value="0" aria-describedby="privacy-desc" <?php checked( '0', get_option( 'blog_public' ) ); ?> /> |
There was a problem hiding this comment.
For attribute consistency with wp-admin/install.php#L201, let's move aria-describedby="privacy-desc" to right after id="blog_public
|
Thanks for updating the attribute ordering, @sawfly! That looks much cleaner now. Just a reminder regarding the remaining part of the review: the radio button section inside To make the accessibility fix complete across all view states, could you also update that branch? In wp-admin/install.php, core handles this exact radio button setup similarly, you can refer from there for consistency check. |
Dear @himanshupathak95 . Thanks for your involving in this PR. I've done as you asked to match install.php. In this comment you had asked to add |
himanshupathak95
left a comment
There was a problem hiding this comment.
To clarify my earlier comment: yes, only #blog-norobots needs aria-describedby, not #blog-public. The description text is a caveat specifically about how the discouraging option works, so linking it only to that input is semantically accurate.
Both branches now look correct. I think the fix is now complete and consistent across both view states.
b177029 to
a25196c
Compare
|
I rebased the PR and updated it to conditionally set aria-describedby in the fieldset. |
a25196c to
a0113ff
Compare
| <td><input name="weblog_title" type="text" id="weblog_title" size="25" value="<?php echo esc_attr( $weblog_title ); ?>" /></td> | ||
| </tr> | ||
| <tr> | ||
| <th scope="row"><label for="user_login"><?php _e( 'Username' ); ?></label></th> |
There was a problem hiding this comment.
This label was not associated with any input when $user_table is truthy, as the input is rendered conditionally.
…ility. Conditionally associates the description of the Search engine visibility setting on the Reading Settings and Install pages: - When the setting shows two radio buttons, the description is associated to the wrapping fieldset element. - Otherwise, it is associated to the checkbox. Additionally: - Fixes an orphaned label element in the Install page. - Replaces implicit labeling (label elements that wrap inputs) with explicit labeling. Developed in #12794 Props sawf1y, abcd95, afercia, joedolson. Fixes #65766. git-svn-id: https://develop.svn.wordpress.org/trunk@63762 602fd350-edb4-49c9-b593-d223f7449a82
…ility. Conditionally associates the description of the Search engine visibility setting on the Reading Settings and Install pages: - When the setting shows two radio buttons, the description is associated to the wrapping fieldset element. - Otherwise, it is associated to the checkbox. Additionally: - Fixes an orphaned label element in the Install page. - Replaces implicit labeling (label elements that wrap inputs) with explicit labeling. Developed in WordPress/wordpress-develop#12794 Props sawf1y, abcd95, afercia, joedolson. Fixes #65766. Built from https://develop.svn.wordpress.org/trunk@63762 git-svn-id: http://core.svn.wordpress.org/trunk@62934 1a063a9b-81f0-0310-95a4-ce76da25c4cd
|
Committed in https://core.trac.wordpress.org/changeset/63762 |
Added attribute aria-describedby
Trac ticket: https://core.trac.wordpress.org/ticket/65766