bug: custom CSS sass translator fails on @supports directive

  • Resolved nealeyoung

    (@nealeyoung)


    10 years, 12 months ago

    Hi, could you add support for the css “@supports” directive to the sass translator (scssphp) in the custom CSS module?

    At the moment it seems to translate any such directive as follows:

    @supports (X) {
    .class1 { background-color: red }
    .class2 { background-color: red }
    }

    gets incorrectly translated to

    .class2 {background-color: red}

    https://www.ads-software.com/plugins/jetpack/

Viewing 4 replies - 1 through 4 (of 4 total)
  • Thread Starter nealeyoung

    (@nealeyoung)

    I’ve narrowed the bug down to the cstidy module, that is, the code in

    wp-content/plugins/jetpack/modules/custom-css/csstidy .

    That module is called by function Jetpack_Custom_CSS::minify, defined in

    wp-content/plugins/jetpack/modules/custom-css/custom-css.php .

    The sass processor in custom-css/custom-css/preprocessors/scss.inc.php produces the correct output, but that output is then munged by csstidy.

    As a workaround, I am patching my copy of Jetpack_Custom_CSS::minify to not call csstidy, but instead to return the css directly as passed back after preprocessing.

    Plugin Author Jeremy Herve

    (@jeherve)

    Jetpack Mechanic ??

    Thanks for the report! I took note of the issue here, and I’ll post again here as soon as the problem is fixed.

    Plugin Contributor Richard Archambault

    (@richardmtl)

    Hi! Just wanted to mention that your example works for me now using Jetpack 2.8, so I have closed the Trac ticket. If you have further issues, feel free to open a new Issue on Github:

    https://github.com/Automattic/jetpack/issues/new

    and have a look at our Contribute document if you want, as well:

    https://jetpack.me/contribute/

    Thanks!

    Thread Starter nealeyoung

    (@nealeyoung)

    Still broken for me in 2.8, so I opened a new ticket (same as the old) at https://github.com/Automattic/jetpack/issues/225

Viewing 4 replies - 1 through 4 (of 4 total)
  • The topic ‘bug: custom CSS sass translator fails on @supports directive’ is closed to new replies.