Skip to:
Content

BuddyPress.org

Opened 4 years ago

Closed 4 years ago

#6624 closed enhancement (fixed)

Accessibility: Form elements in bp-members/

Reported by: mercime Owned by: mercime
Milestone: 2.4 Priority: normal
Severity: normal Version:
Component: Members 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 4 years ago.
bp-members-widgets-checkbox-id.patch (833 bytes) - added by mercime 4 years ago.
bp-members-widgets-maxnum-match-id-for.patch (693 bytes) - added by mercime 4 years ago.
bp-members-widgets-select-id-for.patch (1.8 KB) - added by mercime 4 years ago.
bp-members-widgets-whos-online-ids-fors.patch (1.1 KB) - added by mercime 4 years ago.
bp-members-widgets-rec-act-ids-fors.patch (1.1 KB) - added by mercime 4 years ago.

Download all attachments as: .zip

Change History (24)

#1 @mercime
4 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
4 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
4 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
4 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
4 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
4 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
4 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
4 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
4 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
4 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
4 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
4 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
4 years ago

  • Milestone changed from Awaiting Review to 2.4

#14 @DJPaul
4 years ago

@mercime is this ticket resolved?

#15 @DJPaul
4 years ago

  • Keywords has-patch removed

#16 @mercime
4 years ago

  • Resolution set to fixed
  • Status changed from new to closed

#17 @mercime
4 years ago

  • Resolution fixed deleted
  • Status changed from closed to reopened

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

#18 @mercime
4 years ago

  • Owner set to mercime
  • Resolution set to fixed
  • Status changed from reopened to closed

In 10314:

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

Fixes #6624.

Note: See TracTickets for help on using tickets.