Changeset 54316

Timestamp:

09/26/2022 09:42:12 PM (2 years ago)

Author:

audrasjb

Message:

Menus: Remove bulk action checkboxes when menu is empty.

This changeset removes the bulk action checkboxes when there is no menu item to select.

Follow-up to [51006], [51539].

Props oglekler, krishaweb, costdev, Boniu91, hugodevos, audrasjb.
Fixes #54799.
See #21603, #53654.

Location:

trunk/src

Files:

• js/_enqueues/lib/nav-menu.js (modified) (1 diff)
• wp-admin/nav-menus.php (modified) (2 diffs)

trunk/src/js/_enqueues/lib/nav-menu.js

@@ -1558 +1558 @@
     });
 
+    // Show bulk action
+    $( document ).on( 'menu-item-added', function() {
+        if ( ! $( '.bulk-actions' ).is( ':visible' ) ) {
+            $( '.bulk-actions' ).show();
+        }
+    } );
+
+    // Hide bulk action
+    $( document ).on( 'menu-removing-item', function( e, el ) {
+        var menuElement = $( el ).parents( '#menu-to-edit' );
+        if ( menuElement.find( 'li' ).length === 1 && $( '.bulk-actions' ).is( ':visible' ) ) {
+            $( '.bulk-actions' ).hide();
+        }
+    } );
+
 })(jQuery);

trunk/src/wp-admin/nav-menus.php

@@ -1001 +1001 @@
 
                                 <?php if ( ! $add_new_screen ) : ?>
-                                    <div id="nav-menu-bulk-actions-top" class="bulk-actions">
+                                    <div id="nav-menu-bulk-actions-top" class="bulk-actions" <?php echo $hide_style; ?>>
                                         <label class="bulk-select-button" for="bulk-select-switcher-top">
                                             <input type="checkbox" id="bulk-select-switcher-top" name="bulk-select-switcher-top" class="bulk-select-switcher">
@@ -1036 +1036 @@
 
                             <?php if ( ! $add_new_screen ) : ?>
-                                <div id="nav-menu-bulk-actions-bottom" class="bulk-actions">
+                                <div id="nav-menu-bulk-actions-bottom" class="bulk-actions" <?php echo $hide_style; ?>>
                                     <label class="bulk-select-button" for="bulk-select-switcher-bottom">
                                         <input type="checkbox" id="bulk-select-switcher-bottom" name="bulk-select-switcher-top" class="bulk-select-switcher">