WordPress.org

Make WordPress Core

Opened 3 months ago

Last modified 7 weeks ago

#49587 new defect (bug)

Add error handling for the media manager Ajax response

Reported by: johnbillion Owned by:
Milestone: 5.5 Priority: normal
Severity: normal Version: 3.5
Component: Media Keywords: needs-patch needs-design
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.

Change History (3)

#1 @joemcgill
3 months ago

  • Milestone changed from Awaiting Review to 5.5

Thanks for the report @johnbillion, let's add this to 5.5 to confirm and get some design feedback on the best user experience for this.

#2 @johnbillion
2 months ago

  • Keywords needs-design added

#3 @johnbillion
7 weeks ago

  • Keywords needs-ux removed
Note: See TracTickets for help on using tickets.