[wp-trac] [WordPress Trac] #44675: Media Trash action verbiage inconsistent and causes confusion
WordPress Trac
noreply at wordpress.org
Mon Jul 30 17:20:10 UTC 2018
#44675: Media Trash action verbiage inconsistent and causes confusion
-----------------------------+-----------------------------
Reporter: garrett-eclipse | Owner: (none)
Type: defect (bug) | Status: new
Priority: normal | Milestone: Awaiting Review
Component: I18N | Version: trunk
Severity: normal | Keywords: good-first-bug
Focuses: |
-----------------------------+-----------------------------
Hello,
While looking at the consistency for Bulk Actions I came across the Media
list table uses 'MEDIA_TRASH' to enable it's trash functionality. It's
trash action is labelled as 'Trash' with a context of 'verb' this is
inconsistent with all other similar implementations of trash in the list
tables as they all use 'Move to Trash' to remove the ambiguity. Aside from
the consistency benefit we'll be able to remove the context as it's no
longer needed to distinguish it from the Trash filter or page.
So replacing this line;
https://github.com/WordPress/WordPress/blob/8e01f9f99b9cc39bed623543f77752c819132699
/wp-admin/includes/class-wp-media-list-table.php#L154
With - `$actions['trash'] = __( 'Move to Trash' );`
Will make it consistent with all other implementations;
Comments list table -
https://github.com/WordPress/WordPress/blob/1b5d6c6971c158e38b2f7c4ac6acfa19276aa5df
/wp-admin/includes/class-wp-comments-list-table.php#L351
Posts list table -
https://github.com/WordPress/WordPress/blob/1b5d6c6971c158e38b2f7c4ac6acfa19276aa5df
/wp-admin/includes/class-wp-posts-list-table.php#L413
This will also clear up confusion for translators as it seemed my local
en_CA was replacing Trash with Delete in this case which isn't correct;
https://translate.wordpress.org/projects/wp/dev/en-
ca/default?filters%5Bstatus%5D=either&filters%5Boriginal_id%5D=202108&filters%5Btranslation_id%5D=9657614
*I'm cleaning up the translations.
Thank you
--
Ticket URL: <https://core.trac.wordpress.org/ticket/44675>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list