Changeset 43905

Timestamp:

11/16/2018 12:22:27 AM (8 years ago)

Author:

pento

Message:

Build Tools: Minify the package CSS when building.

CSS coming from the @wordpress packages wasn't being minified in the build. This had the noteable side effect of causing RTL CSS to not be loaded when SCRIPT_DEBUG was set to false.

Props websupporter.
See #45330.

Location:

branches/5.0

Files:

• Gruntfile.js (modified) (2 diffs)
• src/wp-includes/script-loader.php (modified) (2 diffs)

branches/5.0/Gruntfile.js

@@ -198 +198 @@
                                         'wp-includes/css/*.css',
                                         'wp-includes/js/mediaelement/wp-mediaelement.css'
+                                ]
+                        },
+                        dist: {
+                                expand: true,
+                                cwd: BUILD_DIR,
+                                dest: BUILD_DIR,
+                                ext: '.min.css',
+                                src: [
+                                        'wp-includes/css/dist/*/*.css'
                                 ]
                         },
@@ -940 +949 @@
                 'webpack:prod',
                 'webpack:devProdTarget',
+                'cssmin:dist',
                 'jsvalidate:build'
         ] );

branches/5.0/src/wp-includes/script-loader.php

@@ -1676 +1676 @@
         $styles->add( 'wp-editor-font', $fonts_url );
 
-        $styles->add( 'wp-block-library-theme', '/wp-includes/css/dist/block-library/theme.css' );
+        $styles->add( 'wp-block-library-theme', "/wp-includes/css/dist/block-library/theme$suffix.css" );
         $styles->add_data( 'wp-block-library-theme', 'rtl', 'replace' );
 
         $styles->add(
                 'wp-edit-blocks',
-                '/wp-includes/css/dist/block-library/editor.css',
+                "/wp-includes/css/dist/block-library/editor$suffix.css",
                 array(
                         'wp-components',
@@ -1703 +1703 @@
         foreach ( $package_styles as $package => $dependencies ) {
                 $handle  = 'wp-' . $package;
-                $path     = '/wp-includes/css/dist/' . $package . '/style.css';
+                $path     = "/wp-includes/css/dist/$package/style$suffix.css";
 
                 $styles->add( $handle, $path, $dependencies );