Opened 8 years ago
Last modified 6 years ago
#38026 new enhancement
Add Delete link to details modal when file in trash
Reported by: | mahu2401 | Owned by: | |
---|---|---|---|
Milestone: | Awaiting Review | Priority: | normal |
Severity: | normal | Version: | 4.0 |
Component: | Media | Keywords: | has-patch needs-refresh |
Focuses: | ui | Cc: |
Description
First enable media trash by adding define('MEDIA_TRASH', true);
to wp-config.php and move a media file to trash by clicking its "Trash" Link either in grid view or list view.
Then navigate to the trash view. In list view there are two links on each item ("Restore" and "Delete Permanently").
In the details modal of the grid view there's only one link ("Untrash"). In addition this link is red like a delete link, but it actually restores the file. To delete the file you have to close the modal, select it with bulk select and then click the delete button in the toolbar.
I think there should be a second link "Delete Permanently" in the details modal and the "Untrash" link should be blue to avoid confusion.
Attachments (3)
Change History (7)
This ticket was mentioned in Slack in #core by helen. View the logs.
8 years ago
#3
@
8 years ago
- Focuses ui added; administration removed
- Type changed from defect (bug) to enhancement
- Version changed from trunk to 4.0
Not sure how much work we want to do around media trash since it's not a true trash (the file is still accessible) and not very well-supported in general, but if somebody wants to take a look at aligning the two views that seems fine.
#4
@
6 years ago
- Keywords needs-refresh added
@mahu2401 are you still interested in this? 38026.patch is failing to apply cleanly to trunk
. It needs to be refreshed.
Untrash and Delete in list view