SeanJM / atom-css-clean

A plugin for Atom which sorts and aligns CSS and SASS
MIT License
20 stars 2 forks source link

Deleting Code and Formatting Incorrectly #30

Closed dhruveonmars closed 7 years ago

dhruveonmars commented 7 years ago

It was deleting some of my sass inside a selector when it had two comments at the start of the selector, and after removing the comments, and trying to clean the sass, it turned the following: .main-menu_cntr { background: $black; color: $white; border-top-left-radius: 30px; border-bottom-left-radius: 30px; position: absolute; top: 86px; padding-right: calc(50% - #{$biggest}/2 + #{$gapsize}); right: 0;

@include breakpoint($biggest) { padding-right: $gapsize; } } into: .main-menu_cntr { position : absolute; top : 86px; color : $white; border-top-left-radius : 30px; border-bottom-left-radius : 30px; background : $black;

padding-right: calc(50% - #{$biggest}/2 + #{$gapsize}); right: 0; @include breakpoint($biggest) { padding-right : $gapsize; } }

SeanJM commented 7 years ago

Can you show the SASS for all the steps?

dhruveonmars commented 7 years ago

The sass is there, its just pasting without most of the spaces

The first block of code is the original sass, and the second block is the sass that got cleaned

SeanJM commented 7 years ago

Thank you.

On Feb 10, 2017, at 12:25 PM, Dhruve Shah notifications@github.com wrote:

The sass is there, its just pasting without most of the spaces

The first block of code is the original sass, and the second block is the sass that got cleaned

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/SeanJM/css-clean/issues/30#issuecomment-279007198, or mute the thread https://github.com/notifications/unsubscribe-auth/ABUkysO1hf1CJ2gULjpbk1vVxviWZflVks5rbJ2lgaJpZM4L9ROv.

voznik commented 7 years ago

it works wrong! I have following scss: before:

img {
    padding: 10px;
    @include media-breakpoint-up(md) {
      padding: 1rem;
    }
}

after css clean:

img {
    @include   media-breakpoint-up(md);
    padding  : 10px;
  }
SeanJM commented 7 years ago

Hi Vadym, I’ll fix that the next time I have some free time.

On May 29, 2017, at 4:18 AM, Vadym Parakonnyi notifications@github.com wrote:

it works wrong! I have following scss: before:

img { padding: 10px; @include media-breakpoint-up(md) { padding: 1rem; } } after css clean:

img { @include media-breakpoint-up(md); padding : 10px; } — You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub https://github.com/SeanJM/atom-css-clean/issues/30#issuecomment-304601972, or mute the thread https://github.com/notifications/unsubscribe-auth/ABUkyvlK7vldlk1qJEyOYDBWHQ39onxLks5r-n9LgaJpZM4L9ROv.