Opened 7 hours ago
#49587 new defect (bug)
Add error handling for the media manager Ajax response
| Reported by: |
|
Owned by: | |
|---|---|---|---|
| Milestone: | Awaiting Review | Priority: | normal |
| Severity: | normal | Version: | 3.5 |
| Component: | Media | Keywords: | needs-patch needs-ux |
| Focuses: | javascript, administration | Cc: |
Description
When the media manager modal performs an Ajax request to query attachments, if the Ajax call responds with an error then the user is presented with an eternal loading spinner and no indication there has been a problem.
I think there are two places which need a fail() handler for the Ajax promise:
wp.media.model.Attachments.more()wp.media.model.Query.more()
I haven't done any testing yet though to confirm this.
Needs a decision around how best to present an error to a user. Probably needs UI work; maybe one of the UI pieces such as the upload failure message can be reused.
Note: See
TracTickets for help on using
tickets.