Changeset 9605

Timestamp:

03/07/2015 11:25:35 AM (10 years ago)

Author:

djpaul

Message:

Administration: fix valueless <label> element for the Settings > BuddyPress > Components screen.

Originally caused by a simple typo, but we can enhance this further by making the component name the label for the corresponding check box.
Doing so lets us click/touch the component name to toggle its check box.

Fixes #6284

File:

• trunk/src/bp-core/admin/bp-core-admin-components.php (modified) (1 diff)

trunk/src/bp-core/admin/bp-core-admin-components.php

@@ -195 +195 @@
                             <?php endif; ?>
 
-                            <label class="screen-reader-text" for="bp_components[<?php echo esc_attr( $name ); ?>]"><?php sprintf( __( 'Select %s', 'buddypress' ), esc_html( $labels['title'] ) );  ?></label>
                         </th>
                         <td class="plugin-title" style="width: 190px;">
                             <span></span>
-                            <strong><?php echo esc_html( $labels['title'] ); ?></strong>
+                            <label for="bp_components[<?php echo esc_attr( $name ); ?>]">
+                                <strong><?php echo esc_html( $labels['title'] ); ?></strong>
+                            </label>
+
                             <div class="row-actions-visible">