Make WordPress Core

Ticket #35943: 35943.diff

File 35943.diff, 2.7 KB (added by celloexpressions, 9 years ago)

Clean up the new site icon control preview to take up less space, reduce visual clutter, and be more consistent with customizer UI conventions.

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

     
    979979.customize-control-image img,
    980980.customize-control-background img,
    981981.customize-control-cropped_image img,
    982 .customize-control-site_icon img,
    983982.customize-control-header img {
    984983        width: 100%;
    985984        -webkit-border-radius: 2px;
  • src/wp-admin/css/site-icon.css

     
    2424
    2525.site-icon-preview .browser-title {
    2626        left: 109px;
     27        width: 72px;
     28        white-space: nowrap;
    2729}
    2830
    2931.site-icon-preview .app-icon-preview {
     
    3537        width: 64px;
    3638        margin-top: 5px;
    3739}
     40
     41.customize-control-site_icon .favicon-preview {
     42        float: left;
     43        margin-right: 12px;
     44        margin-bottom: 0;
     45}
     46
     47.customize-control-site_icon .app-icon-preview {
     48        margin-top: 9px;
     49}
  • src/wp-includes/customize/class-wp-customize-site-icon-control.php

     
    6161                <# if ( data.attachment && data.attachment.id ) { #>
    6262                <div class="current">
    6363                        <div class="container">
    64                                 <div class="attachment-media-view attachment-media-view-{{ data.attachment.type }} {{ data.attachment.orientation }} site-icon-preview">
    65                                         <strong><?php _e( 'As a browser icon' ); ?></strong>
     64                                <div class="site-icon-preview">
    6665                                        <div class="favicon-preview">
    67                                                 <img src="images/browser.png" class="browser-preview" width="182" height="" alt="" />
     66                                                <img src="images/browser.png" class="browser-preview" width="182" alt="<?php esc_attr_e( 'Browser interface preview' ); ?>" />
    6867
    6968                                                <div class="favicon">
    7069                                                        <img id="preview-favicon" src="{{ data.attachment.sizes.full.url }}" alt="<?php esc_attr_e( 'Preview as a browser icon' ); ?>"/>
     
    7170                                                </div>
    7271                                                <span class="browser-title"><?php bloginfo( 'name' ); ?></span>
    7372                                        </div>
    74 
    75                                         <strong><?php _e( 'As an app icon' ); ?></strong>
    76                                         <p class="app-icon-preview">
    77                                                 <img id="preview-app-icon" src="{{ data.attachment.sizes.full.url }}" alt="<?php esc_attr_e( 'Preview as an app icon' ); ?>"/>
    78                                         </p>
     73                                        <img class="app-icon-preview" src="{{ data.attachment.sizes.full.url }}" alt="<?php esc_attr_e( 'Preview as an app icon' ); ?>"/>
    7974                                </div>
    8075                        </div>
    8176                </div>