stylefmt icon indicating copy to clipboard operation
stylefmt copied to clipboard

selector-list-comma-newline-after [always] not working

Open davegomez opened this issue 7 years ago • 4 comments

The [always] option for the selector-list-comma-newline-after rule in combination with rules like selector-list-comma-space-after|before is not modifying the code correctly.

When the output should be something like this:

.selector-one,
.selector-one, {
  ...
}

Is returning this:

.selector-one, .selector-one, {
  ...
}

Couple of things I notice:

  • the project doesn't have code to deal with the [always] option but do have the unit test file for it
  • there's 44 unit tests failing.

davegomez avatar Jan 27 '17 16:01 davegomez

Going to bump this. Noticing this when trying to get a repo up to standards and that rule is not being correctly recognized. For a repo of a size as the one I am working on, this is generating 300+ warnings.

chryton avatar Jun 05 '17 22:06 chryton

I am having the exact same issue and fixing all these is quite annoying.

stefan-1st avatar Jun 07 '17 09:06 stefan-1st

I just wanted to confirm that this issue is, unfortunately, still present.

AndreCunha1 avatar Jan 02 '18 18:01 AndreCunha1

Eventually, I figured it out by adding

   "selector-list-comma-space-after": null,

luisliuchao avatar Jun 16 '20 11:06 luisliuchao