Changeset 57713 for trunk/src/wp-admin/includes/options.php

Timestamp:

02/26/2024 08:16:09 PM (3 years ago)

Author:

jorbin

Message:

Site icon: Polish up Site Icon on the general settings screen.

This fixes a number of issues, chief among them:

• Updates to the site title are reflected in the preview.
• Improve alt text for preview
• Make string describing site icon more succinct.
• Add inline documentation to JavaScript

Props kebbet, jorbin, swissspidy, afercia, mukesh27, alexstine, jameskoster, andraganescu.
Fixes #54370.

File:

• trunk/src/wp-admin/includes/options.php (modified) (2 diffs)

trunk/src/wp-admin/includes/options.php

@@ -37 +37 @@
         jQuery( function($) {
                 var $siteName = $( '#wp-admin-bar-site-name' ).children( 'a' ).first(),
+                        $siteIconPreview = $('#site-icon-preview-site-title'),
                         homeURL = ( <?php echo wp_json_encode( get_home_url() ); ?> || '' ).replace( /^(https?:\/\/)?(www\.)?/, '' );
 
@@ -48 +49 @@
 
                         $siteName.text( title );
+                        $siteIconPreview.text( title );
                 });