spaceBeforeBrace reports wrong column/line on multi-line selectors

[jwilsson]

Given this code:

.foo,
.bar  { // Note the two spaces
    color: red;
}

And this config:

{
    "spaceBeforeBrace": {
        "enabled": true,
        "style": "one_space"
    }
}

it gives the following report:

Warning: test.less: line 1, col 11, spaceBeforeBrace: Opening curly brace should be preceded by one space.

Which is clearly wrong.

[shellscape]

Tricky, that. The selector technically does start on line 1. Do we want to report the line of the selector that owns the brace or the line the brace is on?

[jwilsson]

Hmm, on one hand I'm thinking that we should try to report the exact position of the error. On the other hand, I started poking at it and it looks like it's quite a lot of work for something that's really edge casey.

[jwilsson]

I'll take another look at this.