mrmlnc / vscode-csscomb

:electric_plug: VS Code plugin for CSScomb — CSS coding style formatter.
https://goo.gl/lKARa5
MIT License
65 stars 11 forks source link

"vendor-prefix-align: true" breaks indent for grid properties #130

Closed sgaponenko closed 4 years ago

sgaponenko commented 4 years ago

Environment

Actual behavior

after i save the .less file, csscomb sets the wrong indent

 .class {

        grid-column: 2; // <-- has one more indent

        grid-row: 1; // <-- has one more indent

    -ms-grid-column: 2;

    -ms-grid-row: 1;
}

Expected behavior

.class {

     grid-column: 2; 

     grid-row: 1; 

     -ms-grid-column: 2;

     -ms-grid-row: 1;

}

Config .csscomb.json

{

    "exclude": [

        ".git/**",

        "node_modules/**",

        "bower_components/**"

    ],

    "always-semicolon": true,

    "block-indent": "    ",

    "color-case": "lower",

    "color-shorthand": true,

    "element-case": "lower",

    "eof-newline": true,

    "leading-zero": false,

    "quotes": "single",

    "remove-empty-rulesets": true,

    "space-after-colon": " ",

    "space-after-combinator": " ",

    "space-after-opening-brace": "\n",

    "space-after-selector-delimiter": "\n",

    "space-before-closing-brace": "\n",

    "space-before-colon": "",

    "space-before-combinator": " ",

    "space-before-opening-brace": " ",

    "space-before-selector-delimiter": "",

    "space-between-declarations": "\n",

    "strip-spaces": true,

    "unitless-zero": true,

    "vendor-prefix-align": true,

    "sort-order": [...] 

}
mrmlnc commented 4 years ago

Hello, @sgaponenko,

It's looks like a bug of CSSComb. We use latest version of CSSComb and most likely it can be reproduced with CSSComb as CLI. You can create issue in the CSSComb repository and after bump CSSComb version I'll update my plugin 👍