Support web pages that use media=print for async CSS (#487)

Co-authored-by: Jack Harrison <jack@jack-harrison.co.uk> Co-authored-by: Ben Zörb <ben@sommerlaune.com>
addyosmani · Jun 5, 2021 · 03d0455 · 03d0455
1 parent 712122d
commit 03d0455
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/src/file.js b/src/file.js
@@ -376,8 +376,11 @@ function getStylesheetHrefs(file) {
 
   const stylesheets = oust.raw(file.contents.toString(), ['stylesheets', 'preload', 'styles']);
 
+  const isNotPrint = (el) =>
+    el.attr('media') !== 'print' || (Boolean(el.attr('onload')) && el.attr('onload').includes('media'));
+
   const hrefs = stylesheets
-    .filter((link) => link.$el.attr('media') !== 'print' && Boolean(link.value))
+    .filter((link) => isNotPrint(link.$el) && Boolean(link.value))
     .map((link) => {
       // support base64 encoded styles
       if (link.value.startsWith('data:')) {