[wp-trac] [WordPress Trac] #49587: Add error handling for the media manager Ajax response

[WordPress Trac]

#49587: Add error handling for the media manager Ajax response
--------------------------+-----------------------------------------
 Reporter:  johnbillion   |       Owner:  (none)
     Type:  defect (bug)  |      Status:  new
 Priority:  normal        |   Milestone:  5.6
Component:  Media         |     Version:  3.5
 Severity:  normal        |  Resolution:
 Keywords:  needs-patch   |     Focuses:  javascript, administration
--------------------------+-----------------------------------------
Changes (by hedgefield):

 * keywords:  needs-patch needs-design-feedback => needs-patch


Comment:

 This looks like a fine solution to me @idbz. It fits nicely in the space
 where the spinner appears as well, with a clear and concise message. I'm
 not sure "HTTP error" is all that helpful, the thing is that for the user,
 the "request failed". The reason is less important, except maybe for
 troubleshooting if there are more ways in which the request can fail. But
 that probably appears in the console?

 Either way, I'd say this design with "Request failed, try again" as copy
 is good to move ahead to dev. Is there a dev available who can write this
 patch?

-- 
Ticket URL: <https://core.trac.wordpress.org/ticket/49587#comment:14>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform