WordPress.org

Make WordPress Core

Changeset 38754


Ignore:
Timestamp:
10/07/16 20:55:25 (8 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.