Changeset 10106

Timestamp:

09/12/2015 06:12:03 AM (10 years ago)

Author:

mercime

Message:

Add 'for' attributes to labels in single Group admin screen.

Accessibility fix binds labels to respective checkboxesin group
admin settings: one to enable discussion forum and the other to delete
the group.

See #6618.

File:

• trunk/src/bp-templates/bp-legacy/buddypress/groups/single/admin.php (modified) (2 diffs)

trunk/src/bp-templates/bp-legacy/buddypress/groups/single/admin.php

@@ -80 +80 @@
 
             <div class="checkbox">
-                <label><input type="checkbox" name="group-show-forum" id="group-show-forum" value="1"<?php bp_group_show_forum_setting(); ?> /> <?php _e( 'Enable discussion forum', 'buddypress' ); ?></label>
+                <label for="group-show-forum"><input type="checkbox" name="group-show-forum" id="group-show-forum" value="1"<?php bp_group_show_forum_setting(); ?> /> <?php _e( 'Enable discussion forum', 'buddypress' ); ?></label>
             </div>
 
@@ -426 +426 @@
     </div>
 
-    <label><input type="checkbox" name="delete-group-understand" id="delete-group-understand" value="1" onclick="if(this.checked) { document.getElementById('delete-group-button').disabled = ''; } else { document.getElementById('delete-group-button').disabled = 'disabled'; }" /> <?php _e( 'I understand the consequences of deleting this group.', 'buddypress' ); ?></label>
+    <label for="delete-group-understand"><input type="checkbox" name="delete-group-understand" id="delete-group-understand" value="1" onclick="if(this.checked) { document.getElementById('delete-group-button').disabled = ''; } else { document.getElementById('delete-group-button').disabled = 'disabled'; }" /> <?php _e( 'I understand the consequences of deleting this group.', 'buddypress' ); ?></label>
 
     <?php