anjlab / bootstrap-rails

Twitter Bootstrap CSS (with Sass flavour) and JS toolkits for Rails 3 projects
https://github.com/anjlab/bootstrap-rails
729 stars 96 forks source link

Change readme for application.css #84

Closed 14113 closed 10 years ago

14113 commented 10 years ago

It's not necessarily to add 'import' into application.css. We can use 'require'. It's faster and better for developing

tough-griff commented 10 years ago

Using the sprockets //= require directive is only advisable if you want to use Bootstrap without any sort of customization or overrides.

From http://guides.rubyonrails.org/asset_pipeline.html:

If you want to use multiple Sass files, you should generally use the Sass @import rule instead of these Sprockets directives. Using Sprockets directives all Sass files exist within their own scope, making variables or mixins only available within the document they were defined in.

So, I would disagree that //= require is "faster and better for developing" in all cases.

yury commented 10 years ago

I totally agree with @tough-griff

yury commented 10 years ago

Closing this for now