Overly aggressive whitespace removal when property value contains var()

[paulhoadley]

This input fragment:

border: var(--bs-border-width) solid var(--bs-border-color);

gets minimised to:

border: var(--bs-border-width)solidvar(--bs-border-color);

by this if-block in CSSMinifier.simplifyQuotesAndCaps():

} else if ((this.contents.length() > 4) && (this.contents.substring(0, 4).equalsIgnoreCase("var("))) {
	this.contents = this.contents.replaceAll("\\s", "");
}

I think perhaps the original use case was pulling whitespace from within a single call to var(), perhaps to remove the whitespace from between the name and the optional default value? Who knows—I'm just guessing.

In any case, I think a reasonable fix is to simply delete this block entirely and accept the occasional missed space reduction.

[paulhoadley]

Actually, we can do marginally better. I see in test-17.css and test-18.css that we've got sample input with long runs of spaces. We can actually just substitute multiple occurrences of whitespace with a single space. This is a nice compromise.