WordPress.org

Make WordPress Core

Changeset 38754


Ignore:
Timestamp:
10/07/2016 08:55:25 PM (20 months ago)
Author:
ocean90
Message:

Menus: Hide controls in the search tab if no items are found.

Props sayedwp, afercia, tywayne, abrightclearweb.
Fixes #35576.

Location:
trunk/src/wp-admin
Files:
3 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/wp-admin/css/nav-menus.css

    r38616 r38754  
    2626#nav-menu-meta .button-controls {
    2727    margin-bottom: 0;
     28}
     29
     30.has-no-menu-item .button-controls {
     31    display: none;
    2832}
    2933
  • trunk/src/wp-admin/includes/nav-menu.php

    r38702 r38754  
    572572                        remove_query_arg( $removed_args )
    573573                    ));
    574                 ?>#posttype-<?php echo $post_type_name; ?>" class="select-all"><?php _e('Select All'); ?></a>
     574                ?>#posttype-<?php echo $post_type_name; ?>" class="select-all aria-button-if-js"><?php _e( 'Select All' ); ?></a>
    575575            </span>
    576576
     
    770770                        remove_query_arg($removed_args)
    771771                    ));
    772                 ?>#taxonomy-<?php echo $taxonomy_name; ?>" class="select-all"><?php _e('Select All'); ?></a>
     772                ?>#taxonomy-<?php echo $taxonomy_name; ?>" class="select-all aria-button-if-js"><?php _e( 'Select All' ); ?></a>
    773773            </span>
    774774
  • trunk/src/wp-admin/js/nav-menu.js

    r38613 r38754  
    10611061                    $('.quick-search', wrapper).focus();
    10621062
     1063                    // Hide controls in the search tab if no items found.
     1064                    if ( ! wrapper.find( '.tabs-panel-active .menu-item-title' ).length ) {
     1065                        wrapper.addClass( 'has-no-menu-item' );
     1066                    } else {
     1067                        wrapper.removeClass( 'has-no-menu-item' );
     1068                    }
     1069
    10631070                    e.preventDefault();
    10641071                } else if ( target.hasClass('select-all') ) {
     
    11931200            pattern = /menu-item[(\[^]\]*/,
    11941201            $items = $('<div>').html(resp).find('li'),
     1202            wrapper = panel.closest( '.accordion-section-content' ),
    11951203            $item;
    11961204
     
    11981206                $('.categorychecklist', panel).html( '<li><p>' + navMenuL10n.noResultsFound + '</p></li>' );
    11991207                $( '.spinner', panel ).removeClass( 'is-active' );
     1208                wrapper.addClass( 'has-no-menu-item' );
    12001209                return;
    12011210            }
     
    12251234            $('.categorychecklist', panel).html( $items );
    12261235            $( '.spinner', panel ).removeClass( 'is-active' );
     1236            wrapper.removeClass( 'has-no-menu-item' );
    12271237        },
    12281238
Note: See TracChangeset for help on using the changeset viewer.