Feature Queries conditions not working

Meanwhile I solved it with this filter:

/**
 * Fixes multiple feature queries issue.
 *
 * @param string $code Minified code.
 */
function prefix_autoptimize_css_after_minify( $code ) {
	return str_replace( ') or(', ') or (', $code );
}
add_filter( 'autoptimize_css_after_minify', 'prefix_autoptimize_css_after_minify' );

interesting, thanks for reporting @ninetienne! what other operators can go in there? and, I guess? any others?

I don’t think there are any others, except for or and and.

Thanks for looking into this.

ah, there’s also not apparently; https://developer.mozilla.org/en-US/docs/Web/CSS/%40supports

I haven’t checked @media queries: https://developer.mozilla.org/en-US/docs/Web/CSS/@media There’s additional only operator.

• This reply was modified 7 years, 5 months ago by ninetienne.

The @supports at-rule associates a block of statements with a supports condition. The supports condition consists of one or more name-value pairs combined by conjunctions (and), disjunctions (or), and/or negations (not). Precedence of operators can be defined with parentheses.

https://developer.mozilla.org/en-US/docs/Web/CSS/@supports

OK, until I fix this in the CSS minifier below code snippet based on @ninetienne ‘s can be used as a generic workaround (not tested thoroughly yet though);

/**
 * Fixes multiple feature queries issue.
 *
 * @param string $code Minified code.
 */
function prefix_autoptimize_css_after_minify( $code ) {
	return preg_replace('#[\s|\)](and|or|any|not)(\()#',' $1 (', $code );
}
add_filter( 'autoptimize_css_after_minify', 'prefix_autoptimize_css_after_minify' );

• This reply was modified 7 years, 5 months ago by Optimizing Matters. Reason: better regex

hmm, above regex is not OK; given [\s|\)] it “eats” a closing bracket preceding the and|or|any|not … this is safer;

return preg_replace('#\s(and|or|any|not)(\()#',' $1 (', $code );

OK, this commit should fix the problem, could you check that out @ninetienne and @carasmo ?

confirmed fixed by @carasmo on github

Sorry, I’ll be away for a couple of days and won’t be able to test the commit ??

Thanks for fixing the issue!