d4rkr00t / css-comb-atom

CssComb plugin for atom
9 stars 0 forks source link

Issue With 'always-semicolon'? #16

Open wylie opened 8 years ago

wylie commented 8 years ago

I'm seeing an issue with the always-semicolon option in the config.

Atom Version: 1.4.0 System: Mac OS X 10.11.2 Thrown From: css-comb package, v2.4.0

My config contains "always-semicolon": true, but it doesn't always add the missing semicolon on individual CSS declarations.

Here are some things I've noticed:

In trying to get my config how I want it I've been playing around with this block of CSS:

.test {
    position: absolute;
    display: block;
    padding: 0 10px 0 0;
    margin: auto;
    color: red;
    border-color: green;
}

Here is a link to my .csscomb.json file: https://www.dropbox.com/s/c4roltd95x4cb1b/csscomb.json?dl=0

Here is an image of my css-comb-atom settings

screen shot 2016-01-21 at 3 02 27 pm
d4rkr00t commented 8 years ago

Hi, thank you for report but it's an issue with csscomb itself, not with the plugin for atom. I copied your issue in csscomb repo — https://github.com/csscomb/csscomb.js/issues/447. And when/if they'll fix it, I'll update plugin.

wylie commented 8 years ago

Ah, okay. Thanks for the update. I hope they get on it because it's a particularly annoying bug.