"selector-newline" false warning

[XhmikosR]

Originally from here.

@mahonnaise @nschonni

This seems to yield wrong results for >, + etc selector.

.latest-news > h1 {
    padding-left: 36px;
}

[XhmikosR]

I think that patch should be reverted if not fixed before 0.11 is out; it's a regression.

The patch seems completely wrong to me from the beginning, but I wasn't able to fix it myself yet.

[nschonni]

Thanks! I've got a reproducable failure with

        "a newline between selectors with child selectors should not result in a warning": function () {
            expectPass(".foo + .bar,\n.foo > .bar{}");

Now if the fix was just that easy 😉 I've tagged this as part of the 0.11.0 release, and you are right that if it cannot be resolved it will need to be reverted before the release.

[XhmikosR]

The problem is that the patch is wrong in its essence. I mean, currentLine and previousLine are both 1.

We need to break the selector in its parts and check if it has any newlines in between.

[XhmikosR]

I think I fixed it, can you try this? https://github.com/XhmikosR/csslint/compare/selector-newline