Make WordPress Core

Ticket #35943: 35943.diff

File 35943.diff, 2.7 KB (added by celloexpressions, 10 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>