mojotech / sass2stylus

Kewl
http://sass2stylus.com/
79 stars 18 forks source link

Doesn't handle multi-line selector specificaitons #86

Closed hiddentao closed 10 years ago

hiddentao commented 10 years ago

Converting:

html, body, div, span, applet, object, iframe,
h1, h2, h3, h4, h5, h6, p, blockquote, pre,
a, abbr, acronym, address, big, cite, code,
del, dfn, em, img, ins, kbd, q, s, samp,
small, strike, strong, sub, sup, tt, var,
b, i, u, center,
dl, dt, dd, ol, ul, li,
fieldset, form, label, legend,
table, caption, tbody, tfoot, thead, tr, th, td,
article, aside, canvas, details, embed, fieldset,
figure, figcaption, footer, header, hgroup, 
menu, nav, output, ruby, section, summary,
time, mark, audio, video {
  margin: 0;
  padding: 0;
  border: 0;
  vertical-align: baseline;
  font: inherit;
  font-size: 100%;
}

Gives:

html, body, div, span, applet, object, iframe,
  margin: 0
  padding: 0
  border: 0
  vertical-align: baseline
  font: inherit
  font-size: 100%

Bad for many reasons, and especially for the trailing comma (,) which trips up the stylus compiler.

jcollum-hcg commented 10 years ago

+1, seeing same behavior today on the website

samccone commented 10 years ago

Thanks for the report guys will dig into this one

demeshkin commented 10 years ago

+1

cpjolicoeur commented 10 years ago

+1

samccone commented 10 years ago

related #85

samccone commented 10 years ago

all deployed!