Make WordPress Core

Opened 7 years ago

Last modified 4 years ago

#42978 new defect (bug)

Automatically clear file upload error messages

Reported by: computerguru's profile ComputerGuru Owned by:
Milestone: Future Release Priority: normal
Severity: normal Version:
Component: Media Keywords: needs-patch
Focuses: ui, administration Cc:

Description

When an error occurs on file upload in the media uploader while drafting a post, the error message persists after both a) another file is successfully uploaded, b) the media manager is closed and re-opened.

In both of these cases, it is my opinion that the user is best served by having the error message automatically dismissed, as

  1. Its positioning is not typical of a "historic/cumulative" error log,
  2. It may be assumed that by successfully uploading another file the error first encountered is no longer relevant, and
  3. An indefinite amount of time may have passed since the error was first encountered and the media manager subsequently closed and the next time the media manager/upload div was re-opened such that the user may have completely forgotten about the error and may be confused to see a red error box and mistakenly think it is associated with some action he/she just took,
  4. As a dialog, nothing about the media manager suggests that its state should persist after being closed and re-opened.

(see attachment as an example of an error)

Attachments (1)

wp-upload-error.png (1.5 MB) - added by ComputerGuru 7 years ago.

Download all attachments as: .zip

Change History (5)

#1 @pento
6 years ago

  • Version trunk deleted

This ticket was mentioned in Slack in #core-media by antpb. View the logs.


6 years ago

#3 @antpb
6 years ago

  • Keywords needs-patch added
  • Milestone changed from Awaiting Review to Future Release

We have a similar issue in Gutenberg notices stacking on uploads. I believe these are very different solutions but should be noted as similar problems: https://github.com/WordPress/gutenberg/issues/6106

Marking this Future Release as it is a great enhancement. :)

#4 @Mista-Flo
4 years ago

Hum, I think the purpose of the dismiss button is to allow the user to hide the errors if he wants to, therefore I'm not sure it's really important to force the removal of errors after a successful upload.

Note: See TracTickets for help on using tickets.