Make WordPress Core

Ticket #59372: 59372.3.2.diff

File 59372.3.2.diff, 6.0 KB (added by mikinc860, 8 months ago)

@petitphp, I have tested in my local with the same configuration that you have used for testing and I found that the button does not have the blue border but it is a focus/active state of the button.I have updated the box-shadow in the new patch.

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

    diff --git a/src/wp-admin/css/customize-controls.css b/src/wp-admin/css/customize-controls.css
    index 4bd63832f9..3548f12e9c 100644
    a b p.customize-section-description { 
    23172317
    23182318.wp-customizer .theme-overlay .theme-actions {
    23192319        text-align: right; /* Because there're only one or two actions, match the UI pattern of media modals and right-align the action. */
    2320         padding: 10px 25px;
     2320        padding: 10px 25px 5px;
    23212321        background: #f0f0f1;
    23222322        border-top: 1px solid #dcdcde;
    23232323}
    p.customize-section-description { 
    23262326        margin-left: 8px;
    23272327}
    23282328
    2329 .control-panel-themes .theme-actions .delete-theme {
    2330         left: 15px; /* these override themes.css on mobile */
    2331         right: auto;
    2332         bottom: auto;
    2333         position: absolute;
    2334 }
    2335 
    23362329.modal-open .in-themes-panel #customize-controls .wp-full-overlay-sidebar-content {
    23372330        overflow: visible; /* Prevent the top-level Customizer controls from becoming visible when elements on the right of the details modal are focused. */
    23382331}
  • src/wp-admin/css/themes.css

    diff --git a/src/wp-admin/css/themes.css b/src/wp-admin/css/themes.css
    index c1a1eb22dd..207f885417 100644
    a b body.folded .theme-browser ~ .theme-overlay .theme-wrap { 
    506506        z-index: 30;
    507507        box-sizing: border-box;
    508508        border-top: 1px solid #f0f0f1;
     509        display: flex;
     510        justify-content: center;
     511        gap: 5px;
    509512}
    510513
    511 .theme-overlay .theme-actions a {
     514.theme-overlay .theme-actions .button {
    512515        margin-right: 5px;
    513516        margin-bottom: 5px;
    514517}
    body.folded .theme-browser ~ .theme-overlay .theme-wrap { 
    521524
    522525.broken-themes a.delete-theme,
    523526.theme-overlay .theme-actions .delete-theme {
    524         color: #d63638;
     527        color: #b32d2e;
    525528        text-decoration: none;
    526529        border-color: transparent;
    527530        box-shadow: none;
    528531        background: transparent;
    529532}
    530533
    531 .theme-overlay .theme-actions .delete-theme {
    532         position: absolute;
    533         right: 10px;
    534         bottom: 5px;
    535 }
    536 
    537534.broken-themes a.delete-theme:hover,
    538535.broken-themes a.delete-theme:focus,
    539536.theme-overlay .theme-actions .delete-theme:hover,
    540537.theme-overlay .theme-actions .delete-theme:focus {
    541         background: #d63638;
     538        background: #b32d2e;
    542539        color: #fff;
    543         border-color: #d63638;
     540        border-color: #b32d2e;
     541        box-shadow: 0 0 0 1px #b32d2e;
    544542}
    545543
    546544.theme-overlay .theme-actions .active-theme,
  • src/wp-admin/includes/theme.php

    diff --git a/src/wp-admin/includes/theme.php b/src/wp-admin/includes/theme.php
    index 6bb450ee1b..ef3aad5bbe 100644
    a b function customize_themes_print_templates() { 
    10581058                                <# if ( data.active ) { #>
    10591059                                        <button type="button" class="button button-primary customize-theme"><?php _e( 'Customize' ); ?></button>
    10601060                                <# } else if ( 'installed' === data.type ) { #>
    1061                                         <?php if ( current_user_can( 'delete_themes' ) ) { ?>
    1062                                                 <# if ( data.actions && data.actions['delete'] ) { #>
    1063                                                         <a href="{{{ data.actions['delete'] }}}" data-slug="{{ data.id }}" class="button button-secondary delete-theme"><?php _e( 'Delete' ); ?></a>
    1064                                                 <# } #>
    1065                                         <?php } ?>
    1066 
     1061                                        <div class="theme-inactive-actions">
    10671062                                        <# if ( data.blockTheme ) { #>
    10681063                                                <?php
    10691064                                                        /* translators: %s: Theme name. */
    function customize_themes_print_templates() { 
    10791074                                                        <button class="button button-primary disabled"><?php _e( 'Live Preview' ); ?></button>
    10801075                                                <# } #>
    10811076                                        <# } #>
     1077                                        </div>
     1078                                        <?php if ( current_user_can( 'delete_themes' ) ) { ?>
     1079                                                <# if ( data.actions && data.actions['delete'] ) { #>
     1080                                                        <a href="{{{ data.actions['delete'] }}}" data-slug="{{ data.id }}" class="button button-secondary delete-theme"><?php _e( 'Delete' ); ?></a>
     1081                                                <# } #>
     1082                                        <?php } ?>
    10821083                                <# } else { #>
    10831084                                        <# if ( data.compatibleWP && data.compatiblePHP ) { #>
    10841085                                                <button type="button" class="button theme-install" data-slug="{{ data.id }}"><?php _e( 'Install' ); ?></button>
  • src/wp-admin/themes.php

    diff --git a/src/wp-admin/themes.php b/src/wp-admin/themes.php
    index d5d8c150e3..80486748f4 100644
    a b function wp_theme_auto_update_setting_template() { 
    12211221                                        /* translators: %s: Theme name. */
    12221222                                        $aria_label = sprintf( _x( 'Activate %s', 'theme' ), '{{ data.name }}' );
    12231223                                        ?>
    1224                                         <# if ( data.actions.activate ) { #>
    1225                                                 <a href="{{{ data.actions.activate }}}" class="button activate" aria-label="<?php echo esc_attr( $aria_label ); ?>"><?php _e( 'Activate' ); ?></a>
    1226                                         <# } #>
    12271224                                        <# if ( ! data.blockTheme ) { #>
    12281225                                                <a href="{{{ data.actions.customize }}}" class="button button-primary load-customize hide-if-no-customize"><?php _e( 'Live Preview' ); ?></a>
    12291226                                        <# } #>
     1227                                        <# if ( data.actions.activate ) { #>
     1228                                                <a href="{{{ data.actions.activate }}}" class="button activate" aria-label="<?php echo esc_attr( $aria_label ); ?>"><?php _e( 'Activate' ); ?></a>
     1229                                        <# } #>
    12301230                                <# } else { #>
    12311231                                        <?php
    12321232                                        /* translators: %s: Theme name. */
    12331233                                        $aria_label = sprintf( _x( 'Cannot Activate %s', 'theme' ), '{{ data.name }}' );
    12341234                                        ?>
    1235                                         <# if ( data.actions.activate ) { #>
    1236                                                 <a class="button disabled" aria-label="<?php echo esc_attr( $aria_label ); ?>"><?php _ex( 'Cannot Activate', 'theme' ); ?></a>
    1237                                         <# } #>
    12381235                                        <# if ( ! data.blockTheme ) { #>
    12391236                                                <a class="button button-primary hide-if-no-customize disabled"><?php _e( 'Live Preview' ); ?></a>
    12401237                                        <# } #>
     1238                                        <# if ( data.actions.activate ) { #>
     1239                                                <a class="button disabled" aria-label="<?php echo esc_attr( $aria_label ); ?>"><?php _ex( 'Cannot Activate', 'theme' ); ?></a>
     1240                                        <# } #>
    12411241                                <# } #>
    12421242                        </div>
    12431243
  • src/wp-includes/css/buttons.css

    diff --git a/src/wp-includes/css/buttons.css b/src/wp-includes/css/buttons.css
    index 6da6da51e0..787efacab6 100644
    a b TABLE OF CONTENTS: 
    385385                vertical-align: inherit;
    386386        }
    387387
     388        .wp-customizer .theme-overlay .theme-actions .button {
     389                margin-bottom: 5px;
     390        }
     391
    388392        .media-modal-content .media-toolbar-primary .media-button {
    389393                margin-top: 10px;
    390394                margin-left: 5px;