graphettion

Works like a charm. Thank you Frank!

Yay! That seemed to work. Thank you @dcooney!

Hrm, perhaps I should keep trying (i.e., copy/paste the same code and hit the “Save changes and empty cache” button?

Yup, the backslashes were the culprit. Works like a charm now. Thank you!

It’s not a big issue. I was just wondering if you could do it.

I did. I even used double quotes instead of single ones. ˉ\_(ツ)_/ˉ

I tried replacing the CSS style, but didn’t work. Am I missing something?:

// Replace CSS Styles
function ao_override_css_after_minify($cssIn) {
  $style1 = '.button.mfp-close:before{font-family:\'Glyphicons Halflings\';content:\'\e014\';font-size:4rem;color:#fff}';
  $style2 = '.button.mfp-close:before{font-family:\'Glyphicons Halflings\';content:\'\e014\';font-size:4rem;color:#333}';
  $cssOut = str_replace($style1, $style2, $cssIn);
  return $cssOut;
}
add_filter('autoptimize_css_after_minify','ao_override_css_after_minify',10,1);

• This reply was modified 7 years, 1 month ago by graphettion.

Errr. It was the single quote thing. Bleh!

I also just want to thank you Frank and everyone who has helped with the AO plugin. You guys are not only great with making websites on wordpress painlessly easy and faster, but you also open source it. Love it.

Thanks again Frank! ??

I ran that code and it didn’t work. It went back to media=”screen” and still shows two css files.

This code works for adding both CSS files as media=”all”:

function replace_style_media_type($htmlIn) {
    $htmlOut = str_replace('media="screen"', 'media="all"', $htmlIn);
    return $htmlOut;
}
add_filter('autoptimize_html_after_minify','replace_style_media_type');

But still shows two css files. :/

I replaced all instances of media=”screen” with media=”all” and it still shows two .css files. Am I missing something?

// Replace media="screen" with media="all"
function replace_style_media_type($htmlIn) {
    $htmlOut = str_replace('media="screen"', 'media="all"', $htmlIn);
    return $htmlOut;
}
add_filter('autoptimize_html_after_minify','replace_style_media_type');

Awesome! Thank you for the speedy response. So you’re saying they should all be either media=”all” or media=”screen”, not both. Correct? If so, which do you think would be better or does it matter?

Ex:
<link href=”path/to/css/file/style-1.css” rel=”stylesheet” type=”text/css” media=”all”>
<link href=”path/to/css/file/style-2.css” rel=”stylesheet” type=”text/css” media=”all”>

Or

Ex:
<link href=”path/to/css/file/style-1.css” rel=”stylesheet” type=”text/css” media=”screen”>
<link href=”path/to/css/file/style-2.css” rel=”stylesheet” type=”text/css” media=”screen”>

https://acceleratedrpc.com/wp-content/cache/autoptimize/css/autoptimize_2d96c469c4c39236b136f849982302d0.css

https://acceleratedrpc.com/wp-content/cache/autoptimize/css/autoptimize_c034505fb01914adad73c64820ef9828.css

No problem! Thank you for the quick reply!

Awesome, I’ll look into that. I know this was kind of off-topic, but it’s been bugging me for a while and thought I would ask because I’m sure you run into this kind of stuff. Anyway, you’re too awesome. Thanks again @dcooney! ??

Ah, I appreciate you looking into it. Thanks @dcooney.