Skip to content

Conversation

@audrasjb
Copy link
Contributor

<?php endif; // Languages. ?>
<tr class="form-field">
<th scope="row">
<label>

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we use for attribute in the label tag? It’s better for accessibility as it links the label to the corresponding input. Like,

<label for="site-privacy-public-on">

<fieldset>
<legend class="screen-reader-text"><?php esc_html_e( 'Site Privacy' ); ?></legend>
<label for="site-privacy-public-on">
<input name="blog[meta][blog_public]" type="radio" id="site-privacy-public-on" value="1"<?php checked( true ); ?> /> <?php echo esc_html_x( 'Yes', 'Multisite new site public privacy on' ); ?>

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we use dynamic check for default selection? Like:

checked( get_option( 'blog_public' ), 1 );

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants