Skip to content

Commit 02b9161

Browse files
committed
Icons library: Register all icons found in manifest
As of WordPress/gutenberg#75684, icons' manifest.php file only lists "public" icons, and in the process the now-redundant `"public" => true` flag was dropped. Accordingly, this removes the check for the `public` flag from the icon registry's registration logic. Follow-up to [61674]. Props mcsf, wildworks, ellatrix. Fixes #64651. git-svn-id: https://develop.svn.wordpress.org/trunk@61675 602fd350-edb4-49c9-b593-d223f7449a82
1 parent 5b7a3ad commit 02b9161

1 file changed

Lines changed: 1 addition & 6 deletions

File tree

src/wp-includes/class-wp-icons-registry.php

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -38,8 +38,7 @@ class WP_Icons_Registry {
3838
*
3939
* These icons are defined in @wordpress/packages (Gutenberg repository) as
4040
* SVG files and as entries in a single manifest file. On init, the
41-
* registry is loaded with those icons in the manifest which are marked
42-
* `public`.
41+
* registry is loaded with those icons listed in the manifest.
4342
*/
4443
private function __construct() {
4544
$icons_directory = __DIR__ . '/icons/';
@@ -77,10 +76,6 @@ private function __construct() {
7776
return;
7877
}
7978

80-
if ( ! ( $icon_data['public'] ?? false ) ) {
81-
continue;
82-
}
83-
8479
$this->register(
8580
'core/' . $icon_name,
8681
array(

0 commit comments

Comments
 (0)