search

maxim / bootswatch-rails

Bootswatches converted to SCSS ready to use in Rails asset pipeline.
MIT License
489 stars 84 forks source link

Fixed buggy calls to bootstrap-sass #13

Closed NZGlitch closed 11 years ago

NZGlitch commented 12 years ago

There were 2 calls to gradient-vertical-three-colors where a value was given as a % when bootstrap-sass seems to prefer a decimal (i.e. 90% should be 0.9)

This was causing an error for me: '9000%*% is not a valid css.....'

I was using bootstrap-sass version 2.1.0.0

The bug exists in some other swatches as well.

mulder commented 11 years ago

:+1:

kurkavojtech commented 11 years ago

Oh man, you just saved me a few hours of my life. Surprisingly, I couldn't find any piece of information about this bug on the net. Good work and I hope that Maxim will take care of this soon. Once again - thanks.

V.

maxim commented 11 years ago

Verified, thanks!

engunneer commented 11 years ago

This appears to be broken again.

maxim commented 11 years ago

@engunneer Where exactly? Not all places need percentage as far as I understood. Are you on latest sass?

engunneer commented 11 years ago

Specifically https://github.com/maxim/bootswatch-rails/blob/master/vendor/assets/stylesheets/bootswatch/cerulean/_bootswatch.scss#L127

I'm using Rails 3.2.12, sass 3.2.6, sass-rails 3.2.6, bootstrap-sass 2.1.0.0, bootswatch-rails 0.4.0. changing line 127 of cerulean from 5% to .05 worked for me.

Edit: I updated to Rails 3.2.12, sass 3.2.7, sass-rails 3.2.6, bootstrap-sass 2.3.1.0 and it still doesn't like the 5%.