Changeset 1467

Timestamp:

04/01/2015 12:59:30 AM (10 years ago)

Author:

obenland

Message:

WP.org Themes: Further optimize the Directory's load time.

This will cut down on the amount of requests for core-bundled JS resources, and
half the size of JS app. I thought about minifying style.css as well but
decided to hold off on that for now. Savings are not very big and it'll
probably be more confusing than helpful.

With the project switching to maintenance mode, we can now also set a saneer
version string.

Location:

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

Files:

• functions.php (modified) (2 diffs)
• js/theme.min.js (added)

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

@@ -38 +38 @@
  */
 function wporg_themes_scripts() {
-    wp_enqueue_style( 'wporg-themes', get_stylesheet_uri(), array(), filemtime( __DIR__ . '/style.css' ) );
+    $script_debug = defined( 'SCRIPT_DEBUG' ) && SCRIPT_DEBUG;
+    $suffix       = $script_debug ? '' : '.min';
+
+    // Concatenates core scripts when possible.
+    if ( ! $script_debug ) {
+        $GLOBALS['concatenate_scripts'] = true;
+    }
+
+    wp_enqueue_style( 'wporg-themes', get_stylesheet_uri(), array(), '1' );
 
     if ( ! is_singular( 'page' ) ) {
         wp_enqueue_script( 'google-jsapi', '//www.google.com/jsapi', array( 'jquery' ), null, true );
-        wp_enqueue_script( 'wporg-theme', get_template_directory_uri() . '/js/theme.js', array( 'wp-backbone' ), filemtime( __DIR__ . '/js/theme.js' ), true );
+        wp_enqueue_script( 'wporg-theme', get_template_directory_uri() . "/js/theme{$suffix}.js", array( 'wp-backbone' ), '1', true );
 
         wp_localize_script( 'wporg-theme', '_wpThemeSettings', array(
@@ -66 +74 @@
 
     // No emoji support needed.
-    remove_action( 'wp_print_styles','print_emoji_styles' );
-    wp_dequeue_script( 'emoji' );
+    remove_action( 'wp_head', 'print_emoji_detection_script', 7 );
+    remove_action( 'wp_print_styles', 'print_emoji_styles' );
 
     // No Jetpack styles needed.