Opened 10 months ago
Closed 10 months ago
#21338 closed enhancement (fixed)
Add the names of Widgets to the Add and Edit buttons on the Accessible Widget Management Screen
| Reported by: |
|
Owned by: |
|
|---|---|---|---|
| Priority: | normal | Milestone: | 3.5 |
| Component: | Accessibility | Version: | 3.4.1 |
| Severity: | normal | Keywords: | has-patch |
| Cc: | cannona@…, xoodrew@… |
Description
When a screen reader user has accessibility mode turned on on the widget management screen, the add and edit buttons are ambiguous. The problem is that it is not obvious what widget each button is associated with.
For example, on my Widgets page, I hear:
Add Heading Level 4 Custom Menu Use this widget to add one of your custom menus as a widget. Add Heading Level 4 Google Analytics Stats Displays Stat Info From Google Analytics Add Heading Level 4 Links Your blogroll Add Heading Level 4 Meta Log in/out, admin, feed and WordPress links
Note that the line-breaks weren't inserted by me. My screen reader inserts them when it detects a division of some sort, usually the start of a new paragraph, or similar.
The ideal solution would be to change the add and edit links to have the name of the effected widget appended to them. So, instead of just "Add" it might read "Add Links".
If it was undesirable to do this for visual design reasons, the appended text could be shown to screen reader users only, using the existing CSS classes for this purpose. However, including such clarifying text could only improve usability for everyone (IMHO).
Attachments (4)
Change History (15)
- Keywords needs-patch added
- Milestone changed from Awaiting Review to 3.5
comment:3
DrewAPicture — 10 months ago
- Cc xoodrew@… added
- Keywords has-patch added; needs-patch removed
This patch does not hide the text for non-screen reader users.
SergeyBiryukov — 10 months ago
SergeyBiryukov — 10 months ago
comment:5
SergeyBiryukov — 10 months ago
21338-1.patch creates a string overlapping (21338-1.png), which is especially noticeable with longer translated strings.
21338.2.patch is an attempt to add a .screen-reader-text clarification.
This patch doesn't quite work for screen reader users. Screan readers now read this as "AddAdd Widget Name". Furthermore, I believe that this is an improper use of the legend tag.
However, thanks for letting me know about the issues with the styling. I am blind, so I don't have a way of determining things like that myself. I'll try to submit a corrected patch this evening.
Thanks.
Aaron
The third patch seems to work well with my screen reader. Hopefully it won't cause problems visually.
The only drawback to the way I did it is that it may not internationalize perfectly in all languages. Basically, while in English, prepending add or edit to the title of the widget works well and is grammatically correct, it may not be so in other languages. However, I suspect the meaning should be obvious regardless, but I'm open to other ideas on how to handle this.
Aaron
comment:8
SergeyBiryukov — 10 months ago
21338-3.patch seems good to me.
- Owner set to azaozz
- Resolution set to fixed
- Status changed from new to closed
In [21355]:
comment:10
nacin — 10 months ago
- Resolution fixed deleted
- Status changed from closed to reopened
In [21355], we need to add context to the 'Edit' and 'Add' strings. This will ensure that it localizes well.
So, _ex( 'Edit', 'widget' ) and _ex( 'Add', 'widget' ).
comment:11
azaozz — 10 months ago
- Resolution set to fixed
- Status changed from reopened to closed
In [21356]:

Patch.