Opened 15 months ago
Closed 13 months ago
#58785 closed enhancement (fixed)
Show dashicons-dismiss instead of the character 'x' when we talk about the dashicon-dismiss
Reported by: | Presskopp | Owned by: | joedolson |
---|---|---|---|
Milestone: | 6.4 | Priority: | normal |
Severity: | normal | Version: | 3.0 |
Component: | Help/About | Keywords: | has-screenshots has-patch commit |
Focuses: | accessibility, ui-copy | Cc: |
Description
The help text on the bulk edit screen explains to click the x next to the name to remove the entry from the bulk edit list. But the x is in fact a dashicon. So why not show the dashicon?
See https://build.trac.wordpress.org/browser/trunk/wp-admin/edit.php?marks=291#L291
Remarks:
When patching to
/* translators: %s is the 'x' icon */
__( 'When using Bulk Edit, you can change the metadata (categories, author, etc.) for all selected posts at once. To remove a post from the grouping, just click the %s next to its name in the Bulk Edit area that appears.' ),
'<span class="dashicons dashicons-dismiss"></span>'
) . '</p>',
the dashicon would be shown too big. It should get
style="font: normal 16px/20px dashicons;"
somehow to make it fit the size the list icons are
Attachments (3)
Change History (17)
#4
@
15 months ago
- Keywords has-patch added
- Milestone changed from Awaiting Review to 6.4
- Severity changed from minor to normal
- Version set to 3.0
This ticket was mentioned in Slack in #accessibility by joedolson. View the logs.
14 months ago
#7
@
14 months ago
The actual name of the button in the bulk edit panel is "Remove {post name} from bulk edit". I think it would appropriate for the dashicon in the help text to be announced as "remove" for screen readers, since that's the common stem of the button name that they'll observe.
#8
@
13 months ago
- Keywords changes-requested added
I think @joedolson has a good point and the name of the button can be added to its symbol.
This ticket was mentioned in Slack in #core by oglekler. View the logs.
13 months ago
This ticket was mentioned in PR #5070 on WordPress/wordpress-develop by matthewfarlymn.
13 months ago
#10
Added screen reader text "remove" to accompany bulk edit help text icon
Trac ticket: [](https://core.trac.wordpress.org/ticket/58785)
#11
@
13 months ago
Changes made on https://github.com/WordPress/wordpress-develop/pull/5070 successfully added "Remove" via the virtual screen reader bulk edit.
Environment
WordPress: 6.4-alpha-20230824.111620
PHP: 8.1.9
Server: nginx
Database: Local WP
Browser: Chrome 115.0.5790.173
Theme: Twenty Twenty One
MU-Plugins: None activated
This might be a good time to address accessibility here. Whether using "x" or the dashicon, the screenreader text for the button doesn't mention "x", so both options are somewhat reliant on sight, or at least require the user to make an educated guess.
Adding
accessibility
so folks with more experience can weigh in with their thoughts on whether this is an accessibility concern, and if so, how best to improve accessibility here.