WordPress.org
Get WordPress

Make WordPress Core

Ticket #33052: 33052.2.diff

File 33052.2.diff, 13.4 KB (added by westonruter, 11 years ago)

Additional change: ​https://github.com/xwp/wordpress-develop/commit/25de145e7dde5054592fdd28eec766a3fad8f801

  • src/wp-admin/css/customize-controls.css 

    diff --git src/wp-admin/css/customize-controls.css src/wp-admin/css/customize-controls.css
index b35bc3b..6b9d910 100644
    body { 
    123123        color: #0073aa;
    124124}
    125125
    126 #customize-controls .customize-info .customize-panel-description {
     126#customize-controls .customize-info .customize-panel-description,
     127#customize-controls .no-widget-areas-rendered-notice {
    127128        color: #555;
    128129        display: none;
    129130        background: #fff;
    130131        padding: 12px 15px;
    131132        border-top: 1px solid #ddd;
    132133}
     134#customize-controls .customize-info .customize-panel-description.open + .no-widget-areas-rendered-notice {
     135        border-top: none;
     136}
    133137
    134138#customize-controls .customize-info .customize-panel-description p:first-child {
    135139        margin-top: 0;

  • src/wp-admin/js/customize-controls.js 

    diff --git src/wp-admin/js/customize-controls.js src/wp-admin/js/customize-controls.js
index e0e4a4d..8394d60 100644
     
    378378                },
    379379
    380380                /**
    381                  * @param {Boolean} expanded
    382                  * @param {Object} [params]
    383                  * @returns {Boolean} false if state already applied
     381                 * Handle the toggle logic for expand/collapse.
     382                 *
     383                 * @param {Boolean}  expanded - The new state to apply.
     384                 * @param {Object}   [params] - Object containing options for expand/collapse.
     385                 * @param {Function} [params.completeCallback] - Function to call when expansion/collapse is complete.
     386                 * @returns {Boolean} false if state already applied or active state is false
    384387                 */
    385                 _toggleExpanded: function ( expanded, params ) {
    386                         var self = this;
     388                _toggleExpanded: function( expanded, params ) {
     389                        var instance = this, previousCompleteCallback;
    387390                        params = params || {};
    388                         var section = this, previousCompleteCallback = params.completeCallback;
    389                         params.completeCallback = function () {
     391                        previousCompleteCallback = params.completeCallback;
     392
     393                        // Short-circuit expand() if the instance is not active.
     394                        if ( expanded && ! instance.active() ) {
     395                                return false;
     396                        }
     397
     398                        params.completeCallback = function() {
    390399                                if ( previousCompleteCallback ) {
    391                                         previousCompleteCallback.apply( section, arguments );
     400                                        previousCompleteCallback.apply( instance, arguments );
    392401                                }
    393402                                if ( expanded ) {
    394                                         section.container.trigger( 'expanded' );
     403                                        instance.container.trigger( 'expanded' );
    395404                                } else {
    396                                         section.container.trigger( 'collapsed' );
     405                                        instance.container.trigger( 'collapsed' );
    397406                                }
    398407                        };
    399                         if ( ( expanded && this.expanded.get() ) || ( ! expanded && ! this.expanded.get() ) ) {
     408                        if ( ( expanded && instance.expanded.get() ) || ( ! expanded && ! instance.expanded.get() ) ) {
    400409                                params.unchanged = true;
    401                                 self.onChangeExpanded( self.expanded.get(), params );
     410                                instance.onChangeExpanded( instance.expanded.get(), params );
    402411                                return false;
    403412                        } else {
    404413                                params.unchanged = false;
    405                                 this.expandedArgumentsQueue.push( params );
    406                                 this.expanded.set( expanded );
     414                                instance.expandedArgumentsQueue.push( params );
     415                                instance.expanded.set( expanded );
    407416                                return true;
    408417                        }
    409418                },
    410419
    411420                /**
    412421                 * @param {Object} [params]
    413                  * @returns {Boolean} false if already expanded
     422                 * @returns {Boolean} false if already expanded or if inactive.
    414423                 */
    415424                expand: function ( params ) {
    416425                        return this._toggleExpanded( true, params );
    … …  
    418427
    419428                /**
    420429                 * @param {Object} [params]
    421                  * @returns {Boolean} false if already collapsed
     430                 * @returns {Boolean} false if already collapsed.
    422431                 */
    423432                collapse: function ( params ) {
    424433                        return this._toggleExpanded( false, params );
    … …  
    539548                        };
    540549                        section.panel.bind( inject );
    541550                        inject( section.panel.get() ); // Since a section may never get a panel, assume that it won't ever get one
     551
     552                        section.deferred.embedded.done(function() {
     553                                // Fix the top margin after reflow.
     554                                api.bind( 'pane-contents-reflowed', _.debounce( function() {
     555                                        section._recalculateTopMargin();
     556                                }, 100 ) );
     557                        });
    542558                },
    543559
    544560                /**
    … …  
    646662                                                // Fix the height after browser resize.
    647663                                                $( window ).on( 'resize.customizer-section', _.debounce( resizeContentHeight, 100 ) );
    648664
    649                                                 // Fix the top margin after reflow.
    650                                                 api.bind( 'pane-contents-reflowed', _.debounce( function() {
    651                                                         var offset = ( content.offset().top - headerActionsHeight );
    652                                                         if ( 0 < offset ) {
    653                                                                 content.css( 'margin-top', ( parseInt( content.css( 'margin-top' ), 10 ) - offset ) );
    654                                                         }
    655                                                 }, 100 ) );
     665                                                section._recalculateTopMargin();
    656666                                        };
    657667                                }
    658668
    … …  
    693703                                        args.completeCallback();
    694704                                }
    695705                        }
     706                },
     707
     708                /**
     709                 * Recalculate the top margin.
     710                 *
     711                 * @since 4.4.0
     712                 * @private
     713                 */
     714                _recalculateTopMargin: function() {
     715                        var section = this, content, offset, headerActionsHeight;
     716                        content = section.container.find( '.accordion-section-content' );
     717                        if ( 0 === content.length ) {
     718                                return;
     719                        }
     720                        headerActionsHeight = $( '#customize-header-actions' ).height();
     721                        offset = ( content.offset().top - headerActionsHeight );
     722                        if ( 0 < offset ) {
     723                                content.css( 'margin-top', ( parseInt( content.css( 'margin-top' ), 10 ) - offset ) );
     724                        }
    696725                }
    697726        });
    698727
    … …  
    11551184                                parentContainer.append( panel.container );
    11561185                                panel.renderContent();
    11571186                        }
     1187
     1188                        api.bind( 'pane-contents-reflowed', _.debounce( function() {
     1189                                panel._recalculateTopMargin();
     1190                        }, 100 ) );
     1191
    11581192                        panel.deferred.embedded.resolve();
    11591193                },
    11601194
    … …  
    12531287                 * @param {Boolean}  expanded
    12541288                 * @param {Object}   args
    12551289                 * @param {Boolean}  args.unchanged
    1256                  * @param {Callback} args.completeCallback
     1290                 * @param {Function} args.completeCallback
    12571291                 */
    12581292                onChangeExpanded: function ( expanded, args ) {
    12591293
    … …  
    12681302                        // Note: there is a second argument 'args' passed
    12691303                        var position, scroll,
    12701304                                panel = this,
    1271                                 section = panel.container.closest( '.accordion-section' ), // This is actually the panel.
    1272                                 overlay = section.closest( '.wp-full-overlay' ),
    1273                                 container = section.closest( '.wp-full-overlay-sidebar-content' ),
     1305                                accordionSection = panel.container.closest( '.accordion-section' ),
     1306                                overlay = accordionSection.closest( '.wp-full-overlay' ),
     1307                                container = accordionSection.closest( '.wp-full-overlay-sidebar-content' ),
    12741308                                siblings = container.find( '.open' ),
    12751309                                topPanel = overlay.find( '#customize-theme-controls > ul > .accordion-section > .accordion-section-title' ),
    1276                                 backBtn = section.find( '.customize-panel-back' ),
    1277                                 panelTitle = section.find( '.accordion-section-title' ).first(),
    1278                                 content = section.find( '.control-panel-content' ),
     1310                                backBtn = accordionSection.find( '.customize-panel-back' ),
     1311                                panelTitle = accordionSection.find( '.accordion-section-title' ).first(),
     1312                                content = accordionSection.find( '.control-panel-content' ),
    12791313                                headerActionsHeight = $( '#customize-header-actions' ).height();
    12801314
    12811315                        if ( expanded ) {
    … …  
    12971331                                        position = content.offset().top;
    12981332                                        scroll = container.scrollTop();
    12991333                                        content.css( 'margin-top', ( headerActionsHeight - position - scroll ) );
    1300                                         section.addClass( 'current-panel' );
     1334                                        accordionSection.addClass( 'current-panel' );
    13011335                                        overlay.addClass( 'in-sub-panel' );
    13021336                                        container.scrollTop( 0 );
    13031337                                        if ( args.completeCallback ) {
    … …  
    13071341                                topPanel.attr( 'tabindex', '-1' );
    13081342                                backBtn.attr( 'tabindex', '0' );
    13091343                                backBtn.focus();
    1310 
    1311                                 // Fix the top margin after reflow.
    1312                                 api.bind( 'pane-contents-reflowed', _.debounce( function() {
    1313                                         content.css( 'margin-top', ( parseInt( content.css( 'margin-top' ), 10 ) - ( content.offset().top - headerActionsHeight ) ) );
    1314                                 }, 100 ) );
     1344                                panel._recalculateTopMargin();
    13151345                        } else {
    13161346                                siblings.removeClass( 'open' );
    1317                                 section.removeClass( 'current-panel' );
     1347                                accordionSection.removeClass( 'current-panel' );
    13181348                                overlay.removeClass( 'in-sub-panel' );
    13191349                                content.delay( 180 ).hide( 0, function() {
    13201350                                        content.css( 'margin-top', 'inherit' ); // Reset
    … …  
    13301360                },
    13311361
    13321362                /**
     1363                 * Recalculate the top margin.
     1364                 *
     1365                 * @since 4.4.0
     1366                 * @private
     1367                 */
     1368                _recalculateTopMargin: function() {
     1369                        var panel = this, headerActionsHeight, content, accordionSection;
     1370                        headerActionsHeight = $( '#customize-header-actions' ).height();
     1371                        accordionSection = panel.container.closest( '.accordion-section' )
     1372                        content = accordionSection.find( '.control-panel-content' );
     1373                        content.css( 'margin-top', ( parseInt( content.css( 'margin-top' ), 10 ) - ( content.offset().top - headerActionsHeight ) ) );
     1374                },
     1375
     1376                /**
    13331377                 * Render the panel from its JS template, if it exists.
    13341378                 *
    13351379                 * The panel's container must already exist in the DOM.

  • src/wp-admin/js/customize-widgets.js 

    diff --git src/wp-admin/js/customize-widgets.js src/wp-admin/js/customize-widgets.js
index bd25757..144365e 100644
     
    15061506        } );
    15071507
    15081508        /**
     1509         * wp.customize.Widgets.WidgetsPanel
     1510         *
     1511         * Customizer panel containing the widget area sections.
     1512         *
     1513         * @since 4.4.0
     1514         */
     1515        api.Widgets.WidgetsPanel = api.Panel.extend({
     1516
     1517                /**
     1518                 * Add and manage the display of the no-rendered-areas notice.
     1519                 *
     1520                 * @since 4.4.0
     1521                 */
     1522                ready: function () {
     1523                        var panel = this;
     1524
     1525                        api.Panel.prototype.ready.call( panel );
     1526
     1527                        panel.deferred.embedded.done(function() {
     1528                                var panelMetaContainer, noRenderedAreasNotice, shouldShowNotice;
     1529                                panelMetaContainer = panel.container.find( '.panel-meta' );
     1530                                noRenderedAreasNotice = $( '<div></div>', {
     1531                                        'class': 'no-widget-areas-rendered-notice'
     1532                                });
     1533                                noRenderedAreasNotice.append( $( '<em></em>', {
     1534                                        text: l10n.noAreasRendered
     1535                                } ) );
     1536                                panelMetaContainer.append( noRenderedAreasNotice );
     1537
     1538                                shouldShowNotice = function() {
     1539                                        return ( 0 === _.filter( panel.sections(), function( section ) {
     1540                                                return section.active();
     1541                                        } ).length );
     1542                                };
     1543
     1544                                /*
     1545                                 * Defer setting visibility of no-rendered-areas-notice until
     1546                                 * preview finishes loading since this is when the active
     1547                                 * sections become available.
     1548                                 */
     1549                                api.previewer.deferred.active.done( function () {
     1550                                        noRenderedAreasNotice.toggle( shouldShowNotice() );
     1551
     1552                                        // Update the visibility of the notice whenever a reflow happens.
     1553                                        api.bind( 'pane-contents-reflowed', function() {
     1554                                                api.previewer.deferred.active.done( function () {
     1555                                                        if ( shouldShowNotice() ) {
     1556                                                                noRenderedAreasNotice.slideDown( 'fast' );
     1557                                                        } else {
     1558                                                                noRenderedAreasNotice.slideUp( 'fast' );
     1559                                                        }
     1560                                                });
     1561                                        });
     1562                                });
     1563                        });
     1564                },
     1565
     1566                /**
     1567                 * Allow an active widgets panel to be contextually active even when it has no active sections (widget areas).
     1568                 *
     1569                 * This ensures that the widgets panel appears even when there are no
     1570                 * sidebars displayed on the URL currently being previewed.
     1571                 *
     1572                 * @since 4.4.0
     1573                 *
     1574                 * @returns {boolean}
     1575                 */
     1576                isContextuallyActive: function() {
     1577                        var panel = this;
     1578                        return panel.active();
     1579                }
     1580        });
     1581
     1582        /**
    15091583         * wp.customize.Widgets.SidebarSection
    15101584         *
    15111585         * Customizer section representing a widget area widget
    … …  
    19682042                }
    19692043        } );
    19702044
    1971         // Register models for custom section and control types
     2045        // Register models for custom panel, section, and control types
     2046        $.extend( api.panelConstructor, {
     2047                widgets: api.Widgets.WidgetsPanel
     2048        });
    19722049        $.extend( api.sectionConstructor, {
    19732050                sidebar: api.Widgets.SidebarSection
    19742051        });

  • src/wp-includes/class-wp-customize-widgets.php 

    diff --git src/wp-includes/class-wp-customize-widgets.php src/wp-includes/class-wp-customize-widgets.php
index a003393..7b7c064 100644
    final class WP_Customize_Widgets { 
    355355                }
    356356
    357357                $this->manager->add_panel( 'widgets', array(
    358                         'title'       => __( 'Widgets' ),
    359                         'description' => __( 'Widgets are independent sections of content that can be placed into widgetized areas provided by your theme (commonly called sidebars).' ),
    360                         'priority'    => 110,
     358                        'type'            => 'widgets',
     359                        'title'           => __( 'Widgets' ),
     360                        'description'     => __( 'Widgets are independent sections of content that can be placed into widgetized areas provided by your theme (commonly called sidebars).' ),
     361                        'priority'        => 110,
     362                        'active_callback' => array( $this, 'is_panel_active' ),
    361363                ) );
    362364
    363365                foreach ( $sidebars_widgets as $sidebar_id => $sidebar_widget_ids ) {
    … … final class WP_Customize_Widgets { 
    455457        }
    456458
    457459        /**
     460         * Return whether the widgets panel is active, based on whether there are sidebars registered.
     461         *
     462         * @since 4.4.0
     463         * @access public
     464         *
     465         * @see WP_Customize_Panel::$active_callback
     466         *
     467         * @global array $wp_registered_sidebars
     468         * @return bool Active.
     469         */
     470        public function is_panel_active() {
     471                global $wp_registered_sidebars;
     472                return ! empty( $wp_registered_sidebars );
     473        }
     474
     475        /**
    458476         * Covert a widget_id into its corresponding Customizer setting ID (option name).
    459477         *
    460478         * @since 3.9.0
    … … final class WP_Customize_Widgets { 
    655673                                'error'            => __( 'An error has occurred. Please reload the page and try again.' ),
    656674                                'widgetMovedUp'    => __( 'Widget moved up' ),
    657675                                'widgetMovedDown'  => __( 'Widget moved down' ),
     676                                'noAreasRendered'  => __( 'There are no widget areas currently rendered in the preview. Navigate in the preview to a template that makes use of a widget area in order to access its widgets here.' ),
    658677                        ),
    659678                        'tpl' => array(
    660679                                'widgetReorderNav' => $widget_reorder_nav_tpl,