WordPress.org

Make WordPress Core

Ticket #31303: 31303.3.diff

File 31303.3.diff, 12.7 KB (added by westonruter, 7 years ago)

https://github.com/xwp/wordpress-develop/pull/73/files

  • src/wp-admin/customize.php

    diff --git src/wp-admin/customize.php src/wp-admin/customize.php
    index d6cec32..04046b6 100644
    do_action( 'customize_controls_print_scripts' ); 
    127127                        ?>
    128128                        <span class="spinner"></span>
    129129                        <a class="customize-controls-preview-toggle" href="#">
    130                                 <span class="controls"><?php _e( 'Customize' ); ?></span>
    131                                 <span class="preview"><?php _e( 'Preview' ); ?></span>
     130                                <span class="controls"><?php esc_html_e( 'Customize' ); ?></span>
     131                                <span class="preview"><?php esc_html_e( 'Preview' ); ?></span>
    132132                        </a>
    133133                        <a class="customize-controls-close" href="<?php echo esc_url( $return ); ?>">
    134                                 <span class="screen-reader-text"><?php _e( 'Cancel' ); ?></span>
     134                                <span class="screen-reader-text"><?php esc_html_e( 'Cancel' ); ?></span>
    135135                        </a>
    136                         <span class="control-panel-back" tabindex="-1"><span class="screen-reader-text"><?php _e( 'Back' ); ?></span></span>
     136                        <span class="control-panel-back" tabindex="-1"><span class="screen-reader-text"><?php esc_html_e( 'Back' ); ?></span></span>
    137137                </div>
    138138
    139139                <div id="widgets-right"><!-- For Widget Customizer, many widgets try to look for instances under div#widgets-right, so we have to add that ID to a container div in the Customizer for compat -->
    do_action( 'customize_controls_print_scripts' ); 
    141141                        <div id="customize-info" class="accordion-section">
    142142                                <div class="accordion-section-title" aria-label="<?php esc_attr_e( 'Customizer Options' ); ?>" tabindex="0">
    143143                                        <span class="preview-notice"><?php
    144                                                 echo sprintf( __( 'You are customizing %s' ), '<strong class="theme-name site-title">' . get_bloginfo( 'name' ) . '</strong>' );
     144                                                echo sprintf( esc_html__( 'You are customizing %s' ), '<strong class="theme-name site-title">' . get_bloginfo( 'name', 'display' ) . '</strong>' );
    145145                                        ?></span>
    146146                                </div>
    147147                                <div class="accordion-section-content"><?php
    148                                         echo __( 'The Customizer allows you to preview changes to your site before publishing them. You can also navigate to different pages on your site to preview them.' );
     148                                        esc_html_e( 'The Customizer allows you to preview changes to your site before publishing them. You can also navigate to different pages on your site to preview them.' );
    149149                                ?></div>
    150150                        </div>
    151151
    do_action( 'customize_controls_print_scripts' ); 
    156156                </div>
    157157
    158158                <div id="customize-footer-actions" class="wp-full-overlay-footer">
    159                         <a href="#" class="collapse-sidebar button-secondary" title="<?php esc_attr_e('Collapse Sidebar'); ?>">
     159                        <a href="#" class="collapse-sidebar button-secondary" title="<?php esc_attr_e( 'Collapse Sidebar' ); ?>">
    160160                                <span class="collapse-sidebar-arrow"></span>
    161                                 <span class="collapse-sidebar-label"><?php _e('Collapse'); ?></span>
     161                                <span class="collapse-sidebar-label"><?php esc_html_e( 'Collapse' ); ?></span>
    162162                        </a>
    163163                </div>
    164164        </form>
  • src/wp-admin/includes/theme.php

    diff --git src/wp-admin/includes/theme.php src/wp-admin/includes/theme.php
    index 3ab1732..dd0cf36 100644
    function customize_themes_print_templates() { 
    498498                <div class="theme-backdrop"></div>
    499499                <div class="theme-wrap">
    500500                        <div class="theme-header">
    501                                 <button type="button" class="left dashicons dashicons-no"><span class="screen-reader-text"><?php _e( 'Show previous theme' ); ?></span></button>
    502                                 <button type="button" class="right dashicons dashicons-no"><span class="screen-reader-text"><?php _e( 'Show next theme' ); ?></span></button>
    503                                 <button type="button" class="close dashicons dashicons-no"><span class="screen-reader-text"><?php _e( 'Close details dialog' ); ?></span></button>
     501                                <button type="button" class="left dashicons dashicons-no"><span class="screen-reader-text"><?php esc_html_e( 'Show previous theme' ); ?></span></button>
     502                                <button type="button" class="right dashicons dashicons-no"><span class="screen-reader-text"><?php esc_html_e( 'Show next theme' ); ?></span></button>
     503                                <button type="button" class="close dashicons dashicons-no"><span class="screen-reader-text"><?php esc_html_e( 'Close details dialog' ); ?></span></button>
    504504                        </div>
    505505                        <div class="theme-about">
    506506                                <div class="theme-screenshots">
    function customize_themes_print_templates() { 
    513513
    514514                                <div class="theme-info">
    515515                                        <# if ( data.active ) { #>
    516                                                 <span class="current-label"><?php _e( 'Current Theme' ); ?></span>
     516                                                <span class="current-label"><?php esc_html_e( 'Current Theme' ); ?></span>
    517517                                        <# } #>
    518                                         <h3 class="theme-name">{{{ data.name }}}<span class="theme-version"><?php printf( __( 'Version: %s' ), '{{ data.version }}' ); ?></span></h3>
    519                                         <h4 class="theme-author"><?php printf( __( 'By %s' ), '{{{ data.authorAndUri }}}' ); ?></h4>
     518                                        <h3 class="theme-name">{{ data.name }}<span class="theme-version"><?php printf( esc_html__( 'Version: %s' ), '{{ data.version }}' ); ?></span></h3>
     519                                        <h4 class="theme-author"><?php printf( esc_html__( 'By %s' ), '{{{ data.authorAndUri }}}' ); ?></h4>
    520520                                        <p class="theme-description">{{{ data.description }}}</p>
    521521
    522522                                        <# if ( data.parent ) { #>
    523                                                 <p class="parent-theme"><?php printf( __( 'This is a child theme of %s.' ), '<strong>{{{ data.parent }}}</strong>' ); ?></p>
     523                                                <p class="parent-theme"><?php printf( esc_html__( 'This is a child theme of %s.' ), '<strong>{{ data.parent }}</strong>' ); ?></p>
    524524                                        <# } #>
    525525
    526526                                        <# if ( data.tags ) { #>
    527                                                 <p class="theme-tags"><span><?php _e( 'Tags:' ); ?></span> {{ data.tags }}</p>
     527                                                <p class="theme-tags"><span><?php esc_html_e( 'Tags:' ); ?></span> {{ data.tags }}</p>
    528528                                        <# } #>
    529529                                </div>
    530530                        </div>
    function customize_themes_print_templates() { 
    532532                        <div class="theme-actions">
    533533                                <# if ( ! data.active ) { #>
    534534                                        <div class="inactive-theme">
    535                                                 <a href="<?php echo add_query_arg( 'theme', '{{ data.id }}', remove_query_arg( 'theme' ) ); ?>" target="_top" class="button button-primary"><?php _e( 'Live Preview' ); ?></a>
     535                                                <a href="<?php echo add_query_arg( 'theme', '{{ data.id }}', remove_query_arg( 'theme' ) ); ?>" target="_top" class="button button-primary"><?php esc_html_e( 'Live Preview' ); ?></a>
    536536                                        </div>
    537537                                <# } #>
    538538                        </div>
  • src/wp-admin/js/customize-controls.js

    diff --git src/wp-admin/js/customize-controls.js src/wp-admin/js/customize-controls.js
    index d95287c..ac2af19 100644
     
    814814                 * @param {Object}   theme
    815815                 */
    816816                showDetails: function ( theme, callback ) {
    817                         var section = this;
     817                        var section = this,
     818                                overlay = section.overlay;
    818819                        callback = callback || function(){};
    819820                        section.currentTheme = theme.id;
    820                         section.overlay.html( section.template( theme ) )
    821                                        .fadeIn( 'fast' )
    822                                        .focus();
     821                        overlay.html( section.template( theme ) );
     822                        overlay.fadeIn( 'fast' );
     823                        overlay.focus();
    823824                        $( 'body' ).addClass( 'modal-open' );
    824825                        section.containFocus( section.overlay );
    825826                        section.updateLimits();
     
    18001801                 * @since 4.2.0
    18011802                 */
    18021803                filter: function( term ) {
    1803                         var control = this,
    1804                             haystack = control.params.theme.name + ' ' +
    1805                                            control.params.theme.description + ' ' +
    1806                                            control.params.theme.tags + ' ' +
    1807                                            control.params.theme.author;
    1808                         haystack = haystack.toLowerCase().replace( '-', ' ' );
     1804                        var control = this, haystack, haystackParts;
     1805                        haystackParts = _.values( _.pick( control.params.theme, [ 'name', 'description', 'tags', 'author' ] ) );
     1806                        haystack = haystackParts.join( ' ' ).toLowerCase().replace( '-', ' ' );
    18091807                        if ( -1 !== haystack.search( term ) ) {
    18101808                                control.activate();
    18111809                        } else {
  • src/wp-includes/class-wp-customize-control.php

    diff --git src/wp-includes/class-wp-customize-control.php src/wp-includes/class-wp-customize-control.php
    index 4447bc2..6d972f9 100644
    class WP_Customize_Theme_Control extends WP_Customize_Control { 
    11581158                        <# } else { #>
    11591159                                <div class="theme-screenshot blank"></div>
    11601160                        <# } #>
    1161                         <span class="more-details" id="{{ data.theme.id }}-action"><?php _e( 'Theme Details' ); ?></span>
    1162                         <div class="theme-author"><?php printf( __( 'By %s' ), '{{ data.theme.author }}' ); ?></div>
     1161                        <span class="more-details" id="{{ data.theme.id }}-action"><?php esc_html_e( 'Theme Details' ); ?></span>
     1162                        <div class="theme-author"><?php printf( esc_html__( 'By %s' ), '{{ data.theme.author }}' ); ?></div>
    11631163
    11641164                        <# if ( data.theme.active ) { #>
    1165                                 <h3 class="theme-name" id="{{ data.theme.id }}-name"><span><?php _ex( 'Previewing:', 'theme' ); ?></span> {{ data.theme.name }}</h3>
     1165                                <h3 class="theme-name" id="{{ data.theme.id }}-name"><span><?php echo esc_html( _x( 'Previewing:', 'theme' ) ); ?></span> {{ data.theme.name }}</h3>
    11661166                        <# } else { #>
    11671167                                <h3 class="theme-name" id="{{ data.theme.id }}-name">{{ data.theme.name }}</h3>
    11681168                        <# } #>
    11691169
    11701170                        <# if ( ! data.theme.active ) { #>
    11711171                                <div class="theme-actions">
    1172                                         <a class="button" href="<?php echo add_query_arg( 'theme', '{{ data.theme.id }}', remove_query_arg( 'theme' ) ); ?>" target="_top"><?php _e( 'Live Preview' ); ?></a>
     1172                                        <a class="button" href="<?php echo add_query_arg( 'theme', '{{ data.theme.id }}', remove_query_arg( 'theme' ) ); ?>" target="_top"><?php esc_html_e( 'Live Preview' ); ?></a>
    11731173                                </div>
    11741174                        <# } #>
    11751175                </div>
    class WP_Customize_New_Theme_Control extends WP_Customize_Control { 
    11921192         * @since 4.2.0
    11931193         */
    11941194        public function render() {
    1195                 if ( is_multisite() || ! current_user_can( 'install_themes') ) {
     1195                if ( is_multisite() || ! current_user_can( 'install_themes' ) ) {
    11961196                        return;
    11971197                }
    11981198                ?>
    11991199                <div class="theme add-new-theme">
    1200                         <a href="<?php echo admin_url( 'theme-install.php' ); ?>" target="_top">
     1200                        <a href="<?php echo esc_url( admin_url( 'theme-install.php' ) ); ?>" target="_top">
    12011201                                <div class="theme-screenshot">
    12021202                                        <span></span>
    12031203                                </div>
    1204                                 <h3 class="theme-name"><?php _e( 'Add New Theme' ); ?></h3>
     1204                                <h3 class="theme-name"><?php esc_html_e( 'Add New Theme' ); ?></h3>
    12051205                        </a>
    12061206                </div>
    12071207                <?php
  • src/wp-includes/class-wp-customize-manager.php

    diff --git src/wp-includes/class-wp-customize-manager.php src/wp-includes/class-wp-customize-manager.php
    index ffdb3f3..be14114 100644
    final class WP_Customize_Manager { 
    11241124                /* Themes */
    11251125
    11261126                $this->add_section( new WP_Customize_Themes_Section( $this, 'themes', array(
    1127                         'title' => sprintf( __( 'Theme: %s' ), $this->theme()->display('Name') ),
     1127                        'title' => sprintf( __( 'Theme: %s' ), $this->theme()->display( 'Name' ) ),
    11281128                        'capability' => 'switch_themes',
    11291129                        'priority' => 0,
    11301130                ) ) );
  • src/wp-includes/class-wp-customize-section.php

    diff --git src/wp-includes/class-wp-customize-section.php src/wp-includes/class-wp-customize-section.php
    index b824173..01a4d2e 100644
    class WP_Customize_Themes_Section extends WP_Customize_Section { 
    343343                <li id="accordion-section-<?php echo esc_attr( $this->id ); ?>" class="<?php echo esc_attr( $classes ); ?>">
    344344                        <h3 class="accordion-section-title" tabindex="0">
    345345                                <?php echo esc_html( $this->title ); ?>
    346                                 <span class="screen-reader-text"><?php _e( 'Press return or enter to expand' ); ?></span>
     346                                <span class="screen-reader-text"><?php esc_html_e( 'Press return or enter to expand' ); ?></span>
    347347                        </h3>
    348348                        <span class="control-panel-back themes-panel-back" tabindex="-1"><span class="screen-reader-text"><?php _e( 'Back' ); ?></span></span>
    349349                        <div class="customize-themes-panel control-panel-content themes-php">
    350350                                <h2><?php esc_html_e( 'Themes' ); ?>
    351351                                        <span class="title-count theme-count"><?php echo count( $this->controls ) - 1; ?></span>
    352352                                <?php if ( ! is_multisite() && current_user_can( 'install_themes' ) ) : ?>
    353                                         <a href="<?php echo admin_url( 'theme-install.php' ); ?>" target="_top" class="add-new-h2"><?php echo esc_html_x( 'Add New', 'Add new theme' ); ?></a>
     353                                        <a href="<?php echo esc_url( admin_url( 'theme-install.php' ) ); ?>" target="_top" class="add-new-h2"><?php echo esc_html_x( 'Add New', 'Add new theme' ); ?></a>
    354354                                <?php endif; ?>
    355355                                </h2>
    356356                                <div class="theme-overlay" tabindex="0" role="dialog" aria-label="<?php esc_attr_e( 'Theme details' ); ?>"></div>
    357357                                <div id="customize-container"></div>
    358358                                <?php if ( 6 < count( $this->controls ) ) : ?>
    359359                                        <p><label for="themes-filter">
    360                                                 <span class="screen-reader-text"><?php _e( 'Search installed themes...' ); ?></span>
     360                                                <span class="screen-reader-text"><?php esc_html_e( 'Search installed themes...' ); ?></span>
    361361                                                <input type="search" id="themes-filter" placeholder="<?php esc_attr_e( 'Search installed themes...' ); ?>" />
    362362                                        </label></p>
    363363                                <?php endif; ?>