stephenbaldwin/fuelux-rails

Some optimization in lib/generators/fuelux/install_generator.rb

snachx opened this issue · 0 comments

if content.match(/require_tree\s+.\s*$/)
# Good enough - that'll include our Fuel UX js

the comment here is not suite for a gem, require_tree . won't automatic include the assets in gem files, we need to include it in application.js as you say in the last section of readme.md.

So the generator should always add a directive 'require fuelux' to the application.js if there has no one.

Also, I don't know why the fuelux.js does not require fuelux/checkbox and the fuelux/loader?