Permalink
Browse files

upgraded to latest layoutmanager, almond, and jquery. updated $.get

call to use "text" Closes #194 and #197
  • Loading branch information...
1 parent 437d0e3 commit 48fcce565f63512c994bfeb365bcb0ceacfea8a2 @tbranyen committed Jan 18, 2013
View
@@ -5,7 +5,7 @@ define([
// `vendor/js` directory, here.
],
-function() {
+function(LayoutManager) {
// Provide a global location to place configuration settings and module
// creation.
@@ -18,7 +18,7 @@ function() {
var JST = window.JST = window.JST || {};
// Configure LayoutManager with Backbone Boilerplate defaults.
- Backbone.LayoutManager.configure({
+ LayoutManager.configure({
// Allow LayoutManager to augment Backbone.View.prototype.
manage: true,
@@ -38,8 +38,8 @@ function() {
// Seek out the template asynchronously.
$.get(app.root + path, function(contents) {
- done(JST[path] = _.template(contents));
- });
+ done(_.template(contents));
@jreyes33

jreyes33 Jan 22, 2013

Contributor

Didn't this change remove JST caching?

@tbranyen

tbranyen Jan 22, 2013

Owner

Sorta, LayoutManager caches internally, wasn't necessary.

+ }, "text");
}
});
View
@@ -15,7 +15,7 @@
<!-- Testing libs -->
<script src="vendor/jasmine.js"></script>
<script src="vendor/jasmine-html.js"></script>
-
+
<!-- Load application -->
<script data-main="../../app/config"
src="../../vendor/jam/require.js"></script>
Oops, something went wrong.

0 comments on commit 48fcce5

Please sign in to comment.