Please remove combined CSS

  • Resolved swinggraphics

    (@swinggraphics)


    9 years, 1 month ago

    I love the minify & combine CSS option! It works wonderfully on my site, …except it is, technically, making the HTML file LARGER by wrapping <!-- --> around external and internal stylesheets. I think it would be great to leave the comments in a debug mode, but render the cached HTML without it. There is no point to combining internal stylesheets with external if it leaves all of it in the HTML; this essentially DOUBLES the size of internal CSS.

    https://wordpress.org/plugins/wp-fastest-cache/

Viewing 5 replies - 1 through 5 (of 5 total)
  • Plugin Author Emre Vona

    (@emrevona)

    9 years, 1 month ago

    The removing comments feature is available in the premium version.

    Thread Starter swinggraphics

    (@swinggraphics)

    9 years, 1 month ago

    It is understandable that you would leave other comments on the page, but you should not leave the massive comment sections you create on the page. It makes no sense to move internal stylesheets to an external file AND leave that code on the page.

    Plugin Author Emre Vona

    (@emrevona)

    9 years, 1 month ago

    yes, you are right but I don’t want to change my code. You can update the file which is css-utilities.php

    Thread Starter swinggraphics

    (@swinggraphics)

    9 years, 1 month ago

    Yep, thanks, will do. For anyone else who’s interested, we edit inc/css-utilities.php

    line 67 change "<!-- <style".$out[1][$i].">".$value."</style> -->"."\n to " (one double quote)

    line 72 change "<!-- <style".$out[1][$i].">".$value."</style> -->"."\n to “ (one double quote)

    line 372 remove "<!-- ".$prevValue." -->"."\n". (note the dot)

    line 376 change "<!-- ".$prevValue." -->" to '' (two single quotes)

    Plugin Author Emre Vona

    (@emrevona)

    9 years, 1 month ago

    thanks 🙂

Viewing 5 replies - 1 through 5 (of 5 total)
  • The topic ‘Please remove combined CSS’ is closed to new replies.