Skip to:
Content

BuddyPress.org

Opened 12 years ago

Closed 12 years ago

#5437 closed enhancement (fixed)

Add labels to BP Group Settings radio buttons

Reported by: tw2113 Owned by: boonebgorges
Priority: normal Milestone: 2.0
Component: Groups Version:
Severity: normal Keywords:
Cc:

Description

Instead of making users click the radio buttons directly, allow clicking the associated text as well. Also helps with accessibility

Attachments (2)

group-settings-label-patch.patch​ (3.1 KB ) - added by tw2113 12 years ago.
group-settings-label-patch-1.patch​ (3.4 KB ) - added by tw2113 12 years ago.
Adds fieldset and legeng to already supplied label fix.

Download all attachments as: .zip

Change History (7)

@tw2113
12 years ago

#1 @boonebgorges
12 years ago

  • Milestone Awaiting Review → 2.0

Awesome, this looks like a good fix.

Your patch also demonstrates that our use of <label for="group-status"> is incorrect. Labels go with individual controls, not entire sets. We should wrap these options in a <fieldset> and then apply a <legend> instead of a top-level label. tw2113, you fancy updating your patch with that change?

#2 @tw2113
12 years ago

Sure, I can give that a go this weekend. I want to make sure the markup is applied correctly. It's been a little while since I used <legend> afterall.

@tw2113
12 years ago

Adds fieldset and legeng to already supplied label fix.

#3 @tw2113
12 years ago

I got ambitious :D

#4 @boonebgorges
12 years ago

Looks great, thanks!

#5 @boonebgorges
12 years ago

  • Owner set to boonebgorges
  • Resolution → fixed
  • Status new → closed

In 8013:

Reconfigure markup for radio buttons on group admin meta boxes

<label> elements should be applied to the individual radio buttons rather
than the entire set, for better accessibility and standards compliance.

The old top-level label is replaced by a fieldset and legend.

Fixes #5437

Props tw2113

Note: See TracTickets for help on using tickets.