Changeset 8266

Timestamp:

02/18/2019 08:20:38 AM (6 years ago)

Author:

dd32

Message:

Developer Theme: Remove the removing of dashicons.

The global header no longer enqueues dashicons if the WordPress has enqueued it.

H/t sebastienserre, daveshine.

See #4179.
Fixes #4182.

File:

• sites/trunk/wordpress.org/public_html/wp-content/themes/pub/wporg-developer/functions.php (modified) (2 diffs)

sites/trunk/wordpress.org/public_html/wp-content/themes/pub/wporg-developer/functions.php

@@ -132 +132 @@
 add_action( 'init', __NAMESPACE__ . '\\init' );
 add_action( 'widgets_init', __NAMESPACE__ . '\\widgets_init' );
-add_action( 'wp_default_styles', __NAMESPACE__ . '\\remove_dashicons_styles', 11 );
 
 function init() {
@@ -156 +155 @@
 
     add_filter( 'syntaxhighlighter_htmlresult', __NAMESPACE__ . '\\syntaxhighlighter_htmlresult' );
-}
-
-/**
- * Makes 'dashicons' a noop script on the frontend to prevent getting enqueued.
- *
- * The global W.org header already includes the dashicons stylesheet.
- *
- * @param WP_Scripts $scripts WP_Scripts object.
- */
-function remove_dashicons_styles( &$scripts ) {
-    if ( ! is_admin() ) {
-        $scripts->remove( 'dashicons' );
-        $scripts->add( 'dashicons', '' );
-    }
 }