search

Sutto / barista

Simple, transparent CoffeeScript support for Rails and Rack apps.
MIT License
398 stars 27 forks source link

Installation and js loading #28

Open marcandre opened 13 years ago

marcandre commented 13 years ago

Barista looks awesome.

I installed it and converted a html :javascript to :coffeescript.

Without any other change (Rails 3.0.4 in development, no fiddling with default javascripts, empty barista config) it doesn't work because the client side coffeescript.js is not loaded (and the source html has the script as "text/coffeescript")

I changed the config with c.embedded_interpreter = false and that bypasses the issue.

1) Wouldn't it be best to add "coffeescript" to the default javascript sources? 2) In the meantime, is there a standard way of embedding "coffeescript.js"? 3) I'd recommend adding the embedded_interpreter option (commented) in the generated config.

Thanks!

Sutto commented 13 years ago
  1. I'm against adding it to the default sources for a couple of reasons (namely, I'm still not sold on embedding the interpreter - but I'm going to change the default to true for the next version).
  2. In rails, the (currently undocumented) coffeescript_interpreter_js helper is what you're looking for.
  3. Excellent idea - I've just added it (and will push out in the next gem release).
marcandre commented 13 years ago

Cool.

As for (1), you could embed it by default only if "emebedded_interpreter" is true. Otherwise, documentation for (2) in the installation notes would do too.

Thanks!