Skip to:
Content

BuddyPress.org

Opened 11 years ago

Closed 11 years ago

#6624 closed enhancement (fixed)

Accessibility: Form elements in bp-members/

Reported by: mercime Owned by: mercime
Priority: normal Milestone: 2.4
Component: Members Version:
Severity: normal Keywords:
Cc:

Description

Bind labels to form controls: add 'id' attributes to form controls as needed, add 'for' attributes to labels, and improve markup as needed.

Attachments (6)

bp-members-widgets-title-match-id-for.patch (698 bytes ) - added by mercime 11 years ago.
bp-members-widgets-checkbox-id.patch (833 bytes ) - added by mercime 11 years ago.
bp-members-widgets-maxnum-match-id-for.patch (693 bytes ) - added by mercime 11 years ago.
bp-members-widgets-select-id-for.patch (1.8 KB ) - added by mercime 11 years ago.
bp-members-widgets-whos-online-ids-fors.patch (1.1 KB ) - added by mercime 11 years ago.
bp-members-widgets-rec-act-ids-fors.patch (1.1 KB ) - added by mercime 11 years ago.

Download all attachments as: .zip

Change History (24)

#1 @mercime
11 years ago

  • Keywords has-patch added

Members Widget: Title - Add 'for' attribute to the label and use the value from the 'id' attribute of the text input field.
bp-members-widgets-title-match-id-for.patch

#2 @mercime
11 years ago

Members Widget: Link Title - Add new 'id' to the checkbox field and use the new 'id' value to replace the 'for' value of the label tag to match.
bp-members-widgets-checkbox-id.patch

#3 @mercime
11 years ago

Members Widget: Max no. members - Replace the 'for' value of the label element with the 'id' value of the text input field.
bp-members-widgets-maxnum-match-id-for.patch

#4 @mercime
11 years ago

Members Widget: Selectbox - Add new 'id' attribute to selectbox and use the 'id' value to replace the 'for' value of the label tag.
bp-members-widgets-select-id-for.patch

#5 @mercime
11 years ago

Who's Online Widget: Replace the 'for' values of the two label tags with the 'id' values of the respective text input fields.
bp-members-widgets-whos-online-ids-fors.patch

#6 @mercime
11 years ago

Recently Active Members Widget: Replace the 'for' values of the two label tags with the 'id' values of the respective text input fields.
bp-members-widgets-rec-act-ids-fors.php

#7 @mercime
11 years ago

In 10118:

Members Widget: Replace the 'for' value with 'id' value of input.

Accessibility fix which replaces the static 'for' value with the
dynamic 'id' value of the input field.

See #6624.

#8 @mercime
11 years ago

In 10119:

Members Widget: Add 'id' attribute to input field and bind to label.

Accessibility fix adds a new dynamic 'id' value to the input field
and replaces the static value of the label 'for' attribute with
the new 'id' value of the checkbox to link widget title to Members
directory

See #6624.

#9 @mercime
11 years ago

In 10120:

Members Widget: Replace the 'for' value to bind label to input.

Accessibility fix which replaces the static 'for' value with the
dynamic 'id' value of the input field.

See #6624.

#10 @mercime
11 years ago

In 10121:

Members Widget: Add 'id' attribute to input and bind to label.

Accessibility fix adds dynamic 'id' value which also replaces the old
'for' value in order to avoid duplicate 'ids' in DOM when more than
one Members Widget is added to a theme's available widget areas. This
also unwraps the label tag from the select field.

See #6624.

#11 @mercime
11 years ago

In 10122:

Who is Online Widget: Replace the 'for' value to bind to input.

Accessibility fix which replaces the static 'for' value with the
dynamic 'id' value of the input field.

See #6624.

#12 @mercime
11 years ago

In 10123:

Recently Active: Replace the 'for' value to bind label to input.

Accessibility fix which replaces two static 'for' values with the
dynamic 'id' values of respective input fields.

See #6624.

#13 @DJPaul
11 years ago

  • Milestone Awaiting Review2.4

#14 @DJPaul
11 years ago

@mercime is this ticket resolved?

#15 @DJPaul
11 years ago

  • Keywords has-patch removed

#16 @mercime
11 years ago

  • Resolutionfixed
  • Status newclosed

#17 @mercime
11 years ago

  • Resolution fixed
  • Status closedreopened

Add one more a11y fix in Extended Profiles Member Type metabox.

#18 @mercime
11 years ago

  • Owner set to mercime
  • Resolutionfixed
  • Status reopenedclosed

In 10314:

Add label for select element in Extended Profiles Member Type metabox.

Fixes #6624.

Note: See TracTickets for help on using tickets.