Changeset 40126

Timestamp:

02/26/2017 06:14:07 PM (8 years ago)

Author:

afercia

Message:

Media: Restore correct upload errors displaying after [37610].

Props codegeass, MatheusGimenez, joemcgill.
Fixes #39516.

Location:

trunk/src/wp-includes/js

Files:

• media-views.js (modified) (1 diff)
• media/views/attachments/browser.js (modified) (1 diff)

trunk/src/wp-includes/js/media-views.js

@@ -3773 +3773 @@
         this.controller.on( 'edit:selection', this.editSelection );
         this.createToolbar();
+        // In the Media Library, the sidebar is used to display errors before the attachments grid.
+        if ( this.options.sidebar && 'errors' === this.options.sidebar ) {
+            this.createSidebar();
+        }
         this.createUploader();
         this.createAttachments();
-        if ( this.options.sidebar ) {
+        // For accessibility reasons, place the normal sidebar after the attachments, see ticket #36909.
+        if ( this.options.sidebar && 'errors' !== this.options.sidebar ) {
             this.createSidebar();
         }

trunk/src/wp-includes/js/media/views/attachments/browser.js

@@ -42 +42 @@
         this.controller.on( 'edit:selection', this.editSelection );
         this.createToolbar();
+        // In the Media Library, the sidebar is used to display errors before the attachments grid.
+        if ( this.options.sidebar && 'errors' === this.options.sidebar ) {
+            this.createSidebar();
+        }
         this.createUploader();
         this.createAttachments();
-        if ( this.options.sidebar ) {
+        // For accessibility reasons, place the normal sidebar after the attachments, see ticket #36909.
+        if ( this.options.sidebar && 'errors' !== this.options.sidebar ) {
             this.createSidebar();
         }