Make WordPress Core

Ticket #35941: 35941.diff

File 35941.diff, 10.0 KB (added by celloexpressions, 8 years ago)

Eliminate the unnecessary WP_Customize_Site_Logo_Control.

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

     
    725725.customize-control-background .current,
    726726.customize-control-cropped_image .current,
    727727.customize-control-site_icon .current,
    728 .customize-control-site_logo .current,
    729728.customize-control-header .current {
    730729        margin-bottom: 8px;
    731730}
     
    767766.customize-control-site_icon .remove-button,
    768767.customize-control-site_icon .default-button,
    769768.customize-control-site_icon .upload-button,
    770 .customize-control-site_logo .remove-button,
    771 .customize-control-site_logo .default-button,
    772 .customize-control-site_logo .upload-button,
    773769.customize-control-header button.new,
    774770.customize-control-header button.remove {
    775771        white-space: normal;
     
    783779.customize-control-background .current .container,
    784780.customize-control-cropped_image .current .container,
    785781.customize-control-site_icon .current .container,
    786 .customize-control-site_logo .current .container,
    787782.customize-control-header .current .container {
    788783        overflow: hidden;
    789784        -webkit-border-radius: 2px;
     
    797792.customize-control-background .current .container,
    798793.customize-control-cropped_image .current .container,
    799794.customize-control-site_icon .current .container,
    800 .customize-control-site_logo .current .container,
    801795.customize-control-image .current .container {
    802796        min-height: 40px;
    803797}
     
    826820.customize-control-background .inner,
    827821.customize-control-cropped_image .inner,
    828822.customize-control-site_icon .inner,
    829 .customize-control-site_logo .inner,
    830823.customize-control-header .inner {
    831824        display: none;
    832825        position: absolute;
     
    842835.customize-control-background .inner,
    843836.customize-control-cropped_image .inner,
    844837.customize-control-site_icon .inner,
    845 .customize-control-site_logo .inner,
    846838.customize-control-image .inner {
    847839        display: block;
    848840        min-height: 40px;
     
    854846.customize-control-background .inner,
    855847.customize-control-cropped_image .inner,
    856848.customize-control-site_icon .inner,
    857 .customize-control-site_logo.inner,
    858849.customize-control-header .inner,
    859850.customize-control-header .inner .dashicons {
    860851        line-height: 20px;
     
    954945        100% { color: #d4b146; }
    955946}
    956947
    957 .customize-control-media .actions,
    958 .customize-control-upload .actions,
    959 .customize-control-image .actions,
    960 .customize-control-background .actions,
    961 .customize-control-cropped_image .actions,
    962 .customize-control-site_icon .actions,
    963 .customize-control-header .actions {
    964         margin-bottom: 32px;
    965 }
    966 
    967948.customize-control-header .choice {
    968949        position: relative;
    969950        display: block;
     
    998979.customize-control-cropped_image .default-button,
    999980.customize-control-site_icon .remove-button,
    1000981.customize-control-site_icon .default-button,
    1001 .customize-control-site_logo .remove-button,
    1002 .customize-control-site_logo .default-button,
    1003982.customize-control-header .remove {
    1004983        float: left;
    1005984        margin-right: 3px;
     
    1011990.customize-control-background .upload-button,
    1012991.customize-control-cropped_image .upload-button,
    1013992.customize-control-site_icon .upload-button,
    1014 .customize-control-site_logo .upload-button,
    1015993.customize-control-header .new {
    1016994        float: right;
    1017995}
  • src/wp-admin/js/customize-controls.js

     
    17901790                                        control.pausePlayer();
    17911791                                });
    17921792
    1793                         // Re-render whenever the control's setting changes.
    1794                         control.setting.bind( function () { control.renderContent(); } );
     1793                        control.setting.bind( function ( value ) {
     1794                                // Send attachment information to the preview for possible use in `postMessage` transport.
     1795                                wp.media.attachment( value ).fetch().done( function() {
     1796                                        wp.customize.previewer.send( control.setting.id + '-attachment-data', this.attributes );
     1797                                } );
     1798
     1799                                // Re-render whenever the control's setting changes.
     1800                                control.renderContent();
     1801                        } );
    17951802                },
    17961803
    17971804                pausePlayer: function () {
     
    23032310        });
    23042311
    23052312        /**
    2306          * A control for selecting Site Logos.
    2307          *
    23082313         * @class
    2309          * @augments wp.customize.MediaControl
    23102314         * @augments wp.customize.Control
    23112315         * @augments wp.customize.Class
    23122316         */
    2313         api.SiteLogoControl = api.MediaControl.extend({
    2314 
    2315                 /**
    2316                  * When the control's DOM structure is ready,
    2317                  * set up internal event bindings.
    2318                  */
    2319                 ready: function() {
    2320                         var control = this;
    2321 
    2322                         // Shortcut so that we don't have to use _.bind every time we add a callback.
    2323                         _.bindAll( control, 'restoreDefault', 'removeFile', 'openFrame', 'select' );
    2324 
    2325                         // Bind events, with delegation to facilitate re-rendering.
    2326                         control.container.on( 'click keydown', '.upload-button', control.openFrame );
    2327                         control.container.on( 'click keydown', '.thumbnail-image img', control.openFrame );
    2328                         control.container.on( 'click keydown', '.default-button', control.restoreDefault );
    2329                         control.container.on( 'click keydown', '.remove-button', control.removeFile );
    2330 
    2331                         control.setting.bind( function( attachmentId ) {
    2332                                 wp.media.attachment( attachmentId ).fetch().done( function() {
    2333                                         wp.customize.previewer.send( 'site-logo-attachment-data', this.attributes );
    2334                                 } );
    2335 
    2336                                 // Re-render whenever the control's setting changes.
    2337                                 control.renderContent();
    2338                         } );
    2339                 }
    2340         });
    2341 
    2342         /**
    2343          * @class
    2344          * @augments wp.customize.Control
    2345          * @augments wp.customize.Class
    2346          */
    23472317        api.HeaderControl = api.Control.extend({
    23482318                ready: function() {
    23492319                        this.btnRemove = $('#customize-control-header_image .actions .remove');
     
    32453215                image:         api.ImageControl,
    32463216                cropped_image: api.CroppedImageControl,
    32473217                site_icon:     api.SiteIconControl,
    3248                 site_logo:     api.SiteLogoControl,
    32493218                header:        api.HeaderControl,
    32503219                background:    api.BackgroundControl,
    32513220                theme:         api.ThemeControl
  • src/wp-includes/class-wp-customize-manager.php

     
    217217                require_once( ABSPATH . WPINC . '/customize/class-wp-customize-background-image-control.php' );
    218218                require_once( ABSPATH . WPINC . '/customize/class-wp-customize-cropped-image-control.php' );
    219219                require_once( ABSPATH . WPINC . '/customize/class-wp-customize-site-icon-control.php' );
    220                 require_once( ABSPATH . WPINC . '/customize/class-wp-customize-site-logo-control.php' );
    221220                require_once( ABSPATH . WPINC . '/customize/class-wp-customize-header-image-control.php' );
    222221                require_once( ABSPATH . WPINC . '/customize/class-wp-customize-theme-control.php' );
    223222                require_once( ABSPATH . WPINC . '/customize/class-wp-widget-area-customize-control.php' );
     
    18611860                $this->register_control_type( 'WP_Customize_Background_Image_Control' );
    18621861                $this->register_control_type( 'WP_Customize_Cropped_Image_Control' );
    18631862                $this->register_control_type( 'WP_Customize_Site_Icon_Control' );
    1864                 $this->register_control_type( 'WP_Customize_Site_Logo_Control' );
    18651863                $this->register_control_type( 'WP_Customize_Theme_Control' );
    18661864
    18671865                /* Themes */
     
    19781976                        'transport'      => 'postMessage',
    19791977                ) );
    19801978
    1981                 $this->add_control( new WP_Customize_Site_Logo_Control( $this, 'site_logo', array(
     1979                $this->add_control( new WP_Customize_Media_Control( $this, 'site_logo', array(
    19821980                        'label'    => __( 'Logo' ),
    19831981                        'section'  => 'title_tagline',
    19841982                        'priority' => 0,
     1983                        'mime_type' => 'image',
     1984                        'button_labels' => array(
     1985                                'select'       => __( 'Select logo' ),
     1986                                'change'       => __( 'Change logo' ),
     1987                                'remove'       => __( 'Remove' ),
     1988                                'default'      => __( 'Default' ),
     1989                                'placeholder'  => __( 'No logo selected' ),
     1990                                'frame_title'  => __( 'Select logo' ),
     1991                                'frame_button' => __( 'Choose logo' ),
     1992                        ),
    19851993                ) ) );
    19861994
    19871995                if ( isset( $this->selective_refresh ) ) {
  • src/wp-includes/customize/class-wp-customize-site-logo-control.php

     
    1 <?php
    2 /**
    3  * Customize API: WP_Customize_Site_Logo_Control class
    4  *
    5  * @package WordPress
    6  * @subpackage Customize
    7  * @since 4.5.0
    8  */
    9 
    10 /**
    11  * Customize Site Logo control class.
    12  *
    13  * Used only for custom functionality in JavaScript.
    14  *
    15  * @since 4.5.0
    16  *
    17  * @see WP_Customize_Image_Control
    18  */
    19 class WP_Customize_Site_Logo_Control extends WP_Customize_Image_Control {
    20 
    21         /**
    22          * Control type.
    23          *
    24          * @since 4.5.0
    25          * @access public
    26          * @var string
    27          */
    28         public $type = 'site_logo';
    29 
    30         /**
    31          * Constructor.
    32          *
    33          * @since 4.5.0
    34          * @access public
    35          *
    36          * @param WP_Customize_Manager $manager Customizer bootstrap instance.
    37          * @param string               $id      Control ID.
    38          * @param array                $args    Optional. Arguments to override class property defaults.
    39          */
    40         public function __construct( $manager, $id, $args = array() ) {
    41                 parent::__construct( $manager, $id, $args );
    42 
    43                 $this->button_labels = array(
    44                         'select'       => __( 'Select logo' ),
    45                         'change'       => __( 'Change logo' ),
    46                         'remove'       => __( 'Remove' ),
    47                         'default'      => __( 'Default' ),
    48                         'placeholder'  => __( 'No logo selected' ),
    49                         'frame_title'  => __( 'Select logo' ),
    50                         'frame_button' => __( 'Choose logo' ),
    51                 );
    52         }
    53 }
  • src/wp-includes/js/customize-preview.js

     
    234234                 *
    235235                 * @since 4.5.0
    236236                 */
    237                 api.preview.bind( 'site-logo-attachment-data', function( attachment ) {
     237                api.preview.bind( 'site_logo-attachment-data', function( attachment ) {
    238238                        var $logo  = $( '.site-logo' ),
    239239                                size   = $logo.data( 'size' ),
    240240                                srcset = [];