Changeset 45701

Timestamp:

07/30/2019 11:19:18 PM (6 years ago)

Author:

karmatosed

Message:

Media: Reduces the bulk media options to have one primary button

This fixes where 2 primary buttons were showing for bulk actions within media trash.

Props garrett-eclipse, afercia, ianbelanger, SergeyBiryukov
Fixes #46757, #46758

Location:

trunk/src

Files:

• js/media/views/attachments/browser.js (modified) (2 diffs)
• js/media/views/button/delete-selected.js (modified) (1 diff)
• wp-includes/css/buttons.css (modified) (1 diff)

trunk/src/js/media/views/attachments/browser.js

@@ -191 +191 @@
                 text: mediaTrash ? l10n.trashSelected : l10n.deletePermanently,
                 controller: this.controller,
-                priority: -60,
+                priority: -80,
                 click: function() {
                     var changed = [], removed = [],
@@ -247 +247 @@
                 this.toolbar.set( 'deleteSelectedPermanentlyButton', new wp.media.view.DeleteSelectedPermanentlyButton({
                     filters: Filters,
-                    style: 'primary',
+                    style: 'link button-link-delete',
                     disabled: true,
                     text: l10n.deletePermanently,

trunk/src/js/media/views/button/delete-selected.js

@@ -23 +23 @@
         }
         this.controller.on( 'selection:toggle', this.toggleDisabled, this );
+        this.controller.on( 'select:activate', this.toggleDisabled, this );
     },
 

trunk/src/wp-includes/css/buttons.css

@@ -225 +225 @@
 .wp-core-ui .button-link-delete:focus {
     color: #dc3232;
+    background: transparent;
+}
+
+.wp-core-ui .button-link-delete:disabled {
+    /* overrides the default buttons disabled background */
+    background: transparent !important;
 }