csscomb-atom icon indicating copy to clipboard operation
csscomb-atom copied to clipboard

"lines-between-rulesets" not included

Open tomagladiator opened this issue 8 years ago • 7 comments

Hello, Thanks for this plugin,

Can we add this useful rule for SCSS ? // Add a new line between selectors newlineBetweenSelectors: false!true

//  before
[♥-component~="accordion"] {
    border-bottom: 1px solid $mine-shaft;
    .a--title {
        align-items: center;
        &:hover,
        &:focus {
            cursor: pointer;
        }
        &.isClose {
            &:before {
                @include triangle(20px, $alto, down);
            }
        }
    }
}

// after
[♥-component~="accordion"] {
    border-bottom: 1px solid $mine-shaft;

    .a--title {
        align-items: center;

        &:hover,
        &:focus {
            cursor: pointer;
        }

        &.isClose {
            &:before {
                @include triangle(20px, $alto, down);
            }
        }
    }
}

Thanks

tomagladiator avatar Jul 28 '16 12:07 tomagladiator

I'm sorry but my plugin is using CSSComb, and all the properties there. I don't add their own, and just use standard.

I think you can try {"lines-between-rulesets": 1} from there options

This is what you need?

jchouse avatar Aug 08 '16 07:08 jchouse

Thanks @jchouse, First, I used this site to generate my config : http://csscomb.com/config But the generated json doesn't have this option : lines-between-rulesets So I added it to the jSon, and nothing new happend

I find that in your package, atom-css-comb/node_modules/csscomb/lib/options/ we don't find https://github.com/csscomb/csscomb.js/blob/dev/src/options/lines-between-rulesets.js

tomagladiator avatar Aug 12 '16 14:08 tomagladiator

OK i'll try to up version, or find some other way to fix that

jchouse avatar Aug 12 '16 16:08 jchouse

Hi,

I write an issue in the repo of csscomb, I think they didn't update their NPM package : https://github.com/csscomb/csscomb.js/issues/482

tomagladiator avatar Aug 14 '16 06:08 tomagladiator

You can use npm install [email protected], but don't forget that this is alpha version.

mrmlnc avatar Aug 18 '16 16:08 mrmlnc

Hi guys,

I'm interested to get this option working as well, if a solution has been found since August, any help would be gladly appreciated 😃

Le-future avatar Dec 09 '16 11:12 Le-future

Guys I tried all the options but looks like that rule is not working. Even if you use csscomb clean. I don't know what to do with it. I'm waiting for your solutions. Or solving bugs https://github.com/csscomb/csscomb.js/issues/209

jchouse avatar Dec 12 '16 09:12 jchouse