SCSS Broken?

[dramalho]

sass --check /usr/local/opt/rbenv/versions/2.1.2/lib/ruby/gems/2.1.0/gems/rails-assets-andy-1.0.1/app/assets/stylesheets/andy.scss
Error: Invalid CSS after "...ratio: $ratio),": expected media query (e.g. print, screen, print and screen), was "@media only scr..."
        on line 219 of /usr/local/opt/rbenv/versions/2.1.2/lib/ruby/gems/2.1.0/gems/rails-assets-andy-1.0.1/app/assets/stylesheets/andy/andy.scss
        from line 1 of /usr/local/opt/rbenv/versions/2.1.2/lib/ruby/gems/2.1.0/gems/rails-assets-andy-1.0.1/app/assets/stylesheets/andy.scss
  Use --trace for backtrace.

Pulling the raw file from github has the same issue. Am I doing something wrong?

[gillesbertaux]

Hm, go ahead and try, there was a small typo on l.219 probably from last merge, I pushed back to master. Let me know if it works now.

[gillesbertaux]

Oops another typo a few lines below from same merge. Sorry for that. Try again?

[dramalho]

humm no, weird - not that I'm a big sass expert. It works fine with either

@media only screen and (max-width: $width) and (min--moz-device-pixel-ratio: $ratio)

or

@media only screen and (max-width: $width) and (-webkit-min-device-pixel-ratio: $ratio)

by themselves, but not with the comma.

Sorry to be bothering :)

[gillesbertaux]

Ok, just pushed some changes. Compiling fine here, what about you?

[dramalho]

Perfect :)

new tag, for the sake of rails-assets?

Cheers Gilles

[gillesbertaux]

Great! Good catch ;)

Yep!

[dramalho]

awesome stuff, thanks Gilles

—
David Ramalho - @dramalho

On Thu, Nov 27, 2014 at 11:30 PM, Gilles Bertaux notifications@github.com
wrote:

Great! Good catch ;)

Yep!

Reply to this email directly or view it on GitHub:
#4 (comment)