diff --git a/app/assets/javascripts/app/views.js b/app/assets/javascripts/app/views.js index 001ad5840..9605dd966 100644 --- a/app/assets/javascripts/app/views.js +++ b/app/assets/javascripts/app/views.js @@ -46,6 +46,10 @@ app.views.Base = Backbone.View.extend({ this.$el .html(this.template(presenter)) .attr("data-template", _.last(this.templateName.split("/"))); + + // add avatar fallback if it can't be loaded + this.$el.find(this.avatars.selector).error(this.avatars.fallback); + this.postRenderTemplate(); }, @@ -136,6 +140,13 @@ app.views.Base = Backbone.View.extend({ }); } }, + + avatars: { + fallback: function(evt) { + $(this).attr("src", ImagePaths.get("user/default.png")); + }, + selector: "img.avatar" + } }); app.views.StaticContentView = app.views.Base.extend({ diff --git a/app/assets/javascripts/view.js b/app/assets/javascripts/view.js index 7220d511c..a833bd035 100644 --- a/app/assets/javascripts/view.js +++ b/app/assets/javascripts/view.js @@ -26,9 +26,6 @@ var View = { .on('click', this.dropdowns.selector, this.dropdowns.click) .on('keypress', this.dropdowns.selector, this.dropdowns.click); - /* Avatars */ - $(this.avatars.selector).error(this.avatars.fallback); - /* Clear forms after successful submit, this is some legacy dan hanson stuff, do we still want it? */ $.fn.clearForm = function() { return this.each(function() { @@ -107,13 +104,6 @@ var View = { }, selector: ".dropdown > .toggle", parentSelector: ".dropdown > .wrapper" - }, - - avatars: { - fallback: function(evt) { - $(this).attr("src", ImagePaths.get("user/default.png")); - }, - selector: "img.avatar" } };