WordPress.org

Make WordPress Core

Changeset 31419


Ignore:
Timestamp:
02/11/2015 08:44:19 PM (5 years ago)
Author:
ocean90
Message:

Customizer: Improve [31252] to show the move-widget buttons only if there is more than one *rendered* sidebar.

fixes #30690.

File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/wp-admin/js/customize-widgets.js

    r31255 r31419  
    622622             */
    623623            updateAvailableSidebars = function() {
    624                 var $sidebarItems = $moveWidgetArea.find( 'li' ), selfSidebarItem;
     624                var $sidebarItems = $moveWidgetArea.find( 'li' ), selfSidebarItem,
     625                    renderedSidebarCount = 0;
    625626
    626627                selfSidebarItem = $sidebarItems.filter( function(){
     
    630631                $sidebarItems.each( function() {
    631632                    var li = $( this ),
    632                         sidebarId,
    633                         sidebar;
     633                        sidebarId, sidebar, sidebarIsRendered;
    634634
    635635                    sidebarId = li.data( 'id' );
    636636                    sidebar = api.Widgets.registeredSidebars.get( sidebarId );
    637 
    638                     li.toggle( sidebar.get( 'is_rendered' ) );
    639 
    640                     if ( li.hasClass( 'selected' ) && ! sidebar.get( 'is_rendered' ) ) {
     637                    sidebarIsRendered = sidebar.get( 'is_rendered' );
     638
     639                    li.toggle( sidebarIsRendered );
     640
     641                    if ( sidebarIsRendered ) {
     642                        renderedSidebarCount += 1;
     643                    }
     644
     645                    if ( li.hasClass( 'selected' ) && ! sidebarIsRendered ) {
    641646                        selectSidebarItem( selfSidebarItem );
    642647                    }
    643648                } );
    644649
    645                 if ( 1 === $sidebarItems.length ) {
     650                if ( renderedSidebarCount > 1 ) {
     651                    self.container.find( '.move-widget' ).show();
     652                } else {
    646653                    self.container.find( '.move-widget' ).hide();
    647                 } else {
    648                     self.container.find( '.move-widget' ).show();
    649654                }
    650655            };
Note: See TracChangeset for help on using the changeset viewer.