YUI Compressor 2.4.8 is removing spaces for Calc property

  • Resolved moxymore

    (@moxymore)


    8 years, 3 months ago

    Hi there,

    This is a well known bug of YUI compressor which delete mandatory spaces for the Calc property.

    Ex :

    .w-1 { width : calc(6.66% – 20px); }

    will be converted into

    .w-1{width:calc(6.66%-20px);} but the thing is that it leads to an nvalid property because spaces between + – * / MUST be there.

    There is an unofficial YUI compressor 2.4.9, which fixes the issue. But when replacing the JAR associated (and renaming the version to the one present in your plugin), it just doesn’t fix the issue.

    Is this because this update doesn’t solve the calc issue, or is this because I have renamed the new yuicompressor-2.4.9.jar to yuicompressor-2.4.8.jar?

    Moreover, I find it strange that there is no compression issue when using 2.4.8.4 here : https://futtta.be/cssmin/gui/

    https://www.ads-software.com/plugins/fast-velocity-minify/

Viewing 4 replies - 1 through 4 (of 4 total)
  • Plugin Author Raul P.

    (@alignak)

    Hi!

    We are using the YUI compressor for JS minification only, but your problem seems to be CSS related.

    Replacing YUI will therefore not solve any CSS problems.

    I am using some regular expressions to minify CSS with PHP only, so that’s probably a bug on my code. I will replace the CSS minification with PHP Minify CSS code and see if it works for you.

    Plugin Author Raul P.

    (@alignak)

    Version 1.1.6, “should” have fixed this. Please update, clear your browser cache (and cloudflare, if you use it) and try again.

    Let me know if it works or not, thank you.

    Thread Starter moxymore

    (@moxymore)

    Version 1.1.6 have fixed the Calc issue. Thank you very much for the fast answer.

    PS : Can you add (in a future update) a feature for excluding folders? This can be usefull in some circumstances. The plugin autoptimize do it, so I’m sure yours will be able to do it soon ??

    Plugin Author Raul P.

    (@alignak)

    Thanks for the info.
    Excluding folders will be considered for some next version, since it should be trivial to implement.

    Marking as solved.

Viewing 4 replies - 1 through 4 (of 4 total)
  • The topic ‘YUI Compressor 2.4.8 is removing spaces for Calc property’ is closed to new replies.