Consider ripping Mustache view autoloading out about tilt HOT 9 CLOSED

I could go either way on this. One of my least favorite parts of Rails is all the autoloading.

But if you take that out, what does Tilt offer for Mustache users?

from tilt.

Between path, template_extension, template_file, and the :namespace option the autoloading is very configurable and simple. Maybe Tilt could provide it and the framework (Sinatra, Rails, whatev) could set up things the way it wants?

from tilt.

Yeah. Mustache is different from most template systems because the template file locating is handled so well by the view class. It seems like Tilt and Mustache make sense in the following scenarios:

1. Cases where you'd use Mustache.render or a template without a specific view subclass. e.g., here's a .html template file and a Hash of values, render that shit. In this way, Mustache works like most other template languages.
2. Cases where the view and template are always side by side in the same directory or where the view is very easily locatable given the template filename.

The case where Tilt probably isn't useful/applicable is when you're writing code directly against a view subclass before rendering. If you already have the view class, tilt doesn't really help anything. I can't think of any reason to use Tilt in that case.

So, if you use Mustache in a template oriented way, Tilt makes sense. If you use Mustache in a view class oriented way (which seems to be more in the spirit of the stache), then Tilt has little to offer.

from tilt.

Or something.

from tilt.

Maybe the autoloading should be in Mustache itself. It does load template files, after all.

from tilt.

I think that makes a lot of sense. We're already doing the autoloading in sinatra and now tilt so it's safe to assume more environments will need to deal with it. Implementing in mustache would make things a bit more consistent across environments.

from tilt.

Try this on for size: http://github.com/defunkt/tilt/commit/39d137a23e19163dcbdb2783b303e4f85080260b

from tilt.

Added some cleanup, too: http://github.com/defunkt/tilt/commits/simpler_mustache_loading

from tilt.

Great patch.

from tilt.