[wp-trac] [WordPress Trac] #39516: Media grid upload errors display below the grid.

WordPress Trac noreply at wordpress.org
Thu Jan 12 19:59:08 UTC 2017


#39516: Media grid upload errors display below the grid.
--------------------------+--------------------------------------------
 Reporter:  joemcgill     |       Owner:
     Type:  defect (bug)  |      Status:  new
 Priority:  normal        |   Milestone:  Awaiting Review
Component:  Media         |     Version:  4.6
 Severity:  normal        |  Resolution:
 Keywords:  needs-patch   |     Focuses:  ui, accessibility, javascript
--------------------------+--------------------------------------------

Comment (by afercia):

 My bad 😬 There are good accessibility reasons to keep the current order
 in the source, see #36909. I'd recommend to don't revert the change.
 Looking for the simplest way to solve this bug, there's probably the need
 to distinguish two cases:

 - in the media modal: the sidebar is still displayed on the right (even if
 it's after the attachments in the source order) so, visually, everything
 works as expected (see screenshot below)
 - in the media Library though, the "sidebar" is displayed at the end of
 the attachments

 Error messages are handled by `wp.media.view.UploaderStatus` and
 `wp.media.view.UploaderStatusError` and displayed inside the "sidebar". I
 guess trying to change the way the `UploaderStatus` is tied to the sidebar
 wouldn't be so easy.

 As far as I see, in the Library view the "sidebar" is only used to display
 error messages so maybe one option could be to just create and place the
 sidebar conditionally:
 - if it's the Library view: create the sidebar before the attachments
 - otherwise: create the sidebar after the attachments

 Any thoughts welcome.

 Screenshot: error messages in the media modal sidebar:

 [[Image(https://cldup.com/xc4JU-50QX.png)]]

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


More information about the wp-trac mailing list