Changeset 57713 for trunk/src/wp-includes/script-loader.php

Timestamp:

02/26/2024 08:16:09 PM (2 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-includes/script-loader.php (modified) (1 diff)

trunk/src/wp-includes/script-loader.php

@@ -850 +850 @@
     $scripts->add( 'wp-lists', "/wp-includes/js/wp-lists$suffix.js", array( 'wp-ajax-response', 'jquery-color' ), false, 1 );
 
-    $scripts->add( 'site-icon', '/wp-admin/js/site-icon.js', array( 'jquery', 'jcrop' ), false, 1 );
+    $scripts->add( 'site-icon', '/wp-admin/js/site-icon.js', array( 'jquery' ), false, 1 );
+    $scripts->set_translations( 'site-icon' );
 
     // WordPress no longer uses or bundles Prototype or script.aculo.us. These are now pulled from an external source.