Dojo CSS build removes too much whitespace

[Rob Retchless]

In IBM Rational Jazz, one of our CSS files (PaginationControl?.css) is formatted so that each class is on a separate line. Example:

.jazz-ui-PaginationControl
.off:hover
{
    color: #666666;
    text-decoration: none;
} 

.jazz-ui-PaginationControl
.on
{
    color: #3087B3;
    cursor: pointer;
} 

This is a perfectly valid way to write child selectors, but the standard dojo compression removes new lines without preserving the whitespace, with the following result:

.jazz-ui-PaginationControl.off:hover{
    color: #666666;
    text-decoration: none;
} 

.jazz-ui-PaginationControl.on{
    color: #3087B3;
    cursor: pointer;
}

The resultant compressed CSS output should be:

.jazz-ui-PaginationControl .off:hover {
    color: #666666;
    text-decoration: none;
} 

.jazz-ui-PaginationControl .on {
    color: #3087B3;
    cursor: pointer;
}

[dante]

fwiw, this (and newline-dependent css hacks) are the reason I always build with

cssOptimize=comments.keepLines

[Rawld Gill]

(In [25319]) fixed bug that cssOptimize keepLines was deleting lines and visa-versa; refs #13148