ben-eb / perfectionist

Beautify CSS files.
MIT License
229 stars 14 forks source link

Incorrect Indention level #7

Closed Mottie closed 8 years ago

Mottie commented 8 years ago

If the css is wrapped in an @ rule, the resulting indention & breaks between rules are incorrect:

Starting with this css:

@-moz-document domain('test.com') {
  .selector 1, .selector 2, .selector 3, .selector 4, .selector 5, .selector 6, .selector 7, .selector 8, .selector 9, .selector 10, .selector 11, .selector 12, .selector 13, .selector 14, .selector 15, .selector 16, .selector 17, .selector 18, .selector 19, .selector 20 {
    background: #000;
  }
}

@media all and (min-width: 480px) and (max-width: 767px) {
  .selector 1, .selector 2, .selector 3, .selector 4, .selector 5, .selector 6, .selector 7, .selector 8, .selector 9, .selector 10, .selector 11, .selector 12, .selector 13, .selector 14, .selector 15, .selector 16, .selector 17, .selector 18, .selector 19, .selector 20 {
    background: #000;
  }
}

with default settings, the result becomes:

@-moz-document domain('test.com') {
    .selector 1, .selector 2, .selector 3, .selector 4, .selector 5, .selector 6,
.selector 7, .selector 8, .selector 9, .selector 10, .selector 11, .selector 12,
.selector 13, .selector 14, .selector 15, .selector 16, .selector 17,
.selector 18, .selector 19, .selector 20 {
        background: #000;
    }
}@media all and (min-width: 480px) and (max-width: 767px) {
    .selector 1, .selector 2, .selector 3, .selector 4, .selector 5, .selector 6,
.selector 7, .selector 8, .selector 9, .selector 10, .selector 11, .selector 12,
.selector 13, .selector 14, .selector 15, .selector 16, .selector 17,
.selector 18, .selector 19, .selector 20 {
        background: #000;
    }
}

Also, is there a way to set the indention to use 2 spaces instead of 4?

ben-eb commented 8 years ago

Yep, can probably add some extra space there too. :+1:

ben-eb commented 8 years ago

Released as 1.3.0.

Mottie commented 8 years ago

Excellent work! :cookie: