Skip to content

Commit 6ea221b

Browse files
committed
Administration: Consistent styling for site icon setting.
In General Settings, set the `remove-site-icon` class in initial state and add border for clearer control boundaries. Props sabernhardt, noruzzaman, madhavishah01, huzaifaalmesbah, hmbashar, shailu25, joedolson. Fixes #64736. git-svn-id: https://develop.svn.wordpress.org/trunk@61770 602fd350-edb4-49c9-b593-d223f7449a82
1 parent 33f013a commit 6ea221b

2 files changed

Lines changed: 2 additions & 2 deletions

File tree

src/wp-admin/css/site-icon.css

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -181,7 +181,7 @@
181181
button.reset.remove-site-icon {
182182
color: var(--site-icon-removal);
183183
text-decoration: none;
184-
border-color: transparent;
184+
border-color: currentColor;
185185
box-shadow: none;
186186
background: transparent;
187187
}

src/wp-admin/options-general.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -202,7 +202,7 @@ class="<?php echo esc_attr( $classes_for_button ); ?>"
202202
<button
203203
id="js-remove-site-icon"
204204
type="button"
205-
<?php echo has_site_icon() ? 'class="button button-secondary reset remove-site-icon"' : 'class="button button-secondary reset hidden"'; ?>
205+
<?php echo has_site_icon() ? 'class="button button-secondary reset remove-site-icon"' : 'class="button button-secondary reset remove-site-icon hidden"'; ?>
206206
>
207207
<?php _e( 'Remove Site Icon' ); ?>
208208
</button>

0 commit comments

Comments
 (0)