Structural optimization fails on browser hacks

[asapach]

Reproducible on Bootstrap 3.3.5:

.caret {
  border-top: 4px dashed;
  border-top: 4px solid \9;
}

Optimized:

.caret {
  border-top: 4px solid \9;
}

The IE8 browser hack breaks the other declaration.

[bc-jasond]

@asapach you beat me to it... I'm seeing this too

[jmcmichael]

Same here. Finally tracked the missing Bootstrap caret to CSSO. (Fabulous package otherwise!) Toggling structural optimization to false unsurprisingly solves the problem (and surprisingly results in a smaller CSS file).

[mourukoski]

I also came across this problem. As jmcmichael said, setting structural optimization to false fortunately fixes this problem.

[lahmatiy]

Fixed by #166

[tkrotoff]

+1

[tkrotoff]

@lahmatiy Sure this is fixed? I'm a strong believer in "What ain't tested, ain't working" :) and PR #166 does not include any unit test

$ ./bin/csso --version
1.3.12
$ echo ".caret { border-top: 4px dashed; border-top: 4px solid \9; }" > test.css
$ ./bin/csso test.css
.caret{border-top:4px solid \9}

The PR integration is probably the cause: look at https://github.com/css/csso/blob/v1.3.12/lib/compressor.js#L125 vs https://github.com/css/csso/pull/166/files#diff-e83fb3368ebbc864f774279d085ac1a8R124
Commit: 42da5bc#diff-e83fb3368ebbc864f774279d085ac1a8L126

[lahmatiy]

Thank you for your feedback.
Will investigate issue and add tests this time.

[lahmatiy]

I see it's another issue now. It's connected with wrong structural optimisation.
Collecting issue #241