selector_separator_newline adds erroneous newline on @extend SCSS statements

[TomAnthony]

Description

The selector_separator_newline option, when set to true, is slightly buggy with SCSS @extend statements, adding a newline when it should not.

Input

The code looked like this before beautification:

	a {
		@extend .color-black, .hover-far-red;
	}

Expected Output

The code should have looked like this after beautification:

	a {
		@extend .color-black, .hover-far-red;
	}

Actual Output

The code actually looked like this after beautification:

	a {
		@extend .color-black,
		.hover-far-red;
	}

The added newline is erroneous.

It works correctly for @import statements with comma delimited entries, and does not add a new line.

Steps to Reproduce

Settings

Example:

{
    "selector_separator_newline": false
}

[damienfa]

Same issue everywhere there is a "comma separated list" .

Example with list variable definition in SCSS :

Expected :
$sizes: 40px, 50px, 80px;

Actual wrong output :

$sizes: 40px,
50px,
80px;

Example with maps variable definition in SCSS :

Expected :
$default-margin: (desktop: 24px, tablet: 16px, phone: 16px);

Actual :

$default-margin: (desktop: 24px,
tablet: 16px,
phone: 16px,
);

Example with @each loop :

Expected :

@each $name, $color in $colorsrefs {
    .#{$name}-bg {
        background-color: $color;
    }
}

Actual wrong output :

@each $name,
$color in $colorsrefs {
    .#{$name}-bg {
        background-color: $color;
    }
}

Etc...

[mhnaeem]

@bitwiseman This was fixed in #1800