kevin-montrose / More

A CSS Compiler
Microsoft Public License
42 stars 4 forks source link

More doesn't clean up unused css #6

Closed Meai1 closed 12 years ago

Meai1 commented 12 years ago

Steps to reproduce:

  1. create test.more
  2. Add
img { width: 10px; @(.avatar); }
p { font-color: grey; }
@media tv {
  p { font-color: black; }
  .avatar { border-width: 1px; @(p); }
  .wrapper { height:10px; @(img); }
}
  1. compile test.more
  2. Now replace everything in test.more with
@reset{
  h1 { margin: 0; }
}
.class h1 { color: blue; }
  1. compile test.more
  2. Weirdly the media rule is still there, it hasn't been cleaned up.
kevin-montrose commented 12 years ago

Fairly certain this was fixed in this commit.

Updated download should contain that fix.

Note that there was another bug with overwriting files also fixed in that download, you should have to use the /f flag to overwrite files that already exist.

Meai1 commented 12 years ago

thanks! the /f flag worked, pretty important. Without it on Linux it just keeps creating new files like test(1).css test(2).css