Opened 12 years ago
Closed 12 years ago
#4380 closed defect (bug) (fixed)
Add screen reader text to checkboxes in list tables
Reported by: | DJPaul | Owned by: | modemlooper |
---|---|---|---|
Milestone: | 1.8 | Priority: | normal |
Severity: | normal | Version: | |
Component: | Administration | Keywords: | needs-refresh |
Cc: |
Description
Add screen reader text to checkboxes in list tables -- currently, only the Activity admin screens use a list table. See #WP21325 for implementation example.
Putting in 1.7-early because this would involve a new string, and 1.6 is in string freeze.
Attachments (1)
Change History (9)
#5
@
12 years ago
- Keywords needs-refresh added; 1.7-early needs-patch has-patch needs-testing removed
- Milestone changed from 1.7 to 1.8
Patch is incomplete; punting to 1.8.
#6
@
12 years ago
modemlooper - I'd say you can use something more English-language descriptive here. Just "Select activity" is probably fine.
Would you mind refreshing this patch for the Groups screen as well? bp-groups/bp-groups-admin.php
#7
@
12 years ago
If those labels are not going to wrap the input forming an 'implicit' relationship then an explicit relationship must be stated i.e the 'for' attr should be used.
Why, out of interest, are those form controls being added to translatable text? does that not run the danger of a bad translated string breaking the markup?
I just added the string "Select activity_id" as the identifier. Not sure what text would be required for a screen reader. Maybe "Activity activity_id" or "Select Activity activity_id" ?